Re: [PATCH 3/3] media: soc_camera: rcar_vin: Add ARGB8888 caputre format support

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

 



Hi Sergei,

2016-01-12 3:35 GMT+09:00 Sergei Shtylyov <sergei.shtylyov@xxxxxxxxxxxxxxxxxx>:
> Hello.
>
> On 01/11/2016 09:00 PM, Yoshihiro Kaneko wrote:
>
>> From: Koji Matsuoka <koji.matsuoka.xm@xxxxxxxxxxx>
>>
>> This patch adds ARGB8888 capture format support for R-Car Gen3.
>>
>> Signed-off-by: Koji Matsuoka <koji.matsuoka.xm@xxxxxxxxxxx>
>> Signed-off-by: Yoshihiro Kaneko <ykaneko0929@xxxxxxxxx>
>> ---
>>   drivers/media/platform/soc_camera/rcar_vin.c | 21 +++++++++++++++++++--
>>   1 file changed, 19 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/media/platform/soc_camera/rcar_vin.c
>> b/drivers/media/platform/soc_camera/rcar_vin.c
>> index cccd859..afe27bb 100644
>> --- a/drivers/media/platform/soc_camera/rcar_vin.c
>> +++ b/drivers/media/platform/soc_camera/rcar_vin.c
>
> [...]
>>
>> @@ -654,6 +654,14 @@ static int rcar_vin_setup(struct rcar_vin_priv *priv)
>>                         dmr = VNDMR_EXRGB;
>>                         break;
>>                 }
>> +       case V4L2_PIX_FMT_ARGB32:
>> +               if (priv->chip == RCAR_GEN3)
>> +                       dmr = VNDMR_EXRGB | VNDMR_DTMD_ARGB;
>> +               else {
>
>
>    Kernel coding style dictates using {} in all *if* branches if at least
> one branch has them.

Got it.
Thanks!

>
>> +                       dev_err(icd->parent, "Not support format\n");
>> +                       return -EINVAL;
>> +               }
>> +               break;
>>         default:
>>                 dev_warn(icd->parent, "Invalid fourcc format (0x%x)\n",
>>                          icd->current_fmt->host_fmt->fourcc);
>
> [...]
>
> MBR, Sergei
>

Thanks,
kaneko
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux