On 05/05/2020 16:15, Mauro Carvalho Chehab wrote:
> Em Tue, 5 May 2020 14:24:22 +0200
> Hans Verkuil <hverkuil@xxxxxxxxx> escreveu:
>
>> On 05/05/2020 14:17, Mauro Carvalho Chehab wrote:
>>> Em Tue, 5 May 2020 10:33:00 +0200
>>> Hans Verkuil <hverkuil@xxxxxxxxx> escreveu:
>>>
>>>> The following changes since commit e51759f56d314d28c25be7606b03791f048e44c7:
>>>>
>>>> media: v4l2-ctrls.h: clarify the p_def argument of v4l2_ctrl_new_std_compound (2020-04-29 12:04:58 +0200)
>>>>
>>>> are available in the Git repository at:
>>>>
>>>> git://linuxtv.org/hverkuil/media_tree.git tags/br-tegra
>>>>
>>>> for you to fetch changes up to 0bba0154ded02babccd59e21c6a27c5ad7d50faf:
>>>>
>>>> MAINTAINERS: Add Tegra Video driver section (2020-05-05 10:26:05 +0200)
>>>>
>>>> ----------------------------------------------------------------
>>>> Tag branch
>>>>
>>>> ----------------------------------------------------------------
>>>> Sowjanya Komatineni (2):
>>>> media: tegra: Add Tegra210 Video input driver
>>>> MAINTAINERS: Add Tegra Video driver section
>>>
>>>
>>> As pointed by checkpatch:
>>>
>>> WARNING: DT compatible string "nvidia,tegra210-csi" appears un-documented -- check ./Documentation/devicetree/bindings/
>>> #628: FILE: drivers/staging/media/tegra-video/csi.c:521:
>>> + { .compatible = "nvidia,tegra210-csi", .data = &tegra210_csi_soc },
>>>
>>> WARNING: DT compatible string "nvidia,tegra210-csi" appears un-documented -- check ./Documentation/devicetree/bindings/
>>> #3238: FILE: drivers/staging/media/tegra-video/video.c:106:
>>> + { .compatible = "nvidia,tegra210-csi", },
>>>
>>> and double-checked with:
>>>
>>> $ git grep "nvidia,tegra210-csi"
>>> drivers/staging/media/tegra-video/csi.c: { .compatible = "nvidia,tegra210-csi", .data = &tegra210_csi_soc },
>>> drivers/staging/media/tegra-video/video.c: { .compatible = "nvidia,tegra210-csi", },
>>>
>>> The DT specs for this driver are missing. I even checked at linux-next:
>>> there's no DT bindings for this.
>>
>> Sorry, I should have mentioned this in the cover letter: the binding file
>> is here: https://patchwork.linuxtv.org/patch/63576/
>>
>> But it will be merged via the tegra subsystem (Thierry Reding) since it is
>> part of the display bindings, not media bindings. Also, Thierry was planning
>> to convert it to yaml, so he offered to merge it instead.
>
> Ok. Please re-send the pull request (or just mark it as New on patchwork)
> once the bindings arrive linux-next.
It's still marked as New in patchwork :-) (https://patchwork.linuxtv.org/patch/63596/)
>
> Alternatively, I don't mind if those patches would also be merged via
> Tegra's tree, together with the bindings. On such case, feel free to
> add, for both patches:
>
> Reviewed-by: Mauro Carvalho Chehab <mchehab+huawei@xxxxxxxxxx>
Thierry, let us know what you want to do. It might not be a bad idea to merge
the whole series through your tree, as Mauro suggested. It keeps everything
together.
Regards,
Hans
