On Wed, Jul 06, 2016 at 10:01:27AM +0900, Kuninori Morimoto wrote:

> +     if (name) {
> +             ret = 0;
> +
> +             dai_link->name =
> +             dai_link->stream_name = name;
> +     }

This code is unclear - it's valid C but the line break in the middle of
the set of assignments makes it look like the first assignment is an
error.  This should be two assignments.

Attachment: signature.asc
Description: PGP signature

Reply via email to