Re: [Intel-gfx] [PATCH 1/2] drm/i915: Optimize HDMI EDID reads

2014-08-17 Thread Sharma, Shashank
; intel-gfx; Vetter, Daniel; Kumar, Shobhit Subject: Re: [Intel-gfx] [PATCH 1/2] drm/i915: Optimize HDMI EDID reads On Thu, Aug 14, 2014 at 02:53:44PM +0530, Sharma, Shashank wrote: Hi Daniel, I can do all the changes accordingly and upload a new patch. This is what I am going to do: 1

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Optimize HDMI EDID reads

2014-08-14 Thread Sharma, Shashank
Regards Shashank On 8/13/2014 5:44 PM, Daniel Vetter wrote: On Tue, Aug 12, 2014 at 06:08:20PM +0530, shashank.sha...@intel.com wrote: From: Shashank Sharma shashank.sha...@intel.com The current hdmi_detect() function is getting called from many places, few of these are: 1. HDMI hot plug

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Optimize HDMI EDID reads

2014-08-14 Thread Daniel Vetter
On Thu, Aug 14, 2014 at 8:15 AM, Sharma, Shashank shashank.sha...@intel.com wrote: Can you please point out what is it, that's unexpected to you ? I think this is what we (you Shobhit) agreed on: 1. Timeout based EDID caching 2. Use of cached EDID within caching duration 3. Separate path for

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Optimize HDMI EDID reads

2014-08-14 Thread Sharma, Shashank
Hi Daniel, I can do all the changes accordingly and upload a new patch. This is what I am going to do: 1. Change the EDID caching time to a second from a minute (probably there was a miscommunication). 2. Remove the gen_check 3. Use the connector-edid pointer to cache EDID. I have only few

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Optimize HDMI EDID reads

2014-08-14 Thread Daniel Vetter
On Thu, Aug 14, 2014 at 02:53:44PM +0530, Sharma, Shashank wrote: Hi Daniel, I can do all the changes accordingly and upload a new patch. This is what I am going to do: 1. Change the EDID caching time to a second from a minute (probably there was a miscommunication). 2. Remove the

Re: [Intel-gfx] [PATCH 1/2] drm/i915: Optimize HDMI EDID reads

2014-08-13 Thread Daniel Vetter
On Tue, Aug 12, 2014 at 06:08:20PM +0530, shashank.sha...@intel.com wrote: From: Shashank Sharma shashank.sha...@intel.com The current hdmi_detect() function is getting called from many places, few of these are: 1. HDMI hot plug interrupt bottom half 2. get_resources() IOCTL family 3.

[Intel-gfx] [PATCH 1/2] drm/i915: Optimize HDMI EDID reads

2014-08-12 Thread shashank . sharma
From: Shashank Sharma shashank.sha...@intel.com The current hdmi_detect() function is getting called from many places, few of these are: 1. HDMI hot plug interrupt bottom half 2. get_resources() IOCTL family 3. drm_helper_probe_single_connector_modes() family 4. output_poll_execute() 5.