Re: [Intel-gfx] [PATCHv2] drm/i915/dp: Change aux_ctl reg read to polling read

2022-12-06 Thread Murthy, Arun R
> -Original Message- > From: Nikula, Jani > Sent: Monday, December 5, 2022 4:08 PM > To: Murthy, Arun R ; intel- > g...@lists.freedesktop.org; ville.syrj...@linux.intel.com; Deak, Imre > > Cc: Murthy, Arun R > Subject: Re: [PATCHv2] drm/i915/dp: Change aux_ctl reg read to polling read >

Re: [Intel-gfx] [PATCHv2] drm/i915/dp: Change aux_ctl reg read to polling read

2022-12-05 Thread Jani Nikula
On Thu, 01 Dec 2022, Arun R Murthy wrote: > The busy timeout logic checks for the AUX BUSY, then waits for the > timeout period and then after timeout reads the register for BUSY set > and fails. That's inaccurate. It checks for busy, waits for the gmbus irq, and on timeout or irq, whichever

[Intel-gfx] [PATCHv2] drm/i915/dp: Change aux_ctl reg read to polling read

2022-11-30 Thread Arun R Murthy
The busy timeout logic checks for the AUX BUSY, then waits for the timeout period and then after timeout reads the register for BUSY set and fails. Instead replace interrupt with polling so as to read the AUX CTL register often before the timeout period. v2: replace interrupt with polling read