On Fri, Jun 02, 2017 at 03:18:42PM +0800, Leilk Liu wrote: > + /* some IC design adjust register define */ > + bool adjust_reg; Can we have a name that's more specific to the particular quirk please? The current name will get confusing if some future chip also needs slightly different register settings. Otherwise this looks good.