[U-Boot] [PATCH 02/23] dm: video: Flush the cache after a puts()
Tom Warren
TWarren at nvidia.com
Tue Jan 19 17:53:09 CET 2016
Simon,
> -----Original Message-----
> From: sjg at google.com [mailto:sjg at google.com] On Behalf Of Simon Glass
> Sent: Monday, January 18, 2016 7:03 PM
> To: Tom Warren <TWarren at nvidia.com>
> Cc: U-Boot Mailing List <u-boot at lists.denx.de>; Stephen Warren
> <swarren at wwwdotorg.org>; Stephen Warren <swarren at nvidia.com>; Marek
> Vasut <marek.vasut at gmail.com>; Anatolij Gustschin <agust at denx.de>; Pavel
> Herrmann <morpheus.ibis at gmail.com>
> Subject: Re: [PATCH 02/23] dm: video: Flush the cache after a puts()
>
> Hi Tom,
>
> On 18 January 2016 at 15:29, Tom Warren <TWarren at nvidia.com> wrote:
> > Simon,
> >
> >> -----Original Message-----
> >> From: Simon Glass [mailto:sjg at google.com] On Behalf Of Simon Glass
> >> Sent: Thursday, January 14, 2016 11:28 AM
> >> To: U-Boot Mailing List <u-boot at lists.denx.de>
> >> Cc: Tom Warren <TWarren at nvidia.com>; Stephen Warren
> >> <swarren at wwwdotorg.org>; Stephen Warren <swarren at nvidia.com>;
> Simon
> >> Glass <sjg at chromium.org>; Marek Vasut <marek.vasut at gmail.com>;
> >> Anatolij Gustschin <agust at denx.de>; Pavel Herrmann
> >> <morpheus.ibis at gmail.com>
> >> Subject: [PATCH 02/23] dm: video: Flush the cache after a puts()
> >>
> >> This helps keep the display consistent. puts() is used when printing
> >> the prompt, so is a useful way to make sure the current display contents is
> visible.
> >>
> >> Signed-off-by: Simon Glass <sjg at chromium.org>
> >> ---
> > This doesn't apply to TOT u-boot-tegra/master (just rebased against TOT u-
> boot/master). Vidconsole-uclass.c is missing:
> >
> > Applying: dm: video: Flush the cache after a puts()
> > error: drivers/video/vidconsole-uclass.c: does not exist in index
> > Patch failed at 0002 dm: video: Flush the cache after a puts() The
> > copy of the patch that failed is found in:
> > /home/tom/denx/uboot-tegra/.git/rebase-apply/patch
> >
>
> Yes this depends on the driver model series - see u-boot-dm/rkf-working for the
> source tree. I've got a few comments on the uclass now so I'll send v2 at some
> point and hopefully apply it soon after that.
OK. But you've assigned these to me in Patchwork (or I've been automatically designated as the Delegate), which makes me think I need to at least try and apply them to u-boot-tegra.
Same with the 23-part display driver-model changes, which don't apply either.
If these are going to go in thru the DM repo, please reassign to yourself. If they're eventually going to apply cleanly to u-boot-tegra, leave them delegated to me and let me know when V2 is ready.
Thanks,
Tom
>
> > Tom
> >
--
nvpublic
> >
> >>
> >> drivers/video/vidconsole-uclass.c | 1 +
> >> 1 file changed, 1 insertion(+)
> >>
> >> diff --git a/drivers/video/vidconsole-uclass.c
> >> b/drivers/video/vidconsole-uclass.c
> >> index 706a189..a4c919e 100644
> >> --- a/drivers/video/vidconsole-uclass.c
> >> +++ b/drivers/video/vidconsole-uclass.c
> >> @@ -139,6 +139,7 @@ static void vidconsole_puts(struct stdio_dev
> >> *sdev, const char *s)
> >>
> >> while (*s)
> >> vidconsole_put_char(dev, *s++);
> >> + video_sync(dev->parent);
> >> }
> >>
> >> /* Set up the number of rows and colours (rotated drivers override
> >> this) */
> >> --
> >> 2.6.0.rc2.230.g3dd15c0
> >
>
> Regards,
> Simon
More information about the U-Boot
mailing list