From: John Keeping <john@metanate.com>
To: Minas Harutyunyan <Minas.Harutyunyan@synopsys.com>
Cc: Felipe Balbi <balbi@kernel.org>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
linux-usb@vger.kernel.org, John Youn <John.Youn@synopsys.com>,
stable <stable@vger.kernel.org>
Subject: Re: [PATCH] usb: dwc2: gadget: Fix ISOC flow for elapsed frames
Date: Fri, 5 Nov 2021 17:27:36 +0000 [thread overview]
Message-ID: <YYVpiEpm+kjDRePK@donbot> (raw)
In-Reply-To: <c356baade6e9716d312d43df08d53ae557cb8037.1636011277.git.Minas.Harutyunyan@synopsys.com>
On Thu, Nov 04, 2021 at 11:36:01AM +0400, Minas Harutyunyan wrote:
> Added updating of request frame number for elapsed frames,
> otherwise frame number will remain as previous use of request.
> This will allow function driver to correctly track frames in
> case of Missed ISOC occurs.
>
> Added setting request actual length to 0 for elapsed frames.
> In Slave mode when pushing data to RxFIFO by dwords, request
> actual length incrementing accordingly. But before whole packet
> will be pushed into RxFIFO and send to host can occurs Missed
> ISOC and data will not send to host. So, in this case request
> actual length should be reset to 0.
>
> Fixes: 91bb163e1e4f ("usb: dwc2: gadget: Fix ISOC flow for BDMA and Slave")
> Signed-off-by: Minas Harutyunyan <Minas.Harutyunyan@synopsys.com>
Reviewed-by: John Keeping <john@metanate.com>
> ---
> drivers/usb/dwc2/gadget.c | 17 ++++++++++++++---
> 1 file changed, 14 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/usb/dwc2/gadget.c b/drivers/usb/dwc2/gadget.c
> index 11d85a6e0b0d..2190225bf3da 100644
> --- a/drivers/usb/dwc2/gadget.c
> +++ b/drivers/usb/dwc2/gadget.c
> @@ -1198,6 +1198,8 @@ static void dwc2_hsotg_start_req(struct dwc2_hsotg *hsotg,
> }
> ctrl |= DXEPCTL_CNAK;
> } else {
> + hs_req->req.frame_number = hs_ep->target_frame;
> + hs_req->req.actual = 0;
> dwc2_hsotg_complete_request(hsotg, hs_ep, hs_req, -ENODATA);
> return;
> }
> @@ -2857,9 +2859,12 @@ static void dwc2_gadget_handle_ep_disabled(struct dwc2_hsotg_ep *hs_ep)
>
> do {
> hs_req = get_ep_head(hs_ep);
> - if (hs_req)
> + if (hs_req) {
> + hs_req->req.frame_number = hs_ep->target_frame;
> + hs_req->req.actual = 0;
> dwc2_hsotg_complete_request(hsotg, hs_ep, hs_req,
> -ENODATA);
> + }
> dwc2_gadget_incr_frame_num(hs_ep);
> /* Update current frame number value. */
> hsotg->frame_number = dwc2_hsotg_read_frameno(hsotg);
> @@ -2912,8 +2917,11 @@ static void dwc2_gadget_handle_out_token_ep_disabled(struct dwc2_hsotg_ep *ep)
>
> while (dwc2_gadget_target_frame_elapsed(ep)) {
> hs_req = get_ep_head(ep);
> - if (hs_req)
> + if (hs_req) {
> + hs_req->req.frame_number = ep->target_frame;
> + hs_req->req.actual = 0;
> dwc2_hsotg_complete_request(hsotg, ep, hs_req, -ENODATA);
> + }
>
> dwc2_gadget_incr_frame_num(ep);
> /* Update current frame number value. */
> @@ -3002,8 +3010,11 @@ static void dwc2_gadget_handle_nak(struct dwc2_hsotg_ep *hs_ep)
>
> while (dwc2_gadget_target_frame_elapsed(hs_ep)) {
> hs_req = get_ep_head(hs_ep);
> - if (hs_req)
> + if (hs_req) {
> + hs_req->req.frame_number = hs_ep->target_frame;
> + hs_req->req.actual = 0;
> dwc2_hsotg_complete_request(hsotg, hs_ep, hs_req, -ENODATA);
> + }
>
> dwc2_gadget_incr_frame_num(hs_ep);
> /* Update current frame number value. */
>
> base-commit: c26f1c109d21f2ea874e4a85c0c76c385b8f46cb
> --
> 2.11.0
>
prev parent reply other threads:[~2021-11-05 17:27 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-11-04 7:36 [PATCH] usb: dwc2: gadget: Fix ISOC flow for elapsed frames Minas Harutyunyan
2021-11-05 17:27 ` John Keeping [this message]
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=YYVpiEpm+kjDRePK@donbot \
--to=john@metanate.com \
--cc=John.Youn@synopsys.com \
--cc=Minas.Harutyunyan@synopsys.com \
--cc=balbi@kernel.org \
--cc=gregkh@linuxfoundation.org \
--cc=linux-usb@vger.kernel.org \
--cc=stable@vger.kernel.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox