From mboxrd@z Thu Jan 1 00:00:00 1970 From: Brian Norris Subject: Re: [PATCH 2/2] ARM: at91: sama5d4: dts: change the nand compatible string Date: Tue, 20 Jan 2015 12:49:15 -0800 Message-ID: <20150120204915.GS9759@ld-irv-0074> References: <1421656387-12538-1-git-send-email-josh.wu@atmel.com> <1421656387-12538-2-git-send-email-josh.wu@atmel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1421656387-12538-2-git-send-email-josh.wu-AIFe0yeh4nAAvxtiuMwx3w@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Josh Wu Cc: linux-mtd-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Nicolas Ferre , linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Alexandre Belloni , Boris Brezillon , devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Jean-Christophe Plagniol-Villard List-Id: devicetree@vger.kernel.org On Mon, Jan 19, 2015 at 04:33:07PM +0800, Josh Wu wrote: > As we introduce a new "atmel,sama5d4-nand" compatible string for sama5d4, > so we need to apply it for sama5d4 chip. > > Signed-off-by: Josh Wu > --- > > arch/arm/boot/dts/sama5d4.dtsi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm/boot/dts/sama5d4.dtsi b/arch/arm/boot/dts/sama5d4.dtsi > index 1b0f30c..02d67c2 100644 > --- a/arch/arm/boot/dts/sama5d4.dtsi > +++ b/arch/arm/boot/dts/sama5d4.dtsi > @@ -267,7 +267,7 @@ > }; > > nand0: nand@80000000 { > - compatible = "atmel,at91rm9200-nand"; > + compatible = "atmel,sama5d4-nand"; Wouldn't it make sense to include both compatible strings, so that old kernels can still bind to it? The newer IP only adds a new ECC feature, but the driver still can work fine without handling it, right? > #address-cells = <1>; > #size-cells = <1>; > ranges; Brian -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html