On Fri, Feb 9, 2018 at 9:21 PM, Andy Shevchenko <andy.shevchenko@xxxxxxxxx> wrote: > On Fri, Feb 9, 2018 at 8:58 AM, Donglin Peng <dolinux.peng@xxxxxxxxx> wrote: >> There is some duplicate code in soc-core.c,and the kernel provides > > 'core.c,and' -> 'core.c, and' > >> +static int codec_list_show(struct seq_file *s, void *v) > > No need to rename m -> s. > >> +static int dai_list_show(struct seq_file *s, void *v) > > Ditto. > >> +static int platform_list_show(struct seq_file *s, void *v) > > Ditto. > > After addressing above, > > Reviewed-by: Andy Shevchenko <andy.shevchenko@xxxxxxxxx> > > -- > With Best Regards, > Andy Shevchenko Thanks for review, and I will send a patch v2. _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
