[PATCH 4/6] stdio: Update to use compiler for Kconfig checks

Simon Glass sjg at chromium.org
Tue Aug 4 16:05:58 CEST 2020


Hi Tom,

I can't see anything other than my patches at that link. But it looks
like I didn't send it, sadly. I'll resend that patch.

Regards,
SImon

On Tue, 4 Aug 2020 at 07:46, Tom Rini <trini at konsulko.com> wrote:
>
> On Tue, Aug 04, 2020 at 07:37:02AM -0600, Simon Glass wrote:
> > Hi Tom,
> >
> > On Mon, 3 Aug 2020 at 20:18, Tom Rini <trini at konsulko.com> wrote:
> > >
> > > On Mon, Aug 03, 2020 at 06:57:05PM -0400, Tom Rini wrote:
> > > > On Fri, Jul 17, 2020 at 09:03:17PM -0600, Simon Glass wrote:
> > > >
> > > > > Drop use of the preprocessor where possible.
> > > > >
> > > > > Signed-off-by: Simon Glass <sjg at chromium.org>
> > > > [snip]
> > > > > +   if (IS_ENABLED(CONFIG_DM_VIDEO)) {
> > > > > +           /*
> > > > > +            * If the console setting is not in environment variables then
> > > > > +            * console_init_r() will not be calling iomux_doenv() (which
> > > > > +            * calls search_device()). So we will not dynamically add
> > > > > +            * devices by calling stdio_probe_device().
> > > > > +            *
> > > > > +            * So just probe all video devices now so that whichever one is
> > > > > +            * required will be available.
> > > > > +            */
> > > > > +           struct udevice *vdev;
> > > > > +           int ret;
> > > > > +
> > > > > +           if (!IS_ENABLED(CONFIG_SYS_CONSOLE_IS_IN_ENV)) {
> > > > > +                   for (ret = uclass_first_device(UCLASS_VIDEO, &vdev);
> > > > > +                        vdev;
> > > > > +                        ret = uclass_next_device(&vdev))
> > > > > +                           ;
> > > > > +                   if (ret)
> > > > > +                           printf("%s: Video device failed (ret=%d)\n",
> > > > > +                                  __func__, ret);
> > > > > +           }
> > > > > +           if (IS_ENABLED(CONFIG_SPLASH_SCREEN) &&
> > > > > +               IS_ENABLED(CONFIG_CMD_BMP))
> > > > > +                   splash_display();
> > > >
> > > > We can't do this yet because CONFIG_SPLASH_SCREEN isn't migrated so the
> > > > test fails and we drop the bmp logo.  I got this run-time tested and
> > > > confirmed on colibri imx6 by my colleague Matt Porter.  So I'm holding
> > > > off on _this_ patch (and then the style clean up patch) until I can push
> > > > SPLASH_SCREEN migration through.
> > >
> > > This is also an issue for "stdio: Tidy up use of
> > > CONFIG_SYS_DEVICE_NULLDEV" as well, so I'm going to put aside the whole
> > > series until I can convert SPLASH_SCREEN.
> >
> > OK, let me know if you'd like me to do something here. My patch 2 was
> > supposed to migrate CONFIG_SPLASH_SCREEN, etc. Did that not work?
>
> I don't see that patch:
> http://patchwork.ozlabs.org/user/todo/uboot/?series=190587
>
> --
> Tom


More information about the U-Boot mailing list