On Mon, Aug 29, 2016 at 10:44:38AM +0200, Takashi Iwai wrote: > On Mon, 29 Aug 2016 08:23:19 +0200, > Subhransu S. Prusty wrote: > > > > Signed-off-by: Subhransu S. Prusty <subhransu.s.prusty@xxxxxxxxx> > > Signed-off-by: Vinod Koul <vinod.koul@xxxxxxxxx> > > --- > > sound/hda/ext/hdac_codec.h | 7 +++++++ > > 1 file changed, 7 insertions(+) > > > > diff --git a/sound/hda/ext/hdac_codec.h b/sound/hda/ext/hdac_codec.h > > index 7f47a7e..9159139 100644 > > --- a/sound/hda/ext/hdac_codec.h > > +++ b/sound/hda/ext/hdac_codec.h > > @@ -46,6 +46,13 @@ struct hdac_codec_widget { > > void *params; /* Widget specific parameters */ > > }; > > > > +static inline struct hdac_ext_device *to_hda_ext_device(struct device *dev) > > +{ > > + struct hdac_device *hdac = dev_to_hdac_dev(dev); > > + > > + return to_ehdac_device(hdac); > > +} > > + > > int snd_hdac_parse_widgets(struct hdac_device *hdac); > > int snd_hdac_codec_init(struct hdac_device *hdac); > > void snd_hdac_codec_cleanup(struct hdac_device *hdac); > > "Move" is to move from point A to B, but you're only adding a new one > here... to_hda_ext_device was defined in hdac_hdmi.c. As ASoC hda codec framework will use this, so moved this to core. The next patch removes this helper from hdac_hdmi.c. As I didn't want intermix two different modules in the same patch, so submitted as separate patches. May be I will rename the commit message to "copy ... " instead of "move". What do you suggest? Regards, Subhransu > > > Takashi -- _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx http://mailman.alsa-project.org/mailman/listinfo/alsa-devel