* [PATCH 1/3] usb: mtu3: fix interval value for intr and isoc @ 2021-12-09 3:14 Chunfeng Yun 2021-12-09 3:14 ` [PATCH 2/3] usb: mtu3: add memory barrier before set GPD's HWO Chunfeng Yun ` (2 more replies) 0 siblings, 3 replies; 11+ messages in thread From: Chunfeng Yun @ 2021-12-09 3:14 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Chunfeng Yun, Matthias Brugger, linux-usb, linux-arm-kernel, linux-mediatek, linux-kernel, Eddie Hung, Yuwen Ng Use the Interval value from isoc/intr endpoint descriptor, no need minus one. But the original code doesn't cause transfer error for normal cases, due to the interval is less than the host request. Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com> --- drivers/usb/mtu3/mtu3_gadget.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/usb/mtu3/mtu3_gadget.c b/drivers/usb/mtu3/mtu3_gadget.c index a9a65b4bbfed..c51be015345b 100644 --- a/drivers/usb/mtu3/mtu3_gadget.c +++ b/drivers/usb/mtu3/mtu3_gadget.c @@ -77,7 +77,7 @@ static int mtu3_ep_enable(struct mtu3_ep *mep) if (usb_endpoint_xfer_int(desc) || usb_endpoint_xfer_isoc(desc)) { interval = desc->bInterval; - interval = clamp_val(interval, 1, 16) - 1; + interval = clamp_val(interval, 1, 16); if (usb_endpoint_xfer_isoc(desc) && comp_desc) mult = comp_desc->bmAttributes; } @@ -89,7 +89,7 @@ static int mtu3_ep_enable(struct mtu3_ep *mep) if (usb_endpoint_xfer_isoc(desc) || usb_endpoint_xfer_int(desc)) { interval = desc->bInterval; - interval = clamp_val(interval, 1, 16) - 1; + interval = clamp_val(interval, 1, 16); mult = usb_endpoint_maxp_mult(desc) - 1; } break; -- 2.18.0 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* [PATCH 2/3] usb: mtu3: add memory barrier before set GPD's HWO 2021-12-09 3:14 [PATCH 1/3] usb: mtu3: fix interval value for intr and isoc Chunfeng Yun @ 2021-12-09 3:14 ` Chunfeng Yun 2021-12-13 14:18 ` Greg Kroah-Hartman 2021-12-09 3:14 ` [PATCH 3/3] usb: mtu3: fix list_head check warning Chunfeng Yun 2021-12-13 14:20 ` [PATCH 1/3] usb: mtu3: fix interval value for intr and isoc Greg Kroah-Hartman 2 siblings, 1 reply; 11+ messages in thread From: Chunfeng Yun @ 2021-12-09 3:14 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Chunfeng Yun, Matthias Brugger, linux-usb, linux-arm-kernel, linux-mediatek, linux-kernel, Eddie Hung, Yuwen Ng, stable There is a seldom issue that the controller access invalid address and trigger devapc or emimpu violation. That is due to memory access is out of order and cause gpd data is not correct. Make sure GPD is fully written before giving it to HW by setting its HWO. Fixes: 48e0d3735aa5 ("usb: mtu3: supports new QMU format") Cc: stable@vger.kernel.org Reported-by: Eddie Hung <eddie.hung@mediatek.com> Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com> --- drivers/usb/mtu3/mtu3_qmu.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/drivers/usb/mtu3/mtu3_qmu.c b/drivers/usb/mtu3/mtu3_qmu.c index 3f414f91b589..34bb5ac67efe 100644 --- a/drivers/usb/mtu3/mtu3_qmu.c +++ b/drivers/usb/mtu3/mtu3_qmu.c @@ -273,6 +273,8 @@ static int mtu3_prepare_tx_gpd(struct mtu3_ep *mep, struct mtu3_request *mreq) gpd->dw3_info |= cpu_to_le32(GPD_EXT_FLAG_ZLP); } + /* make sure GPD is fully written before giving it to HW */ + mb(); gpd->dw0_info |= cpu_to_le32(GPD_FLAGS_IOC | GPD_FLAGS_HWO); mreq->gpd = gpd; @@ -306,6 +308,8 @@ static int mtu3_prepare_rx_gpd(struct mtu3_ep *mep, struct mtu3_request *mreq) gpd->next_gpd = cpu_to_le32(lower_32_bits(enq_dma)); ext_addr |= GPD_EXT_NGP(mtu, upper_32_bits(enq_dma)); gpd->dw3_info = cpu_to_le32(ext_addr); + /* make sure GPD is fully written before giving it to HW */ + mb(); gpd->dw0_info |= cpu_to_le32(GPD_FLAGS_IOC | GPD_FLAGS_HWO); mreq->gpd = gpd; @@ -445,7 +449,8 @@ static void qmu_tx_zlp_error_handler(struct mtu3 *mtu, u8 epnum) return; } mtu3_setbits(mbase, MU3D_EP_TXCR0(mep->epnum), TX_TXPKTRDY); - + /* make sure GPD is fully written before giving it to HW */ + mb(); /* by pass the current GDP */ gpd_current->dw0_info |= cpu_to_le32(GPD_FLAGS_BPS | GPD_FLAGS_HWO); -- 2.18.0 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH 2/3] usb: mtu3: add memory barrier before set GPD's HWO 2021-12-09 3:14 ` [PATCH 2/3] usb: mtu3: add memory barrier before set GPD's HWO Chunfeng Yun @ 2021-12-13 14:18 ` Greg Kroah-Hartman 2021-12-16 8:32 ` Chunfeng Yun 0 siblings, 1 reply; 11+ messages in thread From: Greg Kroah-Hartman @ 2021-12-13 14:18 UTC (permalink / raw) To: Chunfeng Yun Cc: Matthias Brugger, linux-usb, linux-arm-kernel, linux-mediatek, linux-kernel, Eddie Hung, Yuwen Ng, stable On Thu, Dec 09, 2021 at 11:14:23AM +0800, Chunfeng Yun wrote: > There is a seldom issue that the controller access invalid address > and trigger devapc or emimpu violation. That is due to memory access > is out of order and cause gpd data is not correct. > Make sure GPD is fully written before giving it to HW by setting its > HWO. > > Fixes: 48e0d3735aa5 ("usb: mtu3: supports new QMU format") > Cc: stable@vger.kernel.org > Reported-by: Eddie Hung <eddie.hung@mediatek.com> > Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com> > --- > drivers/usb/mtu3/mtu3_qmu.c | 7 ++++++- > 1 file changed, 6 insertions(+), 1 deletion(-) > > diff --git a/drivers/usb/mtu3/mtu3_qmu.c b/drivers/usb/mtu3/mtu3_qmu.c > index 3f414f91b589..34bb5ac67efe 100644 > --- a/drivers/usb/mtu3/mtu3_qmu.c > +++ b/drivers/usb/mtu3/mtu3_qmu.c > @@ -273,6 +273,8 @@ static int mtu3_prepare_tx_gpd(struct mtu3_ep *mep, struct mtu3_request *mreq) > gpd->dw3_info |= cpu_to_le32(GPD_EXT_FLAG_ZLP); > } > > + /* make sure GPD is fully written before giving it to HW */ > + mb(); So this means you are using mmio for this structure? If so, shouldn't you be using normal io memory read/write calls as well and not just "raw" pointers like this: > gpd->dw0_info |= cpu_to_le32(GPD_FLAGS_IOC | GPD_FLAGS_HWO); Are you sure this is ok? Sprinkling around mb() calls is almost never the correct solution. If you need to ensure that a write succeeds, shouldn't you do a read from it afterward? Many busses require this, doesn't yours? > > mreq->gpd = gpd; > @@ -306,6 +308,8 @@ static int mtu3_prepare_rx_gpd(struct mtu3_ep *mep, struct mtu3_request *mreq) > gpd->next_gpd = cpu_to_le32(lower_32_bits(enq_dma)); > ext_addr |= GPD_EXT_NGP(mtu, upper_32_bits(enq_dma)); > gpd->dw3_info = cpu_to_le32(ext_addr); > + /* make sure GPD is fully written before giving it to HW */ > + mb(); Again, mb(); does not ensure that memory-mapped i/o actually hits the HW. Or if it does on your platform, how? mb() is a compiler barrier, not a memory write to a bus barrier. Please read Documentation/memory-barriers.txt for more details. thanks, greg k-h ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 2/3] usb: mtu3: add memory barrier before set GPD's HWO 2021-12-13 14:18 ` Greg Kroah-Hartman @ 2021-12-16 8:32 ` Chunfeng Yun 0 siblings, 0 replies; 11+ messages in thread From: Chunfeng Yun @ 2021-12-16 8:32 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Matthias Brugger, linux-usb, linux-arm-kernel, linux-mediatek, linux-kernel, Eddie Hung, Yuwen Ng, stable On Mon, 2021-12-13 at 15:18 +0100, Greg Kroah-Hartman wrote: > On Thu, Dec 09, 2021 at 11:14:23AM +0800, Chunfeng Yun wrote: > > There is a seldom issue that the controller access invalid address > > and trigger devapc or emimpu violation. That is due to memory > > access > > is out of order and cause gpd data is not correct. > > Make sure GPD is fully written before giving it to HW by setting > > its > > HWO. > > > > Fixes: 48e0d3735aa5 ("usb: mtu3: supports new QMU format") > > Cc: stable@vger.kernel.org > > Reported-by: Eddie Hung <eddie.hung@mediatek.com> > > Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com> > > --- > > drivers/usb/mtu3/mtu3_qmu.c | 7 ++++++- > > 1 file changed, 6 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/usb/mtu3/mtu3_qmu.c > > b/drivers/usb/mtu3/mtu3_qmu.c > > index 3f414f91b589..34bb5ac67efe 100644 > > --- a/drivers/usb/mtu3/mtu3_qmu.c > > +++ b/drivers/usb/mtu3/mtu3_qmu.c > > @@ -273,6 +273,8 @@ static int mtu3_prepare_tx_gpd(struct mtu3_ep > > *mep, struct mtu3_request *mreq) > > gpd->dw3_info |= cpu_to_le32(GPD_EXT_FLAG_ZLP); > > } > > > > + /* make sure GPD is fully written before giving it to HW */ > > + mb(); > > So this means you are using mmio for this structure? No, it's a noncached memory. > If so, shouldn't > you be using normal io memory read/write calls as well and not just > "raw" pointers like this: > > > gpd->dw0_info |= cpu_to_le32(GPD_FLAGS_IOC | GPD_FLAGS_HWO); > > Are you sure this is ok? > > Sprinkling around mb() calls is almost never the correct solution. > > If you need to ensure that a write succeeds, shouldn't you do a read > from it afterward? Many busses require this, doesn't yours? It works for register access. Here is noncache memory access, add mb(), just want to prohibite both the compiler and CPU from reordering read/writes. > > > > > > > mreq->gpd = gpd; > > @@ -306,6 +308,8 @@ static int mtu3_prepare_rx_gpd(struct mtu3_ep > > *mep, struct mtu3_request *mreq) > > gpd->next_gpd = cpu_to_le32(lower_32_bits(enq_dma)); > > ext_addr |= GPD_EXT_NGP(mtu, upper_32_bits(enq_dma)); > > gpd->dw3_info = cpu_to_le32(ext_addr); > > + /* make sure GPD is fully written before giving it to HW */ > > + mb(); > > Again, mb(); does not ensure that memory-mapped i/o actually hits the > HW. Or if it does on your platform, how? Maybe the comment is misleading, I'll change it, here just want to prevent reordering of compiler and cpu. > > mb() is a compiler barrier, not a memory write to a bus > barrier. Please > read Documentation/memory-barriers.txt for more details. Ok, I'll do. Thanks a lot > > thanks, > > greg k-h ^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH 3/3] usb: mtu3: fix list_head check warning 2021-12-09 3:14 [PATCH 1/3] usb: mtu3: fix interval value for intr and isoc Chunfeng Yun 2021-12-09 3:14 ` [PATCH 2/3] usb: mtu3: add memory barrier before set GPD's HWO Chunfeng Yun @ 2021-12-09 3:14 ` Chunfeng Yun 2021-12-09 9:10 ` Sergey Shtylyov 2021-12-13 14:19 ` Greg Kroah-Hartman 2021-12-13 14:20 ` [PATCH 1/3] usb: mtu3: fix interval value for intr and isoc Greg Kroah-Hartman 2 siblings, 2 replies; 11+ messages in thread From: Chunfeng Yun @ 2021-12-09 3:14 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Chunfeng Yun, Matthias Brugger, linux-usb, linux-arm-kernel, linux-mediatek, linux-kernel, Eddie Hung, Yuwen Ng This is caused by uninitialization of list_head. BUG: KASAN: use-after-free in __list_del_entry_valid+0x34/0xe4 Call trace: dump_backtrace+0x0/0x298 show_stack+0x24/0x34 dump_stack+0x130/0x1a8 print_address_description+0x88/0x56c __kasan_report+0x1b8/0x2a0 kasan_report+0x14/0x20 __asan_load8+0x9c/0xa0 __list_del_entry_valid+0x34/0xe4 mtu3_req_complete+0x4c/0x300 [mtu3] mtu3_gadget_stop+0x168/0x448 [mtu3] usb_gadget_unregister_driver+0x204/0x3a0 unregister_gadget_item+0x44/0xa4 Reported-by: Yuwen Ng <yuwen.ng@mediatek.com> Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com> --- drivers/usb/mtu3/mtu3_gadget.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/usb/mtu3/mtu3_gadget.c b/drivers/usb/mtu3/mtu3_gadget.c index c51be015345b..b6c8a4a99c4d 100644 --- a/drivers/usb/mtu3/mtu3_gadget.c +++ b/drivers/usb/mtu3/mtu3_gadget.c @@ -235,6 +235,7 @@ struct usb_request *mtu3_alloc_request(struct usb_ep *ep, gfp_t gfp_flags) mreq->request.dma = DMA_ADDR_INVALID; mreq->epnum = mep->epnum; mreq->mep = mep; + INIT_LIST_HEAD(&mreq->list); trace_mtu3_alloc_request(mreq); return &mreq->request; -- 2.18.0 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH 3/3] usb: mtu3: fix list_head check warning 2021-12-09 3:14 ` [PATCH 3/3] usb: mtu3: fix list_head check warning Chunfeng Yun @ 2021-12-09 9:10 ` Sergey Shtylyov 2021-12-10 1:19 ` Chunfeng Yun 2021-12-13 14:19 ` Greg Kroah-Hartman 1 sibling, 1 reply; 11+ messages in thread From: Sergey Shtylyov @ 2021-12-09 9:10 UTC (permalink / raw) To: Chunfeng Yun, Greg Kroah-Hartman Cc: Matthias Brugger, linux-usb, linux-arm-kernel, linux-mediatek, linux-kernel, Eddie Hung, Yuwen Ng Hello! On 09.12.2021 6:14, Chunfeng Yun wrote: > This is caused by uninitialization of list_head. No such word, suggesting to replace with "not initializing". :-) > BUG: KASAN: use-after-free in __list_del_entry_valid+0x34/0xe4 > > Call trace: > dump_backtrace+0x0/0x298 > show_stack+0x24/0x34 > dump_stack+0x130/0x1a8 > print_address_description+0x88/0x56c > __kasan_report+0x1b8/0x2a0 > kasan_report+0x14/0x20 > __asan_load8+0x9c/0xa0 > __list_del_entry_valid+0x34/0xe4 > mtu3_req_complete+0x4c/0x300 [mtu3] > mtu3_gadget_stop+0x168/0x448 [mtu3] > usb_gadget_unregister_driver+0x204/0x3a0 > unregister_gadget_item+0x44/0xa4 > > Reported-by: Yuwen Ng <yuwen.ng@mediatek.com> > Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com> [...] MBR, Sergey ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 3/3] usb: mtu3: fix list_head check warning 2021-12-09 9:10 ` Sergey Shtylyov @ 2021-12-10 1:19 ` Chunfeng Yun 0 siblings, 0 replies; 11+ messages in thread From: Chunfeng Yun @ 2021-12-10 1:19 UTC (permalink / raw) To: Sergey Shtylyov, Greg Kroah-Hartman Cc: Matthias Brugger, linux-usb, linux-arm-kernel, linux-mediatek, linux-kernel, Eddie Hung, Yuwen Ng On Thu, 2021-12-09 at 12:10 +0300, Sergey Shtylyov wrote: > Hello! > > On 09.12.2021 6:14, Chunfeng Yun wrote: > > > This is caused by uninitialization of list_head. > > No such word, suggesting to replace with "not initializing". :-) Will fix it, thanks > > > BUG: KASAN: use-after-free in __list_del_entry_valid+0x34/0xe4 > > > > Call trace: > > dump_backtrace+0x0/0x298 > > show_stack+0x24/0x34 > > dump_stack+0x130/0x1a8 > > print_address_description+0x88/0x56c > > __kasan_report+0x1b8/0x2a0 > > kasan_report+0x14/0x20 > > __asan_load8+0x9c/0xa0 > > __list_del_entry_valid+0x34/0xe4 > > mtu3_req_complete+0x4c/0x300 [mtu3] > > mtu3_gadget_stop+0x168/0x448 [mtu3] > > usb_gadget_unregister_driver+0x204/0x3a0 > > unregister_gadget_item+0x44/0xa4 > > > > Reported-by: Yuwen Ng <yuwen.ng@mediatek.com> > > Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com> > > [...] > > MBR, Sergey ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 3/3] usb: mtu3: fix list_head check warning 2021-12-09 3:14 ` [PATCH 3/3] usb: mtu3: fix list_head check warning Chunfeng Yun 2021-12-09 9:10 ` Sergey Shtylyov @ 2021-12-13 14:19 ` Greg Kroah-Hartman 2021-12-16 8:43 ` Chunfeng Yun 1 sibling, 1 reply; 11+ messages in thread From: Greg Kroah-Hartman @ 2021-12-13 14:19 UTC (permalink / raw) To: Chunfeng Yun Cc: Matthias Brugger, linux-usb, linux-arm-kernel, linux-mediatek, linux-kernel, Eddie Hung, Yuwen Ng On Thu, Dec 09, 2021 at 11:14:24AM +0800, Chunfeng Yun wrote: > This is caused by uninitialization of list_head. > > BUG: KASAN: use-after-free in __list_del_entry_valid+0x34/0xe4 > > Call trace: > dump_backtrace+0x0/0x298 > show_stack+0x24/0x34 > dump_stack+0x130/0x1a8 > print_address_description+0x88/0x56c > __kasan_report+0x1b8/0x2a0 > kasan_report+0x14/0x20 > __asan_load8+0x9c/0xa0 > __list_del_entry_valid+0x34/0xe4 > mtu3_req_complete+0x4c/0x300 [mtu3] > mtu3_gadget_stop+0x168/0x448 [mtu3] > usb_gadget_unregister_driver+0x204/0x3a0 > unregister_gadget_item+0x44/0xa4 > > Reported-by: Yuwen Ng <yuwen.ng@mediatek.com> > Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com> > --- > drivers/usb/mtu3/mtu3_gadget.c | 1 + > 1 file changed, 1 insertion(+) What commit does this fix? Should it go to stable kernels? thanks, greg k-h ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 3/3] usb: mtu3: fix list_head check warning 2021-12-13 14:19 ` Greg Kroah-Hartman @ 2021-12-16 8:43 ` Chunfeng Yun 0 siblings, 0 replies; 11+ messages in thread From: Chunfeng Yun @ 2021-12-16 8:43 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Matthias Brugger, linux-usb, linux-arm-kernel, linux-mediatek, linux-kernel, Eddie Hung, Yuwen Ng On Mon, 2021-12-13 at 15:19 +0100, Greg Kroah-Hartman wrote: > On Thu, Dec 09, 2021 at 11:14:24AM +0800, Chunfeng Yun wrote: > > This is caused by uninitialization of list_head. > > > > BUG: KASAN: use-after-free in __list_del_entry_valid+0x34/0xe4 > > > > Call trace: > > dump_backtrace+0x0/0x298 > > show_stack+0x24/0x34 > > dump_stack+0x130/0x1a8 > > print_address_description+0x88/0x56c > > __kasan_report+0x1b8/0x2a0 > > kasan_report+0x14/0x20 > > __asan_load8+0x9c/0xa0 > > __list_del_entry_valid+0x34/0xe4 > > mtu3_req_complete+0x4c/0x300 [mtu3] > > mtu3_gadget_stop+0x168/0x448 [mtu3] > > usb_gadget_unregister_driver+0x204/0x3a0 > > unregister_gadget_item+0x44/0xa4 > > > > Reported-by: Yuwen Ng <yuwen.ng@mediatek.com> > > Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com> > > --- > > drivers/usb/mtu3/mtu3_gadget.c | 1 + > > 1 file changed, 1 insertion(+) > > What commit does this fix? Should it go to stable kernels? I add it in next version, thanks > > thanks, > > greg k-h ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 1/3] usb: mtu3: fix interval value for intr and isoc 2021-12-09 3:14 [PATCH 1/3] usb: mtu3: fix interval value for intr and isoc Chunfeng Yun 2021-12-09 3:14 ` [PATCH 2/3] usb: mtu3: add memory barrier before set GPD's HWO Chunfeng Yun 2021-12-09 3:14 ` [PATCH 3/3] usb: mtu3: fix list_head check warning Chunfeng Yun @ 2021-12-13 14:20 ` Greg Kroah-Hartman 2021-12-16 8:45 ` Chunfeng Yun 2 siblings, 1 reply; 11+ messages in thread From: Greg Kroah-Hartman @ 2021-12-13 14:20 UTC (permalink / raw) To: Chunfeng Yun Cc: Matthias Brugger, linux-usb, linux-arm-kernel, linux-mediatek, linux-kernel, Eddie Hung, Yuwen Ng On Thu, Dec 09, 2021 at 11:14:22AM +0800, Chunfeng Yun wrote: > Use the Interval value from isoc/intr endpoint descriptor, no need > minus one. But the original code doesn't cause transfer error for > normal cases, due to the interval is less than the host request. > > Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com> > --- > drivers/usb/mtu3/mtu3_gadget.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) What commit does this fix? ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH 1/3] usb: mtu3: fix interval value for intr and isoc 2021-12-13 14:20 ` [PATCH 1/3] usb: mtu3: fix interval value for intr and isoc Greg Kroah-Hartman @ 2021-12-16 8:45 ` Chunfeng Yun 0 siblings, 0 replies; 11+ messages in thread From: Chunfeng Yun @ 2021-12-16 8:45 UTC (permalink / raw) To: Greg Kroah-Hartman Cc: Matthias Brugger, linux-usb, linux-arm-kernel, linux-mediatek, linux-kernel, Eddie Hung, Yuwen Ng On Mon, 2021-12-13 at 15:20 +0100, Greg Kroah-Hartman wrote: > On Thu, Dec 09, 2021 at 11:14:22AM +0800, Chunfeng Yun wrote: > > Use the Interval value from isoc/intr endpoint descriptor, no need > > minus one. But the original code doesn't cause transfer error for > > normal cases, due to the interval is less than the host request. > > > > Signed-off-by: Chunfeng Yun <chunfeng.yun@mediatek.com> > > --- > > drivers/usb/mtu3/mtu3_gadget.c | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > What commit does this fix? The interval between transfers is less than the Interval value, I add it in commit massage when send out v2. Thanks ^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2021-12-16 8:45 UTC | newest] Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-12-09 3:14 [PATCH 1/3] usb: mtu3: fix interval value for intr and isoc Chunfeng Yun 2021-12-09 3:14 ` [PATCH 2/3] usb: mtu3: add memory barrier before set GPD's HWO Chunfeng Yun 2021-12-13 14:18 ` Greg Kroah-Hartman 2021-12-16 8:32 ` Chunfeng Yun 2021-12-09 3:14 ` [PATCH 3/3] usb: mtu3: fix list_head check warning Chunfeng Yun 2021-12-09 9:10 ` Sergey Shtylyov 2021-12-10 1:19 ` Chunfeng Yun 2021-12-13 14:19 ` Greg Kroah-Hartman 2021-12-16 8:43 ` Chunfeng Yun 2021-12-13 14:20 ` [PATCH 1/3] usb: mtu3: fix interval value for intr and isoc Greg Kroah-Hartman 2021-12-16 8:45 ` Chunfeng Yun
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).