Message ID | 1455213806-21871-9-git-send-email-jonathanh@nvidia.com |
---|---|
State | Superseded, archived |
Headers | show |
On 11/02/16 18:03, Jon Hunter wrote: > For Tegra124 and Tegra210, the GPU partition cannot be toggled on and > off via the APBDEV_PMC_PWRGATE_TOGGLE_0 register. For these devices, the > partition is simply powered up and down via an external regulator. > Describe in the PMC SoC data in which devices the GPU partition can be > controlled via the APBDEV_PMC_PWRGATE_TOGGLE_0 register and ensure that > no one can incorrectly try to toggle the GPU partition via the > APBDEV_PMC_PWRGATE_TOGGLE_0 register. > > Furthermore, because we cannot use the APBDEV_PMC_PWRGATE_STATUS_0 > register to determine if the GPU partition is powered, use the > APBDEV_PMC_CLAMP_STATUS_0 register to determine if the GPU partition > is powered. The APBDEV_PMC_CLAMP_STATUS_0 register is bit wise > compatible with the APBDEV_PMC_PWRGATE_STATUS_0 register and if the > clamp is disabled (ie. the appropriate bit == 0), then this indicates > the partition is powered. > > Signed-off-by: Jon Hunter <jonathanh@nvidia.com> > --- > drivers/soc/tegra/pmc.c | 15 ++++++++++++++- > 1 file changed, 14 insertions(+), 1 deletion(-) > > diff --git a/drivers/soc/tegra/pmc.c b/drivers/soc/tegra/pmc.c > index 8e358dbffaed..442232269df3 100644 > --- a/drivers/soc/tegra/pmc.c > +++ b/drivers/soc/tegra/pmc.c > @@ -52,6 +52,8 @@ > #define DPD_SAMPLE_ENABLE (1 << 0) > #define DPD_SAMPLE_DISABLE (0 << 0) > > +#define CLAMP_STATUS 0x2c > + > #define PWRGATE_TOGGLE 0x30 > #define PWRGATE_TOGGLE_START (1 << 8) > > @@ -109,6 +111,7 @@ struct tegra_pmc_soc { > > bool has_tsense_reset; > bool has_gpu_clamps; > + bool has_gpu_toggle; > }; > > /** > @@ -176,7 +179,10 @@ static void tegra_pmc_writel(u32 value, unsigned long offset) > > static inline bool tegra_powergate_state(int id) > { > - return (tegra_pmc_readl(PWRGATE_STATUS) & BIT(id)) != 0; > + if (id == TEGRA_POWERGATE_3D && !pmc->soc->has_gpu_toggle) > + return (tegra_pmc_readl(CLAMP_STATUS) & BIT(id)) == 0; Well, this is still not right. For devices with !pmc->soc->has_gpu_toggle, these devices have their own separate register for removing the clamp and so the APBDEV_PMC_CLAMP_STATUS_0 register will not tell us the status. I need to look at this some more. Jon -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Fri, Feb 12, 2016 at 09:38:46AM +0000, Jon Hunter wrote: > > On 11/02/16 18:03, Jon Hunter wrote: > > For Tegra124 and Tegra210, the GPU partition cannot be toggled on and > > off via the APBDEV_PMC_PWRGATE_TOGGLE_0 register. For these devices, the > > partition is simply powered up and down via an external regulator. > > Describe in the PMC SoC data in which devices the GPU partition can be > > controlled via the APBDEV_PMC_PWRGATE_TOGGLE_0 register and ensure that > > no one can incorrectly try to toggle the GPU partition via the > > APBDEV_PMC_PWRGATE_TOGGLE_0 register. > > > > Furthermore, because we cannot use the APBDEV_PMC_PWRGATE_STATUS_0 > > register to determine if the GPU partition is powered, use the > > APBDEV_PMC_CLAMP_STATUS_0 register to determine if the GPU partition > > is powered. The APBDEV_PMC_CLAMP_STATUS_0 register is bit wise > > compatible with the APBDEV_PMC_PWRGATE_STATUS_0 register and if the > > clamp is disabled (ie. the appropriate bit == 0), then this indicates > > the partition is powered. > > > > Signed-off-by: Jon Hunter <jonathanh@nvidia.com> > > --- > > drivers/soc/tegra/pmc.c | 15 ++++++++++++++- > > 1 file changed, 14 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/soc/tegra/pmc.c b/drivers/soc/tegra/pmc.c > > index 8e358dbffaed..442232269df3 100644 > > --- a/drivers/soc/tegra/pmc.c > > +++ b/drivers/soc/tegra/pmc.c > > @@ -52,6 +52,8 @@ > > #define DPD_SAMPLE_ENABLE (1 << 0) > > #define DPD_SAMPLE_DISABLE (0 << 0) > > > > +#define CLAMP_STATUS 0x2c > > + > > #define PWRGATE_TOGGLE 0x30 > > #define PWRGATE_TOGGLE_START (1 << 8) > > > > @@ -109,6 +111,7 @@ struct tegra_pmc_soc { > > > > bool has_tsense_reset; > > bool has_gpu_clamps; > > + bool has_gpu_toggle; > > }; > > > > /** > > @@ -176,7 +179,10 @@ static void tegra_pmc_writel(u32 value, unsigned long offset) > > > > static inline bool tegra_powergate_state(int id) > > { > > - return (tegra_pmc_readl(PWRGATE_STATUS) & BIT(id)) != 0; > > + if (id == TEGRA_POWERGATE_3D && !pmc->soc->has_gpu_toggle) > > + return (tegra_pmc_readl(CLAMP_STATUS) & BIT(id)) == 0; > > Well, this is still not right. For devices with > !pmc->soc->has_gpu_toggle, these devices have their own separate > register for removing the clamp and so the APBDEV_PMC_CLAMP_STATUS_0 > register will not tell us the status. I need to look at this some more. Any objections to me applying patches 1-7 while you work on fixing this last one? Thierry
On 12/02/16 17:25, Thierry Reding wrote: > * PGP Signed by an unknown key > > On Fri, Feb 12, 2016 at 09:38:46AM +0000, Jon Hunter wrote: >> >> On 11/02/16 18:03, Jon Hunter wrote: >>> For Tegra124 and Tegra210, the GPU partition cannot be toggled on and >>> off via the APBDEV_PMC_PWRGATE_TOGGLE_0 register. For these devices, the >>> partition is simply powered up and down via an external regulator. >>> Describe in the PMC SoC data in which devices the GPU partition can be >>> controlled via the APBDEV_PMC_PWRGATE_TOGGLE_0 register and ensure that >>> no one can incorrectly try to toggle the GPU partition via the >>> APBDEV_PMC_PWRGATE_TOGGLE_0 register. >>> >>> Furthermore, because we cannot use the APBDEV_PMC_PWRGATE_STATUS_0 >>> register to determine if the GPU partition is powered, use the >>> APBDEV_PMC_CLAMP_STATUS_0 register to determine if the GPU partition >>> is powered. The APBDEV_PMC_CLAMP_STATUS_0 register is bit wise >>> compatible with the APBDEV_PMC_PWRGATE_STATUS_0 register and if the >>> clamp is disabled (ie. the appropriate bit == 0), then this indicates >>> the partition is powered. >>> >>> Signed-off-by: Jon Hunter <jonathanh@nvidia.com> >>> --- >>> drivers/soc/tegra/pmc.c | 15 ++++++++++++++- >>> 1 file changed, 14 insertions(+), 1 deletion(-) >>> >>> diff --git a/drivers/soc/tegra/pmc.c b/drivers/soc/tegra/pmc.c >>> index 8e358dbffaed..442232269df3 100644 >>> --- a/drivers/soc/tegra/pmc.c >>> +++ b/drivers/soc/tegra/pmc.c >>> @@ -52,6 +52,8 @@ >>> #define DPD_SAMPLE_ENABLE (1 << 0) >>> #define DPD_SAMPLE_DISABLE (0 << 0) >>> >>> +#define CLAMP_STATUS 0x2c >>> + >>> #define PWRGATE_TOGGLE 0x30 >>> #define PWRGATE_TOGGLE_START (1 << 8) >>> >>> @@ -109,6 +111,7 @@ struct tegra_pmc_soc { >>> >>> bool has_tsense_reset; >>> bool has_gpu_clamps; >>> + bool has_gpu_toggle; >>> }; >>> >>> /** >>> @@ -176,7 +179,10 @@ static void tegra_pmc_writel(u32 value, unsigned long offset) >>> >>> static inline bool tegra_powergate_state(int id) >>> { >>> - return (tegra_pmc_readl(PWRGATE_STATUS) & BIT(id)) != 0; >>> + if (id == TEGRA_POWERGATE_3D && !pmc->soc->has_gpu_toggle) >>> + return (tegra_pmc_readl(CLAMP_STATUS) & BIT(id)) == 0; >> >> Well, this is still not right. For devices with >> !pmc->soc->has_gpu_toggle, these devices have their own separate >> register for removing the clamp and so the APBDEV_PMC_CLAMP_STATUS_0 >> register will not tell us the status. I need to look at this some more. > > Any objections to me applying patches 1-7 while you work on fixing this > last one? Not at all. Please go ahead. Cheers Jon -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/soc/tegra/pmc.c b/drivers/soc/tegra/pmc.c index 8e358dbffaed..442232269df3 100644 --- a/drivers/soc/tegra/pmc.c +++ b/drivers/soc/tegra/pmc.c @@ -52,6 +52,8 @@ #define DPD_SAMPLE_ENABLE (1 << 0) #define DPD_SAMPLE_DISABLE (0 << 0) +#define CLAMP_STATUS 0x2c + #define PWRGATE_TOGGLE 0x30 #define PWRGATE_TOGGLE_START (1 << 8) @@ -109,6 +111,7 @@ struct tegra_pmc_soc { bool has_tsense_reset; bool has_gpu_clamps; + bool has_gpu_toggle; }; /** @@ -176,7 +179,10 @@ static void tegra_pmc_writel(u32 value, unsigned long offset) static inline bool tegra_powergate_state(int id) { - return (tegra_pmc_readl(PWRGATE_STATUS) & BIT(id)) != 0; + if (id == TEGRA_POWERGATE_3D && !pmc->soc->has_gpu_toggle) + return (tegra_pmc_readl(CLAMP_STATUS) & BIT(id)) == 0; + else + return (tegra_pmc_readl(PWRGATE_STATUS) & BIT(id)) != 0; } static inline bool tegra_powergate_is_valid(int id) @@ -191,6 +197,9 @@ static inline bool tegra_powergate_is_valid(int id) */ static int tegra_powergate_set(unsigned int id, bool new_state) { + if (id == TEGRA_POWERGATE_3D && !pmc->soc->has_gpu_toggle) + return -EINVAL; + mutex_lock(&pmc->powergates_lock); if (tegra_powergate_state(id) == new_state) { @@ -951,6 +960,7 @@ static const struct tegra_pmc_soc tegra30_pmc_soc = { .cpu_powergates = tegra30_cpu_powergates, .has_tsense_reset = true, .has_gpu_clamps = false, + .has_gpu_toggle = true, }; static const char * const tegra114_powergates[] = { @@ -988,6 +998,7 @@ static const struct tegra_pmc_soc tegra114_pmc_soc = { .cpu_powergates = tegra114_cpu_powergates, .has_tsense_reset = true, .has_gpu_clamps = false, + .has_gpu_toggle = true, }; static const char * const tegra124_powergates[] = { @@ -1030,6 +1041,7 @@ static const struct tegra_pmc_soc tegra124_pmc_soc = { .cpu_powergates = tegra124_cpu_powergates, .has_tsense_reset = true, .has_gpu_clamps = true, + .has_gpu_toggle = false, }; static const char * const tegra210_powergates[] = { @@ -1073,6 +1085,7 @@ static const struct tegra_pmc_soc tegra210_pmc_soc = { .cpu_powergates = tegra210_cpu_powergates, .has_tsense_reset = true, .has_gpu_clamps = true, + .has_gpu_toggle = false, }; static const struct of_device_id tegra_pmc_match[] = {
For Tegra124 and Tegra210, the GPU partition cannot be toggled on and off via the APBDEV_PMC_PWRGATE_TOGGLE_0 register. For these devices, the partition is simply powered up and down via an external regulator. Describe in the PMC SoC data in which devices the GPU partition can be controlled via the APBDEV_PMC_PWRGATE_TOGGLE_0 register and ensure that no one can incorrectly try to toggle the GPU partition via the APBDEV_PMC_PWRGATE_TOGGLE_0 register. Furthermore, because we cannot use the APBDEV_PMC_PWRGATE_STATUS_0 register to determine if the GPU partition is powered, use the APBDEV_PMC_CLAMP_STATUS_0 register to determine if the GPU partition is powered. The APBDEV_PMC_CLAMP_STATUS_0 register is bit wise compatible with the APBDEV_PMC_PWRGATE_STATUS_0 register and if the clamp is disabled (ie. the appropriate bit == 0), then this indicates the partition is powered. Signed-off-by: Jon Hunter <jonathanh@nvidia.com> --- drivers/soc/tegra/pmc.c | 15 ++++++++++++++- 1 file changed, 14 insertions(+), 1 deletion(-)