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...@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
