From mboxrd@z Thu Jan 1 00:00:00 1970 From: ezequiel.garcia@free-electrons.com (Ezequiel Garcia) Date: Thu, 7 Nov 2013 13:41:33 -0300 Subject: [PATCH v4 05/31] mtd: nand: pxa3xx: devicetree binding update In-Reply-To: <20131107153210.GK8308@titan.lakedaemon.net> References: <1383837455-30721-1-git-send-email-ezequiel.garcia@free-electrons.com> <1383837455-30721-6-git-send-email-ezequiel.garcia@free-electrons.com> <20131107153210.GK8308@titan.lakedaemon.net> Message-ID: <20131107164132.GA30917@localhost> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Thu, Nov 07, 2013 at 10:32:10AM -0500, Jason Cooper wrote: > On Thu, Nov 07, 2013 at 12:17:09PM -0300, Ezequiel Garcia wrote: > > Since the driver supports the new compatible string, the binding > > documentation must be updated to reflect it. > > > > Signed-off-by: Ezequiel Garcia > > --- > > Documentation/devicetree/bindings/mtd/pxa3xx-nand.txt | 4 +++- > > 1 file changed, 3 insertions(+), 1 deletion(-) > > Thanks for adding this, but please remember to update the Subject > string in the future. Both this patch and the next one are changing the > Documentation/ portion of the tree, not drivers/. > Come again? You mean "[PATCH] Documentation: mtd: nand: Do this" In that case: sorry, I do it differently on purpose, it looks better to me (focus the affected driver), but maybe I'm wrong. > And, > > Acked-by: Jason Cooper > Thanks! > thx, > > Jason. > > > diff --git a/Documentation/devicetree/bindings/mtd/pxa3xx-nand.txt b/Documentation/devicetree/bindings/mtd/pxa3xx-nand.txt > > index f1421e2..bed8390 100644 > > --- a/Documentation/devicetree/bindings/mtd/pxa3xx-nand.txt > > +++ b/Documentation/devicetree/bindings/mtd/pxa3xx-nand.txt > > @@ -2,7 +2,9 @@ PXA3xx NAND DT bindings > > > > Required properties: > > > > - - compatible: Should be "marvell,pxa3xx-nand" > > + - compatible: Should be set to one of the following: > > + marvell,pxa3xx-nand > > + marvell,armada370-nand > > - reg: The register base for the controller > > - interrupts: The interrupt to map > > - #address-cells: Set to <1> if the node includes partitions > > -- > > 1.8.1.5 > > -- Ezequiel Garc?a, Free Electrons Embedded Linux, Kernel and Android Engineering http://free-electrons.com