Thread (21 messages) 21 messages, 6 authors, 2013-07-17

[PATCH v3 6/7] USB: ohci-at91: add usb_clk for transition to common clk framework

From: stern@rowland.harvard.edu (Alan Stern)
Date: 2013-07-16 16:48:57
Also in: lkml

On Tue, 16 Jul 2013, Boris BREZILLON wrote:
The AT91 PMC (Power Management Controller) provides an USB clock used by
USB Full Speed host (ohci) and USB Full Speed device (udc).
The usb drivers (ohci and udc) must configure this clock to 48Mhz.
This configuration was formely done in mach-at91/clock.c, but this
implementation will be removed when moving to common clk framework.

This patch add support for usb clock retrieval and configuration, and is
backward compatible with the current at91 clk implementation (if usb clk
is not found, it does not configure/enable the usb clk).
But it does print a warning in the system log, right?
quoted hunk ↗ jump to hunk
@@ -144,6 +150,11 @@ static int usb_hcd_at91_probe(const struct hc_driver *driver,
 		goto err2;
 	}
 
+	uclk = clk_get(&pdev->dev, "usb_clk");
+	if (IS_ERR(uclk)) {
+		uclk = NULL;
+		dev_warn(&pdev->dev, "failed to get usb_clk\n");
+	}
Is this really what you want for backward compatibility?

Alan Stern
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help