mbox series

[U-Boot,0/3] dm: video: enhancements for Shell.efi

Message ID 20170913221227.21091-1-robdclark@gmail.com
Headers show
Series dm: video: enhancements for Shell.efi | expand

Message

Rob Clark Sept. 13, 2017, 10:12 p.m. UTC
Split out of earlier efi_loader patchset, and updated with some
comments, use of simple_strtol[1], and added kconfig option to
enable ANSI escape sequence support.

[1] not a hard dependency, but to ensure correct behavior in all cases,
    depends on "lib: strto: fix incorrect handling of specified base"

Rob Clark (3):
  dm: video: Fix cache flushes
  dm: video: Add basic ANSI escape sequence support
  dm: video: Add color ANSI escape sequence support

 drivers/video/Kconfig             |   8 ++
 drivers/video/vidconsole-uclass.c | 206 ++++++++++++++++++++++++++++++++++++++
 drivers/video/video-uclass.c      |   4 +-
 include/video.h                   |   7 ++
 include/video_console.h           |  11 ++
 5 files changed, 233 insertions(+), 3 deletions(-)

Comments

Simon Glass Sept. 28, 2017, 12:17 p.m. UTC | #1
Hi Rob,

On 13 September 2017 at 16:12, Rob Clark <robdclark@gmail.com> wrote:
>
> Split out of earlier efi_loader patchset, and updated with some
> comments, use of simple_strtol[1], and added kconfig option to
> enable ANSI escape sequence support.
>
> [1] not a hard dependency, but to ensure correct behavior in all cases,
>     depends on "lib: strto: fix incorrect handling of specified base"
>
> Rob Clark (3):
>   dm: video: Fix cache flushes
>   dm: video: Add basic ANSI escape sequence support
>   dm: video: Add color ANSI escape sequence support
>
>  drivers/video/Kconfig             |   8 ++
>  drivers/video/vidconsole-uclass.c | 206 ++++++++++++++++++++++++++++++++++++++
>  drivers/video/video-uclass.c      |   4 +-
>  include/video.h                   |   7 ++
>  include/video_console.h           |  11 ++
>  5 files changed, 233 insertions(+), 3 deletions(-)

I'd like to apply this series but am waiting for the test update. Did
you get my email about this?

Regards,
Simon
Rob Clark Sept. 28, 2017, 2:40 p.m. UTC | #2
On Thu, Sep 28, 2017 at 8:17 AM, Simon Glass <sjg@chromium.org> wrote:
> Hi Rob,
>
> On 13 September 2017 at 16:12, Rob Clark <robdclark@gmail.com> wrote:
>>
>> Split out of earlier efi_loader patchset, and updated with some
>> comments, use of simple_strtol[1], and added kconfig option to
>> enable ANSI escape sequence support.
>>
>> [1] not a hard dependency, but to ensure correct behavior in all cases,
>>     depends on "lib: strto: fix incorrect handling of specified base"
>>
>> Rob Clark (3):
>>   dm: video: Fix cache flushes
>>   dm: video: Add basic ANSI escape sequence support
>>   dm: video: Add color ANSI escape sequence support
>>
>>  drivers/video/Kconfig             |   8 ++
>>  drivers/video/vidconsole-uclass.c | 206 ++++++++++++++++++++++++++++++++++++++
>>  drivers/video/video-uclass.c      |   4 +-
>>  include/video.h                   |   7 ++
>>  include/video_console.h           |  11 ++
>>  5 files changed, 233 insertions(+), 3 deletions(-)
>
> I'd like to apply this series but am waiting for the test update. Did
> you get my email about this?
>

yes, I guess you missed:
https://patchwork.ozlabs.org/project/uboot/list/?series=5009

BR,
-R
Anatolij Gustschin Sept. 29, 2017, 7:45 p.m. UTC | #3
Hi Rob,

On Wed, 13 Sep 2017 18:12:19 -0400
Rob Clark robdclark@gmail.com wrote:

> Split out of earlier efi_loader patchset, and updated with some
> comments, use of simple_strtol[1], and added kconfig option to
> enable ANSI escape sequence support.
> 
> [1] not a hard dependency, but to ensure correct behavior in all cases,
>     depends on "lib: strto: fix incorrect handling of specified base"
> 
> Rob Clark (3):
>   dm: video: Fix cache flushes
>   dm: video: Add basic ANSI escape sequence support
>   dm: video: Add color ANSI escape sequence support
> 
>  drivers/video/Kconfig             |   8 ++
>  drivers/video/vidconsole-uclass.c | 206 ++++++++++++++++++++++++++++++++++++++
>  drivers/video/video-uclass.c      |   4 +-
>  include/video.h                   |   7 ++
>  include/video_console.h           |  11 ++
>  5 files changed, 233 insertions(+), 3 deletions(-)

Series applied to u-boot-video/master, thanks!

--
Anatolij