[alsa-devel] [PATCH 0/3] make the sgtl5000-codec work
w.sang at pengutronix.de
Tue Jul 19 18:46:39 CEST 2011
On Tue, Jul 19, 2011 at 04:13:37PM +0100, Mark Brown wrote:
> On Tue, Jul 19, 2011 at 11:04:04AM +0200, Wolfram Sang wrote:
> > > pls check http://comments.gmane.org/gmane.linux.alsa.devel/83781
> > >
> > > I think Mark is right, sgtl5000 already declared its register_step is 2, soc_cache.c
> > > should generate a dense cache layout instead of padding it by driver.
> > Thanks for the pointer. I can follow Mark's reasoning. In fact, I was
> > wondering why ASoC does not consider the step, but I assumed it was
> > intentional. With my holidays coming along, nothing I am going to
> > tackle in the next time, though ;)
> The only reason steps are ever used is for AC'97 devices which don't use
> the generic cache code as they're pretty much obsolete so the framework
> support is mostly just carried around as legacy. The step size is used
> in the cache display code which does use it. If someone was interested
> enough to convert AC'97 over that'd be nice.
I am a bit confused now: Do you want to retire the step size in the long run?
Then my sgtl-patch might be the right approach after all.
Pengutronix e.K. | Wolfram Sang |
Industrial Linux Solutions | http://www.pengutronix.de/ |
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 198 bytes
Desc: Digital signature
Url : http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20110719/6128e02b/attachment.sig
More information about the Alsa-devel