From: Phil Edworthy Date: Mon, 3 Dec 2018 11:13:08 +0000 (+0000) Subject: clk: Add comment about __of_clk_get_by_name() error values X-Git-Tag: v5.4-rc1~1399^2~10^2~1 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=5c56dfe63b6e7da3d9fd84a6c553ff4d2753dd1c;p=platform%2Fkernel%2Flinux-rpi.git clk: Add comment about __of_clk_get_by_name() error values It's not immediately obvious from the code that failure to get a clock provider can return either -ENOENT or -EINVAL. Therefore, add a comment to highlight this. Signed-off-by: Phil Edworthy Reviewed-by: Andy Shevchenko Cc: Russell King [sboyd@kernel.org: Fix comment to be proper C with ==] Signed-off-by: Stephen Boyd --- diff --git a/drivers/clk/clkdev.c b/drivers/clk/clkdev.c index 9ab3db8..5284fd1 100644 --- a/drivers/clk/clkdev.c +++ b/drivers/clk/clkdev.c @@ -52,6 +52,12 @@ struct clk *of_clk_get(struct device_node *np, int index) } EXPORT_SYMBOL(of_clk_get); +/* + * Beware the return values when np is valid, but no clock provider is found. + * If name == NULL, the function returns -ENOENT. + * If name != NULL, the function returns -EINVAL. This is because __of_clk_get() + * is called even if of_property_match_string() returns an error. + */ static struct clk *__of_clk_get_by_name(struct device_node *np, const char *dev_id, const char *name)