drivers: usb: nordic: Do not log an error if HFCLK is busy

pawelzadrozniak · carlescufi · commit b2a3e12d861a · 2019-04-15T11:10:19.000+02:00
HFCLK may be requested for multiple modules, i.e. Bluetooth stack. When any other module has requested HFCLK to run, the driver will return -EBUSY on free attempt which is not an error - thie means that free request has been processed, but someone else is still requiring the clock to run. When all clock users free the clock, it may be disabled. Related issue: #15145 Signed-off-by: Paweł Zadrożniak <pawel.zadrozniak@nordicsemi.no>
diff --git a/drivers/usb/device/usb_dc_nrfx.c b/drivers/usb/device/usb_dc_nrfx.c
@@ -557,6 +557,13 @@ static int hf_clock_enable(bool on, bool blocking)
 			return 0;
 		}
 		ret = clock_control_off(clock, (void *)blocking);
+		if (ret == -EBUSY) {
+			/* This is an expected behaviour.
+			 * -EBUSY means that some other module has also
+			 * requested the clock to run.
+			 */
+			ret = 0;
+		}
 	}
 
 	if (ret && (blocking || (ret != -EINPROGRESS))) {

Original file line number	Diff line number	Diff line change
`@@ -557,6 +557,13 @@ static int hf_clock_enable(bool on, bool blocking)`
`557`	`557`	`return 0;`
`558`	`558`	`}`
`559`	`559`	`ret = clock_control_off(clock, (void *)blocking);`
	`560`	`+ if (ret == -EBUSY) {`
	`561`	`+ /* This is an expected behaviour.`
	`562`	`+ * -EBUSY means that some other module has also`
	`563`	`+ * requested the clock to run.`
	`564`	`+ */`
	`565`	`+ ret = 0;`
	`566`	`+ }`
`560`	`567`	`}`
`561`	`568`
`562`	`569`	`if (ret && (blocking \|\| (ret != -EINPROGRESS))) {`