On Thu, Jun 13, 2024 at 8:16 PM Li, Meng <meng...@windriver.com> wrote:
>
>
>
> > -----Original Message-----
> > From: Bruce Ashfield <bruce.ashfi...@gmail.com>
> > Sent: Friday, June 14, 2024 3:24 AM
> > To: Li, Meng <meng...@windriver.com>
> > Cc: linux-yocto@lists.yoctoproject.org
> > Subject: Re: [linux-yocto]: [kernel/kernel-rt]: usb: dwc3: core: remove 
> > lock of
> > otg mode during gadget suspend/resume to avoid deadlock
> >
> > CAUTION: This email comes from a non Wind River email account!
> > Do not click links or open attachments unless you recognize the sender and
> > know the content is safe.
> >
> > In message: [linux-yocto]: [kernel/kernel-rt]: usb: dwc3: core: remove lock 
> > of
> > otg mode during gadget suspend/resume to avoid deadlock on 13/06/2024
> > Meng Li wrote:
> >
> > > From: Limeng <meng...@windriver.com>
> > >
> > > Hi Bruce,
> > >
> > > This patch is used to fix deadlock issues when testing suspend/resume. I 
> > > will
> > send it to mainline upstream in later.
> > > So, could you please help merge these patches into linux-ycoto kernel in
> > advance, both below 2 branchs?
> > > v6.6/standard/nxp-sdk-6.6/nxp-soc
> > > v6.6/standard/preempt-rt/nxp-sdk-6.6/nxp-soc
> >
> > Given the response of upstream, I'll wait until you've replied and an agreed
> > upon solution is acheived.
> >
>
> Ok! I agree your advice, wait for the upstream response from USB maintainer, 
> and then improve my patch.
> I can reproduce this issue on NXP platforms using usb dwc3 driver, for 
> example, ls1028. And I also verify this patch, suspend/resume feature
> works fine with it.
> I think maybe it needs a long period to review this patch in upstream 
> community, and then accept it finally. Maybe it is not accepted by upstream 
> in worse case.
> So, is it possible to merge this patch into NXP branch, not standard/base 
> branch in advance? In this way, it will not affect other BSP.
>

I haven't checked yet today for any more follow ups on the patch upstream, but
let's give it a few more days.

I definitely don't need the patch accepted upstream to merge it, but if we can
at least hear that the approach is generally correct from the maintainers, I'm
happy to merge the work in progress patch to the nxp branches.

Bruce

> Thanks,
> LImeng
>
> > Bruce
> >
> > >
> > > diffstat info as below:
> > >
> > >  core.c |    6 ------
> > >  1 file changed, 6 deletions(-)
> > >
> > >
> > > thanks,
> > > Limeng



-- 
- Thou shalt not follow the NULL pointer, for chaos and madness await
thee at its end
- "Use the force Harry" - Gandalf, Star Trek II
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#14046): 
https://lists.yoctoproject.org/g/linux-yocto/message/14046
Mute This Topic: https://lists.yoctoproject.org/mt/106647367/21656
Group Owner: linux-yocto+ow...@lists.yoctoproject.org
Unsubscribe: https://lists.yoctoproject.org/g/linux-yocto/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to