From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sowjanya Komatineni Subject: Re: [RFC PATCH v1 0/5] media: tegra: Tegra video driver follow-up patches Date: Tue, 28 Apr 2020 07:52:39 -0700 Message-ID: <8ae1bfca-e4bf-ed67-8945-646a6121875a@nvidia.com> References: <1588047650-29402-1-git-send-email-skomatineni@nvidia.com> <5f065fc5-0f45-6b9e-068c-28d0a2e4684e@xs4all.nl> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <5f065fc5-0f45-6b9e-068c-28d0a2e4684e-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org> Content-Language: en-US Sender: linux-tegra-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Hans Verkuil , thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org, frankc-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org Cc: digetx-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, linux-media-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-tegra@vger.kernel.org On 4/28/20 12:38 AM, Hans Verkuil wrote: > External email: Use caution opening links or attachments > > > On 28/04/2020 06:20, Sowjanya Komatineni wrote: >> This is a fllow-up to some last feedback received for Tegra VI driver RFC >> v10 patches. >> https://patchwork.linuxtv.org/cover/63334/ >> >> >> Sowjanya Komatineni (5): >> MAINTAINERS: Rename Tegra Video driver path >> media: tegra: Rename driver path to tegra-video >> media: tegra-video: Move PM runtime handle to streaming >> media: tegra-video: Rearrange SoC specific to Tegra210 and cleanup >> files >> media: tegra-video: Remove module macros for vi and csi driver >> >> MAINTAINERS | 2 +- >> drivers/staging/media/Kconfig | 2 +- >> drivers/staging/media/Makefile | 2 +- >> drivers/staging/media/tegra-video/Kconfig | 12 + >> drivers/staging/media/tegra-video/Makefile | 8 + >> drivers/staging/media/tegra-video/TODO | 10 + >> drivers/staging/media/tegra-video/csi.c | 536 ++++++++++++ >> drivers/staging/media/tegra-video/csi.h | 146 ++++ >> drivers/staging/media/tegra-video/tegra210.c | 1080 ++++++++++++++++++++++++ >> drivers/staging/media/tegra-video/vi.c | 1082 +++++++++++++++++++++++++ >> drivers/staging/media/tegra-video/vi.h | 258 ++++++ >> drivers/staging/media/tegra-video/video.c | 155 ++++ >> drivers/staging/media/tegra-video/video.h | 29 + >> 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 - >> 25 files changed, 3319 insertions(+), 3319 deletions(-) > I thought that the follow-up series was just a rename of a directory > and perhaps one or two smaller changes, but this is too much change. > I prefer to have a v11 instead with all these changes incorporated. > > Sorry about that, > > Hans OK Will send v11. Thanks Hans > >> 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 >> delete mode 100644 drivers/staging/media/tegra/Kconfig >> delete mode 100644 drivers/staging/media/tegra/Makefile >> delete mode 100644 drivers/staging/media/tegra/TODO >> delete mode 100644 drivers/staging/media/tegra/common.h >> delete mode 100644 drivers/staging/media/tegra/csi.c >> delete mode 100644 drivers/staging/media/tegra/csi.h >> delete mode 100644 drivers/staging/media/tegra/tegra210.c >> delete mode 100644 drivers/staging/media/tegra/tegra210.h >> delete mode 100644 drivers/staging/media/tegra/vi.c >> delete mode 100644 drivers/staging/media/tegra/vi.h >> delete mode 100644 drivers/staging/media/tegra/video.c >> delete mode 100644 drivers/staging/media/tegra/video.h >>