[PATCH v2 1/1] usb: dwc3: Fix default mode initialization

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



The default mode, configurable by DT, shall be set before usb role switch
driver is registered. Otherwise there is a race between default mode
and mode set by usb role switch driver.

Fixes: 98ed256a4dbad ("usb: dwc3: Add support for role-switch-default-mode binding")
Signed-off-by: Alexander Stein <alexander.stein@xxxxxxxxxxxxxxx>
---
This is a different approach to [1] to fix the race upon dwc3 role
initialization. dwc3_set_mode() needs to be called in dwc3_setup_role_switch()
to ensure the mode is initialized.
The usb role switch driver probe might be delayed or defered for some unrelated
reason, leaving the dwc3 mode unitialized if dwc3_set_mode() is not called here.

[1] https://lore.kernel.org/linux-usb/2176034.Icojqenx9y@steina-w/T/

 drivers/usb/dwc3/drd.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/usb/dwc3/drd.c b/drivers/usb/dwc3/drd.c
index 039bf241769af..57ddd2e43022e 100644
--- a/drivers/usb/dwc3/drd.c
+++ b/drivers/usb/dwc3/drd.c
@@ -505,6 +505,7 @@ static int dwc3_setup_role_switch(struct dwc3 *dwc)
 		dwc->role_switch_default_mode = USB_DR_MODE_PERIPHERAL;
 		mode = DWC3_GCTL_PRTCAP_DEVICE;
 	}
+	dwc3_set_mode(dwc, mode);
 
 	dwc3_role_switch.fwnode = dev_fwnode(dwc->dev);
 	dwc3_role_switch.set = dwc3_usb_role_switch_set;
@@ -526,7 +527,6 @@ static int dwc3_setup_role_switch(struct dwc3 *dwc)
 		}
 	}
 
-	dwc3_set_mode(dwc, mode);
 	return 0;
 }
 #else
-- 
2.34.1





[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux