On Wed, Feb 01, 2023 at 08:50:50PM +0300, Alexander Shiyan wrote: > Hello! > > Just a nitpick, udelay means milliseconds (udelay_(us)), > which isn't quite right in this case, > so I think we should use the "loops" value here (delay_loops() or so)). > > Thanks! Hi Alexander, Thanks for the review! Are you sure? us means microseconds from what I understand. I've also posted an updated version of this patch just last night. John.