[ 014/135] OMAPFB: fix framebuffer console colors

2012-09-16 Thread Ben Hutchings
3.2-stable review patch.  If anyone has any objections, please let me know.

--

From: Grazvydas Ignotas 

commit c1c52848cef52e157468b8879fc3cae23b6f3a99 upstream.

omapfb does not currently set pseudo palette correctly for color depths
above 16bpp, making red text invisible, command like
  echo -e '\e[0;31mRED' > /dev/tty1
will display nothing on framebuffer console in 24bpp mode.
This is because temporary variable is declared incorrectly, fix it.

Signed-off-by: Grazvydas Ignotas 
Signed-off-by: Tomi Valkeinen 
Signed-off-by: Florian Tobias Schandinat 
Signed-off-by: Ben Hutchings 
---
 drivers/video/omap2/omapfb/omapfb-main.c |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/video/omap2/omapfb/omapfb-main.c 
b/drivers/video/omap2/omapfb/omapfb-main.c
index 08ec1a7..fc671d3 100644
--- a/drivers/video/omap2/omapfb/omapfb-main.c
+++ b/drivers/video/omap2/omapfb/omapfb-main.c
@@ -1192,7 +1192,7 @@ static int _setcolreg(struct fb_info *fbi, u_int regno, 
u_int red, u_int green,
break;
 
if (regno < 16) {
-   u16 pal;
+   u32 pal;
pal = ((red >> (16 - var->red.length)) <<
var->red.offset) |
((green >> (16 - var->green.length)) <<


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[ 014/135] OMAPFB: fix framebuffer console colors

2012-09-16 Thread Ben Hutchings
3.2-stable review patch.  If anyone has any objections, please let me know.

--

From: Grazvydas Ignotas nota...@gmail.com

commit c1c52848cef52e157468b8879fc3cae23b6f3a99 upstream.

omapfb does not currently set pseudo palette correctly for color depths
above 16bpp, making red text invisible, command like
  echo -e '\e[0;31mRED'  /dev/tty1
will display nothing on framebuffer console in 24bpp mode.
This is because temporary variable is declared incorrectly, fix it.

Signed-off-by: Grazvydas Ignotas nota...@gmail.com
Signed-off-by: Tomi Valkeinen tomi.valkei...@ti.com
Signed-off-by: Florian Tobias Schandinat florianschandi...@gmx.de
Signed-off-by: Ben Hutchings b...@decadent.org.uk
---
 drivers/video/omap2/omapfb/omapfb-main.c |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/video/omap2/omapfb/omapfb-main.c 
b/drivers/video/omap2/omapfb/omapfb-main.c
index 08ec1a7..fc671d3 100644
--- a/drivers/video/omap2/omapfb/omapfb-main.c
+++ b/drivers/video/omap2/omapfb/omapfb-main.c
@@ -1192,7 +1192,7 @@ static int _setcolreg(struct fb_info *fbi, u_int regno, 
u_int red, u_int green,
break;
 
if (regno  16) {
-   u16 pal;
+   u32 pal;
pal = ((red  (16 - var-red.length)) 
var-red.offset) |
((green  (16 - var-green.length)) 


--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/