Re: [Intel-gfx] [PATCH 3/3] drm/i915: Read HDMI EDID only when required

2015-07-06 Thread Jindal, Sonika
On 7/6/2015 1:08 PM, Daniel Vetter wrote: On Thu, Jul 02, 2015 at 08:24:12AM +0530, Sharma, Shashank wrote: Regards Shashank On 7/1/2015 6:26 PM, Daniel Vetter wrote: On Tue, Jun 30, 2015 at 09:49:57PM +0530, Shashank Sharma wrote: Userspace always sets force. Are you sure this actually imp

Re: [Intel-gfx] [PATCH 3/3] drm/i915: Read HDMI EDID only when required

2015-07-06 Thread Daniel Vetter
On Thu, Jul 02, 2015 at 08:24:12AM +0530, Sharma, Shashank wrote: > Regards > Shashank > > On 7/1/2015 6:26 PM, Daniel Vetter wrote: > >On Tue, Jun 30, 2015 at 09:49:57PM +0530, Shashank Sharma wrote: > >>>Userspace always sets force. Are you sure this actually improves anything? > >>Yes we do. We

Re: [Intel-gfx] [PATCH 3/3] drm/i915: Read HDMI EDID only when required

2015-07-01 Thread Sharma, Shashank
Regards Shashank On 7/1/2015 6:26 PM, Daniel Vetter wrote: On Tue, Jun 30, 2015 at 09:49:57PM +0530, Shashank Sharma wrote: Userspace always sets force. Are you sure this actually improves anything? Yes we do. We have had this code for commercial projects, and that's really important to have

Re: [Intel-gfx] [PATCH 3/3] drm/i915: Read HDMI EDID only when required

2015-07-01 Thread Shashank Sharma
> Userspace always sets force. Are you sure this actually improves anything? Yes we do. We have had this code for commercial projects, and that's really important to have proper interrupt handling as well as to avoid race condition between multiple HDMI detects from interrupt handler and userspace

Re: [Intel-gfx] [PATCH 3/3] drm/i915: Read HDMI EDID only when required

2015-07-01 Thread shuang . he
Tested-By: Intel Graphics QA PRTS (Patch Regression Test System Contact: shuang...@intel.com) Task id: 6674 -Summary- Platform Delta drm-intel-nightly Series Applied ILK -1

Re: [Intel-gfx] [PATCH 3/3] drm/i915: Read HDMI EDID only when required

2015-07-01 Thread Daniel Vetter
On Tue, Jun 30, 2015 at 09:49:57PM +0530, Shashank Sharma wrote: > > Userspace always sets force. Are you sure this actually improves anything? > Yes we do. We have had this code for commercial projects, and that's really > important to have proper interrupt handling as well as to avoid race > con

Re: [Intel-gfx] [PATCH 3/3] drm/i915: Read HDMI EDID only when required

2015-06-30 Thread Daniel Vetter
On Tue, Jun 30, 2015 at 11:13:58AM +0530, Sonika Jindal wrote: > From: Shashank Sharma > > This patch makes sure that the HDMI detect function > reads EDID only when its forced to do it. All the other > times, it uses the connector->detect_edid which was cached > during hotplug handling in the hd

[Intel-gfx] [PATCH 3/3] drm/i915: Read HDMI EDID only when required

2015-06-29 Thread Sonika Jindal
From: Shashank Sharma This patch makes sure that the HDMI detect function reads EDID only when its forced to do it. All the other times, it uses the connector->detect_edid which was cached during hotplug handling in the hdmi_probe() function. As the probe function gets called before detect in the