On Mon, Sep 30, 2019 at 09:56:13AM -0500, Rob Herring wrote: > On Thu, Sep 12, 2019 at 10:09:06PM +0200, Ludovic Desroches wrote: > > There is a new compatible string for the SAM9X60 sdhci device. It involves > > an update of the properties about the clocks stuff. > > > > Signed-off-by: Ludovic Desroches <[email protected]> > > --- > > .../devicetree/bindings/mmc/sdhci-atmel.txt | 25 ++++++++++++++++--- > > 1 file changed, 22 insertions(+), 3 deletions(-) > > > > diff --git a/Documentation/devicetree/bindings/mmc/sdhci-atmel.txt > > b/Documentation/devicetree/bindings/mmc/sdhci-atmel.txt > > index 1b662d7171a0..364ceea330b6 100644 > > --- a/Documentation/devicetree/bindings/mmc/sdhci-atmel.txt > > +++ b/Documentation/devicetree/bindings/mmc/sdhci-atmel.txt > > @@ -5,12 +5,19 @@ Documentation/devicetree/bindings/mmc/mmc.txt and the > > properties used by the > > sdhci-of-at91 driver. > > > > Required properties: > > -- compatible: Must be "atmel,sama5d2-sdhci". > > +- compatible: Must be "atmel,sama5d2-sdhci" or > > "microchip,sam9x60-sdhci". > > - clocks: Phandlers to the clocks. > > -- clock-names: Must be "hclock", "multclk", "baseclk"; > > +- clock-names: Must be "hclock", "multclk", "baseclk" for > > + "atmel,sama5d2-sdhci". > > + Must be "hclock", "multclk" for > > "microchip,sam9x60-sdhci". > > > > +Optional properties: > > +- assigned-clocks: The same with "multclk". > > +- assigned-clock-rates The rate of "multclk" in order to not rely on > > the > > + gck configuration set by previous components. > > > > -Example: > > + > > +Examples: > > > > sdmmc0: sdio-host@a0000000 { > > compatible = "atmel,sama5d2-sdhci"; > > @@ -18,4 +25,16 @@ sdmmc0: sdio-host@a0000000 { > > interrupts = <31 IRQ_TYPE_LEVEL_HIGH 0>; > > clocks = <&sdmmc0_hclk>, <&sdmmc0_gclk>, <&main>; > > clock-names = "hclock", "multclk", "baseclk"; > > + assigned-clocks = <&sdmmc0_gclk>; > > + assigned-clock-rates = <480000000>; > > +}; > > + > > +sdmmc0: sdio-host@80000000 { > > mmc@... > > Though I don't see much value in a second example. Examples are not a > complete enumeration of all possible dts entries. >
Ok, I'll skip this example. Ludovic > > + compatible = "microchip,sam9x60-sdhci"; > > + reg = <0x80000000 0x300>; > > + interrupts = <12 IRQ_TYPE_LEVEL_HIGH 0>; > > + clocks = <&pmc PMC_TYPE_PERIPHERAL 12>, <&pmc PMC_TYPE_GCK 12>; > > + clock-names = "hclock", "multclk"; > > + assigned-clocks = <&pmc PMC_TYPE_GCK 12>; > > + assigned-clock-rates = <100000000>; > > }; > > -- > > 2.23.0 > >
