Re: [GIT PULL FOR v5.3] Add ZynqMP VCU/Allegro DVT H.264 encoder driver

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

 



On 5/29/19 1:29 PM, Hans Verkuil wrote:
> On 5/29/19 1:07 PM, Mauro Carvalho Chehab wrote:
>> Em Tue, 28 May 2019 20:10:27 +0200
>> Hans Verkuil <hverkuil@xxxxxxxxx> escreveu:
>>
>>> The following changes since commit 9bec226d8c79fcbc95817b082557f72a79d182f5:
>>>
>>>   media: v4l2-pci-skeleton.c: fix doc warning (2019-05-28 13:14:28 -0400)
>>>
>>> are available in the Git repository at:
>>>
>>>   git://linuxtv.org/hverkuil/media_tree.git tags/br-v5.3f
>>>
>>> for you to fetch changes up to 75e52767fa3d58a783dd6862a7fb686e5de67fc6:
>>>
>>>   allegro: add SPS/PPS nal unit writer (2019-05-28 20:00:26 +0200)
>>>
>>> ----------------------------------------------------------------
>>> Tag branch
>>>
>>> ----------------------------------------------------------------
>>> Hans Verkuil (1):
>>>       videobuf2-v4l2: set last_buffer_dequeued in dqbuf
>>>
>>> Michael Tretter (4):
>>>       media: dt-bindings: media: document allegro-dvt bindings
>>>       media: dt-bindings: media: Add vendor prefix for allegro
>>>       allegro: add Allegro DVT video IP core driver
>>>       allegro: add SPS/PPS nal unit writer
>>
>> As this is staging, merged, but there's something wrong with DT here:
>>
>> WARNING: DT compatible string vendor "allegro" appears un-documented -- check ./Documentation/devicetree/bindings/vendor-prefixes.yaml
>> #3013: FILE: drivers/staging/media/allegro-dvt/allegro-core.c:3013:
>> +	{ .compatible = "allegro,al5e-1.1" },
>>
>> Please send a followup patch addressing it.
> 
> Huh? Something went wrong: this is the patch that is in my for-v5.3f branch:
> 
> https://git.linuxtv.org/hverkuil/media_tree.git/commit/?h=for-v5.3f&id=ae4e36dd1945380ccd97090d2099f67be9a976d8
> 
> That's not what you tried to merge.

Never mind.

The cause is that the change to vendor-prefixes.yaml was done after the change
to allegro.txt. Those two patches should have been swapped.

So nothing is wrong.

Regards,

	Hans



[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