From db3cbaa469a21078db8c16f5f10a95643df0e8a2 Mon Sep 17 00:00:00 2001 From: Arend van Spriel Date: Wed, 29 Jun 2011 16:46:36 -0700 Subject: [PATCH] staging: brcm80211: remove otp power control functions For the devices supported by brcmsmac driver the OTP device is always powered. Functions for reading power status and changing it are not needed. Signed-off-by: Arend van Spriel Reviewed-by: Roland Vossen Signed-off-by: Greg Kroah-Hartman --- drivers/staging/brcm80211/brcmsmac/aiutils.c | 14 -------------- drivers/staging/brcm80211/brcmsmac/aiutils.h | 2 -- drivers/staging/brcm80211/brcmsmac/otp.c | 15 ++------------- 3 files changed, 2 insertions(+), 29 deletions(-) diff --git a/drivers/staging/brcm80211/brcmsmac/aiutils.c b/drivers/staging/brcm80211/brcmsmac/aiutils.c index d41104fef9a9..201866450d7d 100644 --- a/drivers/staging/brcm80211/brcmsmac/aiutils.c +++ b/drivers/staging/brcm80211/brcmsmac/aiutils.c @@ -2277,17 +2277,3 @@ bool ai_is_otp_disabled(struct si_pub *sih) return false; } } - -bool ai_is_otp_powered(struct si_pub *sih) -{ - if (PMUCTL_ENAB(sih)) - return si_pmu_is_otp_powered(sih); - return true; -} - -void ai_otp_power(struct si_pub *sih, bool on) -{ - if (PMUCTL_ENAB(sih)) - si_pmu_otp_power(sih, on); - udelay(1000); -} diff --git a/drivers/staging/brcm80211/brcmsmac/aiutils.h b/drivers/staging/brcm80211/brcmsmac/aiutils.h index 66d9471b33d3..965eb94a66dc 100644 --- a/drivers/staging/brcm80211/brcmsmac/aiutils.h +++ b/drivers/staging/brcm80211/brcmsmac/aiutils.h @@ -558,8 +558,6 @@ extern u32 ai_gpiocontrol(struct si_pub *sih, u32 mask, u32 val, /* OTP status */ extern bool ai_is_otp_disabled(struct si_pub *sih); -extern bool ai_is_otp_powered(struct si_pub *sih); -extern void ai_otp_power(struct si_pub *sih, bool on); /* SPROM availability */ extern bool ai_is_sprom_available(struct si_pub *sih); diff --git a/drivers/staging/brcm80211/brcmsmac/otp.c b/drivers/staging/brcm80211/brcmsmac/otp.c index 2cda60f83d8f..4e19b35b9f42 100644 --- a/drivers/staging/brcm80211/brcmsmac/otp.c +++ b/drivers/staging/brcm80211/brcmsmac/otp.c @@ -310,10 +310,7 @@ static void *ipxotp_init(struct si_pub *sih) if (ai_is_otp_disabled(sih)) return NULL; - /* Make sure OTP is powered up */ - if (!ai_is_otp_powered(sih)) - return NULL; - + /* OTP is always powered */ oi = &otpinfo; /* Check for otp size */ @@ -518,15 +515,10 @@ void *otp_init(struct si_pub *sih) int otp_read_region(struct si_pub *sih, int region, u16 *data, uint *wlen) { - bool wasup = false; void *oh; int err = 0; - wasup = ai_is_otp_powered(sih); - if (!wasup) - ai_otp_power(sih, true); - - if (!ai_is_otp_powered(sih) || ai_is_otp_disabled(sih)) { + if (ai_is_otp_disabled(sih)) { err = -EPERM; goto out; } @@ -540,9 +532,6 @@ otp_read_region(struct si_pub *sih, int region, u16 *data, err = (((otpinfo_t *) oh)->fn->read_region) (oh, region, data, wlen); out: - if (!wasup) - ai_otp_power(sih, false); - return err; } -- 2.20.1