Re: [PATCH 04/15] ALSA: hda - Add helper to read eld data

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, 01 Dec 2015 20:12:14 +0100,
Subhransu S. Prusty wrote:
> 
> On Tue, Dec 01, 2015 at 01:39:19PM +0100, Takashi Iwai wrote:
> > On Tue, 01 Dec 2015 18:47:00 +0100,
> > Subhransu S. Prusty wrote:
> > > 
> > > The eld reading APIs are required for ASoC skylake hdmi
> > > driver as well. So moving these definition to core. Once
> > > component ops for reading ELD is available will mark this
> > > as deprecated.
> > > 
> > > Signed-off-by: Subhransu S. Prusty <subhransu.s.prusty@xxxxxxxxx>
> > > Signed-off-by: Vinod Koul <vinod.koul@xxxxxxxxx>
> > 
> > I'd like rather to merge this from my tree, not from asoc tree.
> > Otherwise it makes hard to use this in the legacy HDA driver.
> 
> I have not updated the callers here and not planning to change for now as
> it can be removed I guess once the component framework is available to
> read eld. What do you say?

OK, then you also don't have to put the stuff into the common place,
either, as this stuff won't be used any longer in near future :)


Takashi

> 
> Regards,
> Subhransu
> > 
> > Or make this local in asoc, and clean up later as a common helper.
> > 
> > 
> > thanks,
> > 
> > Takashi
> > 
> > 
> 
> -- 
> 
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux