Re: [PATCH v3 5/5] arm64: dts: ti: k3-j7200*: Add bootph-* properties

[Manorit Chawdhry <m-chawdhry@ti.com>]

Hi Aniket,

On 20:07-20240807, Limaye, Aniket wrote:
> Hi Manorit,
> 
> On 7/30/2024 3:23 PM, Manorit Chawdhry wrote:
> > Adds bootph-* properties to the leaf nodes to enable U-boot to
> > utilise them.
> > 
> > Signed-off-by: Manorit Chawdhry <m-chawdhry@ti.com>
> > ---
> >   .../arm64/boot/dts/ti/k3-j7200-common-proc-board.dts | 20 ++++++++++++++++++++
> >   arch/arm64/boot/dts/ti/k3-j7200-main.dtsi            |  2 ++
> >   arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi      | 10 ++++++++++
> >   arch/arm64/boot/dts/ti/k3-j7200-som-p0.dtsi          |  7 +++++++
> >   4 files changed, 39 insertions(+)
> > 
> > diff --git a/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi b/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
> > index 5097d192c2b2..f8a5ad4737da 100644
> > --- a/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
> > +++ b/arch/arm64/boot/dts/ti/k3-j7200-mcu-wakeup.dtsi
> > @@ -21,16 +21,19 @@ dmsc: system-controller@44083000 {
> 
> Referring to the thread from v2 [0], I see that you have removed the
> bootph-all from the parent dmsc node. With current patch j7200-evm fails to
> boot (Stuck somewhere in tispl or right after that) as discussed in the same
> thread [0].
> 
> I have boot tested this patch (SD boot) and the device boots with an
> additional bootph-all at dmsc node.
> 
> I assume the removal was done intentionally, meaning we intend to fix the
> bug in u-boot or carry the one extra bootph-all there. If NOT, please add
> the same in the parent dmsc node here as well.

Yes, the removal was intentional and there is a fix that can be done in
U-boot for this so we'll take that up there.

Thanks for reviewing!

Regards,
Manorit

> 
> So taking care of the above,
> 
> Reviewed-by: Aniket Limaye <a-limaye@ti.com>
> 
> [0]: https://lore.kernel.org/all/20240709084857.nf7c57mi6miajeau@uda0497581/