Hi Eugen,
On 2 October 2018 at 01:37, Eugen Hristev <eugen.hris...@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 s...@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.
Thanks,
Simon
>
> Eugen
>
>
>>>
>>> Signed-off-by: Simon Glass <s...@chromium.org>
>>
>>
>> Reviewed-by: Anatolij Gustschin <ag...@denx.de>
>>
>>> ---
>>>
>>> board/atmel/common/video_display.c | 5 ++++-
>>> drivers/video/vidconsole-uclass.c | 2 ++
>>> 2 files changed, 6 insertions(+), 1 deletion(-)
>>>
>>> diff --git a/board/atmel/common/video_display.c
>>> b/board/atmel/common/video_display.c
>>> index 7dd7b85556e..e02cb00f866 100644
>>> --- a/board/atmel/common/video_display.c
>>> +++ b/board/atmel/common/video_display.c
>>> @@ -18,6 +18,7 @@ DECLARE_GLOBAL_DATA_PTR;
>>> int at91_video_show_board_info(void)
>>> {
>>> + struct vidconsole_priv *priv;
>>> ulong dram_size, nand_size;
>>> int i;
>>> u32 len = 0;
>>> @@ -63,7 +64,9 @@ int at91_video_show_board_info(void)
>>> if (ret)
>>> return ret;
>>> - vidconsole_position_cursor(con, 0, logo_info.logo_height);
>>> + priv = dev_get_uclass_priv(con);
>>> + vidconsole_position_cursor(con, 0, (logo_info.logo_height +
>>> + con->y_charsize - 1) / con->y_charsize);
>>
>>
>> Shouldn't it be priv->y_charsize? 'con' is struct udevice * and doesn't
>> have y_charsize.
>>
>>
>> --
>> Anatolij
>> _______________________________________________
>> U-Boot mailing list
>> U-Boot@lists.denx.de
>> https://lists.denx.de/listinfo/u-boot
>>
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
