Re: [Intel-gfx] [PATCH v5 2/2] drm/i915/dp: Refactor mav_vswing_tries variable

2018-07-20 Thread Nathan Ciobanu
On Fri, Jul 20, 2018 at 01:19:02PM +0300, Ville Syrjälä wrote: > On Thu, Jul 19, 2018 at 02:48:07PM -0700, Nathan Ciobanu wrote: > > Changes the type and renames the max_vswing_tries variable > > which was declared as an integer but used as a boolean > > making it easy to be confused with a counter

Re: [Intel-gfx] [PATCH v5 2/2] drm/i915/dp: Refactor mav_vswing_tries variable

2018-07-20 Thread Ville Syrjälä
On Thu, Jul 19, 2018 at 02:48:07PM -0700, Nathan Ciobanu wrote: > Changes the type and renames the max_vswing_tries variable > which was declared as an integer but used as a boolean > making it easy to be confused with a counter. > > Changes in v2: > - updated the title and commit message >

[Intel-gfx] [PATCH v5 2/2] drm/i915/dp: Refactor mav_vswing_tries variable

2018-07-19 Thread Nathan Ciobanu
Changes the type and renames the max_vswing_tries variable which was declared as an integer but used as a boolean making it easy to be confused with a counter. Changes in v2: - updated the title and commit message - left the loop exit point in place Cc: Dhinakaran Pandiyan Cc: Rodrigo Vi