[PATCH 0/2] arm: dts: Add Itap Delay Value For High Speed DDR

Nishanth Menon nm at ti.com
Fri Jan 19 15:45:19 CET 2024


On 07:20-20240110, Bryan Brattlof wrote:
> On January 10, 2024 thus sayeth Bhavya Kapoor:
> > 
> > On 08/01/24 7:35 pm, Bryan Brattlof wrote:
> > > Hi Bhavya!
> > > 
> > > On January  8, 2024 thus sayeth Bhavya Kapoor:
> > > > This Series adds Itap Delay Value for DDR52 speed mode for eMMC in
> > > > J7200 SoC and for DDR50 speed mode for MMCSD in J721s2 SoC.
> > > > 
> > > > Bhavya Kapoor (2):
> > > >    arm: dts: k3-j7200-main: Add Itap Delay Value For DDR52 speed mode
> > > >    arm: dts: k3-j721s2-main: Add Itap Delay Value For DDR50 speed mode
> > > > 
> > > >   arch/arm/dts/k3-j7200-main.dtsi  | 1 +
> > > >   arch/arm/dts/k3-j721s2-main.dtsi | 1 +
> > > Because of the periodic syncs with the kernel, modifying these dt files
> > > in U-Boot will cause confusion. (Which node is correct why did we have
> > > to do this in U-Boot and not in the Kernel... bla bla bla) If they
> > > absolutely need to go in now please override these nodes in the
> > > *-u-boot.dtsi files with a comment so we can keep track of these changes
> > > during the next sync with Linux.
> > > 
> > > ~Bryan
> > 
> > Hi Bryan, Fyi, This patch went in kernel as well.
> > 
> > Can be tracked below-
> > 
> > https://lore.kernel.org/all/170266085077.3490141.14935960940418963459.b4-ty@ti.com/
> > 
> > So , kernel and uboot dt files will remain in sync.
> > 
> 
> Sorry I may be missing something. Why do we need these properties in 
> U-Boot now? Why not wait 2 weeks for the v6.8-rc1 tag in Linux and sync 
> everything all at once?

I agree. NAK for the series. Please get this part of dts sync.
-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D


More information about the U-Boot mailing list