Messages in this thread | | | From | Peter Hsiang <> | Date | Wed, 13 Oct 2010 20:18:44 -0700 | Subject | RE: [PATCH] ASoC: Add max98088 CODEC driver |
| |
On Wed, Oct 13, 2010, Mark Brown wrote: > On Tue, Oct 12, 2010 at 06:20:10PM -0700, Peter Hsiang wrote: > > This looks pretty good, waiting for Liam's review but I think we can do > further stuff incrementally. Minor comments: > > > + > > + /* Setup an array of texts for the equalizer enum. > > + * This is based on Mark Brown's equalizer driver code. > > + * It has been extended to support multiple equalizers. > > + */ > > The code that you're basing this on supports multiple equalizers too - > things like the WM8994 have several. Given that all those supported by > the original code are identical there didn't seem to be any point in > restricting which EQ a given setup can be deployed on - it's just one > more thing that can go wrong.
Thanks - that is a very good idea. I have implemented it and will submit it shortly.
| |