[alsa-devel] [PATCH 0/3] make the sgtl5000-codec work
w.sang at pengutronix.de
Tue Jul 19 11:04:04 CEST 2011
> 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 ;)
> But after check soc_cache.c, ASoC mix up index and register address to index cache,
> and many places need to modify to correct it, so I think we should re-consider
> if register_step != 1 is not a common case.
I think it should be properly fixed in ASoC. It should be carefully
done, but otherwise not be a major task IMO. Note that sgtl5000 will
need adaptions nonetheless. Which brings me back to the question: In
what setup did the driver work for you? Did you have regulators on that
board? I am still trying to understand the side-effects of what I am
Mark: Will you pick up patches 1 and 2 nonetheless?
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/da9f7aa8/attachment.sig
More information about the Alsa-devel