Re: [PATCH v3 0/2] media: rcar-vin: Mask access to VNCSI_IFMD register

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

 



On 03/12/2020 07:58, Mauro Carvalho Chehab wrote:
> Em Thu, 26 Nov 2020 08:47:55 +0100
> Jacopo Mondi <jacopo+renesas@xxxxxxxxxx> escreveu:
> 
>> As reported in patch 2/2 commit message the the VNCSI_IFMD register
>> has the following limitations according to chip manual revision 2.20
>>
>> - V3M, V3H and E3 do not support the DES1 field has they do not feature
>> a CSI20 receiver.
>> - D3 only supports parallel input, and the whole register shall always
>> be written as 0.
>>
>> This patch upports the BSP change commit f54697394457
>> ("media: rcar-vin: Fix VnCSI_IFMD register access for r8a77990") from
>> Koji Matsuoka
> 
> As checkpatch warned:
> 
> 	-:22: WARNING: Unknown commit id 'f54697394457', maybe rebased or not pulled?
> 	
> There's no such commit upstream. Are you referring to an OOT patch?
> 
> If so, you should provide an URL from where the patch is stored,
> as otherwise this comment won't make any sense upstream.
> 
> If you can't provide such URL, please adjust the comment to provide
> a description that won't be dependent of such OOT commit.

Read the commit message:

>> This patch upports the BSP change commit f54697394457
>> ("media: rcar-vin: Fix VnCSI_IFMD register access for r8a77990") from
>> Koji Matsuoka

This commit is from the rcar BSP package, and does not refer to a kernel
commit. So it is correct that the commit ID is unknown.

Regards,

	Hans

> 
>>
>> Tested on r-car E3 Ebisu.
>>
>> v2 -> v3:
>> - Remove a few comments and add Niklas' tag to [2/2]
>>
>> v1 -> v2:
>> - Inspect the channel routing table to deduce the availability of DES1/DES0
>>   bits as suggested by Niklas.
>>
>> Jacopo Mondi (2):
>>   media: rcar-vin: Remove unused macro
>>   media: rcar-vin: Mask VNCSI_IFMD register
>>
>>  drivers/media/platform/rcar-vin/rcar-dma.c | 26 ++++++++++++++++++----
>>  1 file changed, 22 insertions(+), 4 deletions(-)
>>
>> --
>> 2.29.1
>>
> 
> 
> 
> Thanks,
> Mauro
> 





[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