* [U-Boot] [PATCH] lcd: fix console address is not initialized
@ 2015-01-21 4:37 Bo Shen
2015-01-22 13:10 ` Nikita Kiryanov
0 siblings, 1 reply; 9+ messages in thread
From: Bo Shen @ 2015-01-21 4:37 UTC (permalink / raw)
To: u-boot
This commit 904672e (lcd: refactor lcd console stuff into its
own file), which cause lcd console address is not initialized.
This patch split lcd console address initialize and lcd logo
display into two functions.
Signed-off-by: Bo Shen <voice.shen@atmel.com>
---
common/lcd.c | 11 ++++++++---
1 file changed, 8 insertions(+), 3 deletions(-)
diff --git a/common/lcd.c b/common/lcd.c
index cc34b8a..f435e2a 100644
--- a/common/lcd.c
+++ b/common/lcd.c
@@ -82,7 +82,8 @@ DECLARE_GLOBAL_DATA_PTR;
static int lcd_init(void *lcdbase);
-static void *lcd_logo(void);
+static void lcd_logo(void);
+static void *lcd_console_address(void);
static void lcd_setfgcolor(int color);
static void lcd_setbgcolor(int color);
@@ -268,7 +269,8 @@ void lcd_clear(void)
console_rows = panel_info.vl_row / VIDEO_FONT_HEIGHT;
#endif
console_cols = panel_info.vl_col / VIDEO_FONT_WIDTH;
- lcd_init_console(lcd_logo(), console_rows, console_cols);
+ lcd_init_console(lcd_console_address(), console_rows, console_cols);
+ lcd_logo();
lcd_sync();
}
@@ -849,7 +851,7 @@ int lcd_display_bitmap(ulong bmp_image, int x, int y)
}
#endif
-static void *lcd_logo(void)
+static void lcd_logo(void)
{
#ifdef CONFIG_SPLASH_SCREEN
char *s;
@@ -879,7 +881,10 @@ static void *lcd_logo(void)
lcd_set_row(LCD_INFO_Y / VIDEO_FONT_HEIGHT);
lcd_show_board_info();
#endif /* CONFIG_LCD_INFO */
+}
+static void *lcd_console_address(void)
+{
#if defined(CONFIG_LCD_LOGO) && !defined(CONFIG_LCD_INFO_BELOW_LOGO)
return (void *)((ulong)lcd_base + BMP_LOGO_HEIGHT * lcd_line_length);
#else
--
2.3.0.rc0
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] lcd: fix console address is not initialized
2015-01-21 4:37 [U-Boot] [PATCH] lcd: fix console address is not initialized Bo Shen
@ 2015-01-22 13:10 ` Nikita Kiryanov
2015-01-23 1:20 ` Bo Shen
0 siblings, 1 reply; 9+ messages in thread
From: Nikita Kiryanov @ 2015-01-22 13:10 UTC (permalink / raw)
To: u-boot
Hi Bo,
On 01/21/2015 06:37 AM, Bo Shen wrote:
> This commit 904672e (lcd: refactor lcd console stuff into its
> own file), which cause lcd console address is not initialized.
Based on your fix, I'm certain that the bug was introduced in a
previous patch, perhaps 140beb9 (lcd: expand console api).
Also, can you provide a more detailed explanation of when this
happens and how?
>
> This patch split lcd console address initialize and lcd logo
> display into two functions.
>
> Signed-off-by: Bo Shen <voice.shen@atmel.com>
> ---
>
> common/lcd.c | 11 ++++++++---
> 1 file changed, 8 insertions(+), 3 deletions(-)
>
> diff --git a/common/lcd.c b/common/lcd.c
> index cc34b8a..f435e2a 100644
> --- a/common/lcd.c
> +++ b/common/lcd.c
> @@ -82,7 +82,8 @@ DECLARE_GLOBAL_DATA_PTR;
>
> static int lcd_init(void *lcdbase);
>
> -static void *lcd_logo(void);
> +static void lcd_logo(void);
> +static void *lcd_console_address(void);
>
> static void lcd_setfgcolor(int color);
> static void lcd_setbgcolor(int color);
> @@ -268,7 +269,8 @@ void lcd_clear(void)
> console_rows = panel_info.vl_row / VIDEO_FONT_HEIGHT;
> #endif
> console_cols = panel_info.vl_col / VIDEO_FONT_WIDTH;
> - lcd_init_console(lcd_logo(), console_rows, console_cols);
> + lcd_init_console(lcd_console_address(), console_rows, console_cols);
> + lcd_logo();
> lcd_sync();
> }
>
> @@ -849,7 +851,7 @@ int lcd_display_bitmap(ulong bmp_image, int x, int y)
> }
> #endif
>
> -static void *lcd_logo(void)
> +static void lcd_logo(void)
> {
> #ifdef CONFIG_SPLASH_SCREEN
> char *s;
> @@ -879,7 +881,10 @@ static void *lcd_logo(void)
> lcd_set_row(LCD_INFO_Y / VIDEO_FONT_HEIGHT);
> lcd_show_board_info();
> #endif /* CONFIG_LCD_INFO */
> +}
>
> +static void *lcd_console_address(void)
> +{
> #if defined(CONFIG_LCD_LOGO) && !defined(CONFIG_LCD_INFO_BELOW_LOGO)
> return (void *)((ulong)lcd_base + BMP_LOGO_HEIGHT * lcd_line_length);
> #else
>
I would like to see some mention of why it's ok to redefine the console
address in such a way. At first glance it looks like there is a slight change of
behavior (the value no longer depends on whether splash image was loaded
or not), but it seems to be ok if you go over the various cases. The only
instance where I see the difference could manifest itself is if lcd console
would start writing to the frame buffer while the splash screen is still on,
but that doesn't look good regardless of where the console starts, so I'm ok
with that.
--
Regards,
Nikita Kiryanov
^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] lcd: fix console address is not initialized
2015-01-22 13:10 ` Nikita Kiryanov
@ 2015-01-23 1:20 ` Bo Shen
2015-01-26 5:55 ` Bo Shen
0 siblings, 1 reply; 9+ messages in thread
From: Bo Shen @ 2015-01-23 1:20 UTC (permalink / raw)
To: u-boot
Hi Nikita Kiryanov,
On 01/22/2015 09:10 PM, Nikita Kiryanov wrote:
> Hi Bo,
>
> On 01/21/2015 06:37 AM, Bo Shen wrote:
>> This commit 904672e (lcd: refactor lcd console stuff into its
>> own file), which cause lcd console address is not initialized.
>
> Based on your fix, I'm certain that the bug was introduced in a
> previous patch, perhaps 140beb9 (lcd: expand console api).
>
> Also, can you provide a more detailed explanation of when this
> happens and how?
It will cause the system hang.
Before this patch, lcd_logo -> lcd_show_board_info (CONFIG_LCD_INFO) ->
.. -> lcd_drawchars. It has the following lines:
--->8---
dest = (uchar *)(lcd_base + y * lcd_line_length + x * NBITS(LCD_BPP)/8);
---8<---
while with the patch,
--->8---
dest = (uchar *)(lcd_console_address +
y * lcd_line_length + x * NBITS(LCD_BPP) / 8);
---8<---
As the lcd_console_address is initialized after lcd_logo return, so the
lcd_console_address is not initialized, it is 0. When try to write to
address 0, the system hang.
>>
>> This patch split lcd console address initialize and lcd logo
>> display into two functions.
>>
>> Signed-off-by: Bo Shen <voice.shen@atmel.com>
>> ---
>>
>> common/lcd.c | 11 ++++++++---
>> 1 file changed, 8 insertions(+), 3 deletions(-)
>>
>> diff --git a/common/lcd.c b/common/lcd.c
>> index cc34b8a..f435e2a 100644
>> --- a/common/lcd.c
>> +++ b/common/lcd.c
>> @@ -82,7 +82,8 @@ DECLARE_GLOBAL_DATA_PTR;
>>
>> static int lcd_init(void *lcdbase);
>>
>> -static void *lcd_logo(void);
>> +static void lcd_logo(void);
>> +static void *lcd_console_address(void);
>>
>> static void lcd_setfgcolor(int color);
>> static void lcd_setbgcolor(int color);
>> @@ -268,7 +269,8 @@ void lcd_clear(void)
>> console_rows = panel_info.vl_row / VIDEO_FONT_HEIGHT;
>> #endif
>> console_cols = panel_info.vl_col / VIDEO_FONT_WIDTH;
>> - lcd_init_console(lcd_logo(), console_rows, console_cols);
>> + lcd_init_console(lcd_console_address(), console_rows, console_cols);
>> + lcd_logo();
>> lcd_sync();
>> }
>>
>> @@ -849,7 +851,7 @@ int lcd_display_bitmap(ulong bmp_image, int x, int y)
>> }
>> #endif
>>
>> -static void *lcd_logo(void)
>> +static void lcd_logo(void)
>> {
>> #ifdef CONFIG_SPLASH_SCREEN
>> char *s;
>> @@ -879,7 +881,10 @@ static void *lcd_logo(void)
>> lcd_set_row(LCD_INFO_Y / VIDEO_FONT_HEIGHT);
>> lcd_show_board_info();
>> #endif /* CONFIG_LCD_INFO */
>> +}
>>
>> +static void *lcd_console_address(void)
>> +{
>> #if defined(CONFIG_LCD_LOGO) && !defined(CONFIG_LCD_INFO_BELOW_LOGO)
>> return (void *)((ulong)lcd_base + BMP_LOGO_HEIGHT *
>> lcd_line_length);
>> #else
>>
>
> I would like to see some mention of why it's ok to redefine the console
> address in such a way. At first glance it looks like there is a slight
> change of
> behavior (the value no longer depends on whether splash image was loaded
> or not), but it seems to be ok if you go over the various cases. The only
> instance where I see the difference could manifest itself is if lcd console
> would start writing to the frame buffer while the splash screen is still
> on,
> but that doesn't look good regardless of where the console starts, so
> I'm ok
> with that.
>
Thanks for reminder this, I will check whether this will break the
splash screen.
Best Regards,
Bo Shen
^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] lcd: fix console address is not initialized
2015-01-23 1:20 ` Bo Shen
@ 2015-01-26 5:55 ` Bo Shen
2015-01-27 14:45 ` Nikita Kiryanov
0 siblings, 1 reply; 9+ messages in thread
From: Bo Shen @ 2015-01-26 5:55 UTC (permalink / raw)
To: u-boot
Hi Nikita Kiryanov,
+ Andreas, Tom
On 01/23/2015 09:20 AM, Bo Shen wrote:
> Hi Nikita Kiryanov,
>
> On 01/22/2015 09:10 PM, Nikita Kiryanov wrote:
>> Hi Bo,
>>
>> On 01/21/2015 06:37 AM, Bo Shen wrote:
>>> This commit 904672e (lcd: refactor lcd console stuff into its
>>> own file), which cause lcd console address is not initialized.
>>
>> Based on your fix, I'm certain that the bug was introduced in a
>> previous patch, perhaps 140beb9 (lcd: expand console api).
>>
>> Also, can you provide a more detailed explanation of when this
>> happens and how?
>
> It will cause the system hang.
>
> Before this patch, lcd_logo -> lcd_show_board_info (CONFIG_LCD_INFO) ->
> .. -> lcd_drawchars. It has the following lines:
> --->8---
> dest = (uchar *)(lcd_base + y * lcd_line_length + x * NBITS(LCD_BPP)/8);
> ---8<---
>
> while with the patch,
> --->8---
> dest = (uchar *)(lcd_console_address +
> y * lcd_line_length + x * NBITS(LCD_BPP) / 8);
> ---8<---
>
> As the lcd_console_address is initialized after lcd_logo return, so the
> lcd_console_address is not initialized, it is 0. When try to write to
> address 0, the system hang.
>
>>>
>>> This patch split lcd console address initialize and lcd logo
>>> display into two functions.
>>>
>>> Signed-off-by: Bo Shen <voice.shen@atmel.com>
>>> ---
>>>
>>> common/lcd.c | 11 ++++++++---
>>> 1 file changed, 8 insertions(+), 3 deletions(-)
>>>
>>> diff --git a/common/lcd.c b/common/lcd.c
>>> index cc34b8a..f435e2a 100644
>>> --- a/common/lcd.c
>>> +++ b/common/lcd.c
>>> @@ -82,7 +82,8 @@ DECLARE_GLOBAL_DATA_PTR;
>>>
>>> static int lcd_init(void *lcdbase);
>>>
>>> -static void *lcd_logo(void);
>>> +static void lcd_logo(void);
>>> +static void *lcd_console_address(void);
>>>
>>> static void lcd_setfgcolor(int color);
>>> static void lcd_setbgcolor(int color);
>>> @@ -268,7 +269,8 @@ void lcd_clear(void)
>>> console_rows = panel_info.vl_row / VIDEO_FONT_HEIGHT;
>>> #endif
>>> console_cols = panel_info.vl_col / VIDEO_FONT_WIDTH;
>>> - lcd_init_console(lcd_logo(), console_rows, console_cols);
>>> + lcd_init_console(lcd_console_address(), console_rows,
>>> console_cols);
>>> + lcd_logo();
>>> lcd_sync();
>>> }
>>>
>>> @@ -849,7 +851,7 @@ int lcd_display_bitmap(ulong bmp_image, int x,
>>> int y)
>>> }
>>> #endif
>>>
>>> -static void *lcd_logo(void)
>>> +static void lcd_logo(void)
>>> {
>>> #ifdef CONFIG_SPLASH_SCREEN
>>> char *s;
>>> @@ -879,7 +881,10 @@ static void *lcd_logo(void)
>>> lcd_set_row(LCD_INFO_Y / VIDEO_FONT_HEIGHT);
>>> lcd_show_board_info();
>>> #endif /* CONFIG_LCD_INFO */
>>> +}
>>>
>>> +static void *lcd_console_address(void)
>>> +{
>>> #if defined(CONFIG_LCD_LOGO) && !defined(CONFIG_LCD_INFO_BELOW_LOGO)
>>> return (void *)((ulong)lcd_base + BMP_LOGO_HEIGHT *
>>> lcd_line_length);
>>> #else
>>>
>>
>> I would like to see some mention of why it's ok to redefine the console
>> address in such a way. At first glance it looks like there is a slight
>> change of
>> behavior (the value no longer depends on whether splash image was loaded
>> or not), but it seems to be ok if you go over the various cases. The only
>> instance where I see the difference could manifest itself is if lcd
>> console
>> would start writing to the frame buffer while the splash screen is still
>> on,
>> but that doesn't look good regardless of where the console starts, so
>> I'm ok
>> with that.
>>
>
> Thanks for reminder this, I will check whether this will break the
> splash screen.
Can we use the following patch to fix this issue?
--->8---
diff --git a/common/lcd.c b/common/lcd.c
index cc34b8a..1195a54 100644
--- a/common/lcd.c
+++ b/common/lcd.c
@@ -268,6 +268,7 @@ void lcd_clear(void)
console_rows = panel_info.vl_row / VIDEO_FONT_HEIGHT;
#endif
console_cols = panel_info.vl_col / VIDEO_FONT_WIDTH;
+ lcd_init_console(lcd_base, console_rows, console_cols);
lcd_init_console(lcd_logo(), console_rows, console_cols);
lcd_sync();
}
---8<---
It first initializes the lcd console with LCD base, if the splash screen
is used, new address is updated.
I need this kind of fix to be applied as soon as possible, or else, most
Atmel related board are broken on u-boot master branch.
Best Regards,
Bo Shen
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] lcd: fix console address is not initialized
2015-01-26 5:55 ` Bo Shen
@ 2015-01-27 14:45 ` Nikita Kiryanov
2015-01-28 1:08 ` Bo Shen
0 siblings, 1 reply; 9+ messages in thread
From: Nikita Kiryanov @ 2015-01-27 14:45 UTC (permalink / raw)
To: u-boot
Hi Bo,
On 01/26/2015 07:55 AM, Bo Shen wrote:
> Hi Nikita Kiryanov,
> + Andreas, Tom
>
> On 01/23/2015 09:20 AM, Bo Shen wrote:
>> Hi Nikita Kiryanov,
>>
>> On 01/22/2015 09:10 PM, Nikita Kiryanov wrote:
>>> Hi Bo,
>>>
>>> On 01/21/2015 06:37 AM, Bo Shen wrote:
>>>> This commit 904672e (lcd: refactor lcd console stuff into its
>>>> own file), which cause lcd console address is not initialized.
>>>
>>> Based on your fix, I'm certain that the bug was introduced in a
>>> previous patch, perhaps 140beb9 (lcd: expand console api).
>>>
>>> Also, can you provide a more detailed explanation of when this
>>> happens and how?
>>
>> It will cause the system hang.
>>
>> Before this patch, lcd_logo -> lcd_show_board_info (CONFIG_LCD_INFO) ->
>> .. -> lcd_drawchars. It has the following lines:
>> --->8---
>> dest = (uchar *)(lcd_base + y * lcd_line_length + x * NBITS(LCD_BPP)/8);
>> ---8<---
>>
>> while with the patch,
>> --->8---
>> dest = (uchar *)(lcd_console_address +
>> y * lcd_line_length + x * NBITS(LCD_BPP) / 8);
>> ---8<---
>>
>> As the lcd_console_address is initialized after lcd_logo return, so the
>> lcd_console_address is not initialized, it is 0. When try to write to
>> address 0, the system hang.
>>
>>>>
>>>> This patch split lcd console address initialize and lcd logo
>>>> display into two functions.
>>>>
>>>> Signed-off-by: Bo Shen <voice.shen@atmel.com>
>>>> ---
>>>>
>>>> common/lcd.c | 11 ++++++++---
>>>> 1 file changed, 8 insertions(+), 3 deletions(-)
>>>>
>>>> diff --git a/common/lcd.c b/common/lcd.c
>>>> index cc34b8a..f435e2a 100644
>>>> --- a/common/lcd.c
>>>> +++ b/common/lcd.c
>>>> @@ -82,7 +82,8 @@ DECLARE_GLOBAL_DATA_PTR;
>>>>
>>>> static int lcd_init(void *lcdbase);
>>>>
>>>> -static void *lcd_logo(void);
>>>> +static void lcd_logo(void);
>>>> +static void *lcd_console_address(void);
>>>>
>>>> static void lcd_setfgcolor(int color);
>>>> static void lcd_setbgcolor(int color);
>>>> @@ -268,7 +269,8 @@ void lcd_clear(void)
>>>> console_rows = panel_info.vl_row / VIDEO_FONT_HEIGHT;
>>>> #endif
>>>> console_cols = panel_info.vl_col / VIDEO_FONT_WIDTH;
>>>> - lcd_init_console(lcd_logo(), console_rows, console_cols);
>>>> + lcd_init_console(lcd_console_address(), console_rows,
>>>> console_cols);
>>>> + lcd_logo();
>>>> lcd_sync();
>>>> }
>>>>
>>>> @@ -849,7 +851,7 @@ int lcd_display_bitmap(ulong bmp_image, int x,
>>>> int y)
>>>> }
>>>> #endif
>>>>
>>>> -static void *lcd_logo(void)
>>>> +static void lcd_logo(void)
>>>> {
>>>> #ifdef CONFIG_SPLASH_SCREEN
>>>> char *s;
>>>> @@ -879,7 +881,10 @@ static void *lcd_logo(void)
>>>> lcd_set_row(LCD_INFO_Y / VIDEO_FONT_HEIGHT);
>>>> lcd_show_board_info();
>>>> #endif /* CONFIG_LCD_INFO */
>>>> +}
>>>>
>>>> +static void *lcd_console_address(void)
>>>> +{
>>>> #if defined(CONFIG_LCD_LOGO) && !defined(CONFIG_LCD_INFO_BELOW_LOGO)
>>>> return (void *)((ulong)lcd_base + BMP_LOGO_HEIGHT *
>>>> lcd_line_length);
>>>> #else
>>>>
>>>
>>> I would like to see some mention of why it's ok to redefine the console
>>> address in such a way. At first glance it looks like there is a slight
>>> change of
>>> behavior (the value no longer depends on whether splash image was loaded
>>> or not), but it seems to be ok if you go over the various cases. The only
>>> instance where I see the difference could manifest itself is if lcd
>>> console
>>> would start writing to the frame buffer while the splash screen is still
>>> on,
>>> but that doesn't look good regardless of where the console starts, so
>>> I'm ok
>>> with that.
>>>
>>
>> Thanks for reminder this, I will check whether this will break the
>> splash screen.
>
> Can we use the following patch to fix this issue?
> --->8---
> diff --git a/common/lcd.c b/common/lcd.c
> index cc34b8a..1195a54 100644
> --- a/common/lcd.c
> +++ b/common/lcd.c
> @@ -268,6 +268,7 @@ void lcd_clear(void)
> console_rows = panel_info.vl_row / VIDEO_FONT_HEIGHT;
> #endif
> console_cols = panel_info.vl_col / VIDEO_FONT_WIDTH;
> + lcd_init_console(lcd_base, console_rows, console_cols);
> lcd_init_console(lcd_logo(), console_rows, console_cols);
> lcd_sync();
> }
> ---8<---
>
> It first initializes the lcd console with LCD base, if the splash screen is used, new address is updated.
I think this is the best approach. I am very close to posting the next step in my refactor of lcd.c, and I can
incorporate it in the series. I will greatly appreciate your help in testing this series, since it involves Atmel
related changes.
>
> I need this kind of fix to be applied as soon as possible, or else, most Atmel related board are broken on u-boot master branch.
>
> Best Regards,
> Bo Shen
--
Regards,
Nikita Kiryanov
^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] lcd: fix console address is not initialized
2015-01-27 14:45 ` Nikita Kiryanov
@ 2015-01-28 1:08 ` Bo Shen
0 siblings, 0 replies; 9+ messages in thread
From: Bo Shen @ 2015-01-28 1:08 UTC (permalink / raw)
To: u-boot
Hi Nikita Kiryanov,
On 01/27/2015 10:45 PM, Nikita Kiryanov wrote:
>> Can we use the following patch to fix this issue?
>> --->8---
>> diff --git a/common/lcd.c b/common/lcd.c
>> index cc34b8a..1195a54 100644
>> --- a/common/lcd.c
>> +++ b/common/lcd.c
>> @@ -268,6 +268,7 @@ void lcd_clear(void)
>> console_rows = panel_info.vl_row / VIDEO_FONT_HEIGHT;
>> #endif
>> console_cols = panel_info.vl_col / VIDEO_FONT_WIDTH;
>> + lcd_init_console(lcd_base, console_rows, console_cols);
>> lcd_init_console(lcd_logo(), console_rows, console_cols);
>> lcd_sync();
>> }
>> ---8<---
>>
>> It first initializes the lcd console with LCD base, if the splash
>> screen is used, new address is updated.
>
>
> I think this is the best approach. I am very close to posting the next
Thanks for review. I think send this fix patch first, and then you can
based on this patch for your next step, or else, we can not know how
long it will break most Atmel SoC related boards on master branch. So, I
will send this patch and try to get it applied as soon as possible.
Thanks again.
> step in my refactor of lcd.c, and I can
> incorporate it in the series. I will greatly appreciate your help in
> testing this series, since it involves Atmel
> related changes.
>
>>
>> I need this kind of fix to be applied as soon as possible, or else,
>> most Atmel related board are broken on u-boot master branch.
>>
>> Best Regards,
>> Bo Shen
>
> --
> Regards,
> Nikita Kiryanov
Best Regards,
Bo Shen
^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] lcd: fix console address is not initialized
2015-01-29 8:51 ` Anatolij Gustschin
@ 2015-01-29 8:55 ` Bo Shen
0 siblings, 0 replies; 9+ messages in thread
From: Bo Shen @ 2015-01-29 8:55 UTC (permalink / raw)
To: u-boot
Hi Anatolij,
On 01/29/2015 04:51 PM, Anatolij Gustschin wrote:
> Hi,
>
> On Wed, 28 Jan 2015 09:13:22 +0800
> Bo Shen <voice.shen@atmel.com> wrote:
>
>> This commit 904672e (lcd: refactor lcd console stuff into its
>> own file), which cause lcd console address is not initialized.
>>
>> This patch initialize the lcd console use the default value,
>> will be update when splash screen is enabled.
>>
>> Signed-off-by: Bo Shen <voice.shen@atmel.com>
>> ---
>> Hi Tom,
>> If no objection of this patch, can you apply it as soon as possible.
>> Or else most Atmel SoC based boards (which enable lcd info) will be
>> broken.
>> Thanks.
>>
>> common/lcd.c | 1 +
>> 1 file changed, 1 insertion(+)
>
> Slightly modified the commit log and applied to u-boot-video/master.
Thanks.
> Thanks!
>
> Anatolij
>
Best Regards,
Bo Shen
^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] lcd: fix console address is not initialized
2015-01-28 1:13 Bo Shen
@ 2015-01-29 8:51 ` Anatolij Gustschin
2015-01-29 8:55 ` Bo Shen
0 siblings, 1 reply; 9+ messages in thread
From: Anatolij Gustschin @ 2015-01-29 8:51 UTC (permalink / raw)
To: u-boot
Hi,
On Wed, 28 Jan 2015 09:13:22 +0800
Bo Shen <voice.shen@atmel.com> wrote:
> This commit 904672e (lcd: refactor lcd console stuff into its
> own file), which cause lcd console address is not initialized.
>
> This patch initialize the lcd console use the default value,
> will be update when splash screen is enabled.
>
> Signed-off-by: Bo Shen <voice.shen@atmel.com>
> ---
> Hi Tom,
> If no objection of this patch, can you apply it as soon as possible.
> Or else most Atmel SoC based boards (which enable lcd info) will be
> broken.
> Thanks.
>
> common/lcd.c | 1 +
> 1 file changed, 1 insertion(+)
Slightly modified the commit log and applied to u-boot-video/master.
Thanks!
Anatolij
^ permalink raw reply [flat|nested] 9+ messages in thread
* [U-Boot] [PATCH] lcd: fix console address is not initialized
@ 2015-01-28 1:13 Bo Shen
2015-01-29 8:51 ` Anatolij Gustschin
0 siblings, 1 reply; 9+ messages in thread
From: Bo Shen @ 2015-01-28 1:13 UTC (permalink / raw)
To: u-boot
This commit 904672e (lcd: refactor lcd console stuff into its
own file), which cause lcd console address is not initialized.
This patch initialize the lcd console use the default value,
will be update when splash screen is enabled.
Signed-off-by: Bo Shen <voice.shen@atmel.com>
---
Hi Tom,
If no objection of this patch, can you apply it as soon as possible.
Or else most Atmel SoC based boards (which enable lcd info) will be
broken.
Thanks.
common/lcd.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/common/lcd.c b/common/lcd.c
index cc34b8a..1195a54 100644
--- a/common/lcd.c
+++ b/common/lcd.c
@@ -268,6 +268,7 @@ void lcd_clear(void)
console_rows = panel_info.vl_row / VIDEO_FONT_HEIGHT;
#endif
console_cols = panel_info.vl_col / VIDEO_FONT_WIDTH;
+ lcd_init_console(lcd_base, console_rows, console_cols);
lcd_init_console(lcd_logo(), console_rows, console_cols);
lcd_sync();
}
--
2.3.0.rc0
^ permalink raw reply related [flat|nested] 9+ messages in thread
end of thread, other threads:[~2015-01-29 8:55 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-01-21 4:37 [U-Boot] [PATCH] lcd: fix console address is not initialized Bo Shen
2015-01-22 13:10 ` Nikita Kiryanov
2015-01-23 1:20 ` Bo Shen
2015-01-26 5:55 ` Bo Shen
2015-01-27 14:45 ` Nikita Kiryanov
2015-01-28 1:08 ` Bo Shen
2015-01-28 1:13 Bo Shen
2015-01-29 8:51 ` Anatolij Gustschin
2015-01-29 8:55 ` Bo Shen
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.