* [GIT PULL FOR v5.8] Add Tegra210 Video input driver @ 2020-05-05 8:33 Hans Verkuil 2020-05-05 8:42 ` [GIT PULL FOR v5.8] Add Tegra210 Video input driver (#63596) Jenkins 2020-05-05 12:17 ` [GIT PULL FOR v5.8] Add Tegra210 Video input driver Mauro Carvalho Chehab 0 siblings, 2 replies; 9+ messages in thread From: Hans Verkuil @ 2020-05-05 8:33 UTC (permalink / raw) To: Linux Media Mailing List; +Cc: Sowjanya Komatineni 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 MAINTAINERS | 10 + drivers/staging/media/Kconfig | 2 + drivers/staging/media/Makefile | 1 + drivers/staging/media/tegra-video/Kconfig | 12 + drivers/staging/media/tegra-video/Makefile | 8 + drivers/staging/media/tegra-video/TODO | 11 + drivers/staging/media/tegra-video/csi.c | 539 ++++++++++++++++++++++++++++ drivers/staging/media/tegra-video/csi.h | 147 ++++++++ drivers/staging/media/tegra-video/tegra210.c | 978 ++++++++++++++++++++++++++++++++++++++++++++++++++ drivers/staging/media/tegra-video/vi.c | 1074 +++++++++++++++++++++++++++++++++++++++++++++++++++++++ drivers/staging/media/tegra-video/vi.h | 257 +++++++++++++ drivers/staging/media/tegra-video/video.c | 155 ++++++++ drivers/staging/media/tegra-video/video.h | 29 ++ 13 files changed, 3223 insertions(+) create mode 100644 drivers/staging/media/tegra-video/Kconfig create mode 100644 drivers/staging/media/tegra-video/Makefile create mode 100644 drivers/staging/media/tegra-video/TODO create mode 100644 drivers/staging/media/tegra-video/csi.c create mode 100644 drivers/staging/media/tegra-video/csi.h create mode 100644 drivers/staging/media/tegra-video/tegra210.c create mode 100644 drivers/staging/media/tegra-video/vi.c create mode 100644 drivers/staging/media/tegra-video/vi.h create mode 100644 drivers/staging/media/tegra-video/video.c create mode 100644 drivers/staging/media/tegra-video/video.h ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [GIT PULL FOR v5.8] Add Tegra210 Video input driver (#63596) 2020-05-05 8:33 [GIT PULL FOR v5.8] Add Tegra210 Video input driver Hans Verkuil @ 2020-05-05 8:42 ` Jenkins 2020-05-05 12:17 ` [GIT PULL FOR v5.8] Add Tegra210 Video input driver Mauro Carvalho Chehab 1 sibling, 0 replies; 9+ messages in thread From: Jenkins @ 2020-05-05 8:42 UTC (permalink / raw) To: mchehab+samsung, linux-media; +Cc: builder From: builder@linuxtv.org Pull request: https://patchwork.linuxtv.org/patch/63596/ Build log: https://builder.linuxtv.org/job/patchwork/49600/ Build time: 00:02:11 Link: https://lore.kernel.org/linux-media/c95abf33-bfd4-489f-2c20-536e76b848ce@xs4all.nl gpg: Signature made Tue 05 May 2020 08:31:46 AM UTC gpg: using RSA key AAA7FFBA4D2D77EF4CAEA1421326E0CD23ABDCE5 gpg: Good signature from "Hans Verkuil <hverkuil-cisco@xs4all.nl>" [unknown] gpg: aka "Hans Verkuil <hverkuil@xs4all.nl>" [full] Summary: 2 patches and/or PDF generation with issues, being 0 at build time Error/warnings: Error #256 when running cat patches/0001-media-tegra-Add-Tegra210-Video-input-driver.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict: $ cat patches/0001-media-tegra-Add-Tegra210-Video-input-driver.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict -:76: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? -:650: WARNING: DT compatible string "nvidia,tegra210-csi" appears un-documented -- check ./Documentation/devicetree/bindings/ -:3260: WARNING: DT compatible string "nvidia,tegra210-csi" appears un-documented -- check ./Documentation/devicetree/bindings/ ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [GIT PULL FOR v5.8] Add Tegra210 Video input driver 2020-05-05 8:33 [GIT PULL FOR v5.8] Add Tegra210 Video input driver Hans Verkuil 2020-05-05 8:42 ` [GIT PULL FOR v5.8] Add Tegra210 Video input driver (#63596) Jenkins @ 2020-05-05 12:17 ` Mauro Carvalho Chehab 2020-05-05 12:24 ` Hans Verkuil 1 sibling, 1 reply; 9+ messages in thread From: Mauro Carvalho Chehab @ 2020-05-05 12:17 UTC (permalink / raw) To: Hans Verkuil; +Cc: Linux Media Mailing List, Sowjanya Komatineni, Rob Herring Em Tue, 5 May 2020 10:33:00 +0200 Hans Verkuil <hverkuil@xs4all.nl> 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. While the code sounds ok, I'll reject this PR. Please submit a new one together with the DT file properly reviewed by DT maintainers. Thanks, Mauro ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [GIT PULL FOR v5.8] Add Tegra210 Video input driver 2020-05-05 12:17 ` [GIT PULL FOR v5.8] Add Tegra210 Video input driver Mauro Carvalho Chehab @ 2020-05-05 12:24 ` Hans Verkuil 2020-05-05 14:15 ` Mauro Carvalho Chehab 0 siblings, 1 reply; 9+ messages in thread From: Hans Verkuil @ 2020-05-05 12:24 UTC (permalink / raw) To: Mauro Carvalho Chehab Cc: Linux Media Mailing List, Sowjanya Komatineni, Rob Herring, Thierry Reding On 05/05/2020 14:17, Mauro Carvalho Chehab wrote: > Em Tue, 5 May 2020 10:33:00 +0200 > Hans Verkuil <hverkuil@xs4all.nl> 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. Regards, Hans > > While the code sounds ok, I'll reject this PR. Please submit a new one > together with the DT file properly reviewed by DT maintainers. > > Thanks, > Mauro > ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [GIT PULL FOR v5.8] Add Tegra210 Video input driver 2020-05-05 12:24 ` Hans Verkuil @ 2020-05-05 14:15 ` Mauro Carvalho Chehab 2020-05-05 14:26 ` Hans Verkuil 0 siblings, 1 reply; 9+ messages in thread From: Mauro Carvalho Chehab @ 2020-05-05 14:15 UTC (permalink / raw) To: Hans Verkuil Cc: Linux Media Mailing List, Sowjanya Komatineni, Rob Herring, Thierry Reding Em Tue, 5 May 2020 14:24:22 +0200 Hans Verkuil <hverkuil@xs4all.nl> escreveu: > On 05/05/2020 14:17, Mauro Carvalho Chehab wrote: > > Em Tue, 5 May 2020 10:33:00 +0200 > > Hans Verkuil <hverkuil@xs4all.nl> 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. 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@kernel.org> Thanks, Mauro ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [GIT PULL FOR v5.8] Add Tegra210 Video input driver 2020-05-05 14:15 ` Mauro Carvalho Chehab @ 2020-05-05 14:26 ` Hans Verkuil 2020-05-05 20:22 ` Thierry Reding 0 siblings, 1 reply; 9+ messages in thread From: Hans Verkuil @ 2020-05-05 14:26 UTC (permalink / raw) To: Mauro Carvalho Chehab Cc: Linux Media Mailing List, Sowjanya Komatineni, Rob Herring, Thierry Reding On 05/05/2020 16:15, Mauro Carvalho Chehab wrote: > Em Tue, 5 May 2020 14:24:22 +0200 > Hans Verkuil <hverkuil@xs4all.nl> escreveu: > >> On 05/05/2020 14:17, Mauro Carvalho Chehab wrote: >>> Em Tue, 5 May 2020 10:33:00 +0200 >>> Hans Verkuil <hverkuil@xs4all.nl> 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@kernel.org> 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 ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [GIT PULL FOR v5.8] Add Tegra210 Video input driver 2020-05-05 14:26 ` Hans Verkuil @ 2020-05-05 20:22 ` Thierry Reding 2020-05-05 20:39 ` Mauro Carvalho Chehab 0 siblings, 1 reply; 9+ messages in thread From: Thierry Reding @ 2020-05-05 20:22 UTC (permalink / raw) To: Hans Verkuil, Mauro Carvalho Chehab Cc: Linux Media Mailing List, Sowjanya Komatineni, Rob Herring [-- Attachment #1: Type: text/plain, Size: 4060 bytes --] On Tue, May 05, 2020 at 04:26:04PM +0200, Hans Verkuil wrote: > On 05/05/2020 16:15, Mauro Carvalho Chehab wrote: > > Em Tue, 5 May 2020 14:24:22 +0200 > > Hans Verkuil <hverkuil@xs4all.nl> escreveu: > > > >> On 05/05/2020 14:17, Mauro Carvalho Chehab wrote: > >>> Em Tue, 5 May 2020 10:33:00 +0200 > >>> Hans Verkuil <hverkuil@xs4all.nl> 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@kernel.org> Mauro, do you mind if I turn that into an Acked-by? I think the ARM SoC maintainers may prefer that as being more of an explicit statement than Reviewed-by: that it's fine to take this through ARM SoC. > > 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. I can do that. For ARM SoC I typically create a for-X.Y/dt-bindings branch where I collect device tree bindings patches that don't have a better place anywhere else, so I'll probably stick the DT binding update into that branch and then I can base a for-5.8/media branch on top of that to resolve the checkpatch warnings. Actually, once I have that I could even just send a PR to Mauro to take this through the media tree since the dependencies should all be resolved. But with an Acked-by I'm sure the ARM SoC maintainers won't mind picking this up. Thierry [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [GIT PULL FOR v5.8] Add Tegra210 Video input driver 2020-05-05 20:22 ` Thierry Reding @ 2020-05-05 20:39 ` Mauro Carvalho Chehab 0 siblings, 0 replies; 9+ messages in thread From: Mauro Carvalho Chehab @ 2020-05-05 20:39 UTC (permalink / raw) To: Thierry Reding Cc: Hans Verkuil, Linux Media Mailing List, Sowjanya Komatineni, Rob Herring Em Tue, 5 May 2020 22:22:20 +0200 Thierry Reding <thierry.reding@gmail.com> escreveu: > On Tue, May 05, 2020 at 04:26:04PM +0200, Hans Verkuil wrote: > > On 05/05/2020 16:15, Mauro Carvalho Chehab wrote: > > > Em Tue, 5 May 2020 14:24:22 +0200 > > > Hans Verkuil <hverkuil@xs4all.nl> escreveu: > > > > > >> On 05/05/2020 14:17, Mauro Carvalho Chehab wrote: > > >>> Em Tue, 5 May 2020 10:33:00 +0200 > > >>> Hans Verkuil <hverkuil@xs4all.nl> 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@kernel.org> > > Mauro, do you mind if I turn that into an Acked-by? I think the ARM SoC > maintainers may prefer that as being more of an explicit statement than > Reviewed-by: that it's fine to take this through ARM SoC. Sure, go ahead. > > > > > 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. > > I can do that. For ARM SoC I typically create a for-X.Y/dt-bindings > branch where I collect device tree bindings patches that don't have > a better place anywhere else, so I'll probably stick the DT binding > update into that branch and then I can base a for-5.8/media branch > on top of that to resolve the checkpatch warnings. > > Actually, once I have that I could even just send a PR to Mauro to > take this through the media tree since the dependencies should all > be resolved. But with an Acked-by I'm sure the ARM SoC maintainers > won't mind picking this up. Whatever works best. I don't mind to either pick it together with the DT, or to merge it via some other tree with either my acked-by or reviewed-by. Thanks, Mauro ^ permalink raw reply [flat|nested] 9+ messages in thread
* [GIT PULL FOR v5.8] Add Tegra210 Video input driver @ 2020-04-24 12:00 Hans Verkuil 0 siblings, 0 replies; 9+ messages in thread From: Hans Verkuil @ 2020-04-24 12:00 UTC (permalink / raw) To: Linux Media Mailing List; +Cc: Sowjanya Komatineni, Thierry Reding Hi Mauro, This adds the Tegra video input driver. It currently only supports the test pattern generator, the next phase will add sensor support. Sowjanya, thank you for all your hard work on this driver! Regards, Hans PS: The dt-bindings patch https://patchwork.linuxtv.org/patch/63331 will go through Thierry, together with other non-media patches. The bindings for this device are a bit unusual since they are under bindings/display due to the HW design. The following changes since commit 4bdbff4da40584ec2225bb429b7c66ad54d19cda: media: tuners: Kconfig: add some missing VIDEO_V4L2 dependencies (2020-04-23 07:57:27 +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 81ac3132ed589757063d65928b13148bb32b0f08: MAINTAINERS: Add Tegra Video driver section (2020-04-24 13:15:18 +0200) ---------------------------------------------------------------- Tag branch ---------------------------------------------------------------- Sowjanya Komatineni (2): media: tegra: Add Tegra210 Video input driver MAINTAINERS: Add Tegra Video driver section MAINTAINERS | 10 + drivers/staging/media/Kconfig | 2 + drivers/staging/media/Makefile | 1 + drivers/staging/media/tegra/Kconfig | 12 + drivers/staging/media/tegra/Makefile | 8 + drivers/staging/media/tegra/TODO | 10 + drivers/staging/media/tegra/common.h | 259 ++++++++++++++ drivers/staging/media/tegra/csi.c | 604 +++++++++++++++++++++++++++++++++ drivers/staging/media/tegra/csi.h | 144 ++++++++ drivers/staging/media/tegra/tegra210.c | 708 ++++++++++++++++++++++++++++++++++++++ drivers/staging/media/tegra/tegra210.h | 190 +++++++++++ drivers/staging/media/tegra/vi.c | 1127 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ drivers/staging/media/tegra/vi.h | 72 ++++ drivers/staging/media/tegra/video.c | 153 +++++++++ drivers/staging/media/tegra/video.h | 29 ++ 15 files changed, 3329 insertions(+) create mode 100644 drivers/staging/media/tegra/Kconfig create mode 100644 drivers/staging/media/tegra/Makefile create mode 100644 drivers/staging/media/tegra/TODO create mode 100644 drivers/staging/media/tegra/common.h create mode 100644 drivers/staging/media/tegra/csi.c create mode 100644 drivers/staging/media/tegra/csi.h create mode 100644 drivers/staging/media/tegra/tegra210.c create mode 100644 drivers/staging/media/tegra/tegra210.h create mode 100644 drivers/staging/media/tegra/vi.c create mode 100644 drivers/staging/media/tegra/vi.h create mode 100644 drivers/staging/media/tegra/video.c create mode 100644 drivers/staging/media/tegra/video.h ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2020-05-05 20:39 UTC | newest] Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2020-05-05 8:33 [GIT PULL FOR v5.8] Add Tegra210 Video input driver Hans Verkuil 2020-05-05 8:42 ` [GIT PULL FOR v5.8] Add Tegra210 Video input driver (#63596) Jenkins 2020-05-05 12:17 ` [GIT PULL FOR v5.8] Add Tegra210 Video input driver Mauro Carvalho Chehab 2020-05-05 12:24 ` Hans Verkuil 2020-05-05 14:15 ` Mauro Carvalho Chehab 2020-05-05 14:26 ` Hans Verkuil 2020-05-05 20:22 ` Thierry Reding 2020-05-05 20:39 ` Mauro Carvalho Chehab -- strict thread matches above, loose matches on Subject: below -- 2020-04-24 12:00 Hans Verkuil
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.