[PATCH 3/8] dtoc: Allow multiple warnings for a driver

Simon Glass sjg at chromium.org
Mon Jul 12 04:23:43 CEST 2021


On Sun, 11 Jul 2021 at 17:00, Simon Glass <sjg at chromium.org> wrote:
>
> Hi Simon,
>
> On 7/4/21 3:19 PM, Simon Glass wrote:
> > At present we show when a driver is missing but this is not always that
> > useful. There are various reasons way a driver may appear to be missing,
>
> Did you mean "why" instead of "way"?
>
> > such as a parse error in the source code or a missing field in the driver
> > declaration.
> >
> > Update the implementation to record all warnings for each driver, showing
> > only those which relate to drivers that are actually used. This avoids
> > spamming the user with warnings related to a driver for a different board.
> >
> > Signed-off-by: Simon Glass <sjg at chromium.org>
> > ---
> >
> >   tools/dtoc/src_scan.py | 24 ++++++++++++++++++++----
> >   1 file changed, 20 insertions(+), 4 deletions(-)
>
> Reviewed-by: Walter Lozano <walter.lozano at collabora.com>
>
> Thank you, it is indeed something worth to be added!
>
> Walter
>
> >
> Applied to u-boot-dm, thanks!

(Note: I fixed the nits in this patch and other other one when
applying...thanks for the review!)


More information about the U-Boot mailing list