On Tue Jan 13 07:57:22 2026 +0000, jempty.liang wrote:
> The subdevices registered by the Mali-C55 driver do not have their
> 'struct device *dev' member initialized. This is visibile when looking
> at debug message, as in example:
> 
> "(NULL device *): collect_streams: sub-device 'mali-c55 tpg' does not
>  support streams"
> 
> Fix this by initializing the *dev field for each subdevice registered
> by the Mali-C55 driver.
> 
> Signed-off-by: jempty.liang <[email protected]>
> Reviewed-by: Daniel Scally <[email protected]>
> Reviewed-by: Jacopo Mondi <[email protected]>
> Signed-off-by: Jacopo Mondi <[email protected]>
> Signed-off-by: Hans Verkuil <[email protected]>

Patch committed.

Thanks,
Hans Verkuil

 drivers/media/platform/arm/mali-c55/mali-c55-isp.c     | 1 +
 drivers/media/platform/arm/mali-c55/mali-c55-resizer.c | 1 +
 drivers/media/platform/arm/mali-c55/mali-c55-tpg.c     | 1 +
 3 files changed, 3 insertions(+)

---

diff --git a/drivers/media/platform/arm/mali-c55/mali-c55-isp.c 
b/drivers/media/platform/arm/mali-c55/mali-c55-isp.c
index 4c0fd1ec741c..dfa5dbe68c46 100644
--- a/drivers/media/platform/arm/mali-c55/mali-c55-isp.c
+++ b/drivers/media/platform/arm/mali-c55/mali-c55-isp.c
@@ -583,6 +583,7 @@ int mali_c55_register_isp(struct mali_c55 *mali_c55)
        sd->entity.ops = &mali_c55_isp_media_ops;
        sd->entity.function = MEDIA_ENT_F_PROC_VIDEO_ISP;
        sd->internal_ops = &mali_c55_isp_internal_ops;
+       sd->dev = mali_c55->dev;
        strscpy(sd->name, MALI_C55_DRIVER_NAME " isp", sizeof(sd->name));
 
        isp->pads[MALI_C55_ISP_PAD_SINK_VIDEO].flags = MEDIA_PAD_FL_SINK |
diff --git a/drivers/media/platform/arm/mali-c55/mali-c55-resizer.c 
b/drivers/media/platform/arm/mali-c55/mali-c55-resizer.c
index a8d739af74b6..c4f46651dcee 100644
--- a/drivers/media/platform/arm/mali-c55/mali-c55-resizer.c
+++ b/drivers/media/platform/arm/mali-c55/mali-c55-resizer.c
@@ -1070,6 +1070,7 @@ static int mali_c55_register_resizer(struct mali_c55 
*mali_c55,
        sd->flags |= V4L2_SUBDEV_FL_HAS_DEVNODE | V4L2_SUBDEV_FL_STREAMS;
        sd->entity.function = MEDIA_ENT_F_PROC_VIDEO_SCALER;
        sd->internal_ops = &mali_c55_resizer_internal_ops;
+       sd->dev = mali_c55->dev;
 
        rsz->pads[MALI_C55_RSZ_SINK_PAD].flags = MEDIA_PAD_FL_SINK;
        rsz->pads[MALI_C55_RSZ_SOURCE_PAD].flags = MEDIA_PAD_FL_SOURCE;
diff --git a/drivers/media/platform/arm/mali-c55/mali-c55-tpg.c 
b/drivers/media/platform/arm/mali-c55/mali-c55-tpg.c
index 1af5d2759a83..894f4cf377af 100644
--- a/drivers/media/platform/arm/mali-c55/mali-c55-tpg.c
+++ b/drivers/media/platform/arm/mali-c55/mali-c55-tpg.c
@@ -370,6 +370,7 @@ int mali_c55_register_tpg(struct mali_c55 *mali_c55)
        sd->flags |= V4L2_SUBDEV_FL_HAS_DEVNODE | V4L2_SUBDEV_FL_HAS_EVENTS;
        sd->entity.function = MEDIA_ENT_F_CAM_SENSOR;
        sd->internal_ops = &mali_c55_tpg_internal_ops;
+       sd->dev = mali_c55->dev;
        strscpy(sd->name, MALI_C55_DRIVER_NAME " tpg", sizeof(sd->name));
 
        pad->flags = MEDIA_PAD_FL_SOURCE;
_______________________________________________
linuxtv-commits mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to