Messages in this thread Patch in this message | | | From | Laurentiu Palcu <> | Subject | [PATCH] media: i2c: rdacm2x: properly set subdev entity function | Date | Tue, 8 Mar 2022 14:55:49 +0200 |
| |
The subdevice entity function was left unset, which produces a warning when probing the device:
mxc-md bus@58000000:camera: Entity type for entity rdacm20 19-0051 was not initialized!
Also, set the entity flags to MEDIA_ENT_FL_DEFAULT instead of MEDIA_ENT_F_CAM_SENSOR which will be used for entity's function instead.
Signed-off-by: Laurentiu Palcu <laurentiu.palcu@oss.nxp.com> --- Hi,
Only tested the rdacm20 part but I believe rdacm21 should work as well.
Cheers, laurentiu
drivers/media/i2c/rdacm20.c | 3 ++- drivers/media/i2c/rdacm21.c | 3 ++- 2 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/drivers/media/i2c/rdacm20.c b/drivers/media/i2c/rdacm20.c index 025a610de893..ea5b7d5151ee 100644 --- a/drivers/media/i2c/rdacm20.c +++ b/drivers/media/i2c/rdacm20.c @@ -611,7 +611,8 @@ static int rdacm20_probe(struct i2c_client *client) goto error_free_ctrls; dev->pad.flags = MEDIA_PAD_FL_SOURCE; - dev->sd.entity.flags |= MEDIA_ENT_F_CAM_SENSOR; + dev->sd.entity.flags |= MEDIA_ENT_FL_DEFAULT; + dev->sd.entity.function = MEDIA_ENT_F_CAM_SENSOR; ret = media_entity_pads_init(&dev->sd.entity, 1, &dev->pad); if (ret < 0) goto error_free_ctrls; diff --git a/drivers/media/i2c/rdacm21.c b/drivers/media/i2c/rdacm21.c index 12ec5467ed1e..be89bd43e88b 100644 --- a/drivers/media/i2c/rdacm21.c +++ b/drivers/media/i2c/rdacm21.c @@ -583,7 +583,8 @@ static int rdacm21_probe(struct i2c_client *client) goto error_free_ctrls; dev->pad.flags = MEDIA_PAD_FL_SOURCE; - dev->sd.entity.flags |= MEDIA_ENT_F_CAM_SENSOR; + dev->sd.entity.flags |= MEDIA_ENT_FL_DEFAULT; + dev->sd.entity.function = MEDIA_ENT_F_CAM_SENSOR; ret = media_entity_pads_init(&dev->sd.entity, 1, &dev->pad); if (ret < 0) goto error_free_ctrls; -- 2.33.0
| |