28 Aug
2012
28 Aug
'12
10:47 a.m.
Hello Stefano,
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.
The hbp, hfp, hsw, vsw all have a "(program to value minus 1)." vbp and vfp don't, you can set those to zero.
So while the layout is the same, the meaning isn't. To make the values match the lcd specification, I would prefer to have the separate defines.
Regards, Jeroen