Re: [PATCH] ARM: dts: imx23/8: Rename apbh and apbx nodes

[Shawn Guo <shawnguo2@xxxxxxxx>]

On Wed, Aug 28, 2024 at 04:49:19PM -0300, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@xxxxxxx>
> 
> According to simple-bus.yaml, apbh and apbx are not valid bus names.
> 
> Rename them to apbh-bus and apbx-bus to fix the following dt-schema
> warnings:
> 
> 'apbh@80000000' does not match '^([a-z][a-z0-9\\-]+-bus|bus|localbus|soc|axi|ahb|apb)(@.+)?$'
> 'apbx@80040000' does not match '^([a-z][a-z0-9\\-]+-bus|bus|localbus|soc|axi|ahb|apb)(@.+)?$'
> 
> Signed-off-by: Fabio Estevam <festevam@xxxxxxx>
> ---
>  arch/arm/boot/dts/nxp/mxs/imx23-evk.dts           | 4 ++--
>  arch/arm/boot/dts/nxp/mxs/imx23-olinuxino.dts     | 4 ++--
>  arch/arm/boot/dts/nxp/mxs/imx23-sansa.dts         | 4 ++--
>  arch/arm/boot/dts/nxp/mxs/imx23-stmp378x_devb.dts | 4 ++--
>  arch/arm/boot/dts/nxp/mxs/imx23-xfi3.dts          | 4 ++--
>  arch/arm/boot/dts/nxp/mxs/imx23.dtsi              | 4 ++--
>  arch/arm/boot/dts/nxp/mxs/imx28-cfa10037.dts      | 2 +-

I think imx28-cfa10037.dts missed the change below.
@@ -14,7 +14,7 @@ / {
        compatible = "crystalfontz,cfa10037", "crystalfontz,cfa10036", "fsl,imx28";
 
        apb@80000000 {
-               apbh@80000000 {
+               apbh-bus@80000000 {
                        pinctrl@80018000 {
                                usb_pins_cfa10037: usb-10037@0 {
                                        reg = <0>;

I fixed it up.

Shawn

>  arch/arm/boot/dts/nxp/mxs/imx28.dtsi              | 4 ++--
>  8 files changed, 15 insertions(+), 15 deletions(-)