| Message ID | 20241023-b4-upstream-bootph-all-v5-2-a974d06370ab@ti.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | Add bootph-all property for J7 boards | expand |
On 23/10/24 12:27, Manorit Chawdhry wrote: > Adding bootph properties on leaf nodes imply that they are applicable to > the parent nodes as well. Bootloaders can derive the parent nodes when > bootph is available in the leaf nodes. > > Remove the bootph-* properties from parent nodes as they are redundant. > > Signed-off-by: Manorit Chawdhry <m-chawdhry@ti.com> Reviewed-by: Aniket Limaye <a-limaye@ti.com> > --- > > Notes: > It wasn't existing previously in U-boot but the following patch fixes it [0] > > [0]: https://lore.kernel.org/u-boot/20231217163627.2339802-10-sjg@chromium.org/ > > arch/arm64/boot/dts/ti/k3-j784s4-j742s2-mcu-wakeup-common.dtsi | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-mcu-wakeup-common.dtsi b/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-mcu-wakeup-common.dtsi > index 9899da73a905..46bc2a3e4aea 100644 > --- a/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-mcu-wakeup-common.dtsi > +++ b/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-mcu-wakeup-common.dtsi > @@ -7,7 +7,6 @@ > > &cbass_mcu_wakeup { > sms: system-controller@44083000 { > - bootph-all; > compatible = "ti,k2g-sci"; > ti,host-id = <12>; > > @@ -39,7 +38,6 @@ k3_reset: reset-controller { > }; > > wkup_conf: bus@43000000 { > - bootph-all; > compatible = "simple-bus"; > #address-cells = <1>; > #size-cells = <1>; > @@ -458,7 +456,6 @@ mcu_spi2: spi@40320000 { > }; > > mcu_navss: bus@28380000 { > - bootph-all; > compatible = "simple-bus"; > #address-cells = <2>; > #size-cells = <2>; >
diff --git a/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-mcu-wakeup-common.dtsi b/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-mcu-wakeup-common.dtsi index 9899da73a905..46bc2a3e4aea 100644 --- a/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-mcu-wakeup-common.dtsi +++ b/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-mcu-wakeup-common.dtsi @@ -7,7 +7,6 @@ &cbass_mcu_wakeup { sms: system-controller@44083000 { - bootph-all; compatible = "ti,k2g-sci"; ti,host-id = <12>; @@ -39,7 +38,6 @@ k3_reset: reset-controller { }; wkup_conf: bus@43000000 { - bootph-all; compatible = "simple-bus"; #address-cells = <1>; #size-cells = <1>; @@ -458,7 +456,6 @@ mcu_spi2: spi@40320000 { }; mcu_navss: bus@28380000 { - bootph-all; compatible = "simple-bus"; #address-cells = <2>; #size-cells = <2>;
Adding bootph properties on leaf nodes imply that they are applicable to the parent nodes as well. Bootloaders can derive the parent nodes when bootph is available in the leaf nodes. Remove the bootph-* properties from parent nodes as they are redundant. Signed-off-by: Manorit Chawdhry <m-chawdhry@ti.com> --- Notes: It wasn't existing previously in U-boot but the following patch fixes it [0] [0]: https://lore.kernel.org/u-boot/20231217163627.2339802-10-sjg@chromium.org/ arch/arm64/boot/dts/ti/k3-j784s4-j742s2-mcu-wakeup-common.dtsi | 3 --- 1 file changed, 3 deletions(-)