Thread (6 messages) 6 messages, 3 authors, 2017-07-24

Re: [PATCH 1/3] can: m_can: Make hclk optional

From: Sergei Shtylyov <hidden>
Date: 2017-07-20 09:42:48
Also in: linux-can, linux-devicetree, lkml

Hello!

On 7/20/2017 2:09 AM, Franklin S Cooper Jr wrote:
quoted hunk ↗ jump to hunk
Hclk is the MCAN's interface clock. However, for OMAP based devices such as
DRA7 SoC family the interface clock is handled by hwmod. Therefore, this
interface clock is managed by hwmod driver via pm_runtime_get and
pm_runtime_put calls. Therefore, this interface clock isn't defined in DT
and thus the driver shouldn't fail if this clock isn't found.

Signed-off-by: Franklin S Cooper Jr <redacted>
---
  drivers/net/can/m_can/m_can.c | 9 +++++++--
  1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/drivers/net/can/m_can/m_can.c b/drivers/net/can/m_can/m_can.c
index f4947a7..7fe9145 100644
--- a/drivers/net/can/m_can/m_can.c
+++ b/drivers/net/can/m_can/m_can.c
@@ -1568,8 +1568,13 @@ static int m_can_plat_probe(struct platform_device *pdev)
  	hclk = devm_clk_get(&pdev->dev, "hclk");
  	cclk = devm_clk_get(&pdev->dev, "cclk");
  
-	if (IS_ERR(hclk) || IS_ERR(cclk)) {
-		dev_err(&pdev->dev, "no clock found\n");
+	if (IS_ERR(hclk)) {
+		dev_warn(&pdev->dev, "can't find hclk\n");
+		hclk = 0;
    s/0/NULL/.

[...]

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