On Wed, Apr 12, 2017 at 11:15:59AM -0400, Sven Van Asbroeck wrote:
> +     pm_runtime_set_active(&client->dev);
> +     /* enable will put the chip into suspend, which is what we
> +      * want as all outputs are disabled at this point
> +      */

This comment should also be in the standard block comment format.

Other than that, the patch looks good to me.

Reviewed-by: Mika Westerberg <[email protected]>

Reply via email to