Thread (20 messages) 20 messages, 2 authors, 2021-04-16

Re: [PATCH v3 net-next 08/10] net: korina: Get mdio input clock via common clock framework

From: Andrew Lunn <andrew@lunn.ch>
Date: 2021-04-15 23:53:08
Also in: lkml

quoted hunk ↗ jump to hunk
@@ -1079,6 +1078,14 @@ static int korina_probe(struct platform_device *pdev)
 			eth_hw_addr_random(dev);
 	}
 
+	clk = devm_clk_get(&pdev->dev, NULL);
You should use a name here. It makes future expansion of the binding
easier. devm_clk_get_optional() is probably better. If there is a real
error it will return an error. If the clock does not exist, you get a
NULL. Real errors should cause the problem to fail, but with a NULL
you can use the fallback value.

You also need to document the device tree binding.

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