Message ID | 1542032768-20453-3-git-send-email-jonathanh@nvidia.com |
---|---|
State | Accepted |
Headers | show |
Series | [1/3] clk: tegra: Fix maximum audio sync clock for Tegra124/210 | expand |
On Mon, Nov 12, 2018 at 02:26:08PM +0000, Jon Hunter wrote: > Now there are no more external users of tegra_powergate_is_powered(), > make this a local function. > > Signed-off-by: Jon Hunter <jonathanh@nvidia.com> > --- > drivers/soc/tegra/pmc.c | 2 +- > include/soc/tegra/pmc.h | 6 ------ > 2 files changed, 1 insertion(+), 7 deletions(-) Given the dependency on patch 2/3, I think it may be best to wait until that has been applied and apply this cleanup patch in a subsequent release. Oh, and I just noticed that you didn't Cc the clock maintainers on patches 1 & 2. Perhaps resend both of them to the clock maintainers with my Acked-by and then we'll deal with this patch (3/3) when the time is right. Thierry
diff --git a/drivers/soc/tegra/pmc.c b/drivers/soc/tegra/pmc.c index 1fa840e3d930..5490e3936f17 100644 --- a/drivers/soc/tegra/pmc.c +++ b/drivers/soc/tegra/pmc.c @@ -536,7 +536,7 @@ EXPORT_SYMBOL(tegra_powergate_power_off); * tegra_powergate_is_powered() - check if partition is powered * @id: partition ID */ -int tegra_powergate_is_powered(unsigned int id) +static int tegra_powergate_is_powered(unsigned int id) { int status; diff --git a/include/soc/tegra/pmc.h b/include/soc/tegra/pmc.h index 562426812ab2..a242ea5b46e4 100644 --- a/include/soc/tegra/pmc.h +++ b/include/soc/tegra/pmc.h @@ -142,7 +142,6 @@ enum tegra_io_pad { #define TEGRA_IO_RAIL_LVDS TEGRA_IO_PAD_LVDS #ifdef CONFIG_SOC_TEGRA_PMC -int tegra_powergate_is_powered(unsigned int id); int tegra_powergate_power_on(unsigned int id); int tegra_powergate_power_off(unsigned int id); int tegra_powergate_remove_clamping(unsigned int id); @@ -163,11 +162,6 @@ void tegra_pmc_set_suspend_mode(enum tegra_suspend_mode mode); void tegra_pmc_enter_suspend_mode(enum tegra_suspend_mode mode); #else -static inline int tegra_powergate_is_powered(unsigned int id) -{ - return -ENOSYS; -} - static inline int tegra_powergate_power_on(unsigned int id) { return -ENOSYS;
Now there are no more external users of tegra_powergate_is_powered(), make this a local function. Signed-off-by: Jon Hunter <jonathanh@nvidia.com> --- drivers/soc/tegra/pmc.c | 2 +- include/soc/tegra/pmc.h | 6 ------ 2 files changed, 1 insertion(+), 7 deletions(-)