On 07/11/2016 10:18, Arnd Bergmann wrote: > On Tuesday, October 25, 2016 3:15:50 PM CET Marc Gonzalez wrote: >> Add the tango NAND Flash Controller dt bindings documentation. >> >> Signed-off-by: Marc Gonzalez <marc_gonzalez@xxxxxxxxxxxxxxxx> >> --- >> Documentation/devicetree/bindings/mtd/tango-nand.txt | 38 ++++++++++++++++++++++++++++++ >> 1 file changed, 38 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/mtd/tango-nand.txt b/Documentation/devicetree/bindings/mtd/tango-nand.txt >> new file mode 100644 >> index 000000000000..3cbf95d6595a >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/mtd/tango-nand.txt >> @@ -0,0 +1,38 @@ >> +Sigma Designs Tango4 NAND Flash Controller (NFC) >> + >> +Required properties: >> + >> +- compatible: "sigma,smp8758-nand" >> +- reg: address/size of nfc_reg, nfc_mem, and pbus_reg >> +- dmas: reference to the DMA channel used by the controller >> +- dma-names: "nfc_sbox" > > Drop the "nfc_" prefix here, it seems redundant. I took a closer look at Documentation/devicetree/bindings and arch/arm/boot/dts and it seems the overwhelming nomenclature is "rx", "tx" (with variants around these two). Should I just use "rxtx" for my driver? >> +- clocks: reference to the system clock >> +- #address-cells: <1> >> +- #size-cells: <0> >> + >> +Children nodes represent the available NAND chips. >> +See Documentation/devicetree/bindings/mtd/nand.txt for generic bindings. >> + >> +Example: >> + >> + nand: nand@2c000 { >> + compatible = "sigma,smp8758-nand"; >> + reg = <0x2c000 0x30 0x2d000 0x800 0x20000 0x1000>; > > It would be nicer to write this as > > reg = <0x2c000 0x30>, <0x2d000 0x800>, <0x20000 0x1000>; > > which is identical in binary format. This is indeed a marked improvement. Boris, if I fix these two nits, can I submit a single patch, or do you want them split? Regards. -- 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