Re: [PATCH v2 1/2] media: i2c: imx283: Enter full standby when stopping streaming

[Kieran Bingham]

Quoting Jai Luthra (2026-02-14 13:05:21)
> Use IMX283_STANDBY (bit 0) instead of IMX283_STBLOGIC (bit 1) when
> stopping streaming. STBLOGIC only puts the sensor logic into standby but
> leaves the MIPI interface (along with other components) in an
> indeterminate state.
> 
> This (presumably) causes the CSI receiver (e.g. Raspberry Pi's CFE) to
> miss the LP-11 to HS transition when streaming restarts, resulting in a
> hang of 10+ seconds. The issue is most visible when immediately
> restarting a full-resolution stream after stopping a 3x3 binned one, so
> that runtime suspend hasn't yet been triggered.
> 
> Writing IMX283_STANDBY puts the entire sensor into standby. The
> imx283_standby_cancel() sequence already handles the full wakeup from
> this suspended state.
> 
> Cc: stable@xxxxxxxxxxxxxxx
> Link: https://github.com/raspberrypi/linux/issues/7153
> Link: https://github.com/will127534/OneInchEye/issues/12
> Fixes: ccb4eb4496fa ("media: i2c: Add imx283 camera sensor driver")
> Signed-off-by: Jai Luthra <jai.luthra@xxxxxxxxxxxxxxxx>

Thank you for digging into this! Indeed - I think this series fixes all
my current known issues with this module.

Tested-by: Kieran Bingham <kieran.bingham@xxxxxxxxxxxxxxxx>
Reviewed-by: Kieran Bingham <kieran.bingham@xxxxxxxxxxxxxxxx>

> ---
>  drivers/media/i2c/imx283.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/media/i2c/imx283.c b/drivers/media/i2c/imx283.c
> index 8ab63ad8f385f6e2a2d7432feff0af09a5356dc4..9b3094a57873c6e8ff8c41d058ea3b0012642cac 100644
> --- a/drivers/media/i2c/imx283.c
> +++ b/drivers/media/i2c/imx283.c
> @@ -1158,7 +1158,7 @@ static int imx283_disable_streams(struct v4l2_subdev *sd,
>         if (pad != IMAGE_PAD)
>                 return -EINVAL;
>  
> -       ret = cci_write(imx283->cci, IMX283_REG_STANDBY, IMX283_STBLOGIC, NULL);
> +       ret = cci_write(imx283->cci, IMX283_REG_STANDBY, IMX283_STANDBY, NULL);
>         if (ret)
>                 dev_err(imx283->dev, "Failed to stop stream\n");
>  
> 
> -- 
> 2.53.0
>