Re: [PATCH v2 06/11] media: i2c: max9286: Rename MAX9286_DATATYPE_RAW11 to RAW12

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Laurent,

On Sat, Jan 01, 2022 at 08:28:01PM +0200, Laurent Pinchart wrote:
> The MAX9286_DATATYPE_RAW11 value is used to configure the MAX9286 for
> 11-bit or 12-bit input data. While 11-bit data is supported on the GMSL
> side, CSI-2 doesn't have a RAW11 format. 11-bit data is transferred over
> CSI-2 as RAW12. Rename the macro accordingly to avoid confusion.
>
> Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@xxxxxxxxxxxxxxxx>

Makes sense
Reviewed-by: Jacopo Mondi <jacopo+renesas@xxxxxxxxxx>

Thanks
  j

> ---
>  drivers/media/i2c/max9286.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/media/i2c/max9286.c b/drivers/media/i2c/max9286.c
> index 75374034724f..576d9c6fac14 100644
> --- a/drivers/media/i2c/max9286.c
> +++ b/drivers/media/i2c/max9286.c
> @@ -71,7 +71,7 @@
>  #define MAX9286_DATATYPE_USER_YUV_12BIT	(10 << 0)
>  #define MAX9286_DATATYPE_USER_24BIT	(9 << 0)
>  #define MAX9286_DATATYPE_RAW14		(8 << 0)
> -#define MAX9286_DATATYPE_RAW11		(7 << 0)
> +#define MAX9286_DATATYPE_RAW12		(7 << 0)
>  #define MAX9286_DATATYPE_RAW10		(6 << 0)
>  #define MAX9286_DATATYPE_RAW8		(5 << 0)
>  #define MAX9286_DATATYPE_YUV422_10BIT	(4 << 0)
> --
> Regards,
>
> Laurent Pinchart
>



[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux