diff options
| author | Sakari Ailus <sakari.ailus@linux.intel.com> | 2018-09-24 10:24:43 -0400 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab+samsung@kernel.org> | 2018-10-03 11:55:44 -0400 |
| commit | 654be7dde76b1897450c49d526af11067601e5b9 (patch) | |
| tree | a1e6e53992bfc878dd8ba2ff34743a18082f90f9 /drivers/media/i2c/s5k4ecgx.c | |
| parent | 4158757395b300b6eb308fc20b96d1d231484413 (diff) | |
| download | linux-654be7dde76b1897450c49d526af11067601e5b9.tar.gz linux-654be7dde76b1897450c49d526af11067601e5b9.tar.bz2 linux-654be7dde76b1897450c49d526af11067601e5b9.zip | |
media: v4l: i2c: Add a comment not to use static sub-device names in the future
A number of sub-device drivers used a static name for the sub-device, and
thus the media entity. As the entity name must be unique within a media
device, this makes it impossible to have more than one instance of each
device in a media device. This is a rather severe limitation.
Instead of fixing these drivers, add a comment to the drivers noting that
such static names may not be used in the future.
The alternative of fixing the drivers is troublesome as the entity (as
well as sub-device) name is part of the uAPI. Changing that is almost
certain to break something. As these devices are old but no-one has
encountered a problem with the static names, leave it as-is.
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
Diffstat (limited to 'drivers/media/i2c/s5k4ecgx.c')
| -rw-r--r-- | drivers/media/i2c/s5k4ecgx.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/media/i2c/s5k4ecgx.c b/drivers/media/i2c/s5k4ecgx.c index 8c0dca6cb20c..79aa2740edc4 100644 --- a/drivers/media/i2c/s5k4ecgx.c +++ b/drivers/media/i2c/s5k4ecgx.c @@ -954,6 +954,7 @@ static int s5k4ecgx_probe(struct i2c_client *client, sd = &priv->sd; /* Registering subdev */ v4l2_i2c_subdev_init(sd, client, &s5k4ecgx_ops); + /* Static name; NEVER use in new drivers! */ strscpy(sd->name, S5K4ECGX_DRIVER_NAME, sizeof(sd->name)); sd->internal_ops = &s5k4ecgx_subdev_internal_ops; |