> > Whether the dt is used or not, almost all the simple card information > > for the DAI link and sound card are initialized in the simple card driver. > > > > And for the platform caller, the snd_link and snd_card are of no use, so > > move them from struct asoc_simple_card_info to struct asoc_simple_card_data. > > > > And now only one DAI link is supported for simple card. > > > > Suggested-by: Jean-Francois Moine <moin...@free.fr> > > Signed-off-by: Xiubo Li <li.xi...@freescale.com> > > --- > > This patch seems have many this kind of lines > - info->xxx > + sdata->info->xxx > > But, how about add this line to each function ? > > struct asoc_simple_card_info *info = &sdata->info; > > Patch will be more readable >
Yes, that's looks perfect. See the next version please. Thanks, -- Best Regards, Xiubo -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/