Hi Kuninori,
>> > + component = kzalloc(sizeof(*component), GFP_KERNEL);
>> > + if (!component) {
>> > + dev_err(dev, "ASoC: Failed to allocate memory\n");
>> > + return -ENOMEM;
>>
>> No need to print an error message if kzalloc fails. The core will print it.
>
> Thanks. Yes I had noticed this warning from checkpatch.
> The main purpose of this patch is separate "register" function into
> "register" and "add".
> Thus, I keeped existing all code.
> I think "remove unneeded message" should be increment patch,
> but can you agree ?
Makes sense. It's better to be sent as a separate patch.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
