On Thu Jun 12 08:54:10 2025 +0200, Matthias Fend wrote:
> Just some minor coding style fixes reported by checkpatch.
> 
> Signed-off-by: Matthias Fend <matthias.f...@emfend.at>
> Signed-off-by: Sakari Ailus <sakari.ai...@linux.intel.com>
> Signed-off-by: Hans Verkuil <hverk...@xs4all.nl>

Patch committed.

Thanks,
Hans Verkuil

 drivers/media/i2c/dw9714.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

---

diff --git a/drivers/media/i2c/dw9714.c b/drivers/media/i2c/dw9714.c
index 2ddd7daa79e2..1392cddab738 100644
--- a/drivers/media/i2c/dw9714.c
+++ b/drivers/media/i2c/dw9714.c
@@ -144,7 +144,7 @@ static int dw9714_probe(struct i2c_client *client)
 
        dw9714_dev = devm_kzalloc(&client->dev, sizeof(*dw9714_dev),
                                  GFP_KERNEL);
-       if (dw9714_dev == NULL)
+       if (!dw9714_dev)
                return -ENOMEM;
 
        dw9714_dev->vcc = devm_regulator_get(&client->dev, "vcc");
@@ -247,7 +247,7 @@ static int __maybe_unused dw9714_vcm_suspend(struct device 
*dev)
  * The lens position is gradually moved in units of DW9714_CTRL_STEPS,
  * to make the movements smoothly.
  */
-static int  __maybe_unused dw9714_vcm_resume(struct device *dev)
+static int __maybe_unused dw9714_vcm_resume(struct device *dev)
 {
        struct i2c_client *client = to_i2c_client(dev);
        struct v4l2_subdev *sd = i2c_get_clientdata(client);
@@ -271,7 +271,7 @@ static int  __maybe_unused dw9714_vcm_resume(struct device 
*dev)
                                       DW9714_VAL(val, DW9714_DEFAULT_S));
                if (ret)
                        dev_err_ratelimited(dev, "%s I2C failure: %d",
-                                               __func__, ret);
+                                           __func__, ret);
                usleep_range(DW9714_CTRL_DELAY_US, DW9714_CTRL_DELAY_US + 10);
        }
 

Reply via email to