[U-Boot] [PATCH] video: Add support for TI's AM335x LCD-Controller

[Heiko Schocher <hs@denx.de>]

Hello Hannes,

Am 06.03.2014 15:24, schrieb Hannes Petermaier:
> On 2014-03-06 14:58, Heiko Schocher wrote:
>> Hello Hannes,
>>
>> Am 06.03.2014 14:39, schrieb Hannes Petermaier:
>>> - Adds support for a minimal framebuffer driver of TI's AM335x SoC
>>> to be compatible with Wolfgang Denk's LCD-Framework (CONFIG_LCD,
>>> common/lcd.c)
>>>
>>> Signed-off-by: Hannes Petermaier<oe5hpm@oevsv.at>
>>> ---
>>> drivers/video/Makefile | 1 +
>>> drivers/video/am335x-fb.c | 169 +++++++++++++++++++++++++++++++++++++++++++++
>>> drivers/video/am335x-fb.h | 67 ++++++++++++++++++
>>> 3 files changed, 237 insertions(+)
>>> create mode 100644 drivers/video/am335x-fb.c
>>> create mode 100644 drivers/video/am335x-fb.h
>>
>> Why you cannot use:
>>
>> u-boot:drivers/video/da8xx-fb.c ?
>>
>> bye,
>> Heiko
> Hi Heiko,
>
> for my opinion this clone of the linux-driver is very overloaded and difficult to use/configure.
> With the words 'small-is-beautiful' and 'keep-it-simple' i've wrote a few lines which do the minimum:
> - configure raster-controller
> - setup DMA
> - powerON Display
>
> maybe we can use the small-version in other projects too.

Why is it difficult to use/configure the existing driver?

Look for example into the board/siemens/pxm2/board.c board, which uses
this driver. You have to define:

static struct da8xx_panel lcd_panels[]
static const struct display_panel disp_panel
static const struct lcd_ctrl_config lcd_cfg

and call "da8xx_video_init(&lcd_panels[0], &lcd_cfg, lcd_cfg.bpp);"

Thats all ...

bye,
Heiko
-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany