Message ID | 1430456052-8888-4-git-send-email-stripathi@apm.com (mailing list archive) |
---|---|
State | Not Applicable |
Delegated to: | Michael Ellerman |
Headers | show |
On 05/01/2015 06:54 AM, Suman Tripathi wrote: > This patch updates Arasan SDHC documentation to support > 4.9a version of Arasan SDHC controller. > > Signed-off-by: Suman Tripathi <stripathi@apm.com> > --- > Documentation/devicetree/bindings/mmc/arasan,sdhci.txt | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/mmc/arasan,sdhci.txt b/Documentation/devicetree/bindings/mmc/arasan,sdhci.txt > index 98ee2ab..f01d41a 100644 > --- a/Documentation/devicetree/bindings/mmc/arasan,sdhci.txt > +++ b/Documentation/devicetree/bindings/mmc/arasan,sdhci.txt > @@ -8,7 +8,8 @@ Device Tree Bindings for the Arasan SDHCI Controller > [3] Documentation/devicetree/bindings/interrupt-controller/interrupts.txt > > Required Properties: > - - compatible: Compatibility string. Must be 'arasan,sdhci-8.9a' > + - compatible: Compatibility string. Must be 'arasan,sdhci-8.9a' or > + 'arasan,sdhci-4.9a' > - reg: From mmc bindings: Register location and length. > - clocks: From clock bindings: Handles to clock inputs. > - clock-names: From clock bindings: Tuple including "clk_xin" and "clk_ahb" > @@ -18,7 +19,7 @@ Required Properties: > > Example: > sdhci@e0100000 { > - compatible = "arasan,sdhci-8.9a"; > + compatible = "arasan,sdhci-8.9a", "arasan,sdhci-4.9a"; Is there any reason to change this example? You probably want to have just one compatible string not both. Thanks, Michal
Hi, On Mon, May 4, 2015 at 11:30 AM, Michal Simek <michal.simek@xilinx.com> wrote: > On 05/01/2015 06:54 AM, Suman Tripathi wrote: >> This patch updates Arasan SDHC documentation to support >> 4.9a version of Arasan SDHC controller. >> >> Signed-off-by: Suman Tripathi <stripathi@apm.com> >> --- >> Documentation/devicetree/bindings/mmc/arasan,sdhci.txt | 5 +++-- >> 1 file changed, 3 insertions(+), 2 deletions(-) >> >> diff --git a/Documentation/devicetree/bindings/mmc/arasan,sdhci.txt b/Documentation/devicetree/bindings/mmc/arasan,sdhci.txt >> index 98ee2ab..f01d41a 100644 >> --- a/Documentation/devicetree/bindings/mmc/arasan,sdhci.txt >> +++ b/Documentation/devicetree/bindings/mmc/arasan,sdhci.txt >> @@ -8,7 +8,8 @@ Device Tree Bindings for the Arasan SDHCI Controller >> [3] Documentation/devicetree/bindings/interrupt-controller/interrupts.txt >> >> Required Properties: >> - - compatible: Compatibility string. Must be 'arasan,sdhci-8.9a' >> + - compatible: Compatibility string. Must be 'arasan,sdhci-8.9a' or >> + 'arasan,sdhci-4.9a' >> - reg: From mmc bindings: Register location and length. >> - clocks: From clock bindings: Handles to clock inputs. >> - clock-names: From clock bindings: Tuple including "clk_xin" and "clk_ahb" >> @@ -18,7 +19,7 @@ Required Properties: >> >> Example: >> sdhci@e0100000 { >> - compatible = "arasan,sdhci-8.9a"; >> + compatible = "arasan,sdhci-8.9a", "arasan,sdhci-4.9a"; > > Is there any reason to change this example? Thought that if dts is updated so why not update the binding too. > You probably want to have just one compatible string not both. Okay will remove in next version. > > Thanks, > Michal > > > >
On 05/04/2015 08:03 AM, Suman Tripathi wrote: > Hi, > > On Mon, May 4, 2015 at 11:30 AM, Michal Simek <michal.simek@xilinx.com> wrote: >> On 05/01/2015 06:54 AM, Suman Tripathi wrote: >>> This patch updates Arasan SDHC documentation to support >>> 4.9a version of Arasan SDHC controller. >>> >>> Signed-off-by: Suman Tripathi <stripathi@apm.com> >>> --- >>> Documentation/devicetree/bindings/mmc/arasan,sdhci.txt | 5 +++-- >>> 1 file changed, 3 insertions(+), 2 deletions(-) >>> >>> diff --git a/Documentation/devicetree/bindings/mmc/arasan,sdhci.txt b/Documentation/devicetree/bindings/mmc/arasan,sdhci.txt >>> index 98ee2ab..f01d41a 100644 >>> --- a/Documentation/devicetree/bindings/mmc/arasan,sdhci.txt >>> +++ b/Documentation/devicetree/bindings/mmc/arasan,sdhci.txt >>> @@ -8,7 +8,8 @@ Device Tree Bindings for the Arasan SDHCI Controller >>> [3] Documentation/devicetree/bindings/interrupt-controller/interrupts.txt >>> >>> Required Properties: >>> - - compatible: Compatibility string. Must be 'arasan,sdhci-8.9a' >>> + - compatible: Compatibility string. Must be 'arasan,sdhci-8.9a' or >>> + 'arasan,sdhci-4.9a' >>> - reg: From mmc bindings: Register location and length. >>> - clocks: From clock bindings: Handles to clock inputs. >>> - clock-names: From clock bindings: Tuple including "clk_xin" and "clk_ahb" >>> @@ -18,7 +19,7 @@ Required Properties: >>> >>> Example: >>> sdhci@e0100000 { >>> - compatible = "arasan,sdhci-8.9a"; >>> + compatible = "arasan,sdhci-8.9a", "arasan,sdhci-4.9a"; >> >> Is there any reason to change this example? > > Thought that if dts is updated so why not update the binding too. It is just about updating example. It is fine to update binding. Thanks, Michal
diff --git a/Documentation/devicetree/bindings/mmc/arasan,sdhci.txt b/Documentation/devicetree/bindings/mmc/arasan,sdhci.txt index 98ee2ab..f01d41a 100644 --- a/Documentation/devicetree/bindings/mmc/arasan,sdhci.txt +++ b/Documentation/devicetree/bindings/mmc/arasan,sdhci.txt @@ -8,7 +8,8 @@ Device Tree Bindings for the Arasan SDHCI Controller [3] Documentation/devicetree/bindings/interrupt-controller/interrupts.txt Required Properties: - - compatible: Compatibility string. Must be 'arasan,sdhci-8.9a' + - compatible: Compatibility string. Must be 'arasan,sdhci-8.9a' or + 'arasan,sdhci-4.9a' - reg: From mmc bindings: Register location and length. - clocks: From clock bindings: Handles to clock inputs. - clock-names: From clock bindings: Tuple including "clk_xin" and "clk_ahb" @@ -18,7 +19,7 @@ Required Properties: Example: sdhci@e0100000 { - compatible = "arasan,sdhci-8.9a"; + compatible = "arasan,sdhci-8.9a", "arasan,sdhci-4.9a"; reg = <0xe0100000 0x1000>; clock-names = "clk_xin", "clk_ahb"; clocks = <&clkc 21>, <&clkc 32>;
This patch updates Arasan SDHC documentation to support 4.9a version of Arasan SDHC controller. Signed-off-by: Suman Tripathi <stripathi@apm.com> --- Documentation/devicetree/bindings/mmc/arasan,sdhci.txt | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-)