Hi Mark > > +void snd_soc_of_parse_node_prefix(struct device_node *np, > > struct snd_soc_codec_conf *codec_conf, > > struct device_node *of_node, > > const char *propname); > > +#define snd_soc_of_parse_audio_prefix(card, conf, node, name) \ > > + snd_soc_of_parse_node_prefix((card)->dev->of_node, \ > > + (conf), (node), (name)) > > This is basically fine but it'd be better to make this a static inline > rather than a #define - it helps with error messages and type safety. > I'll apply the series since otherwise this looks good, please send a > followup patch fixing this. OK, thanks. Yes, I will post it soon Best regards --- Kuninori Morimoto _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel