[PATCH 5/5] fsl-diu-fb: Support setting display mode using EDID
Anatolij Gustschin
agust at denx.de
Fri Apr 30 17:43:43 EST 2010
On Thu, 29 Apr 2010 20:44:12 -0500
Timur Tabi <timur.tabi at gmail.com> wrote:
> On Thu, Apr 29, 2010 at 6:49 PM, Anatolij Gustschin <agust at denx.de> wrote:
>
> > +Optional properties:
> > +- EDID : verbatim EDID data block describing attached display.
> > + Data from the detailed timing descriptor will be used to
> > + program the display controller.
>
> The property name should be lower-case.
Will change to lower-case.
> > /*
> > * These parameters give default parameters
> > @@ -217,6 +218,7 @@ struct mfb_info {
> > int x_aoi_d; /* aoi display x offset to physical screen */
> > int y_aoi_d; /* aoi display y offset to physical screen */
> > struct fsl_diu_data *parent;
> > + char *edid_data;
>
> edid_data should be "u8 *". "char *" is should be used only for
> strings or arrays of characters.
Will fix it, too.
> > + /* Still not found, use preferred mode from database if any */
> > + if (!has_default_mode && info->monspecs.modedb != NULL) {
>
> No need for the "!= NULL"
Ok, I will simplify this.
Thanks,
Anatolij
More information about the Linuxppc-dev
mailing list