Messages in this thread | | | Date | Wed, 9 May 2012 21:50:41 +0200 | From | Samuel Ortiz <> | Subject | Re: [PATCH 1/2] twl-regulator: define all feature flags in one place. |
| |
Hi Neil,
On Wed, May 02, 2012 at 10:08:49AM +0100, Mark Brown wrote: > On Wed, May 02, 2012 at 06:44:50AM +1000, NeilBrown wrote: > > twl-regulator has a collection of feature flags, some defined > > in twl-core.c and one defined in i2c/twl.h. > > This is confusing for anyone adding a new feature flag. > > > > So collect them together and place them in twl.h immediately > > after the structure in which they are initially set. > > Please remember to always CC maintainers on patches and always use > subject lines appropriate for the subsystem. Adding Samuel since this > is not a regulator patch. Could you please re-send this patch to me ?
Cheers, Samuel.
-- Intel Open Source Technology Centre http://oss.intel.com/
| |