On Wed, 2018-06-06 at 10:39 +0200, Marek Vasut wrote:
> On 05/24/2018 07:04 AM, tien.fong.c...@intel.com wrote:
> >
> > From: Tien Fong Chee <tien.fong.c...@intel.com>
> >
> > This is file system generic loader which can be used to load
> > the file image from the storage into target such as memory.
> > The consumer driver would then use this loader to program whatever,
> > ie. the FPGA device.
> >
> > Signed-off-by: Tien Fong Chee <tien.fong.c...@intel.com>
> > ---
> [...]
> >
> > +static int fs_loader_probe(struct udevice *dev)
> > +{
> > + return 0;
> > +};
> > +
> > +static const struct udevice_id fs_loader_ids[] = {
> > + { .compatible = "fs_loader"},
> Why exactly is there a DT compatible for a firmware loader?
>
Correct me if i'm wrong, this is required to look the platform data
from DTS, right? Details of DTS in patch 2.
> >
> > + { }
> > +};
> > +
> > +U_BOOT_DRIVER(fs_loader) = {
> > + .name = "fs_loader",
> > + .id = UCLASS_FS_FIRMWARE_LOADER,
> > + .of_match = fs_loader_ids,
> > + .probe = fs_loader_probe,
> > + .ofdata_to_platdata = fs_loader_ofdata_to_platdata,
> > + .platdata_auto_alloc_size = sizeof(struct
> > device_platdata),
> > +};
> > +
> > +UCLASS_DRIVER(fs_loader) = {
> > + .id = UCLASS_FS_FIRMWARE_LOADER,
> > + .name = "fs_loader",
> > +};
> [...]
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
