On 06/11/2018 01:53 PM, Chee, Tien Fong wrote: > 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.
And why can we not do as Linux does ? -- Best regards, Marek Vasut _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
