On 10/29/2014 05:24 AM, Vinod Koul wrote:
On Wed, Oct 29, 2014 at 10:50:19AM +0800, Greg Kroah-Hartman wrote:
cepted into staging until this is resolved? If you are willing to abide by the staging rules:
- incremental patches only doing one thing at a time
- never break the build
- constantly moving forward to getting merged (i.e. no new features being added)
I think it will be easier for you to do the work outside of the tree as you are going to be changing the API, which is not going to be easy to do in an incremental patch series.
And yes, this isn't going to be a trivial amount of work, sorry.
I am not sure if it can fit into staging model. The whole design of driver would need rework and all the infrastructure added here for exposing stuff to usermode will be thrown away, so this driver would simply need an major overhaul.
If this was ALSA driver and we would need to plumb it for acceptance then would have made sense in include, but thats not the fact today
Thanks
Thanks for all the comments Vinod. Looks like I made a bad judgement, clearly I wouldn't want to rewrite the code twice :) I'll take a closer look at using the framework and maybe start with the features that we know will work and build up the rest incrementally and address problems from there.
Thanks, Qais