Hi Dafna, On Fri, 2020-06-05 at 19:26 +0200, Dafna Hirschfeld wrote: [...] > --- a/include/uapi/linux/v4l2-subdev.h > +++ b/include/uapi/linux/v4l2-subdev.h > @@ -66,6 +66,7 @@ struct v4l2_subdev_crop { > }; > > #define V4L2_SUBDEV_MBUS_CODE_CSC_YCBCR_ENC 0x00000001 > +#define V4L2_SUBDEV_MBUS_CODE_CSC_HSV_ENC 0x00000001 Hans' comment on patch 4 applies here as well, it would probably be better to write: #define V4L2_SUBDEV_MBUS_CODE_CSC_HSV_ENC V4L2_SUBDEV_MBUS_CODE_CSC_YCBCR_ENC regards Philipp