On Thu, 16 Dec 2010 10:47:53 -0600 Timur Tabi <timur@xxxxxxxxxxxxx> wrote: > On Fri, Jul 23, 2010 at 9:00 AM, Anatolij Gustschin <agust@xxxxxxx> wrote: > > Adds support for encoding display mode information > > in the device tree using verbatim EDID block. > > > > If the EDID entry in the DIU node is present, the > > driver will build mode database using EDID data > > and allow setting the display modes from this database. > > Otherwise display mode will be set using mode > > entries from driver's internal database as usual. > > > > This patch also updates device tree bindings. > > > > Signed-off-by: Anatolij Gustschin <agust@xxxxxxx> > > Acked-by: Timur Tabi <timur@xxxxxxxxxxxxx> > > Cc: devicetree-discuss@xxxxxxxxxxxxxxxx > > Anatolij, > > I know this patch is old, but I'm now getting back to working on the > DIU driver. One question I have: why are you reading the EDID data > from the device tree? Why not just read it directly from the device > using I2C? Who is supposed to put the EDID data into the device tree > in the first place? Many embedded boards only hard-wire a panel which does not provide an i2c edid channel. For such boards the EDID data can be inserted by the bootloader or encoded in the board's .dts. Look at pdm360ng U-Boot board code, it inserts the EDID data into device tree using fdt_add_edid(). Anatolij -- To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html