On Thu, Dec 22, 2016 at 02:32:42PM -0600, Rob Herring wrote: > On Mon, Dec 19, 2016 at 01:15:52PM +0100, Jan Glauber wrote: > > Add description of Cavium Octeon and ThunderX SOC device tree bindings. > > > > CC: Ulf Hansson <ulf.hansson@xxxxxxxxxx> > > CC: Rob Herring <robh+dt@xxxxxxxxxx> > > CC: Mark Rutland <mark.rutland@xxxxxxx> > > CC: devicetree@xxxxxxxxxxxxxxx > > > > Signed-off-by: Jan Glauber <jglauber@xxxxxxxxxx> > > --- > > .../devicetree/bindings/mmc/octeon-mmc.txt | 59 ++++++++++++++++++++++ > > Perhaps cavium-mmc.txt would be more appropriate now. Yes, forgot to rename it. > > 1 file changed, 59 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/mmc/octeon-mmc.txt > > > > diff --git a/Documentation/devicetree/bindings/mmc/octeon-mmc.txt b/Documentation/devicetree/bindings/mmc/octeon-mmc.txt > > new file mode 100644 > > index 0000000..aad02eb > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/mmc/octeon-mmc.txt > > @@ -0,0 +1,59 @@ > > +* Cavium Octeon & ThunderX MMC controller > > + > > +The highspeed MMC host controller on Caviums SoCs provides an interface > > +for MMC and SD types of memory cards. > > + > > +Supported maximum speeds are the ones of the eMMC standard 4.41 as well > > +as the speed of SD standard 4.0. Only 3.3 Volt is supported. > > + > > +Required properties: > > + - compatible : should be one of: > > + * "cavium,octeon-6130-mmc" > > + * "cavium,octeon-6130-mmc-slot" > > + * "cavium,octeon-7890-mmc" > > + * "cavium,octeon-7890-mmc-slot" > > + * "cavium,thunder-8190-mmc" > > + * "cavium,thunder-8190-mmc-slot" > > + * "cavium,thunder-8390-mmc" > > + * "cavium,thunder-8390-mmc-slot" > > + - reg : mmc controller base registers > > Following PCI addressing? Yes for the host controller, for the child nodes we use reg as the physical slot number. Maybe I should describe the child node properties separately? > > + - clocks : phandle > > + > > +Optional properties: > > + - for cd, bus-width and additional generic mmc parameters > > + please refer to mmc.txt within this directory > > + - "cavium,cmd-clk-skew" : number of coprocessor clocks before sampling command > > + - "cavium,dat-clk-skew" : number of coprocessor clocks before sampling data > > + > > +Deprecated properties: > > +- spi-max-frequency : use max-frequency instead > > +- "cavium,bus-max-width" : use bus-width instead > > Drop the quotes. OK, droped them also above. > > + > > +Examples: > > + - Within .dtsi: > > Don't show the division between files in the example. OK. > > + mmc_1_4: mmc@1,4 { > > + compatible = "cavium,thunder-8390-mmc"; > > + reg = <0x0c00 0 0 0 0>; /* DEVFN = 0x0c (1:4) */ > > + #address-cells = <1>; > > + #size-cells = <0>; > > + clocks = <&sclk>; > > + }; > > + > > + - Within dts: > > + mmc-slot@0 { > > Need to show this is a child node. OK. > > + compatible = "cavium,thunder-8390-mmc-slot"; > > + reg = <0>; > > + voltage-ranges = <3300 3300>; > > + max-frequency = <42000000>; > > + bus-width = <4>; > > + cap-sd-highspeed; > > + }; > > + mmc-slot@1 { > > + compatible = "cavium,thunder-8390-mmc-slot"; > > + reg = <1>; > > + voltage-ranges = <3300 3300>; > > + max-frequency = <42000000>; > > + bus-width = <8>; > > + cap-mmc-highspeed; > > + non-removable; > > + }; > > -- > > 2.9.0.rc0.21.g7777322 > > -- 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