R: [PATCH] media: i2c: cvs: Register subdev nodes upon binding remote sensor

[Mauro Marinoni]

Thanks for the pointer! 
I was testing on v7.3-rc5 and missed that commit in media-committers/next. 
I can confirm that commit 7456b40bc992 properly addresses the missing subdev node registration. I've tested on an XPS13 9350.

Sorry for the noise, and thanks again for the review.

Best regards,
Mauro

________________________________________
Da: sakari.ailus@xxxxxxxxxxxxxxx <sakari.ailus@xxxxxxxxxxxxxxx>
Inviato: mercoledì 30 settembre 2026 13:59
A: Mauro Marinoni
Cc: miguel.vadillo@xxxxxxxxx; mchehab@xxxxxxxxxx; linux-media@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx
Oggetto: Re: [PATCH] media: i2c: cvs: Register subdev nodes upon binding remote sensor

Hi Mauro,

On Wed, Sep 30, 2026 at 11:28:08AM +0000, Mauro Marinoni wrote:
> The Intel CVS driver acts as an intermediate CSI-2 bridge between
> the host receiver (e.g. IPU7 ISYS) and the downstream camera sensor.
> When the host driver initializes, its async notifier completes upon
> binding the CVS bridge and registers its initial subdev nodes.
>
> However, when the CVS bridge subsequently binds the downstream sensor
> via its own async notifier callback (cvs_csi_notify_bound), it creates
> the media pad link but omits calling v4l2_device_register_subdev_nodes().
> As a consequence, downstream sensors declaring the flag
> V4L2_SUBDEV_FL_HAS_DEVNODE (such as ov02c10) never have their
> /dev/v4l-subdev* character device created, preventing userspace
> frameworks like libcamera from opening the sensor.
>
> Call v4l2_device_register_subdev_nodes() in cvs_csi_notify_bound() once
> the media pad link is established to expose newly bound subdevices.
>
> Signed-off-by: Mauro Nino Marinoni <mauro.marinoni@xxxxxxxxxxxxxxx>

Please post patches against the media committers tree next branch or
linux-next master. This issue has been fixed by commit
7456b40bc992a3fc34b587545716729923d507b9 .

--
Regards,

Sakari Ailus