Re: [PATCH 4/5 v3] mmc: add a function to get a regulator, supplying card's Vdd

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Mark

On Mon, 28 May 2012, Mark Brown wrote:

> On Fri, May 25, 2012 at 05:14:52PM +0200, Guennadi Liakhovetski wrote:
> > Add a function to get a regulator, supplying card's Vdd on a specific host.
> > If such a regulator is found, the function checks, whether a valid OCR mask
> > can be obtained from this regulator.
> 
> Reviwed-by: Mark Brown <broonie@xxxxxxxxxxxxxxxxxxxxxxxxxxx>

Thanks for the review!

> but
> 
> > +EXPORT_SYMBOL(mmc_regulator_get_vmmc);
> 
> regulator is all _GPL...

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...

Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux