On Thu, Apr 29, 2010 at 6:49 PM, Anatolij Gustschin <ag...@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.
> /*
> * 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.
> + /* Still not found, use preferred mode from database if any */
> + if (!has_default_mode && info->monspecs.modedb != NULL) {
No need for the "!= NULL"
--
Timur Tabi
Linux kernel developer at Freescale
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev
