[PATCH 1/2] drm: bridge/dw_hdmi: Cache edid data

2015-08-08 Thread Russell King - ARM Linux
On Sat, Aug 08, 2015 at 02:35:40PM +0100, Russell King - ARM Linux wrote: > I have a whole bunch of further patches for dw-hdmi, so if you don't > mind, I'll take your two, merge them on the front of my series, and > re-post them. I think we're getting close to the time where we can > see initial

[PATCH 1/2] drm: bridge/dw_hdmi: Cache edid data

2015-08-08 Thread Russell King - ARM Linux
On Fri, Aug 07, 2015 at 11:40:28PM +0100, Russell King - ARM Linux wrote: > However, if we do this, then we might as well just modify > dw_hdmi_connector_get_modes(), and arrange for a HPD de-assertion to > NULL and free hdmi->edid so the next get_modes() call triggers a re-read. Testing with the

[PATCH 1/2] drm: bridge/dw_hdmi: Cache edid data

2015-08-07 Thread Russell King - ARM Linux
On Fri, Aug 07, 2015 at 04:20:47PM +0200, Lucas Stach wrote: > Am Freitag, den 07.08.2015, 15:13 +0100 schrieb Russell King - ARM > Linux: > > On Fri, Aug 07, 2015 at 03:59:20PM +0200, Sascha Hauer wrote: > > > Instead of rereading the edid data each time userspace asks for them > > > read them onc

[PATCH 1/2] drm: bridge/dw_hdmi: Cache edid data

2015-08-07 Thread Lucas Stach
Am Freitag, den 07.08.2015, 15:13 +0100 schrieb Russell King - ARM Linux: > On Fri, Aug 07, 2015 at 03:59:20PM +0200, Sascha Hauer wrote: > > Instead of rereading the edid data each time userspace asks for them > > read them once and cache them in the previously unused edid field in > > struct dw_h

[PATCH 1/2] drm: bridge/dw_hdmi: Cache edid data

2015-08-07 Thread Sascha Hauer
Instead of rereading the edid data each time userspace asks for them read them once and cache them in the previously unused edid field in struct dw_hdmi. This makes the code a little bit more efficient. Signed-off-by: Sascha Hauer --- drivers/gpu/drm/bridge/dw_hdmi.c | 41 +++

[PATCH 1/2] drm: bridge/dw_hdmi: Cache edid data

2015-08-07 Thread Russell King - ARM Linux
On Fri, Aug 07, 2015 at 03:59:20PM +0200, Sascha Hauer wrote: > Instead of rereading the edid data each time userspace asks for them > read them once and cache them in the previously unused edid field in > struct dw_hdmi. This makes the code a little bit more efficient. How has this been tested?