[U-Boot] [PATCH] misc: fs_loader: Replace label with DT phandle

Simon Glass sjg at chromium.org
Tue Mar 19 01:22:48 UTC 2019


Hi Tien Fong,

On Mon, 11 Mar 2019 at 12:28, Chee, Tien Fong <tien.fong.chee at intel.com> wrote:
>
> On Sun, 2019-03-10 at 15:51 -0600, Simon Glass wrote:
> > Hi Tien Fong,
> >
> > On Tue, 26 Feb 2019 at 05:37, Chee, Tien Fong <tien.fong.chee at intel.c
> > om> wrote:
> > >
> > >
> > > On Fri, 2019-02-15 at 14:35 +0800, tien.fong.chee at intel.com wrote:
> > > >
> > > > From: Tien Fong Chee <tien.fong.chee at intel.com>
> > > >
> > > > In previously label which will be expanded to the node's full
> > > > path
> > > > was
> > > > used, and now replacing label with most commonly used DT phandle.
> > > > The
> > > > codes were changed accordingly to the use of DT phandle and
> > > > supporting
> > > > multiple instances.
> > > >
> > > > Signed-off-by: Tien Fong Chee <tien.fong.chee at intel.com>
> > > > ---
> > > >  doc/driver-model/fs_firmware_loader.txt |   58
> > > > +++++++++++++++++++++++++------
> > > >  drivers/misc/fs_loader.c                |   36 ++++++++---------
> > > > --
> > > >  2 files changed, 62 insertions(+), 32 deletions(-)
> > This seems OK to me, but I think this feature needs a test.
> Yes, i have ran the test passed with FPGA driver.

No I mean that you should add a test in test/py/tests

Regards,
Simon


More information about the U-Boot mailing list