mbox series

[0/4] ARM: dts: at91: calao_usb: fix various naming problems

Message ID 20250131210236.36212-6-wsa+renesas@sang-engineering.com (mailing list archive)
Headers show
Series ARM: dts: at91: calao_usb: fix various naming problems | expand

Message

Wolfram Sang Jan. 31, 2025, 9:02 p.m. UTC
Two patches deal with the either missed or mistyped vendor 'Calao'. The
other two patches deal with improper node names as reported by
dtbs_check.

Wolfram Sang (4):
  ARM: dts: at91: usb_a9263: fix wrong vendor
  ARM: dts: at91: use correct vendor name for Calao boards
  ARM: dts: at91: calao_usb: fix button nodes
  ARM: dts: at91: usb_a9g20_lpw: use proper mmc node name

 arch/arm/boot/dts/microchip/tny_a9260.dts          |  2 +-
 arch/arm/boot/dts/microchip/tny_a9260_common.dtsi  |  2 +-
 arch/arm/boot/dts/microchip/tny_a9263.dts          |  2 +-
 arch/arm/boot/dts/microchip/tny_a9g20.dts          |  2 +-
 arch/arm/boot/dts/microchip/usb_a9260.dts          |  2 +-
 arch/arm/boot/dts/microchip/usb_a9260_common.dtsi  |  6 ++----
 arch/arm/boot/dts/microchip/usb_a9263.dts          |  8 +++-----
 arch/arm/boot/dts/microchip/usb_a9g20-dab-mmx.dtsi | 10 ++++------
 arch/arm/boot/dts/microchip/usb_a9g20.dts          |  2 +-
 arch/arm/boot/dts/microchip/usb_a9g20_common.dtsi  |  2 +-
 arch/arm/boot/dts/microchip/usb_a9g20_lpw.dts      |  4 ++--
 11 files changed, 18 insertions(+), 24 deletions(-)

Comments

Rob Herring (Arm) Jan. 31, 2025, 9:23 p.m. UTC | #1
On Fri, 31 Jan 2025 22:02:35 +0100, Wolfram Sang wrote:
> Two patches deal with the either missed or mistyped vendor 'Calao'. The
> other two patches deal with improper node names as reported by
> dtbs_check.
> 
> Wolfram Sang (4):
>   ARM: dts: at91: usb_a9263: fix wrong vendor
>   ARM: dts: at91: use correct vendor name for Calao boards
>   ARM: dts: at91: calao_usb: fix button nodes
>   ARM: dts: at91: usb_a9g20_lpw: use proper mmc node name
> 
>  arch/arm/boot/dts/microchip/tny_a9260.dts          |  2 +-
>  arch/arm/boot/dts/microchip/tny_a9260_common.dtsi  |  2 +-
>  arch/arm/boot/dts/microchip/tny_a9263.dts          |  2 +-
>  arch/arm/boot/dts/microchip/tny_a9g20.dts          |  2 +-
>  arch/arm/boot/dts/microchip/usb_a9260.dts          |  2 +-
>  arch/arm/boot/dts/microchip/usb_a9260_common.dtsi  |  6 ++----
>  arch/arm/boot/dts/microchip/usb_a9263.dts          |  8 +++-----
>  arch/arm/boot/dts/microchip/usb_a9g20-dab-mmx.dtsi | 10 ++++------
>  arch/arm/boot/dts/microchip/usb_a9g20.dts          |  2 +-
>  arch/arm/boot/dts/microchip/usb_a9g20_common.dtsi  |  2 +-
>  arch/arm/boot/dts/microchip/usb_a9g20_lpw.dts      |  4 ++--
>  11 files changed, 18 insertions(+), 24 deletions(-)
> 
> --
> 2.45.2
> 
> 
> 


My bot found new DTB warnings on the .dts files added or changed in this
series.

Some warnings may be from an existing SoC .dtsi. Or perhaps the warnings
are fixed by another series. Ultimately, it is up to the platform
maintainer whether these warnings are acceptable or not. No need to reply
unless the platform maintainer has comments.

