diff mbox

[1/4] ASoC: Add snd_soc_component_{get, set}_drvdata()

Message ID 1408532929-26978-2-git-send-email-lars@metafoo.de (mailing list archive)
State Accepted
Commit c815dbb47758bd469927849fdd45fed3ce206e73
Headers show

Commit Message

Lars-Peter Clausen Aug. 20, 2014, 11:08 a.m. UTC
Add Add snd_soc_component_{get,set}_drvdata() similar to
snd_soc_codec_{get,set}_drvdata() and snd_soc_platform_{get,set}_drvdata().
Also update them to use the new functions internally.

Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
---
 include/sound/soc.h | 19 +++++++++++++++----
 1 file changed, 15 insertions(+), 4 deletions(-)

Comments

Vinod Koul Sept. 3, 2014, 12:22 p.m. UTC | #1
On Wed, Aug 20, 2014 at 01:08:46PM +0200, Lars-Peter Clausen wrote:
> Add Add snd_soc_component_{get,set}_drvdata() similar to
> snd_soc_codec_{get,set}_drvdata() and snd_soc_platform_{get,set}_drvdata().
> Also update them to use the new functions internally.
> 
> Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
Reviewed-by: Vinod Koul <vinod.koul@intel.com>
Lars-Peter Clausen Sept. 3, 2014, 7:06 p.m. UTC | #2
On 09/03/2014 02:22 PM, Vinod Koul wrote:
> On Wed, Aug 20, 2014 at 01:08:46PM +0200, Lars-Peter Clausen wrote:
>> Add Add snd_soc_component_{get,set}_drvdata() similar to
>> snd_soc_codec_{get,set}_drvdata() and snd_soc_platform_{get,set}_drvdata().
>> Also update them to use the new functions internally.
>>
>> Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
> Reviewed-by: Vinod Koul <vinod.koul@intel.com>

Is this for the first patch or for the whole series?

Thanks,
- Lars
Vinod Koul Sept. 4, 2014, 6:33 a.m. UTC | #3
On Wed, Sep 03, 2014 at 09:06:49PM +0200, Lars-Peter Clausen wrote:
> On 09/03/2014 02:22 PM, Vinod Koul wrote:
> >On Wed, Aug 20, 2014 at 01:08:46PM +0200, Lars-Peter Clausen wrote:
> >>Add Add snd_soc_component_{get,set}_drvdata() similar to
> >>snd_soc_codec_{get,set}_drvdata() and snd_soc_platform_{get,set}_drvdata().
> >>Also update them to use the new functions internally.
> >>
> >>Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
> >Reviewed-by: Vinod Koul <vinod.koul@intel.com>
> 
> Is this for the first patch or for the whole series?
Patch only. The HSW driver changes, I think Liam should ack
diff mbox

Patch

diff --git a/include/sound/soc.h b/include/sound/soc.h
index e43fbb6..e6440d8 100644
--- a/include/sound/soc.h
+++ b/include/sound/soc.h
@@ -1299,26 +1299,37 @@  static inline void *snd_soc_card_get_drvdata(struct snd_soc_card *card)
 	return card->drvdata;
 }
 
+static inline void snd_soc_component_set_drvdata(struct snd_soc_component *c,
+		void *data)
+{
+	dev_set_drvdata(c->dev, data);
+}
+
+static inline void *snd_soc_component_get_drvdata(struct snd_soc_component *c)
+{
+	return dev_get_drvdata(c->dev);
+}
+
 static inline void snd_soc_codec_set_drvdata(struct snd_soc_codec *codec,
 		void *data)
 {
-	dev_set_drvdata(codec->dev, data);
+	snd_soc_component_set_drvdata(&codec->component, data);
 }
 
 static inline void *snd_soc_codec_get_drvdata(struct snd_soc_codec *codec)
 {
-	return dev_get_drvdata(codec->dev);
+	return snd_soc_component_get_drvdata(&codec->component);
 }
 
 static inline void snd_soc_platform_set_drvdata(struct snd_soc_platform *platform,
 		void *data)
 {
-	dev_set_drvdata(platform->dev, data);
+	snd_soc_component_set_drvdata(&platform->component, data);
 }
 
 static inline void *snd_soc_platform_get_drvdata(struct snd_soc_platform *platform)
 {
-	return dev_get_drvdata(platform->dev);
+	return snd_soc_component_get_drvdata(&platform->component);
 }
 
 static inline void snd_soc_pcm_set_drvdata(struct snd_soc_pcm_runtime *rtd,