media: i2c: ov7670: Use the devm_clk_get_optional() helper

Use devm_clk_get_optional() instead of hand writing it.
This saves some loC and improves the semantic.

Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>

diff --git a/drivers/media/i2c/ov7670.c b/drivers/media/i2c/ov7670.c
index b1bb0833571e82b04036f2d18386850155ada080..ecbded4f07653e76432dfcfcaa863ffbfa6fdbbf 100644 (file)
--- a/drivers/media/i2c/ov7670.c
+++ b/drivers/media/i2c/ov7670.c
@@ -1894,14 +1894,9 @@ static int ov7670_probe(struct i2c_client *client)
                        info->pclk_hb_disable = true;
        }
 
-       info->clk = devm_clk_get(&client->dev, "xclk"); /* optional */
-       if (IS_ERR(info->clk)) {
-               ret = PTR_ERR(info->clk);
-               if (ret == -ENOENT)
-                       info->clk = NULL;
-               else
-                       return ret;
-       }
+       info->clk = devm_clk_get_optional(&client->dev, "xclk");
+       if (IS_ERR(info->clk))
+               return PTR_ERR(info->clk);
 
        ret = ov7670_init_gpio(client, info);
        if (ret)