* Woodruff, Richard <r-woodruff2@xxxxxx> [160301 16:24]: > Hi Tony, > > > From: Tony Lindgren [mailto:tony@xxxxxxxxxxx] > > Sent: Thursday, February 11, 2016 10:21 AM > > > Care to check again with the mainline kernel? All the mainline > > code was doing was incrementing these. > > > > > The kick counters were there to work around HW quirks/bugs not for any > > kind of debugging. Code in this area was added reactively to customer > > production failures not for expirmentation. > > I was able to check code over time. > > You are correct the counters were debug only. The DLL wait spin and kick was added to work around a HW errata for OMAP3430. This work around is mandatory. The kick_counter was used as a debug aid to help characterize the time of unlock condition. My initial comment wrongly conflated the kick function with the counter. These are separate. Sorry for any misdirection and delay in being able to check more carefully. OK thanks for checking! Regards, Tony -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html