]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
media: ov08x40: Add missing '\n' to ov08x40_check_hwcfg() error messages
authorHans de Goede <hdegoede@redhat.com>
Fri, 20 Dec 2024 14:41:29 +0000 (15:41 +0100)
committerHans Verkuil <hverkuil@xs4all.nl>
Sat, 15 Feb 2025 14:22:40 +0000 (15:22 +0100)
A number of dev_err() error messages miss a terminating '\n', add
the missing '\n' to these.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl>
drivers/media/i2c/ov08x40.c

index 7d00740222c184248b8e54a1da4f8e23bb9c25a6..e7012ee84822f68493171cc55c6c17f0fb4b344c 100644 (file)
@@ -2194,27 +2194,27 @@ static int ov08x40_check_hwcfg(struct ov08x40 *ov08x, struct device *dev)
                ret = fwnode_property_read_u32(dev_fwnode(dev), "clock-frequency",
                                               &xvclk_rate);
                if (ret) {
-                       dev_err(dev, "can't get clock frequency");
+                       dev_err(dev, "can't get clock frequency\n");
                        goto out_err;
                }
        }
 
        if (xvclk_rate != OV08X40_XVCLK) {
-               dev_err(dev, "external clock %d is not supported",
+               dev_err(dev, "external clock %d is not supported\n",
                        xvclk_rate);
                ret = -EINVAL;
                goto out_err;
        }
 
        if (bus_cfg.bus.mipi_csi2.num_data_lanes != OV08X40_DATA_LANES) {
-               dev_err(dev, "number of CSI2 data lanes %d is not supported",
+               dev_err(dev, "number of CSI2 data lanes %d is not supported\n",
                        bus_cfg.bus.mipi_csi2.num_data_lanes);
                ret = -EINVAL;
                goto out_err;
        }
 
        if (!bus_cfg.nr_of_link_frequencies) {
-               dev_err(dev, "no link frequencies defined");
+               dev_err(dev, "no link frequencies defined\n");
                ret = -EINVAL;
                goto out_err;
        }
@@ -2227,7 +2227,7 @@ static int ov08x40_check_hwcfg(struct ov08x40 *ov08x, struct device *dev)
                }
 
                if (j == bus_cfg.nr_of_link_frequencies) {
-                       dev_err(dev, "no link frequency %lld supported",
+                       dev_err(dev, "no link frequency %lld supported\n",
                                link_freq_menu_items[i]);
                        ret = -EINVAL;
                        goto out_err;
@@ -2252,7 +2252,7 @@ static int ov08x40_probe(struct i2c_client *client)
        /* Check HW config */
        ret = ov08x40_check_hwcfg(ov08x, &client->dev);
        if (ret) {
-               dev_err(&client->dev, "failed to check hwcfg: %d", ret);
+               dev_err(&client->dev, "failed to check hwcfg: %d\n", ret);
                return ret;
        }