On Thu, Jan 28, 2016 at 04:54:20PM +0000, Damien Horsley wrote:

> +     ret = clk_set_rate(pcm3168a->scki, freq);
> +     if (ret)
> +             return ret;
> +
>       pcm3168a->sysclk = freq;

Since the clock appears to be mandatory in this driver (good!) why are
we even storing sysclk?  Just remove it, the driver never actually used
it anyway.

Attachment: signature.asc
Description: PGP signature

Reply via email to