Hi Felipe, On Tue, Aug 07, 2018 at 08:48:53PM +0200, Eugeniu Rosca wrote: > Hello Felipe, > > On Fri, Jul 27, 2018 at 08:57:29PM +0200, Eugeniu Rosca wrote: > > Hi Felipe, > > > > On Thu, Jul 26, 2018 at 01:51:35PM +0300, Felipe Balbi wrote: > > > Eugeniu Rosca <erosca@xxxxxxxxxxxxxx> writes: > > > > > > > From: Andreas Pape <apape@xxxxxxxxxxxxxx> > > > > > > > > Via p_chmask/c_chmask the user can define whether uac2 shall support > > > > playback and/or capture. This has only effect on the created ALSA device, > > > > but not on the USB descriptor. This patch adds playback/capture descriptors > > > > dependent on that parameter. > > > > > > > > Signed-off-by: Andreas Pape <apape@xxxxxxxxxxxxxx> > > > > Signed-off-by: Eugeniu Rosca <erosca@xxxxxxxxxxxxxx> > > > > > > unfortunately doesn't apply: > > > > > > checking file drivers/usb/gadget/function/f_uac2.c > > > Hunk #13 FAILED at 678. > > > 1 out of 14 hunks FAILED > > > > > > Please rebase on testing/next after a couple hours > > > > I've checked the reason for which you are experiencing the conflict and > > what I can see is that your testing/next branch [1] (currently based on > > v4.18-rc6) lacks two uac2 fixes which already reached vanilla master via > > v4.18-rc6+ merge commit [2]: > > > > $ git log --oneline --right-only --cherry-pick \ > > balbi/testing/next..linux/master drivers/usb/gadget/function/f_uac2.c > > eec24f2a0d4d usb: gadget: f_uac2: fix endianness of 'struct cntrl_*_lay3' > > e87581fe0509 usb: gadget: f_uac2: fix error handling in afunc_bind (again) > > > > So, the conflicts are caused by an older state of testing/next > > relative to mainline. If I rebase [3] on top of [1], you will then face > > additional conflicts when rebasing [1] on top of v4.18. > > > > Let me know if we are on the same page. > > > > Best regards, > > Eugeniu. > > > > [1] https://git.kernel.org/pub/scm/linux/kernel/git/balbi/usb.git/log/?h=testing/next > > [2] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=cd3f77d74ac3 > > [3] https://patchwork.kernel.org/patch/10480073/ > > I can see that the testing/next branch is still based on v4.18-rc6. > Could you please feedback if you are going to rebase it to a later -rc, > so that patch "[6/7] usb: gadget: f_uac2: disable IN/OUT ep if unused" > from my series applies without conflicts? This patch (i.e. [3] above) now applies cleanly on top of your testing/next branch (since the latter is rebased against v4.19-rc1). Since there are no conflicts anymore, I hope you can pick the patch. Many thanks in advance. > Best regards, > Eugeniu.