| Message ID | 20231026-dwc3-v2-1-1d4fd5c3e067@chromium.org (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 8bbae288a85abed6a1cf7d185d8b9dc2f5dcb12c |
| Headers | show |
| Series | [v2] usb: dwc3: set the dma max_seg_size | expand |
On Fri, Oct 27, 2023, Ricardo Ribalda wrote: > Allow devices to have dma operations beyond 4K, and avoid warnings such > as: > > DMA-API: dwc3 a600000.usb: mapping sg segment longer than device claims to support [len=86016] [max=65536] > > Cc: stable@vger.kernel.org > Fixes: 72246da40f37 ("usb: Introduce DesignWare USB3 DRD Driver") > Reported-by: Zubin Mithra <zsm@chromium.org> > Signed-off-by: Ricardo Ribalda <ribalda@chromium.org> > --- > Found while running > yavta -f YUYV -s 1280x720 -c /dev/video0 > > with: > > CONFIG_DMA_API_DEBUG=y > --- > Changes in v2: > - Add stable tag > - Link to v1: https://urldefense.com/v3/__https://lore.kernel.org/r/20231026-dwc3-v1-1-643c74771599@chromium.org__;!!A4F2R9G_pg!b22b7Wx1QOpnFtMi7rJ0TS-rhOnMmy_sOCX3kdfln7ezNjslyMhBwHvWaXbK8D5BkKPjjn3HdID-7Ls5R3eR_4o$ > --- > drivers/usb/dwc3/core.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c > index 343d2570189f..65f73dd8ef47 100644 > --- a/drivers/usb/dwc3/core.c > +++ b/drivers/usb/dwc3/core.c > @@ -1918,6 +1918,8 @@ static int dwc3_probe(struct platform_device *pdev) > > pm_runtime_put(dev); > > + dma_set_max_seg_size(dev, UINT_MAX); > + > return 0; > > err_exit_debugfs: > > --- > base-commit: 611da07b89fdd53f140d7b33013f255bf0ed8f34 > change-id: 20231026-dwc3-fac74fcb3b2a > > Best regards, > -- > Ricardo Ribalda <ribalda@chromium.org> > Probably better to have the Closes: tag with the link to the reported issue. Regardless, Acked-by: Thinh Nguyen <Thinh.Nguyen@synopsys.com> Thanks, Thinh
Hi Thinh On Wed, 1 Nov 2023 at 00:22, Thinh Nguyen <Thinh.Nguyen@synopsys.com> wrote: > > On Fri, Oct 27, 2023, Ricardo Ribalda wrote: > > Allow devices to have dma operations beyond 4K, and avoid warnings such > > as: > > > > DMA-API: dwc3 a600000.usb: mapping sg segment longer than device claims to support [len=86016] [max=65536] > > > > Cc: stable@vger.kernel.org > > Fixes: 72246da40f37 ("usb: Introduce DesignWare USB3 DRD Driver") > > Reported-by: Zubin Mithra <zsm@chromium.org> > > Signed-off-by: Ricardo Ribalda <ribalda@chromium.org> > > --- > > Found while running > > yavta -f YUYV -s 1280x720 -c /dev/video0 > > > > with: > > > > CONFIG_DMA_API_DEBUG=y > > --- > > Changes in v2: > > - Add stable tag > > - Link to v1: https://urldefense.com/v3/__https://lore.kernel.org/r/20231026-dwc3-v1-1-643c74771599@chromium.org__;!!A4F2R9G_pg!b22b7Wx1QOpnFtMi7rJ0TS-rhOnMmy_sOCX3kdfln7ezNjslyMhBwHvWaXbK8D5BkKPjjn3HdID-7Ls5R3eR_4o$ > > --- > > drivers/usb/dwc3/core.c | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c > > index 343d2570189f..65f73dd8ef47 100644 > > --- a/drivers/usb/dwc3/core.c > > +++ b/drivers/usb/dwc3/core.c > > @@ -1918,6 +1918,8 @@ static int dwc3_probe(struct platform_device *pdev) > > > > pm_runtime_put(dev); > > > > + dma_set_max_seg_size(dev, UINT_MAX); > > + > > return 0; > > > > err_exit_debugfs: > > > > --- > > base-commit: 611da07b89fdd53f140d7b33013f255bf0ed8f34 > > change-id: 20231026-dwc3-fac74fcb3b2a > > > > Best regards, > > -- > > Ricardo Ribalda <ribalda@chromium.org> > > > > Probably better to have the Closes: tag with the link to the reported > issue. Regardless, It was reported internally, so I have no link to share. In this cases it seems that Reported-by with no link in good enough: https://lore.kernel.org/all/ab557ae6-7550-189a-81dd-6e3346d84620@leemhuis.info/ > > Acked-by: Thinh Nguyen <Thinh.Nguyen@synopsys.com> Thanks! > > Thanks, > Thinh
On Fri, Nov 10, 2023, Ricardo Ribalda wrote: > > Probably better to have the Closes: tag with the link to the reported > > issue. Regardless, > > It was reported internally, so I have no link to share. > I see. Thanks, Thinh
Greg: Friendly ping, just want to make sure that the patch did not fell into the cracks thanks! On Fri, 10 Nov 2023 at 23:56, Thinh Nguyen <Thinh.Nguyen@synopsys.com> wrote: > > On Fri, Nov 10, 2023, Ricardo Ribalda wrote: > > > Probably better to have the Closes: tag with the link to the reported > > > issue. Regardless, > > > > It was reported internally, so I have no link to share. > > > > I see. > > Thanks, > Thinh
On Fri, Nov 17, 2023 at 01:48:47PM +0100, Ricardo Ribalda wrote: > Greg: Friendly ping, just want to make sure that the patch did not > fell into the cracks The merge window just ended, and we have been away at the plumbers conference all week long. Give us a chance to travel back home and start to catch up in the next few weeks. In the meantime, please take some time and review the pending dwc3 patches on the list to make my review process easier and allow me to get through them all to yours quicker. thanks, greg k-h
diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c index 343d2570189f..65f73dd8ef47 100644 --- a/drivers/usb/dwc3/core.c +++ b/drivers/usb/dwc3/core.c @@ -1918,6 +1918,8 @@ static int dwc3_probe(struct platform_device *pdev) pm_runtime_put(dev); + dma_set_max_seg_size(dev, UINT_MAX); + return 0; err_exit_debugfs:
Allow devices to have dma operations beyond 4K, and avoid warnings such as: DMA-API: dwc3 a600000.usb: mapping sg segment longer than device claims to support [len=86016] [max=65536] Cc: stable@vger.kernel.org Fixes: 72246da40f37 ("usb: Introduce DesignWare USB3 DRD Driver") Reported-by: Zubin Mithra <zsm@chromium.org> Signed-off-by: Ricardo Ribalda <ribalda@chromium.org> --- Found while running yavta -f YUYV -s 1280x720 -c /dev/video0 with: CONFIG_DMA_API_DEBUG=y --- Changes in v2: - Add stable tag - Link to v1: https://lore.kernel.org/r/20231026-dwc3-v1-1-643c74771599@chromium.org --- drivers/usb/dwc3/core.c | 2 ++ 1 file changed, 2 insertions(+) --- base-commit: 611da07b89fdd53f140d7b33013f255bf0ed8f34 change-id: 20231026-dwc3-fac74fcb3b2a Best regards,