Re: [PATCH 00/11] usb: gadget: Introduce handle_interrupts ops to USB_GADGET_GENERIC uclass
Hi, On Fri, 14 Jun 2024 02:51:15 +0200, Marek Vasut wrote: > Introduce .ops for USB_GADGET_GENERIC uclass. The first new ops is > .handle_interrupts which must be implemented by DM capable USB gadget > controller drivers and must implement interrupt handling similar to > dm_usb_gadget_handle_interrupts(). For DM USB gadget drivers this is > a replacement for dm_usb_gadget_handle_interrupts(). Convert the DM > USB gadget drivers to this new ops instead. > > [...] Thanks, Applied to https://source.denx.de/u-boot/custodians/u-boot-dfu (u-boot-dfu) [01/11] usb: gadget: Introduce handle_interrupts ops to USB_GADGET_GENERIC uclass https://source.denx.de/u-boot/custodians/u-boot-dfu/-/commit/d36ef5cbedd08e9a11c6a719d853f1003173133e [02/11] usb: gadget: cdns3: Convert interrupt handling to usb_gadget_generic_ops https://source.denx.de/u-boot/custodians/u-boot-dfu/-/commit/356542d6dd72f66031bc0fbe174a99c360ee318a [03/11] usb: gadget: dwc2: Convert interrupt handling to usb_gadget_generic_ops https://source.denx.de/u-boot/custodians/u-boot-dfu/-/commit/2329109fbd7e48c559594c089238dc57d7fea166 [04/11] usb: gadget: dwc3: Convert interrupt handling to usb_gadget_generic_ops https://source.denx.de/u-boot/custodians/u-boot-dfu/-/commit/75f94ed82ff0a8c021decaa9a0eda56d48cba716 [05/11] usb: gadget: max3420: Convert interrupt handling to usb_gadget_generic_ops https://source.denx.de/u-boot/custodians/u-boot-dfu/-/commit/a75753327399d821ba70d8cb493d1bc12d1034cc [06/11] usb: gadget: mtu3: Convert interrupt handling to usb_gadget_generic_ops https://source.denx.de/u-boot/custodians/u-boot-dfu/-/commit/2c2d7c2bab0ec6e4cd93db242822a57986680917 [07/11] usb: gadget: omap2430: Convert interrupt handling to usb_gadget_generic_ops https://source.denx.de/u-boot/custodians/u-boot-dfu/-/commit/521bc756cb5fea0ca8c159bd812da063e08bd351 [08/11] usb: gadget: musb: Convert interrupt handling to usb_gadget_generic_ops https://source.denx.de/u-boot/custodians/u-boot-dfu/-/commit/281eaf1ed83a09bfce6b82b86775e11d4269ad08 [09/11] usb: gadget: ux500: Convert interrupt handling to usb_gadget_generic_ops https://source.denx.de/u-boot/custodians/u-boot-dfu/-/commit/ac4bf5d48a9e85e79ced3b7195a8599c21c0a16d [10/11] usb: gadget: sandbox: Drop dm_usb_gadget_handle_interrupts() https://source.denx.de/u-boot/custodians/u-boot-dfu/-/commit/ed8e638de1ef0ab2bf87c6c83882b275b66b292f [11/11] usb: gadget: Mark dm_usb_gadget_handle_interrupts as non-weak for DM_USB_GADGET https://source.denx.de/u-boot/custodians/u-boot-dfu/-/commit/9e2c14804e4aaefe7cbb43d444f5dc83ba76c86d -- Mattijs
Re: [PATCH 00/11] usb: gadget: Introduce handle_interrupts ops to USB_GADGET_GENERIC uclass
Hi Marek, Thank you for the series. On ven., juin 14, 2024 at 02:51, Marek Vasut wrote: > Introduce .ops for USB_GADGET_GENERIC uclass. The first new ops is > .handle_interrupts which must be implemented by DM capable USB gadget > controller drivers and must implement interrupt handling similar to > dm_usb_gadget_handle_interrupts(). For DM USB gadget drivers this is > a replacement for dm_usb_gadget_handle_interrupts(). Convert the DM > USB gadget drivers to this new ops instead. > > DEPENDS: https://patchwork.ozlabs.org/project/uboot/list/?series=410150 > > Marek Vasut (11): > usb: gadget: Introduce handle_interrupts ops to USB_GADGET_GENERIC > uclass > usb: gadget: cdns3: Convert interrupt handling to > usb_gadget_generic_ops > usb: gadget: dwc2: Convert interrupt handling to > usb_gadget_generic_ops > usb: gadget: dwc3: Convert interrupt handling to > usb_gadget_generic_ops > usb: gadget: max3420: Convert interrupt handling to > usb_gadget_generic_ops > usb: gadget: mtu3: Convert interrupt handling to > usb_gadget_generic_ops > usb: gadget: omap2430: Convert interrupt handling to > usb_gadget_generic_ops > usb: gadget: musb: Convert interrupt handling to > usb_gadget_generic_ops > usb: gadget: ux500: Convert interrupt handling to > usb_gadget_generic_ops > usb: gadget: sandbox: Drop dm_usb_gadget_handle_interrupts() > usb: gadget: Mark dm_usb_gadget_handle_interrupts as non-weak for > DM_USB_GADGET > > drivers/usb/cdns3/core.c| 24 > drivers/usb/cdns3/gadget-export.h | 2 ++ > drivers/usb/cdns3/gadget.c | 11 +-- > drivers/usb/dwc3/dwc3-generic.c | 23 ++- > drivers/usb/dwc3/dwc3-layerscape.c | 21 + > drivers/usb/gadget/dwc2_udc_otg.c | 20 +++- > drivers/usb/gadget/max3420_udc.c| 19 --- > drivers/usb/gadget/udc/udc-uclass.c | 24 > drivers/usb/host/usb-sandbox.c | 7 +-- > drivers/usb/mtu3/mtu3_plat.c| 23 ++- > drivers/usb/musb-new/omap2430.c | 26 -- > drivers/usb/musb-new/ti-musb.c | 23 ++- > drivers/usb/musb-new/ux500.c| 22 ++ > include/linux/usb/gadget.h | 8 > 14 files changed, 172 insertions(+), 81 deletions(-) Tested on Khadas vim3 using fastboot, ums and usb storage scanning. Tested-by: Mattijs Korpershoek # vim3 > > --- > Cc: Alexander Sverdlin > Cc: Felipe Balbi > Cc: Lukasz Majewski > Cc: Mattijs Korpershoek > Cc: Nishanth Menon > Cc: Simon Glass > Cc: Thinh Nguyen > Cc: Tom Rini > Cc: u-boot@lists.denx.de > > -- > 2.43.0
Re: [PATCH 00/11] usb: gadget: Introduce handle_interrupts ops to USB_GADGET_GENERIC uclass
On 6/14/24 12:24 PM, Sverdlin, Alexander wrote: Hi Marek! Hi, On Fri, 2024-06-14 at 02:51 +0200, Marek Vasut wrote: Introduce .ops for USB_GADGET_GENERIC uclass. The first new ops is .handle_interrupts which must be implemented by DM capable USB gadget controller drivers and must implement interrupt handling similar to dm_usb_gadget_handle_interrupts(). For DM USB gadget drivers this is a replacement for dm_usb_gadget_handle_interrupts(). Convert the DM USB gadget drivers to this new ops instead. DEPENDS: https://patchwork.ozlabs.org/project/uboot/list/?series=410150 I've applied both the dependency and this series and tested on AM62x, which I believe would cover patches 1,4,11 (32-bit SPL, 64-bit SPL + U-Boot proper, booted over USB DFU): I appreciate the testing, thanks !
Re: [PATCH 00/11] usb: gadget: Introduce handle_interrupts ops to USB_GADGET_GENERIC uclass
Hi Marek! On Fri, 2024-06-14 at 02:51 +0200, Marek Vasut wrote: > Introduce .ops for USB_GADGET_GENERIC uclass. The first new ops is > .handle_interrupts which must be implemented by DM capable USB gadget > controller drivers and must implement interrupt handling similar to > dm_usb_gadget_handle_interrupts(). For DM USB gadget drivers this is > a replacement for dm_usb_gadget_handle_interrupts(). Convert the DM > USB gadget drivers to this new ops instead. > > DEPENDS: https://patchwork.ozlabs.org/project/uboot/list/?series=410150 I've applied both the dependency and this series and tested on AM62x, which I believe would cover patches 1,4,11 (32-bit SPL, 64-bit SPL + U-Boot proper, booted over USB DFU): > Marek Vasut (11): > usb: gadget: Introduce handle_interrupts ops to USB_GADGET_GENERIC > uclass > usb: gadget: dwc3: Convert interrupt handling to > usb_gadget_generic_ops > usb: gadget: Mark dm_usb_gadget_handle_interrupts as non-weak for > DM_USB_GADGET For these, Tested-by: Alexander Sverdlin -- Alexander Sverdlin Siemens AG www.siemens.com
[PATCH 00/11] usb: gadget: Introduce handle_interrupts ops to USB_GADGET_GENERIC uclass
Introduce .ops for USB_GADGET_GENERIC uclass. The first new ops is .handle_interrupts which must be implemented by DM capable USB gadget controller drivers and must implement interrupt handling similar to dm_usb_gadget_handle_interrupts(). For DM USB gadget drivers this is a replacement for dm_usb_gadget_handle_interrupts(). Convert the DM USB gadget drivers to this new ops instead. DEPENDS: https://patchwork.ozlabs.org/project/uboot/list/?series=410150 Marek Vasut (11): usb: gadget: Introduce handle_interrupts ops to USB_GADGET_GENERIC uclass usb: gadget: cdns3: Convert interrupt handling to usb_gadget_generic_ops usb: gadget: dwc2: Convert interrupt handling to usb_gadget_generic_ops usb: gadget: dwc3: Convert interrupt handling to usb_gadget_generic_ops usb: gadget: max3420: Convert interrupt handling to usb_gadget_generic_ops usb: gadget: mtu3: Convert interrupt handling to usb_gadget_generic_ops usb: gadget: omap2430: Convert interrupt handling to usb_gadget_generic_ops usb: gadget: musb: Convert interrupt handling to usb_gadget_generic_ops usb: gadget: ux500: Convert interrupt handling to usb_gadget_generic_ops usb: gadget: sandbox: Drop dm_usb_gadget_handle_interrupts() usb: gadget: Mark dm_usb_gadget_handle_interrupts as non-weak for DM_USB_GADGET drivers/usb/cdns3/core.c| 24 drivers/usb/cdns3/gadget-export.h | 2 ++ drivers/usb/cdns3/gadget.c | 11 +-- drivers/usb/dwc3/dwc3-generic.c | 23 ++- drivers/usb/dwc3/dwc3-layerscape.c | 21 + drivers/usb/gadget/dwc2_udc_otg.c | 20 +++- drivers/usb/gadget/max3420_udc.c| 19 --- drivers/usb/gadget/udc/udc-uclass.c | 24 drivers/usb/host/usb-sandbox.c | 7 +-- drivers/usb/mtu3/mtu3_plat.c| 23 ++- drivers/usb/musb-new/omap2430.c | 26 -- drivers/usb/musb-new/ti-musb.c | 23 ++- drivers/usb/musb-new/ux500.c| 22 ++ include/linux/usb/gadget.h | 8 14 files changed, 172 insertions(+), 81 deletions(-) --- Cc: Alexander Sverdlin Cc: Felipe Balbi Cc: Lukasz Majewski Cc: Mattijs Korpershoek Cc: Nishanth Menon Cc: Simon Glass Cc: Thinh Nguyen Cc: Tom Rini Cc: u-boot@lists.denx.de -- 2.43.0
