| Message ID | 20250225212031.188987-1-demonsingur@gmail.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | media: platform: rpi1-cfe: fix pad in call to get_mbus_config() | expand |
On 2/25/25 11:20 PM, 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: e7bad98c205d ("media: v4l: Convert the users of v4l2_get_link_freq to call it on a pad") I used the wrong Fixes tag, this is the correct one: Fixes: 6edb685abb2a ("media: raspberrypi: Add support for RP1-CFE") > Signed-off-by: Cosmin Tanislav <demonsingur@gmail.com> > --- > 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;
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: e7bad98c205d ("media: v4l: Convert the users of v4l2_get_link_freq to call it on a pad") Signed-off-by: Cosmin Tanislav <demonsingur@gmail.com> --- drivers/media/platform/raspberrypi/rp1-cfe/cfe.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)