Wed, 30 Apr 2014 15:46:50 +0100 от Lee Jones <lee.jones@xxxxxxxxxx>: > > > > > > > > > > > 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.. --- ��.n��������+%������w��{.n����z�{��ܨ}���Ơz�j:+v�����w����ޙ��&�)ߡ�a����z�ޗ���ݢj��w�f