Re: [U-Boot] [PATCH 2/3] dm: video: Add basic ANSI escape sequence support
Hi Rob, On 17 September 2017 at 13:39, Rob Clarkwrote: > On Sun, Sep 17, 2017 at 3:30 PM, Simon Glass wrote: >> Hi Rob, >> >> On 17 September 2017 at 13:26, Rob Clark wrote: >>> On Sun, Sep 17, 2017 at 1:55 PM, Simon Glass wrote: On 13 September 2017 at 16:12, Rob Clark wrote: > Really just the subset that is needed by efi_console. Perhaps more will > be added later, for example color support would be useful to implement > efi_cout_set_attribute(). > > Signed-off-by: Rob Clark > --- > drivers/video/Kconfig | 8 +++ > drivers/video/vidconsole-uclass.c | 109 > ++ > drivers/video/video-uclass.c | 4 +- > include/video.h | 7 +++ > include/video_console.h | 11 > 5 files changed, 136 insertions(+), 3 deletions(-) Reviewed-by: Simon Glass I don't see the test though - is that in another patch? >>> >>> well, at this point test is load/bootefi Shell.efi and does it look >>> messed up on screen. If you have better ideas, let me know. >> >> It should be easy enough to update test/dm/video.o to use the new >> feature in a new test. This code has nothing to do with EFI really. >> >> The tests are fairly slow in that they gzip the display to check that >> it is correctly, but they work. >> > > > ok, I guess snapshotting fb and comparing to a reference is a way.. > where do I look to figure out how to build/run these tests (and > presumable update reference screenshots?) make tests will run the tests. To run the video tests, something like: ./test/py/test.py -k video --build-dir sandbox/ BTW there are no real reference screenshots. Just run the test once to get the value, check that the display looks good (run sandbox with -l) and then update your test with that value. > > I'd be inclined to add any test as a patch on top of the following > patch to cover color escape sequences at the same time.. > Yes that sounds good. - Simon ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH 2/3] dm: video: Add basic ANSI escape sequence support
On Sun, Sep 17, 2017 at 3:30 PM, Simon Glasswrote: > Hi Rob, > > On 17 September 2017 at 13:26, Rob Clark wrote: >> On Sun, Sep 17, 2017 at 1:55 PM, Simon Glass wrote: >>> On 13 September 2017 at 16:12, Rob Clark wrote: Really just the subset that is needed by efi_console. Perhaps more will be added later, for example color support would be useful to implement efi_cout_set_attribute(). Signed-off-by: Rob Clark --- drivers/video/Kconfig | 8 +++ drivers/video/vidconsole-uclass.c | 109 ++ drivers/video/video-uclass.c | 4 +- include/video.h | 7 +++ include/video_console.h | 11 5 files changed, 136 insertions(+), 3 deletions(-) >>> >>> Reviewed-by: Simon Glass >>> >>> I don't see the test though - is that in another patch? >> >> well, at this point test is load/bootefi Shell.efi and does it look >> messed up on screen. If you have better ideas, let me know. > > It should be easy enough to update test/dm/video.o to use the new > feature in a new test. This code has nothing to do with EFI really. > > The tests are fairly slow in that they gzip the display to check that > it is correctly, but they work. > ok, I guess snapshotting fb and comparing to a reference is a way.. where do I look to figure out how to build/run these tests (and presumable update reference screenshots?) I'd be inclined to add any test as a patch on top of the following patch to cover color escape sequences at the same time.. BR, -R ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH 2/3] dm: video: Add basic ANSI escape sequence support
Hi Rob, On 17 September 2017 at 13:26, Rob Clarkwrote: > On Sun, Sep 17, 2017 at 1:55 PM, Simon Glass wrote: >> On 13 September 2017 at 16:12, Rob Clark wrote: >>> Really just the subset that is needed by efi_console. Perhaps more will >>> be added later, for example color support would be useful to implement >>> efi_cout_set_attribute(). >>> >>> Signed-off-by: Rob Clark >>> --- >>> drivers/video/Kconfig | 8 +++ >>> drivers/video/vidconsole-uclass.c | 109 >>> ++ >>> drivers/video/video-uclass.c | 4 +- >>> include/video.h | 7 +++ >>> include/video_console.h | 11 >>> 5 files changed, 136 insertions(+), 3 deletions(-) >> >> Reviewed-by: Simon Glass >> >> I don't see the test though - is that in another patch? > > well, at this point test is load/bootefi Shell.efi and does it look > messed up on screen. If you have better ideas, let me know. It should be easy enough to update test/dm/video.o to use the new feature in a new test. This code has nothing to do with EFI really. The tests are fairly slow in that they gzip the display to check that it is correctly, but they work. Regards, Simon ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH 2/3] dm: video: Add basic ANSI escape sequence support
On Sun, Sep 17, 2017 at 1:55 PM, Simon Glasswrote: > On 13 September 2017 at 16:12, Rob Clark wrote: >> Really just the subset that is needed by efi_console. Perhaps more will >> be added later, for example color support would be useful to implement >> efi_cout_set_attribute(). >> >> Signed-off-by: Rob Clark >> --- >> drivers/video/Kconfig | 8 +++ >> drivers/video/vidconsole-uclass.c | 109 >> ++ >> drivers/video/video-uclass.c | 4 +- >> include/video.h | 7 +++ >> include/video_console.h | 11 >> 5 files changed, 136 insertions(+), 3 deletions(-) > > Reviewed-by: Simon Glass > > I don't see the test though - is that in another patch? well, at this point test is load/bootefi Shell.efi and does it look messed up on screen. If you have better ideas, let me know. BR, -R ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH 2/3] dm: video: Add basic ANSI escape sequence support
On 13 September 2017 at 16:12, Rob Clarkwrote: > Really just the subset that is needed by efi_console. Perhaps more will > be added later, for example color support would be useful to implement > efi_cout_set_attribute(). > > Signed-off-by: Rob Clark > --- > drivers/video/Kconfig | 8 +++ > drivers/video/vidconsole-uclass.c | 109 > ++ > drivers/video/video-uclass.c | 4 +- > include/video.h | 7 +++ > include/video_console.h | 11 > 5 files changed, 136 insertions(+), 3 deletions(-) Reviewed-by: Simon Glass I don't see the test though - is that in another patch? ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot