Documentation to v4l2_i2c_subdev_set_name says I²C device name if devname parameter is NULL. But instead the I²C driver name is assigned. Fix this by actually using the i2c_client->name for the subdev name. Fixes: 0658293012af ("media: v4l: subdev: Add a function to set an I²C sub-device's name") Signed-off-by: Alexander Stein <alexander.stein@xxxxxxxxxxxxxxx> --- Having a sensor node in DT with compatible = "ovti,ov9281" the subdev name is "ov9282" without this patch. With this patch applied it is "ov9281". The i2c_client->name is "ov9281" in every case. drivers/media/v4l2-core/v4l2-i2c.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/v4l2-core/v4l2-i2c.c b/drivers/media/v4l2-core/v4l2-i2c.c index b4acca75644b..90c3c799317a 100644 --- a/drivers/media/v4l2-core/v4l2-i2c.c +++ b/drivers/media/v4l2-core/v4l2-i2c.c @@ -33,7 +33,7 @@ void v4l2_i2c_subdev_set_name(struct v4l2_subdev *sd, const char *devname, const char *postfix) { if (!devname) - devname = client->dev.driver->name; + devname = client->name; if (!postfix) postfix = ""; -- 2.25.1