> @@ -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

Reply via email to