| Message ID | 20250404090026.2986810-1-demonsingur@gmail.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | [v2] media: platform: rpi1-cfe: fix pad in call to get_mbus_config() | expand |
Hi, On 04/04/2025 12:00, Cosmin Tanislav wrote: > The source subdevice might be using a source pad not equal to 0. > > Use the already existing source_pad field of cfe. > > Fixes: 6edb685abb2a ("media: raspberrypi: Add support for RP1-CFE") > Signed-off-by: Cosmin Tanislav <demonsingur@gmail.com> > --- > > V2: > * correct Fixes tag > > drivers/media/platform/raspberrypi/rp1-cfe/cfe.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/media/platform/raspberrypi/rp1-cfe/cfe.c b/drivers/media/platform/raspberrypi/rp1-cfe/cfe.c > index 69a5f23e7954..7db4fe5e0fd4 100644 > --- a/drivers/media/platform/raspberrypi/rp1-cfe/cfe.c > +++ b/drivers/media/platform/raspberrypi/rp1-cfe/cfe.c > @@ -1206,8 +1206,8 @@ static int cfe_start_streaming(struct vb2_queue *vq, unsigned int count) > cfg_reg_write(cfe, MIPICFG_INTE, > MIPICFG_INT_CSI_DMA | MIPICFG_INT_PISP_FE); > > - ret = v4l2_subdev_call(cfe->source_sd, pad, get_mbus_config, 0, > - &mbus_config); > + ret = v4l2_subdev_call(cfe->source_sd, pad, get_mbus_config, > + cfe->source_pad, &mbus_config); > if (ret < 0 && ret != -ENOIOCTLCMD) { > cfe_err(cfe, "g_mbus_config failed\n"); > goto err_clear_inte; Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com> Tomi
diff --git a/drivers/media/platform/raspberrypi/rp1-cfe/cfe.c b/drivers/media/platform/raspberrypi/rp1-cfe/cfe.c index 69a5f23e7954..7db4fe5e0fd4 100644 --- a/drivers/media/platform/raspberrypi/rp1-cfe/cfe.c +++ b/drivers/media/platform/raspberrypi/rp1-cfe/cfe.c @@ -1206,8 +1206,8 @@ static int cfe_start_streaming(struct vb2_queue *vq, unsigned int count) cfg_reg_write(cfe, MIPICFG_INTE, MIPICFG_INT_CSI_DMA | MIPICFG_INT_PISP_FE); - ret = v4l2_subdev_call(cfe->source_sd, pad, get_mbus_config, 0, - &mbus_config); + ret = v4l2_subdev_call(cfe->source_sd, pad, get_mbus_config, + cfe->source_pad, &mbus_config); if (ret < 0 && ret != -ENOIOCTLCMD) { cfe_err(cfe, "g_mbus_config failed\n"); goto err_clear_inte;
The source subdevice might be using a source pad not equal to 0. Use the already existing source_pad field of cfe. Fixes: 6edb685abb2a ("media: raspberrypi: Add support for RP1-CFE") Signed-off-by: Cosmin Tanislav <demonsingur@gmail.com> --- V2: * correct Fixes tag drivers/media/platform/raspberrypi/rp1-cfe/cfe.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)