Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751107AbbBUKXq (ORCPT ); Sat, 21 Feb 2015 05:23:46 -0500 Received: from opensource.wolfsonmicro.com ([80.75.67.52]:54181 "EHLO opensource.wolfsonmicro.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750770AbbBUKXo (ORCPT ); Sat, 21 Feb 2015 05:23:44 -0500 Date: Sat, 21 Feb 2015 10:23:41 +0000 From: Charles Keepax To: Mark Brown Cc: lee.jones@linaro.org, robh+dt@kernel.org, pawel.moll@arm.com, mark.rutland@arm.com, ijc+devicetree@hellion.org.ok, galak@codeaurora.org, sameo@linux.intel.org, lgirdwood@gmail.com, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org Subject: Re: [PATCH] mfd: arizona: Move useful defines into a dt-binding include Message-ID: <20150221102341.GA15374@opensource.wolfsonmicro.com> References: <1424257419-28000-1-git-send-email-ckeepax@opensource.wolfsonmicro.com> <20150221092308.GC6236@finisterre.sirena.org.uk> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150221092308.GC6236@finisterre.sirena.org.uk> User-Agent: Mutt/1.5.17+20080114 (2008-01-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 919 Lines: 26 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 > > > > #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 linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/