On Mon, 2018-06-11 at 11:39 +0200, Marek Vasut wrote: > On 06/11/2018 07:01 AM, Chee, Tien Fong wrote: > [...] > > > > > > > > > > > > > > > > > > > This is a configuration. You do not need (new) a DT > > > > > compatible > > > > > for > > > > > that. > > > > > So why is the DT compatible even needed in the FW loader at > > > > > all ? > > > > > > > > > I thought DT compatible is used by driver to find the fs_loader > > > > node in > > > > DTS. May be i am wrong. > > > There should be no FW loader in the DTS. Why would there be one ? > > > > > I added DTS support for user to define storage type and default > > partition. So you want me to remove the DTS? > > Removing the DTS, then user can only set storage type and partition > > through dev instance. So, this design OK for you? > How does Linux do it ? > Linux firmware loading method is different with this Linux loading firmware with 1. Firmware search paths - which is hardcoded in root file system 2. Built-in firmware - part of kernel binaries
This patch loading firmware with 1. Storage type and partition specified in DTS, but storage type can be overridden dev instance and partition through U-boot variable environment. Or: 2. Storage type can be set through dev instance, and partition through U-boot variable environment. No DTS is required. > [...] _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
