Re: [PATCH 6/7] usb: gadget: f_uac2: disable IN/OUT ep if unused

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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.



[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux