Wed, 30 Apr 2014 15:46:50 +0100 от Lee Jones <[email protected]>:
> > > > > > > > > > > This patch adds devicetree support for mc13783-codec.
> > > > > > > > > > 
> > > > > > > > > > Applied, thanks.
> > > > > > > > > 
> > > > > > > > > No Mark, no!
> > > > > > > > 
> > > > > > > > What a problem, Lee?
> > > > > > > 
> > > > > > > There's no problem with the patch, don't worry.
> > > > > > > 
> > > > > > > Neither Mark, nor I can apply this patch independently, we need to
> > > > > > > create a shared branch somewhere, or we'll risk conflicts at merge
> > > > > > > time.
> > > > > > 
> > > > > > I can separate this patch for ASoC and MFD if need.
> > > > > > But this will work only if both is applied.
> > > > > 
> > > > > Are there build dependencies, or only functional ones?
> > > > 
> > > > Functional only.
> > > > At this time "fsl,mc13xxx-uses-codec" option for PMIC produce a kernel 
> > > > crash :)
> > > 
> > > If patches can be broken up notwithstanding build errors, they should
> > > be.  It makes things sooo much easier.
> > 
> > In fact that the patch for MFD is not obvious if the codec have no DT 
> > support...
> > 
> > If you do we see this problem (pdata == NULL and we pass this to the
> > driver in DT case), for me it not worst to separate this. What we do?
> > Should this be a two patches in one series or one for MFD and one for ASoC 
> > then?
> 
> You can put them into a patch-set and I will take the MFD one, Mark
> will take the other.

OK. Mark, please revert this patch then.
I 'll send new version tomorrow..

---

Reply via email to