diff options
| author | Wolfgang Denk <wd@denx.de> | 2013-01-05 09:45:48 +0000 | 
|---|---|---|
| committer | Anatolij Gustschin <agust@denx.de> | 2013-03-21 09:05:08 +0100 | 
| commit | 46d1d5dd43e4d7c78f1fd6a5ee958d7b8a8626eb (patch) | |
| tree | b7f742aa84d55cb10254b2dfdcc5ce5a9f1b8531 /drivers | |
| parent | 8b906a9f0b3fd0d4421e08c4fa62f61a01289611 (diff) | |
| download | olio-uboot-2014.01-46d1d5dd43e4d7c78f1fd6a5ee958d7b8a8626eb.tar.xz olio-uboot-2014.01-46d1d5dd43e4d7c78f1fd6a5ee958d7b8a8626eb.zip | |
common/lcd.c: cleanup use of global variables
lcd_color_fg and lcd_color_bg had to be declared in board specific
code, but were not actually used there; in addition, we have getter /
setter functions for these, which were not used either.
Get rid of the global variables, and use the getter function where
needed (so far no setter calls are needed).
Signed-off-by: Wolfgang Denk <wd@denx.de>
Cc: Alessandro Rubini <rubini@unipv.it>
Cc: Anatolij Gustschin <agust@denx.de>
Cc: Bo Shen <voice.shen@atmel.com>
Cc: Haavard Skinnemoen <haavard.skinnemoen@atmel.com>
Cc: Kyungmin Park <kyungmin.park@samsung.com>
Cc: Marek Vasut <marek.vasut@gmail.com>
Cc: Minkyu Kang <mk7.kang@samsung.com>
Cc: Nikita Kiryanov <nikita@compulab.co.il>
Cc: Simon Glass <sjg@chromium.org>
Cc: Stelian Pop <stelian@popies.net>
Cc: Tom Warren <twarren@nvidia.com>
Acked-by: Simon Glass <sjg@chromium.org>
Acked-by: Jeroen Hofstee <jeroen@myspectrum.nl>
[agust: also fixed cm_t35 board while rebasing]
Signed-off-by: Anatolij Gustschin <agust@denx.de>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/video/amba.c | 2 | ||||
| -rw-r--r-- | drivers/video/atmel_hlcdfb.c | 2 | ||||
| -rw-r--r-- | drivers/video/atmel_lcdfb.c | 2 | ||||
| -rw-r--r-- | drivers/video/exynos_fb.c | 2 | ||||
| -rw-r--r-- | drivers/video/tegra.c | 4 | 
5 files changed, 1 insertions, 11 deletions
| diff --git a/drivers/video/amba.c b/drivers/video/amba.c index ffa1c399e..b4fb47da4 100644 --- a/drivers/video/amba.c +++ b/drivers/video/amba.c @@ -29,8 +29,6 @@  /* These variables are required by lcd.c -- although it sets them by itself */  int lcd_line_length; -int lcd_color_fg; -int lcd_color_bg;  void *lcd_base;  void *lcd_console_address;  short console_col; diff --git a/drivers/video/atmel_hlcdfb.c b/drivers/video/atmel_hlcdfb.c index b10ca4b67..e74eb65f3 100644 --- a/drivers/video/atmel_hlcdfb.c +++ b/drivers/video/atmel_hlcdfb.c @@ -30,8 +30,6 @@  #include <atmel_hlcdc.h>  int lcd_line_length; -int lcd_color_fg; -int lcd_color_bg;  void *lcd_base;				/* Start of framebuffer memory	*/  void *lcd_console_address;		/* Start of console buffer	*/ diff --git a/drivers/video/atmel_lcdfb.c b/drivers/video/atmel_lcdfb.c index c02ffd803..d96f175c5 100644 --- a/drivers/video/atmel_lcdfb.c +++ b/drivers/video/atmel_lcdfb.c @@ -30,8 +30,6 @@  #include <atmel_lcdc.h>  int lcd_line_length; -int lcd_color_fg; -int lcd_color_bg;  void *lcd_base;				/* Start of framebuffer memory	*/  void *lcd_console_address;		/* Start of console buffer	*/ diff --git a/drivers/video/exynos_fb.c b/drivers/video/exynos_fb.c index ee0ed06d6..4536c5c0e 100644 --- a/drivers/video/exynos_fb.c +++ b/drivers/video/exynos_fb.c @@ -34,8 +34,6 @@  #include "exynos_fb.h"  int lcd_line_length; -int lcd_color_fg; -int lcd_color_bg;  void *lcd_base;  void *lcd_console_address; diff --git a/drivers/video/tegra.c b/drivers/video/tegra.c index afcb00881..00d855356 100644 --- a/drivers/video/tegra.c +++ b/drivers/video/tegra.c @@ -61,8 +61,6 @@ enum {  };  int lcd_line_length; -int lcd_color_fg; -int lcd_color_bg;  void *lcd_base;			/* Start of framebuffer memory	*/  void *lcd_console_address;	/* Start of console buffer	*/ @@ -108,7 +106,7 @@ void lcd_toggle_cursor(void)  		for (i = 0; i < lcd_cursor_width; ++i) {  			color = *d; -			color ^= lcd_color_fg; +			color ^= lcd_getfgcolor();  			*d = color;  			++d;  		} |