On Thu, Feb 7, 2013 at 1:09 PM, Rahul Sharma <rahul.sharma@xxxxxxxxxxx> wrote: > It exposes generic interface from drm_edid.c to get the edid data and length > by any display entity. Once I get clear idea about edid handling in CDF, I need > to revert these temporary changes. Just a quick reply about edid reading: One of the key results (at least imo) of the fosdem cdf discussion was that we need to split up the different parts of it clearly (i.e. abstract panel interface, dsi support, discovery/dev matching, ...) to have more flexibility. One idea is also to not use the panel interface for e.g. hdmi transcoders, but only use the bus support (like dsi), since transcoders which connect to external devices like hdmi need to expose _much_ more features to the master driver and so it's better to have tighter integration. Some of the things which need close cooperation between drivers are e.g. edid reading, hotplug handling, bpp/colorspace/restricted range stuff, ... I didn't read through the patch which requires the exported drm edid stuff, but maybe this helps a bit. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel