From: Thierry Reding <thierry.reding@gmail.com> To: Dmitry Osipenko <digetx@gmail.com> Cc: devel@driverdev.osuosl.org, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Jonathan Hunter <jonathanh@nvidia.com>, linux-tegra@vger.kernel.org, Mauro Carvalho Chehab <mchehab@kernel.org>, linux-media@vger.kernel.org Subject: Re: [PATCH 01/14] staging: media: tegra-vde: Support BSEV clock and reset Date: Tue, 14 Aug 2018 16:21:24 +0200 [thread overview] Message-ID: <20180814142124.GA21075@ulmo> (raw) In-Reply-To: <2754354.GStWHyBo4g@dimapc> [-- Attachment #1.1: Type: text/plain, Size: 1181 bytes --] On Mon, Aug 13, 2018 at 06:09:46PM +0300, Dmitry Osipenko wrote: > On Monday, 13 August 2018 17:50:14 MSK Thierry Reding wrote: > > From: Thierry Reding <treding@nvidia.com> > > > > The BSEV clock has a separate gate bit and can not be assumed to be > > always enabled. Add explicit handling for the BSEV clock and reset. > > > > This fixes an issue on Tegra124 where the BSEV clock is not enabled > > by default and therefore accessing the BSEV registers will hang the > > CPU if the BSEV clock is not enabled and the reset not deasserted. > > > > Signed-off-by: Thierry Reding <treding@nvidia.com> > > --- > > Are you sure that BSEV clock is really needed for T20/30? I've tried already > to disable the clock explicitly and everything kept working, though I'll try > again. I think you're right that these aren't strictly required for VDE to work on Tegra20 and Tegra30. However, the BSEV clock and reset do exist on those platforms, so I didn't see a reason why they shouldn't be handled uniformly across all generations. > The device-tree changes should be reflected in the binding documentation. Indeed, I forgot to update that. Thierry [-- Attachment #1.2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] [-- Attachment #2: Type: text/plain, Size: 169 bytes --] _______________________________________________ devel mailing list devel@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
WARNING: multiple messages have this Message-ID (diff)
From: Thierry Reding <thierry.reding@gmail.com> To: Dmitry Osipenko <digetx@gmail.com> Cc: Mauro Carvalho Chehab <mchehab@kernel.org>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Jonathan Hunter <jonathanh@nvidia.com>, linux-media@vger.kernel.org, linux-tegra@vger.kernel.org, devel@driverdev.osuosl.org Subject: Re: [PATCH 01/14] staging: media: tegra-vde: Support BSEV clock and reset Date: Tue, 14 Aug 2018 16:21:24 +0200 [thread overview] Message-ID: <20180814142124.GA21075@ulmo> (raw) In-Reply-To: <2754354.GStWHyBo4g@dimapc> [-- Attachment #1: Type: text/plain, Size: 1181 bytes --] On Mon, Aug 13, 2018 at 06:09:46PM +0300, Dmitry Osipenko wrote: > On Monday, 13 August 2018 17:50:14 MSK Thierry Reding wrote: > > From: Thierry Reding <treding@nvidia.com> > > > > The BSEV clock has a separate gate bit and can not be assumed to be > > always enabled. Add explicit handling for the BSEV clock and reset. > > > > This fixes an issue on Tegra124 where the BSEV clock is not enabled > > by default and therefore accessing the BSEV registers will hang the > > CPU if the BSEV clock is not enabled and the reset not deasserted. > > > > Signed-off-by: Thierry Reding <treding@nvidia.com> > > --- > > Are you sure that BSEV clock is really needed for T20/30? I've tried already > to disable the clock explicitly and everything kept working, though I'll try > again. I think you're right that these aren't strictly required for VDE to work on Tegra20 and Tegra30. However, the BSEV clock and reset do exist on those platforms, so I didn't see a reason why they shouldn't be handled uniformly across all generations. > The device-tree changes should be reflected in the binding documentation. Indeed, I forgot to update that. Thierry [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --]
next prev parent reply other threads:[~2018-08-14 14:21 UTC|newest] Thread overview: 72+ messages / expand[flat|nested] mbox.gz Atom feed top 2018-08-13 14:50 [PATCH 00/14] staging: media: tegra-vdea: Add Tegra124 support Thierry Reding 2018-08-13 14:50 ` Thierry Reding 2018-08-13 14:50 ` [PATCH 01/14] staging: media: tegra-vde: Support BSEV clock and reset Thierry Reding 2018-08-13 14:50 ` Thierry Reding 2018-08-13 15:09 ` Dmitry Osipenko 2018-08-13 15:09 ` Dmitry Osipenko 2018-08-14 14:21 ` Thierry Reding [this message] 2018-08-14 14:21 ` Thierry Reding 2018-08-14 15:05 ` Dmitry Osipenko 2018-08-14 15:05 ` Dmitry Osipenko 2018-08-14 15:16 ` Dmitry Osipenko 2018-08-14 15:16 ` Dmitry Osipenko 2018-08-13 14:50 ` [PATCH 02/14] staging: media: tegra-vde: Support reference picture marking Thierry Reding 2018-08-13 14:50 ` Thierry Reding 2018-08-18 12:48 ` Dmitry Osipenko 2018-08-18 12:48 ` Dmitry Osipenko 2018-08-13 14:50 ` [PATCH 03/14] staging: media: tegra-vde: Prepare for interlacing support Thierry Reding 2018-08-13 14:50 ` Thierry Reding 2018-08-18 12:48 ` Dmitry Osipenko 2018-08-18 12:48 ` Dmitry Osipenko 2018-08-30 8:56 ` Dan Carpenter 2018-08-30 8:56 ` Dan Carpenter 2018-08-13 14:50 ` [PATCH 04/14] staging: media: tegra-vde: Use DRM/KMS framebuffer modifiers Thierry Reding 2018-08-13 14:50 ` Thierry Reding 2018-08-18 12:53 ` Dmitry Osipenko 2018-08-18 12:53 ` Dmitry Osipenko 2018-08-13 14:50 ` [PATCH 05/14] staging: media: tegra-vde: Properly mark invalid entries Thierry Reding 2018-08-13 14:50 ` Thierry Reding 2018-08-18 12:45 ` Dmitry Osipenko 2018-08-18 12:45 ` Dmitry Osipenko 2018-08-13 14:50 ` [PATCH 06/14] staging: media: tegra-vde: Print out invalid FD Thierry Reding 2018-08-13 14:50 ` Thierry Reding 2018-08-18 12:45 ` Dmitry Osipenko 2018-08-18 12:45 ` Dmitry Osipenko 2018-08-13 14:50 ` [PATCH 07/14] staging: media: tegra-vde: Add some clarifying comments Thierry Reding 2018-08-13 14:50 ` Thierry Reding 2018-08-18 12:50 ` Dmitry Osipenko 2018-08-18 12:50 ` Dmitry Osipenko 2018-08-13 14:50 ` [PATCH 08/14] staging: media: tegra-vde: Track struct device * Thierry Reding 2018-08-13 14:50 ` Thierry Reding 2018-08-18 12:49 ` Dmitry Osipenko 2018-08-18 12:49 ` Dmitry Osipenko 2018-08-18 15:39 ` Dmitry Osipenko 2018-08-18 15:39 ` Dmitry Osipenko 2018-08-13 14:50 ` [PATCH 09/14] staging: media: tegra-vde: Add IOMMU support Thierry Reding 2018-08-13 14:50 ` Thierry Reding 2018-08-18 12:50 ` Dmitry Osipenko 2018-08-18 12:50 ` Dmitry Osipenko 2018-08-18 13:07 ` Dmitry Osipenko 2018-08-18 13:07 ` Dmitry Osipenko 2018-08-18 13:29 ` Dmitry Osipenko 2018-08-18 13:29 ` Dmitry Osipenko 2018-08-13 14:50 ` [PATCH 10/14] staging: media: tegra-vde: Keep VDE in reset when unused Thierry Reding 2018-08-13 14:50 ` Thierry Reding 2018-08-18 12:50 ` Dmitry Osipenko 2018-08-18 12:50 ` Dmitry Osipenko 2018-08-13 14:50 ` [PATCH 11/14] ARM: tegra: Enable VDE on Tegra124 Thierry Reding 2018-08-13 14:50 ` Thierry Reding 2018-08-18 12:45 ` Dmitry Osipenko 2018-08-18 12:45 ` Dmitry Osipenko 2018-08-13 14:50 ` [PATCH 12/14] ARM: tegra: Add BSEV clock and reset for VDE on Tegra20 Thierry Reding 2018-08-13 14:50 ` Thierry Reding 2018-08-13 14:50 ` [PATCH 13/14] ARM: tegra: Add BSEV clock and reset for VDE on Tegra30 Thierry Reding 2018-08-13 14:50 ` Thierry Reding 2018-08-13 14:50 ` [PATCH 14/14] ARM: tegra: Enable SMMU for VDE on Tegra124 Thierry Reding 2018-08-13 14:50 ` Thierry Reding 2018-08-18 12:45 ` Dmitry Osipenko 2018-08-18 12:45 ` Dmitry Osipenko 2018-09-03 12:18 ` [PATCH 00/14] staging: media: tegra-vdea: Add Tegra124 support Hans Verkuil 2018-09-03 12:18 ` Hans Verkuil 2018-09-03 13:12 ` Thierry Reding 2018-09-03 13:12 ` Thierry Reding
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20180814142124.GA21075@ulmo \ --to=thierry.reding@gmail.com \ --cc=devel@driverdev.osuosl.org \ --cc=digetx@gmail.com \ --cc=gregkh@linuxfoundation.org \ --cc=jonathanh@nvidia.com \ --cc=linux-media@vger.kernel.org \ --cc=linux-tegra@vger.kernel.org \ --cc=mchehab@kernel.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
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.