[U-Boot] [PATCH 43/45] video: at91: Adjust vidconsole_position_cursor() to use char pos

[sjg at google.com]

On 09.10.2018 06:40, Simon Glass wrote:
> Hi Eugen,
>
> On 2 October 2018 at 01:37, Eugen Hristev <eugen.hristev at microchip.com> wrote:
>>
>>
>>
>> On 01.10.2018 23:22, Anatolij Gustschin wrote:
>>>
>>> Hi Simon,
>>>
>>> On Mon,  1 Oct 2018 12:22:47 -0600
>>> Simon Glass sjg at chromium.org wrote:
>>>
>>>> At present this function uses pixels but it seems more useful for it to
>>>> position in terms of characters on the screen. This also matches the
>>>> comment to the function. Update this.
>>>>
>>>> Unfortunately there is one user of this function (at91). Have a crack at
>>>> fixing this, since I cannot test it.
>>
>>
>> Hello Simon,
>>
>> I will gladly test this for you on at91 board,
>> but I am having some issues applying your patch series:
>>
>> Applying: binman: Move to three-digit test-file numbers
>> error: patch failed: tools/binman/entry_test.py:25
>> error: tools/binman/entry_test.py: patch does not apply
>> error: patch failed: tools/binman/ftest.py:712
>> error: tools/binman/ftest.py: patch does not apply
>> Patch failed at 0026 binman: Move to three-digit test-file numbers
>>
>> Do you have them in some public tree I can pull from ?
>
> Yes you can try u-boot-dm/testing
>
>>
>> Also, any specific tests you would like except just checking the video console ?
>
> It looks like this code runs when the board boots up, so just starting
> it should be enough.

I made a build on your branch and tested it on at91 sama5d2_xplained
board, and the logo and text appears on the display.

Tested-by: Eugen Hristev <eugen.hristev at microchip.com>

Let me know if you want me to do more tests.

Eugen
>
> Thanks,
> Simon
>
>
>>
>> Eugen
>>
>>
>>>>
>>>> Signed-off-by: Simon Glass <sjg at chromium.org>
>>>
>>>
>>> Reviewed-by: Anatolij Gustschin <agust at denx.de>
>>>
>>>> ---
>>>>
>>>>    board/atmel/common/video_display.c | 5 ++++-
>>>>    drivers/video/vidconsole-uclass.c  | 2 ++
>>>>    2 files changed, 6 insertions(+), 1 deletion(-)
>>>>
Applied to u-boot-dm