On 28/08/2012 10:11, Heiko Schocher wrote:
> Hello Stefano,
>
Hi Heiko,
>> +static struct panel_config lcd_cfg[] = {
>> + {
>> + .timing_h = ((4 /* hpb */ - 1)<< 20) |
>> + ((8 /*hfp */- 1)<< 8) |
>> + (41 /* hsw */ - 1), /* Horizontal timing */
>
> Could we use here a define, something like this:
>
> #define PANEL_TIMING_H(bp, fp, sw) (((bp - 1) << 20) | ((fp - 1) << 8) |
> (sw - 1))
Agree - and put it in a common header (arch/dss.h), making it accessible
to all boards. And we need only one macro PANEL_TIMING, the register has
the same layout.
I'll do in V2.
>
> .panel_config = {
> .timing_h = PANEL_TIMING_H(4, 8, 41),
>
>> + .timing_v = (2 /*vbp */<< 24) |
> ^^
> Is this correct? In the manual I
> see for the timing H and V
> settings
> the same bitpositions!
>
This is not correct, thanks. It has no influence on my hardware because
it simply increments vbp, but I will fix it.
Stefano
--
=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=====================================================================
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
