media: rp1: fe: Use ~0, not -1, when working with unsigned values
authorTomi Valkeinen <tomi.valkeinen@ideasonboard.com>
Thu, 21 Sep 2023 13:03:07 +0000 (16:03 +0300)
committerDom Cobley <popcornmix@gmail.com>
Mon, 19 Feb 2024 11:35:02 +0000 (11:35 +0000)
Use ~0, not -1, when working with unsigned values (-1 is not unsigned).

Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
drivers/media/platform/raspberrypi/rp1_cfe/pisp_fe.c

index 91a3b5f..a97d010 100644 (file)
@@ -372,7 +372,7 @@ void pisp_fe_submit_job(struct pisp_fe_device *fe, struct vb2_buffer **vb2_bufs,
 void pisp_fe_start(struct pisp_fe_device *fe)
 {
        pisp_fe_reg_write(fe, FE_CONTROL, FE_CONTROL_RESET);
-       pisp_fe_reg_write(fe, FE_INT_STATUS, -1);
+       pisp_fe_reg_write(fe, FE_INT_STATUS, ~0);
        pisp_fe_reg_write(fe, FE_INT_EN, FE_INT_EOF | FE_INT_SOF | FE_INT_LINES0 | FE_INT_LINES1);
        fe->inframe_count = 0;
 }
@@ -383,7 +383,7 @@ void pisp_fe_stop(struct pisp_fe_device *fe)
        pisp_fe_reg_write(fe, FE_CONTROL, FE_CONTROL_ABORT);
        usleep_range(1000, 2000);
        WARN_ON(pisp_fe_reg_read(fe, FE_STATUS));
-       pisp_fe_reg_write(fe, FE_INT_STATUS, -1);
+       pisp_fe_reg_write(fe, FE_INT_STATUS, ~0);
 }
 
 static struct pisp_fe_device *to_pisp_fe_device(struct v4l2_subdev *subdev)