[UPSTREAM - RFC] pcf50633_use_regulator_api.patch
Harald Welte
laforge at openmoko.org
Thu Oct 9 00:15:07 CEST 2008
Dear Balaji,
first of all thanks for all your work.
However:
On Wed, Oct 08, 2008 at 07:38:45PM +0530, Balaji Rao wrote:
> pcf50633_use_regulator_api.patch
I don't really like the split betwee the i2c-pcf5633 part and the regulator
part. It means that you need to publicly expose/export a number of low level
register read/write/set/clearbit functions, plus put the entire register
definitions into the public linux/pcf50633.h file.
It is usually not a good idea to expose that many internal details to the
kernel. Either you just put the two files into one driver by directly linking
multiple .c/.o files into one .ko, or you find some other way how to find
a high-level API between the two drivers.
I personally think putting the i2c and the regulator bits together is a good
idea.
--
- Harald Welte <laforge at openmoko.org> http://openmoko.org/
============================================================================
Software for the world's first truly open Free Software mobile phone
More information about the openmoko-kernel
mailing list