Hi Alexander, Thank you for the patch.
On ven., avril 12, 2024 at 22:26, "A. Sverdlin" <alexander.sverd...@siemens.com> wrote: > From: Felipe Balbi <felipe.ba...@linux.intel.com> > > Upstream Linux commit 5999914f227b. > > The cmd argument we pass to > dwc3_send_gadget_ep_cmd() could contain extra > arguments embedded. When checking for StartTransfer > command, we need to make sure to match only lower 4 > bits which contain the actual command and ignore the > rest. > > Reported-by: Janusz Dziedzic <januszx.dzied...@intel.com> > Signed-off-by: Felipe Balbi <felipe.ba...@linux.intel.com> > [A. Sverdlin: cherry-picked only DWC3_DEPCMD_CMD() define] > Signed-off-by: Alexander Sverdlin <alexander.sverd...@siemens.com> Reviewed-by: Mattijs Korpershoek <mkorpersh...@baylibre.com> > --- > drivers/usb/dwc3/core.h | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/usb/dwc3/core.h b/drivers/usb/dwc3/core.h > index 1e7eda89a34c9..7709ab793f36d 100644 > --- a/drivers/usb/dwc3/core.h > +++ b/drivers/usb/dwc3/core.h > @@ -405,6 +405,8 @@ > #define DWC3_DEPCMD_SETTRANSFRESOURCE (0x02 << 0) > #define DWC3_DEPCMD_SETEPCONFIG (0x01 << 0) > > +#define DWC3_DEPCMD_CMD(x) ((x) & 0xf) > + > /* The EP number goes 0..31 so ep0 is always out and ep1 is always in */ > #define DWC3_DALEPENA_EP(n) (1 << n) > > -- > 2.44.0
