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 <ezequiel.garcia@xxxxxxxxxxxxxxxxxx> > > --- > > 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 <jason@xxxxxxxxxxxxxx> > 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 -- 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
