From: John Keeping <john@metanate.com>
To: Pavel Hofman <pavel.hofman@ivitera.com>
Cc: linux-usb@vger.kernel.org, Felipe Balbi <balbi@kernel.org>
Subject: Re: [PATCH] usb: gadget: u_audio: Fix high-speed max packet size
Date: Fri, 31 Jan 2020 11:27:33 +0000 [thread overview]
Message-ID: <20200131112733.758ab77c.john@metanate.com> (raw)
In-Reply-To: <4588dcf7-7e2a-8bdd-48e5-e835f3503f5a@ivitera.com>
On Fri, 31 Jan 2020 11:30:11 +0100
Pavel Hofman <pavel.hofman@ivitera.com> wrote:
> Dne 24. 01. 20 v 13:16 Pavel Hofman napsal(a):
> >
> > Dne 19. 01. 20 v 15:53 Pavel Hofman napsal(a):
> >>
> >> Dne 17. 01. 20 v 11:40 John Keeping napsal(a):
> >>> On Thu, 16 Jan 2020 16:39:50 +0100
> >>> Pavel Hofman <pavel.hofman@ivitera.com> wrote:
> >>>
> >>>>> I've taken a look at this and the patch below fixes it in my simple
> >>>>> testing. But note that this doesn't adjust the PCM's min_period_bytes
> >>>>> which will be necessary if you want to minimize latency with an
> >>>>> adjusted
> >>>>> high-speed bInterval setting.
> >>>>
> >>>> Please can I ask you to submit your patch? IMO your perhaps slightly
> >>>> suboptimal solution is much better than the current broken version.
> >>>
> >>> Yes, the patch is definitely an improvement. I thought it would be
> >>> picked up from the earlier mail, but I think Patchwork requires the
> >>> subject to match, so I'm including it again here.
> >>>
> >>> Are you able to provide a Tested-by for this change?
> >>
> >>
> >> Testing looks OK, thanks a lot!
> >>
> >> Tested-by: Pavel Hofman <pavel.hofman@ivitera.com>
> >>
> >
>
> I apologize for a basic question - please which official repository to
> check status of a gadget patch after being accepted? Thanks a lot for
> the information.
If you have a kernel tree, you can ask the MAINTAINERS file:
./scripts/get_maintainer.pl --scm -f drivers/usb/gadget/function/u_audio.c
I'd expect this to appear in Felipe's tree first at:
https://git.kernel.org/pub/scm/linux/kernel/git/balbi/usb.git
but I don't see it yet. I guess it won't be picked up until after the
merge window.
Regards,
John
next prev parent reply other threads:[~2020-01-31 11:27 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-01-10 7:29 USB:UAC2: Incorrect req->length > maxpacket*mc Pavel Hofman
2020-01-10 11:28 ` [PATCH] usb: gadget: f_uac2: fix packet size calculation John Keeping
2020-01-11 9:12 ` Pavel Hofman
2020-01-11 9:31 ` USB:UAC2: Incorrect req->length > maxpacket*mc - cause likely found Pavel Hofman
2020-01-11 9:49 ` Pavel Hofman
2020-01-14 18:39 ` Pavel Hofman
2020-01-14 20:04 ` John Keeping
2020-01-14 20:21 ` Pavel Hofman
2020-01-16 15:39 ` Pavel Hofman
2020-01-17 10:40 ` [PATCH] usb: gadget: u_audio: Fix high-speed max packet size John Keeping
2020-01-19 14:53 ` Pavel Hofman
2020-01-24 12:16 ` Pavel Hofman
2020-01-31 10:30 ` Pavel Hofman
2020-01-31 11:27 ` John Keeping [this message]
2020-01-31 12:47 ` Pavel Hofman
2020-01-31 13:09 ` Felipe Balbi
2020-01-24 12:52 ` Felipe Balbi
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200131112733.758ab77c.john@metanate.com \
--to=john@metanate.com \
--cc=balbi@kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=pavel.hofman@ivitera.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.