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.