Hi Valentine, Thank you for the patch. Two small comments below. On Friday 08 November 2013 00:04:02 Valentine Barshak wrote: > These bindings can be used to register SATA devices found on R-Car SoC. > > Signed-off-by: Valentine Barshak <valentine.barshak@xxxxxxxxxxxxxxxxxx> > Acked-by: Kumar Gala <galak@xxxxxxxxxxxxxx> > --- > Documentation/devicetree/bindings/ata/sata_rcar.txt | 18 ++++++++++++++++++ > 1 file changed, 18 insertions(+) > create mode 100644 Documentation/devicetree/bindings/ata/sata_rcar.txt > > diff --git a/Documentation/devicetree/bindings/ata/sata_rcar.txt > b/Documentation/devicetree/bindings/ata/sata_rcar.txt new file mode 100644 > index 0000000..d6b20a6 > --- /dev/null > +++ b/Documentation/devicetree/bindings/ata/sata_rcar.txt > @@ -0,0 +1,18 @@ > +* Renesas R-Car SATA > + > +Required properties: > +- compatible : should contain one of the following: > + - "renesas,sata-r8a7779" for R-Car H1 > + - "renesas,sata-r8a7790" for R-Car H2 > + - "renesas,sata-r8a7791" for R-Car M2 > +- reg : offset and length of the SATA registers; Maybe address instead of offset ? > +- interrupts : must consist of one interrupt specifier. > + > +Example: > + > +sata: sata@fc600000 { > + compatible = "renesas,sata-r8a7779"; > + reg = <0xfc600000 0x2000>; > + interrupt-parent = <&gic>; > + interrupts = <0 100 0x4>; Please use IRQ_TYPE_LEVEL_HIGH instead of 0x4. > +}; -- Regards, Laurent Pinchart -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html