From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chris Ball Date: Mon, 11 Jun 2012 13:41:00 +0000 Subject: Re: [PATCH 4/5 v3] mmc: add a function to get a regulator, supplying card's Vdd Message-Id: <871ulm7yqb.fsf@octavius.laptop.org> List-Id: References: <20120528144502.GG4032@opensource.wolfsonmicro.com> In-Reply-To: (Guennadi Liakhovetski's message of "Mon, 11 Jun 2012 13:03:26 +0200 (CEST)") MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Guennadi Liakhovetski Cc: Mark Brown , linux-mmc@vger.kernel.org, Magnus Damm , linux-sh@vger.kernel.org Hi, On Mon, Jun 11 2012, Guennadi Liakhovetski wrote: > Yes, but existing mmc-regulator functions use EXPORT_SYMBOL(), e.g., > > EXPORT_SYMBOL(mmc_regulator_get_ocrmask); > > which indeed seem to do the "old evil binary-only module trick": re-export > functions, dropping the "_GPL" suffix. Looks like they should be fixed... Yuck, thanks for noticing that. Happy to fix it: Subject: [PATCH] mmc: core: Export regulator_* functions as GPL The regulator API functions we're wrapping are exported as GPL, so our wrappers for the same functions should be too. Reported-by: Guennadi Liakhovetski Signed-off-by: Chris Ball --- drivers/mmc/core/core.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/mmc/core/core.c b/drivers/mmc/core/core.c index 0b6141d..c689877 100644 --- a/drivers/mmc/core/core.c +++ b/drivers/mmc/core/core.c @@ -941,7 +941,7 @@ int mmc_regulator_get_ocrmask(struct regulator *supply) return result; } -EXPORT_SYMBOL(mmc_regulator_get_ocrmask); +EXPORT_SYMBOL_GPL(mmc_regulator_get_ocrmask); /** * mmc_regulator_set_ocr - set regulator to match host->ios voltage @@ -1011,7 +1011,7 @@ int mmc_regulator_set_ocr(struct mmc_host *mmc, "could not set regulator OCR (%d)\n", result); return result; } -EXPORT_SYMBOL(mmc_regulator_set_ocr); +EXPORT_SYMBOL_GPL(mmc_regulator_set_ocr); #endif /* CONFIG_REGULATOR */ -- Chris Ball One Laptop Per Child