[1/2] ARM: dts: versatile: Fix up interrupt controller node names

Once the new schema interrupt-controller/arm,vic.yaml is added, we get
the below warnings:

        arch/arm/boot/dts/versatile-ab.dt.yaml:
        intc@10140000: $nodename:0: 'intc@10140000' does not match
        '^interrupt-controller(@[0-9a-f,]+)*$'

	arch/arm/boot/dts/versatile-ab.dt.yaml:
	intc@10140000: 'clear-mask' does not match any of the regexes

Fix the node names for the interrupt controller to conform
to the standard node name interrupt-controller@.. Also drop invalid
clear-mask property.

Cc: Linus Walleij <linus.walleij@linaro.org>
Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>

---
 arch/arm/boot/dts/versatile-ab.dts | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

-- 
2.25.1

On Thu, Jun 17, 2021 at 11:08 PM Sudeep Holla <sudeep.holla@arm.com> wrote:

> Once the new schema interrupt-controller/arm,vic.yaml is added, we get

> the below warnings:

>

>         arch/arm/boot/dts/versatile-ab.dt.yaml:

>         intc@10140000: $nodename:0: 'intc@10140000' does not match

>         '^interrupt-controller(@[0-9a-f,]+)*$'

>

>         arch/arm/boot/dts/versatile-ab.dt.yaml:

>         intc@10140000: 'clear-mask' does not match any of the regexes

>

> Fix the node names for the interrupt controller to conform

> to the standard node name interrupt-controller@.. Also drop invalid

> clear-mask property.

>

> Cc: Linus Walleij <linus.walleij@linaro.org>

> Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>


Thanks Sudeep, I relayed the patch to the SoC tree!

Yours,
Linus Walleij

Hi Sudeep,

I love your patch! Yet something to improve:

[auto build test ERROR on robh/for-next]
[also build test ERROR on pinctrl/devel v5.13 next-20210630]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/0day-ci/linux/commits/Sudeep-Holla/ARM-dts-versatile-Fix-up-interrupt-controller-node-names/20210618-050956
base:   https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next
config: arm-allmodconfig (attached as .config)
compiler: arm-linux-gnueabi-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # https://github.com/0day-ci/linux/commit/39c2a1b84ddd856b2a8015abc7952b9ca79a5c4c
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Sudeep-Holla/ARM-dts-versatile-Fix-up-interrupt-controller-node-names/20210618-050956
        git checkout 39c2a1b84ddd856b2a8015abc7952b9ca79a5c4c
        # save the attached .config to linux build tree
        mkdir build_dir
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross O=build_dir ARCH=arm SHELL=/bin/bash

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All errors (new ones prefixed by >>):

   arch/arm/boot/dts/versatile-pb.dts:10.22-17.5: ERROR (duplicate_label): /amba/intc@10003000: Duplicate label 'sic' on /amba/intc@10003000 and /amba/interrupt-controller@10003000
>> ERROR: Input tree has errors, aborting (use -f to force output)


---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

Hi Linus,

Sorry for this. For some reason I had assumed versatile is in
multi_v5_defconfig, but it is not and I missed to see that.

I will post v2 to SoC team directly with your ack instead.

On Thu, Jul 01, 2021 at 06:53:53PM +0800, kernel test robot wrote:
> Hi Sudeep,

> 

> I love your patch! Yet something to improve:

> 

> [auto build test ERROR on robh/for-next]

> [also build test ERROR on pinctrl/devel v5.13 next-20210630]

> [If your patch is applied to the wrong git tree, kindly drop us a note.

> And when submitting patch, we suggest to use '--base' as documented in

> https://git-scm.com/docs/git-format-patch]

> 

> url:    https://github.com/0day-ci/linux/commits/Sudeep-Holla/ARM-dts-versatile-Fix-up-interrupt-controller-node-names/20210618-050956

> base:   https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next

> config: arm-allmodconfig (attached as .config)

> compiler: arm-linux-gnueabi-gcc (GCC) 9.3.0

> reproduce (this is a W=1 build):

>         wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross

>         chmod +x ~/bin/make.cross

>         # https://github.com/0day-ci/linux/commit/39c2a1b84ddd856b2a8015abc7952b9ca79a5c4c

>         git remote add linux-review https://github.com/0day-ci/linux

>         git fetch --no-tags linux-review Sudeep-Holla/ARM-dts-versatile-Fix-up-interrupt-controller-node-names/20210618-050956

>         git checkout 39c2a1b84ddd856b2a8015abc7952b9ca79a5c4c

>         # save the attached .config to linux build tree

>         mkdir build_dir

>         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross O=build_dir ARCH=arm SHELL=/bin/bash

> 

> If you fix the issue, kindly add following tag as appropriate

> Reported-by: kernel test robot <lkp@intel.com>

> 

> All errors (new ones prefixed by >>):

> 

>    arch/arm/boot/dts/versatile-pb.dts:10.22-17.5: ERROR (duplicate_label): /amba/intc@10003000: Duplicate label 'sic' on /amba/intc@10003000 and /amba/interrupt-controller@10003000

> >> ERROR: Input tree has errors, aborting (use -f to force output)

> 

> ---

> 0-DAY CI Kernel Test Service, Intel Corporation

> https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org




-- 
Regards,
Sudeep