public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot] [RESEND PATCH] common/lcd_console: fix console/logo regression
@ 2015-08-04 13:49 Marcel Ziswiler
  2015-08-09 15:07 ` Simon Glass
  2015-08-13 13:23 ` [U-Boot] [U-Boot, RESEND] " Tom Rini
  0 siblings, 2 replies; 5+ messages in thread
From: Marcel Ziswiler @ 2015-08-04 13:49 UTC (permalink / raw)
  To: u-boot

From: Marcel Ziswiler <marcel.ziswiler@toradex.com>

The following commit changed the order of the column vs. row parameter
to the lcd_init_console() function but missed actually changing it as
well the second time it is called from lcd_clear() which resulted in a
garbled text console which this patch fixes.

commit 604c7d4a5a3cf70949f6e6094bf0d52ee3b4804d
common/lcd_console: introduce display/framebuffer rotation

Tested on Colibri T20 with my latest assortment of tegra
fixes/enhancements patch set.

Signed-off-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
---
 common/lcd.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/common/lcd.c b/common/lcd.c
index 5a52fe4..d29308a 100644
--- a/common/lcd.c
+++ b/common/lcd.c
@@ -234,8 +234,8 @@ void lcd_clear(void)
 	lcd_logo();
 #if defined(CONFIG_LCD_LOGO) && !defined(CONFIG_LCD_INFO_BELOW_LOGO)
 	addr = (ulong)lcd_base + BMP_LOGO_HEIGHT * lcd_line_length;
-	lcd_init_console((void *)addr, panel_info.vl_row,
-			 panel_info.vl_col, panel_info.vl_rot);
+	lcd_init_console((void *)addr, panel_info.vl_col,
+			 panel_info.vl_row, panel_info.vl_rot);
 #endif
 	lcd_sync();
 }
-- 
2.4.3

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [U-Boot] [RESEND PATCH] common/lcd_console: fix console/logo regression
  2015-08-04 13:49 [U-Boot] [RESEND PATCH] common/lcd_console: fix console/logo regression Marcel Ziswiler
@ 2015-08-09 15:07 ` Simon Glass
  2015-08-13 11:57   ` Marcel Ziswiler
  2015-08-13 13:23 ` [U-Boot] [U-Boot, RESEND] " Tom Rini
  1 sibling, 1 reply; 5+ messages in thread
From: Simon Glass @ 2015-08-09 15:07 UTC (permalink / raw)
  To: u-boot

On 4 August 2015 at 07:49, Marcel Ziswiler <marcel@ziswiler.com> wrote:
> From: Marcel Ziswiler <marcel.ziswiler@toradex.com>
>
> The following commit changed the order of the column vs. row parameter
> to the lcd_init_console() function but missed actually changing it as
> well the second time it is called from lcd_clear() which resulted in a
> garbled text console which this patch fixes.
>
> commit 604c7d4a5a3cf70949f6e6094bf0d52ee3b4804d
> common/lcd_console: introduce display/framebuffer rotation
>
> Tested on Colibri T20 with my latest assortment of tegra
> fixes/enhancements patch set.
>
> Signed-off-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> ---
>  common/lcd.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)

Acked-by: Simon Glass <sjg@chromium.org>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [U-Boot] [RESEND PATCH] common/lcd_console: fix console/logo regression
  2015-08-09 15:07 ` Simon Glass
@ 2015-08-13 11:57   ` Marcel Ziswiler
  2015-08-13 12:10     ` Anatolij Gustschin
  0 siblings, 1 reply; 5+ messages in thread
From: Marcel Ziswiler @ 2015-08-13 11:57 UTC (permalink / raw)
  To: u-boot

On Sun, 2015-08-09 at 09:07 -0600, Simon Glass wrote:

> Acked-by: Simon Glass <sjg@chromium.org>

Thanks, Simon.

Could any of you guys please pull that one in or how do we proceed?

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [U-Boot] [RESEND PATCH] common/lcd_console: fix console/logo regression
  2015-08-13 11:57   ` Marcel Ziswiler
@ 2015-08-13 12:10     ` Anatolij Gustschin
  0 siblings, 0 replies; 5+ messages in thread
From: Anatolij Gustschin @ 2015-08-13 12:10 UTC (permalink / raw)
  To: u-boot

Hi,

On Thu, 13 Aug 2015 13:57:37 +0200
Marcel Ziswiler <marcel@ziswiler.com> wrote:
...
> Could any of you guys please pull that one in or how do we proceed?

Sorry for delay, I'll merge the patch soon.

Thanks,

Anatolij

^ permalink raw reply	[flat|nested] 5+ messages in thread

* [U-Boot] [U-Boot, RESEND] common/lcd_console: fix console/logo regression
  2015-08-04 13:49 [U-Boot] [RESEND PATCH] common/lcd_console: fix console/logo regression Marcel Ziswiler
  2015-08-09 15:07 ` Simon Glass
@ 2015-08-13 13:23 ` Tom Rini
  1 sibling, 0 replies; 5+ messages in thread
From: Tom Rini @ 2015-08-13 13:23 UTC (permalink / raw)
  To: u-boot

On Tue, Aug 04, 2015 at 03:49:50PM +0200, Marcel Ziswiler wrote:

> From: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> 
> The following commit changed the order of the column vs. row parameter
> to the lcd_init_console() function but missed actually changing it as
> well the second time it is called from lcd_clear() which resulted in a
> garbled text console which this patch fixes.
> 
> commit 604c7d4a5a3cf70949f6e6094bf0d52ee3b4804d
> common/lcd_console: introduce display/framebuffer rotation
> 
> Tested on Colibri T20 with my latest assortment of tegra
> fixes/enhancements patch set.
> 
> Signed-off-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
> Acked-by: Simon Glass <sjg@chromium.org>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150813/db2c2a6e/attachment.sig>

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2015-08-13 13:23 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-08-04 13:49 [U-Boot] [RESEND PATCH] common/lcd_console: fix console/logo regression Marcel Ziswiler
2015-08-09 15:07 ` Simon Glass
2015-08-13 11:57   ` Marcel Ziswiler
2015-08-13 12:10     ` Anatolij Gustschin
2015-08-13 13:23 ` [U-Boot] [U-Boot, RESEND] " Tom Rini

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox