[alsa-devel] [PATCH] ASoC: da7210: convert to soc-cache

Ashish Chavan ashish.chavan at kpitcummins.com
Wed Oct 12 13:36:58 CEST 2011


On Wed, 2011-10-12 at 11:47 +0100, Mark Brown wrote:
> On Wed, Oct 12, 2011 at 03:43:35PM +0530, Ashish Chavan wrote:
> > Hi Axel,
> 
> Don't top post and fix your mail client to thread mails properly; I
> didn't actually see your mail until just now due to the lack of
> threading.
> 

Ooops! I didn't actually realize that.

> > I am thinking of going forward with my posting which is based on current
> > version of da7210 mainline driver. In am ready to take the
> > responsibility of updating(to base it on my patch series) and testing
> > your patch on the hardware that we have.
> 
> > Other option is to wait till your changes gets approved and then base my
> > patch series on updated driver with your changes. But as this may take
> > some time and I definitely want to get more eyes on the patches early,
> > this would be least preferred.
> 
> There's no reason you couldn't do both.

Do you mean I can continue posting the patches to get them reviewed and
update them as per Axel's patch after review part is over?

> 
> I have to be honest here and say that even if I'd seen your mail before
> I applied Axel's patch there's no way I'd have held off on it due to
> your unpublished work.  Axel's patch is here now, is a clear win and
> he's got a strong track record of doing this sort of valuable cleanup
> work.  In contrast you guys have been working for something like a year
> to get the driver for one of your PMICs integrated and haven't yet sent
> any patches for audio at all.  Without a track record of contributions
> it's hard to have confidence that the changes you're proposing will
> arrive when you say, or to have any idea about the level of difficulty
> in integrating them.
> 

I agree with you here. what you are saying is correct as per your
experience. But for da7210 codec, I already have the patch series ready
with some confidence.




More information about the Alsa-devel mailing list