diff options
| author |   Phil Edworthy <[email protected]> | 2018-12-03 11:13:08 +0000 |
|---|---|---|
| committer |   Stephen Boyd <[email protected]> | 2019-02-06 10:32:42 -0800 |
| commit | 5c56dfe63b6e7da3d9fd84a6c553ff4d2753dd1c (patch) | |
| tree | c75798256a923e36f9f9273e9e0eb1aff5e1504d /drivers/clk/clkdev.c | |
| parent | Linux 5.0-rc1 (diff) | |
| download | wireguard-linux-5c56dfe63b6e7da3d9fd84a6c553ff4d2753dd1c.tar.xz wireguard-linux-5c56dfe63b6e7da3d9fd84a6c553ff4d2753dd1c.zip | |
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 <[email protected]>
Reviewed-by: Andy Shevchenko <[email protected]>
Cc: Russell King <[email protected]>
[[email protected]: Fix comment to be proper C with ==]
Signed-off-by: Stephen Boyd <[email protected]>
Diffstat (limited to 'drivers/clk/clkdev.c')
| -rw-r--r-- | drivers/clk/clkdev.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/drivers/clk/clkdev.c b/drivers/clk/clkdev.c index 9ab3db8b3988..5284fd191cca 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) |