On Wed, Nov 08, 2017 at 12:24:03PM -0500, Alex Deucher wrote: > regmap_read(regmap, RT5645_VENDOR_ID2, &val); > > + /* > + * Read after 400msec, as it is the interval required between > + * read and power On. > + */ > + msleep(TIME_TO_POWER_MS); > + regmap_read(regmap, RT5645_VENDOR_ID2, &val); > + This leaves the original read in there so we've both got the early read (which might upset things potentially) and the delayed read. Shouldn't we just be adding a msleep() before the existing read? -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 488 bytes Desc: not available URL: <https://lists.freedesktop.org/archives/amd-gfx/attachments/20171108/8a34208d/attachment.sig>
