lkml.org 
[lkml]   [2010]   [Oct]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRE: [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.



\
 
 \ /
  Last update: 2010-10-14 05:21    [W:1.292 / U:0.408 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site