On Sat, Feb 21, 2015 at 06:23:08PM +0900, Mark Brown wrote: > On Wed, Feb 18, 2015 at 11:03:39AM +0000, Charles Keepax wrote: > > > --- a/include/linux/mfd/arizona/pdata.h > > +++ b/include/linux/mfd/arizona/pdata.h > > @@ -7,61 +7,17 @@ > > * it under the terms of the GNU General Public License version 2 as > > * published by the Free Software Foundation. > > */ > > +#include <dt-bindings/mfd/arizona.h> > > > > #ifndef _ARIZONA_PDATA_H > > #define _ARIZONA_PDATA_H > > It's a bit odd to have the include outside the guards, is there any > great reason for doing so? There is not, simple error on my part, I will respin and move it inside. Thanks, Charles -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html