| Message ID | 20230929134646.214781-7-rogerq@kernel.org |
|---|---|
| State | Accepted |
| Commit | 6d35682fb61ef5933eb3ddcf57acf87eaa076634 |
| Delegated to: | Tom Rini |
| Headers | show |
| Series | arm: dts: k3-am64 / k3-am62: Fix boot | expand |
On 16:46-20230929, Roger Quadros wrote: > CPSW node needs PHY, MDIO, pinmux, DMA and INTC nodes. > main_conf is required for phy_gmii_sel. > Mark them as 'bootph-all' so they are available in all > pre-relocation phases. > > Fixes the below dts warnings: > > <stdout>: Warning (reg_format): /bus@f0000/syscon@100000/phy@4044:reg: property has invalid length (8 bytes) (#address-cells == 2, #size-cells == 1) > <stdout>: Warning (reg_format): /bus@f0000/ethernet@8000000/ethernet-ports/port@1:reg: property has invalid length (4 bytes) (#address-cells == 2, #size-cells == 1) > <stdout>: Warning (unit_address_vs_reg): /bus@f0000/syscon@100000: node has a unit name, but no reg or ranges property > <stdout>: Warning (pci_device_reg): Failed prerequisite 'reg_format' > <stdout>: Warning (pci_device_bus_num): Failed prerequisite 'reg_format' > <stdout>: Warning (simple_bus_reg): Failed prerequisite 'reg_format' > <stdout>: Warning (i2c_bus_reg): Failed prerequisite 'reg_format' > <stdout>: Warning (spi_bus_reg): Failed prerequisite 'reg_format' > <stdout>: Warning (avoid_default_addr_size): /bus@f0000/syscon@100000/phy@4044: Relying on default #address-cells value > <stdout>: Warning (avoid_default_addr_size): /bus@f0000/syscon@100000/phy@4044: Relying on default #size-cells value > <stdout>: Warning (avoid_default_addr_size): /bus@f0000/ethernet@8000000/ethernet-ports/port@1: Relying on default #address-cells value > <stdout>: Warning (avoid_default_addr_size): /bus@f0000/ethernet@8000000/ethernet-ports/port@1: Relying on default #size-cells value > <stdout>: Warning (avoid_unnecessary_addr_size): Failed prerequisite 'avoid_default_addr_size' > <stdout>: Warning (unique_unit_address): Failed prerequisite 'avoid_default_addr_size' > > Signed-off-by: Roger Quadros <rogerq@kernel.org> > --- > arch/arm/dts/k3-am625-sk-u-boot.dtsi | 39 ++++++++++++++++++++++++++++ > 1 file changed, 39 insertions(+) > > diff --git a/arch/arm/dts/k3-am625-sk-u-boot.dtsi b/arch/arm/dts/k3-am625-sk-u-boot.dtsi > index c658506256..b78d34b459 100644 > --- a/arch/arm/dts/k3-am625-sk-u-boot.dtsi > +++ b/arch/arm/dts/k3-am625-sk-u-boot.dtsi > @@ -21,6 +21,10 @@ > }; > }; > > +&main_conf { > + bootph-all; > +}; > + > &cbass_main { > bootph-all; > }; > @@ -118,8 +122,43 @@ > }; > }; > > +&inta_main_dmss { > + bootph-all; > +}; > + > +&main_pktdma { > + bootph-all; > +}; > + > +&cpsw3g_mdio { > + bootph-all; > +}; > + > +&cpsw3g_phy0 { > + bootph-all; > +}; > + > +&cpsw3g_phy1 { > + bootph-all; > +}; > + > +&main_rgmii1_pins_default { > + bootph-all; > +}; > + > +&main_rgmii2_pins_default { > + bootph-all; > +}; > + > +&phy_gmii_sel { > + bootph-all; > +}; > + > &cpsw3g { > bootph-all; > + ethernet-ports { > + bootph-all; > + }; > }; > > &cpsw_port1 { > -- > 2.34.1 > Reviewed-by: Nishanth Menon <nm@ti.com>
diff --git a/arch/arm/dts/k3-am625-sk-u-boot.dtsi b/arch/arm/dts/k3-am625-sk-u-boot.dtsi index c658506256..b78d34b459 100644 --- a/arch/arm/dts/k3-am625-sk-u-boot.dtsi +++ b/arch/arm/dts/k3-am625-sk-u-boot.dtsi @@ -21,6 +21,10 @@ }; }; +&main_conf { + bootph-all; +}; + &cbass_main { bootph-all; }; @@ -118,8 +122,43 @@ }; }; +&inta_main_dmss { + bootph-all; +}; + +&main_pktdma { + bootph-all; +}; + +&cpsw3g_mdio { + bootph-all; +}; + +&cpsw3g_phy0 { + bootph-all; +}; + +&cpsw3g_phy1 { + bootph-all; +}; + +&main_rgmii1_pins_default { + bootph-all; +}; + +&main_rgmii2_pins_default { + bootph-all; +}; + +&phy_gmii_sel { + bootph-all; +}; + &cpsw3g { bootph-all; + ethernet-ports { + bootph-all; + }; }; &cpsw_port1 {
CPSW node needs PHY, MDIO, pinmux, DMA and INTC nodes. main_conf is required for phy_gmii_sel. Mark them as 'bootph-all' so they are available in all pre-relocation phases. Fixes the below dts warnings: <stdout>: Warning (reg_format): /bus@f0000/syscon@100000/phy@4044:reg: property has invalid length (8 bytes) (#address-cells == 2, #size-cells == 1) <stdout>: Warning (reg_format): /bus@f0000/ethernet@8000000/ethernet-ports/port@1:reg: property has invalid length (4 bytes) (#address-cells == 2, #size-cells == 1) <stdout>: Warning (unit_address_vs_reg): /bus@f0000/syscon@100000: node has a unit name, but no reg or ranges property <stdout>: Warning (pci_device_reg): Failed prerequisite 'reg_format' <stdout>: Warning (pci_device_bus_num): Failed prerequisite 'reg_format' <stdout>: Warning (simple_bus_reg): Failed prerequisite 'reg_format' <stdout>: Warning (i2c_bus_reg): Failed prerequisite 'reg_format' <stdout>: Warning (spi_bus_reg): Failed prerequisite 'reg_format' <stdout>: Warning (avoid_default_addr_size): /bus@f0000/syscon@100000/phy@4044: Relying on default #address-cells value <stdout>: Warning (avoid_default_addr_size): /bus@f0000/syscon@100000/phy@4044: Relying on default #size-cells value <stdout>: Warning (avoid_default_addr_size): /bus@f0000/ethernet@8000000/ethernet-ports/port@1: Relying on default #address-cells value <stdout>: Warning (avoid_default_addr_size): /bus@f0000/ethernet@8000000/ethernet-ports/port@1: Relying on default #size-cells value <stdout>: Warning (avoid_unnecessary_addr_size): Failed prerequisite 'avoid_default_addr_size' <stdout>: Warning (unique_unit_address): Failed prerequisite 'avoid_default_addr_size' Signed-off-by: Roger Quadros <rogerq@kernel.org> --- arch/arm/dts/k3-am625-sk-u-boot.dtsi | 39 ++++++++++++++++++++++++++++ 1 file changed, 39 insertions(+)