On Fri, 2008-09-05 at 10:13 -0700, Tony Lindgren wrote:
> * stanley.miao <[EMAIL PROTECTED]> [080902 02:32]:
> > Thanks. I resolved the style problem. If gpio function needs to be
> > changed, I think it should be in a single patch.
> >
> > Re-send the patch again.
> >
> > Fix omapfb's problem on OMAP3430
> >
> > Signed-off-by: Stanley.Miao <[EMAIL PROTECTED]>
> > ---
> > drivers/video/omap/lcd_2430sdp.c | 17 +++++++++++++++++
> > 1 files changed, 17 insertions(+), 0 deletions(-)
> >
[...]
> >
> > #define t2_out(c, r, v) twl4030_i2c_write_u8(c, r, v)
> >
> > @@ -77,6 +82,13 @@ static int sdp2430_panel_enable(struct lcd_panel
> > *panel)
> > u8 ded_val, ded_reg;
> > u8 grp_val, grp_reg;
> >
> > + if (is_sil_rev_greater_than(OMAP3430_REV_ES1_0)) {
> > + t2_out(PM_RECEIVER, ENABLE_VPLL2_DEDICATED,
> > + TWL4030_VPLL2_DEDICATED);
> > + t2_out(PM_RECEIVER, ENABLE_VPLL2_DEV_GRP,
> > + TWL4030_VPLL2_DEV_GRP);
> > + }
> > +
> > if (machine_is_omap_3430sdp()) {
> > ded_reg = TWL4030_VAUX3_DEDICATED;
> > ded_val = ENABLE_VAUX3_DEDICATED;
>
> How about putting the is_sel_rev_greater_than test within the
> machine_is_omap3430sdp test?
Good, this will run fewer instructions on 2430sdp.
Stanley.
>
> Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
