Hi Fabio, > Hi Lukasz, > > On Mon, Mar 25, 2024 at 5:48 AM Lukasz Majewski <lu...@denx.de> wrote: > > > The case here is that I'm modifying the *-u-boot.dts{i} files only. > > In > > The diff below shows that you are creating imx28-xea-1.dts and > imx28-xea-2.dts for U-Boot consumption and renaming the upstream > imx28-xea.dts to imx28-xea.dts. > > create mode 100644 arch/arm/dts/imx28-xea-1.dts > create mode 100644 arch/arm/dts/imx28-xea-2-u-boot.dtsi > create mode 100644 arch/arm/dts/imx28-xea-2.dts > rename arch/arm/dts/{imx28-xea.dts => imx28-xea.dtsi} (100%) >
Yes, exactly. > > other words, u-boot will not support features described in Linux > > DTS. > > That's OK and this happens frequently. > > For example, upstream devicetree may describes audio codec, > but U-Boot does not support audio playback. > > Devicetree should be OS agnostic. > Ok. > In U-Boot, we want to re-use the upstream Linux devicetree files > 'as-is'. > Yes, I'm fully aware of this. However, the U-Boot size increases rapidly... > Adding -u-boot.dtsi files is OK though. > Yes, this is a good way (with /delete-node) to reduce the size. > Can you convert the imx28-xea board to OF_UPSTREAM available in the > U-Boot next branch? I will check if XEA can be moved to OF_UPSTREAM. > > > Hence, the rename of files (which would be in sync with Linux at > > some point) looks like not related to Linux DTS (as even after > > re-sync with upstream Linux those changes will not be in Linux > > DTS). > > I did not understand this part, do you mean that Linux will also do > the imx28-xea.dts => imx28-xea.dtsi rename and will also have the new > imx28-xea-1.dts and imx28-xea-2.dts? Yes. Exactly. I started the conversion from u-boot. Updating Linux DTS will be next. > > Regards, > > Fabio Estevam Best regards, Lukasz Majewski -- DENX Software Engineering GmbH, Managing Director: Erika Unter HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-59 Fax: (+49)-8142-66989-80 Email: lu...@denx.de
pgppYXgAbZoRO.pgp
Description: OpenPGP digital signature