If you already ran DT checks and didn't see these error(s), then
make sure dt-schema is up to date:

  pip3 install dtschema --upgrade


New warnings running 'make CHECK_DTBS=y for arch/arm/boot/dts/microchip/' for 20250131210236.36212-6-wsa+renesas@sang-engineering.com:

arch/arm/boot/dts/microchip/usb_a9263.dtb: /: failed to match any schema with compatible: ['calao,usb-a9263', 'atmel,at91sam9263', 'atmel,at91sam9']
Nicolas Ferre Feb. 3, 2025, 1:08 p.m. UTC | #2
On 31/01/2025 at 22:02, Wolfram Sang wrote:
> Two patches deal with the either missed or mistyped vendor 'Calao'. The
> other two patches deal with improper node names as reported by
> dtbs_check.
> 
> Wolfram Sang (4):
>    ARM: dts: at91: usb_a9263: fix wrong vendor
>    ARM: dts: at91: use correct vendor name for Calao boards
>    ARM: dts: at91: calao_usb: fix button nodes
>    ARM: dts: at91: usb_a9g20_lpw: use proper mmc node name

Series looks good to me:
Acked-by: Nicolas Ferre <nicolas.ferre@microchip.com>

Thanks Wolfram. Regards,
   Nicolas

>   arch/arm/boot/dts/microchip/tny_a9260.dts          |  2 +-
>   arch/arm/boot/dts/microchip/tny_a9260_common.dtsi  |  2 +-
>   arch/arm/boot/dts/microchip/tny_a9263.dts          |  2 +-
>   arch/arm/boot/dts/microchip/tny_a9g20.dts          |  2 +-
>   arch/arm/boot/dts/microchip/usb_a9260.dts          |  2 +-
>   arch/arm/boot/dts/microchip/usb_a9260_common.dtsi  |  6 ++----
>   arch/arm/boot/dts/microchip/usb_a9263.dts          |  8 +++-----
>   arch/arm/boot/dts/microchip/usb_a9g20-dab-mmx.dtsi | 10 ++++------
>   arch/arm/boot/dts/microchip/usb_a9g20.dts          |  2 +-
>   arch/arm/boot/dts/microchip/usb_a9g20_common.dtsi  |  2 +-
>   arch/arm/boot/dts/microchip/usb_a9g20_lpw.dts      |  4 ++--
>   11 files changed, 18 insertions(+), 24 deletions(-)
> 
> --
> 2.45.2
>
Wolfram Sang Feb. 4, 2025, 10:43 a.m. UTC | #3
> arch/arm/boot/dts/microchip/usb_a9263.dtb: /: failed to match any schema with compatible: ['calao,usb-a9263', 'atmel,at91sam9263', 'atmel,at91sam9']

I forgot to mention there is a dependency. As I need to resend the
dependency anyhow, I will put it into the series.
claudiu beznea (tuxon) Feb. 17, 2025, 9:14 a.m. UTC | #4
Hi, Wolfram,

> Wolfram Sang (4):
>   ARM: dts: at91: usb_a9263: fix wrong vendor

For this, I'll be waiting a follow up on
https://lore.kernel.org/all/20250131162713.33524-2-wsa+renesas@sang-engineering.com/

>   ARM: dts: at91: use correct vendor name for Calao boards
>   ARM: dts: at91: calao_usb: fix button nodes
>   ARM: dts: at91: usb_a9g20_lpw: use proper mmc node name

These were applied to at91-dt, thanks!
Wolfram Sang Feb. 17, 2025, 9:53 a.m. UTC | #5
> For this, I'll be waiting a follow up on
> https://lore.kernel.org/all/20250131162713.33524-2-wsa+renesas@sang-engineering.com/

I have the patches ready. Will send v2 after I rebased on top of your
at91-next tree.

> These were applied to at91-dt, thanks!

Thank you!