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?

Attachment: signature.asc
Description: PGP signature

_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to