aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJon Hunter <jonathanh@nvidia.com>2016-02-11 13:03:25 -0500
committerThierry Reding <treding@nvidia.com>2016-04-05 09:22:50 -0400
commitc3ea297260b77a99a2383c1029a381d125f788fe (patch)
tree1ba83e7adc05685999649a2c886e787d88151b53
parent0a243bd438ce3d44e03140adf58df4df11dce978 (diff)
soc/tegra: pmc: Remove additional check for a valid partition
The function tegra_powergate_is_powered() verifies that the partition being queried is valid and so there is no need to check this before calling tegra_powergate_is_powered() in powergate_show(). So remove this extra check. Signed-off-by: Jon Hunter <jonathanh@nvidia.com> Reviewed-by: Mathieu Poirier <mathieu.poirier@linaro.org> Signed-off-by: Thierry Reding <treding@nvidia.com>
-rw-r--r--drivers/soc/tegra/pmc.c6
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/soc/tegra/pmc.c b/drivers/soc/tegra/pmc.c
index e75782b47267..8e358dbffaed 100644
--- a/drivers/soc/tegra/pmc.c
+++ b/drivers/soc/tegra/pmc.c
@@ -434,16 +434,18 @@ static struct notifier_block tegra_pmc_restart_handler = {
434static int powergate_show(struct seq_file *s, void *data) 434static int powergate_show(struct seq_file *s, void *data)
435{ 435{
436 unsigned int i; 436 unsigned int i;
437 int status;
437 438
438 seq_printf(s, " powergate powered\n"); 439 seq_printf(s, " powergate powered\n");
439 seq_printf(s, "------------------\n"); 440 seq_printf(s, "------------------\n");
440 441
441 for (i = 0; i < pmc->soc->num_powergates; i++) { 442 for (i = 0; i < pmc->soc->num_powergates; i++) {
442 if (!pmc->soc->powergates[i]) 443 status = tegra_powergate_is_powered(i);
444 if (status < 0)
443 continue; 445 continue;
444 446
445 seq_printf(s, " %9s %7s\n", pmc->soc->powergates[i], 447 seq_printf(s, " %9s %7s\n", pmc->soc->powergates[i],
446 tegra_powergate_is_powered(i) ? "yes" : "no"); 448 status ? "yes" : "no");
447 } 449 }
448 450
449 return 0; 451 return 0;