lcd: allow overriding lcd_get_size()

Remove the redundant lcd_line_length initialisation which
sneaked in when an earlier version of the patch of commit
6d330719 has been rebased.

Some lcd drivers need to setup lcd_line_length not from the
panel_info parameters but by different means. Make the
lcd_get_size() weak to allow setting lcd_line_length in
a driver specific way.

Signed-off-by: Anatolij Gustschin <agust@denx.de>
Cc: Stephen Warren <swarren@wwwdotorg.org>
This commit is contained in:
Anatolij Gustschin 2013-11-09 11:00:09 +01:00
parent e2788afe67
commit cefa471712

View file

@ -386,8 +386,13 @@ static void test_pattern(void)
/************************************************************************/ /************************************************************************/
/* ** GENERIC Initialization Routines */ /* ** GENERIC Initialization Routines */
/************************************************************************/ /************************************************************************/
/*
int lcd_get_size(int *line_length) * With most lcd drivers the line length is set up
* by calculating it from panel_info parameters. Some
* drivers need to calculate the line length differently,
* so make the function weak to allow overriding it.
*/
__weak int lcd_get_size(int *line_length)
{ {
*line_length = (panel_info.vl_col * NBITS(panel_info.vl_bpix)) / 8; *line_length = (panel_info.vl_col * NBITS(panel_info.vl_bpix)) / 8;
return *line_length * panel_info.vl_row; return *line_length * panel_info.vl_row;
@ -495,7 +500,6 @@ static int lcd_init(void *lcdbase)
debug("[LCD] Using LCD frambuffer at %p\n", lcd_base); debug("[LCD] Using LCD frambuffer at %p\n", lcd_base);
lcd_get_size(&lcd_line_length); lcd_get_size(&lcd_line_length);
lcd_line_length = (panel_info.vl_col * NBITS(panel_info.vl_bpix)) / 8;
lcd_is_enabled = 1; lcd_is_enabled = 1;
lcd_clear(); lcd_clear();
lcd_enable(); lcd_enable();