diff mbox

[v2,6/7] devicetree: bindings: Document supported Allwinner sunxi SoCs

Message ID 1411570141-29960-7-git-send-email-wens@csie.org
State Superseded, archived
Headers show

Commit Message

Chen-Yu Tsai Sept. 24, 2014, 2:49 p.m. UTC
This adds a list of supported Allwinner SoC bindings.

Signed-off-by: Chen-Yu Tsai <wens@csie.org>
---
 Documentation/devicetree/bindings/arm/sunxi.txt | 12 ++++++++++++
 1 file changed, 12 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/arm/sunxi.txt

Comments

Maxime Ripard Sept. 25, 2014, 1:36 p.m. UTC | #1
On Wed, Sep 24, 2014 at 10:49:00PM +0800, Chen-Yu Tsai wrote:
> This adds a list of supported Allwinner SoC bindings.
> 
> Signed-off-by: Chen-Yu Tsai <wens@csie.org>
> ---
>  Documentation/devicetree/bindings/arm/sunxi.txt | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/arm/sunxi.txt
> 
> diff --git a/Documentation/devicetree/bindings/arm/sunxi.txt b/Documentation/devicetree/bindings/arm/sunxi.txt
> new file mode 100644
> index 0000000..d7d1039
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/arm/sunxi.txt
> @@ -0,0 +1,12 @@
> +Allwinner sunXi Platforms Device Tree Bindings
> +
> +Each device tree must specify which Allwinner SoC it uses,
> +using one of the following compatible stirngs:
                                         ^ strings

> +
> +  allwinner,sun4i-a10
> +  allwinner,sun5i-a10s
> +  allwinner,sun5i-a13
> +  allwinner,sun6i-a31
> +  allwinner,sun7i-a20
> +  allwinner,sun8i-a23
> +  allwinner,sun9i-a80

Maybe you can just add this info to the already existing README?
Chen-Yu Tsai Sept. 27, 2014, 1:08 p.m. UTC | #2
On Thu, Sep 25, 2014 at 9:36 PM, Maxime Ripard
<maxime.ripard@free-electrons.com> wrote:
> On Wed, Sep 24, 2014 at 10:49:00PM +0800, Chen-Yu Tsai wrote:
>> This adds a list of supported Allwinner SoC bindings.
>>
>> Signed-off-by: Chen-Yu Tsai <wens@csie.org>
>> ---
>>  Documentation/devicetree/bindings/arm/sunxi.txt | 12 ++++++++++++
>>  1 file changed, 12 insertions(+)
>>  create mode 100644 Documentation/devicetree/bindings/arm/sunxi.txt
>>
>> diff --git a/Documentation/devicetree/bindings/arm/sunxi.txt b/Documentation/devicetree/bindings/arm/sunxi.txt
>> new file mode 100644
>> index 0000000..d7d1039
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/arm/sunxi.txt
>> @@ -0,0 +1,12 @@
>> +Allwinner sunXi Platforms Device Tree Bindings
>> +
>> +Each device tree must specify which Allwinner SoC it uses,
>> +using one of the following compatible stirngs:
>                                          ^ strings

Will fix.

>> +
>> +  allwinner,sun4i-a10
>> +  allwinner,sun5i-a10s
>> +  allwinner,sun5i-a13
>> +  allwinner,sun6i-a31
>> +  allwinner,sun7i-a20
>> +  allwinner,sun8i-a23
>> +  allwinner,sun9i-a80
>
> Maybe you can just add this info to the already existing README?

AFAIK the bindings are where they're supposed to be.

Putting the bindings outside of devicetree/bindings is unexpected
and confusing.


Cheers
ChenYu
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Maxime Ripard Sept. 29, 2014, 9:52 a.m. UTC | #3
On Sat, Sep 27, 2014 at 09:08:46PM +0800, Chen-Yu Tsai wrote:
> On Thu, Sep 25, 2014 at 9:36 PM, Maxime Ripard
> <maxime.ripard@free-electrons.com> wrote:
> > On Wed, Sep 24, 2014 at 10:49:00PM +0800, Chen-Yu Tsai wrote:
> >> This adds a list of supported Allwinner SoC bindings.
> >>
> >> Signed-off-by: Chen-Yu Tsai <wens@csie.org>
> >> ---
> >>  Documentation/devicetree/bindings/arm/sunxi.txt | 12 ++++++++++++
> >>  1 file changed, 12 insertions(+)
> >>  create mode 100644 Documentation/devicetree/bindings/arm/sunxi.txt
> >>
> >> diff --git a/Documentation/devicetree/bindings/arm/sunxi.txt b/Documentation/devicetree/bindings/arm/sunxi.txt
> >> new file mode 100644
> >> index 0000000..d7d1039
> >> --- /dev/null
> >> +++ b/Documentation/devicetree/bindings/arm/sunxi.txt
> >> @@ -0,0 +1,12 @@
> >> +Allwinner sunXi Platforms Device Tree Bindings
> >> +
> >> +Each device tree must specify which Allwinner SoC it uses,
> >> +using one of the following compatible stirngs:
> >                                          ^ strings
> 
> Will fix.
> 
> >> +
> >> +  allwinner,sun4i-a10
> >> +  allwinner,sun5i-a10s
> >> +  allwinner,sun5i-a13
> >> +  allwinner,sun6i-a31
> >> +  allwinner,sun7i-a20
> >> +  allwinner,sun8i-a23
> >> +  allwinner,sun9i-a80
> >
> > Maybe you can just add this info to the already existing README?
> 
> AFAIK the bindings are where they're supposed to be.
> 
> Putting the bindings outside of devicetree/bindings is unexpected
> and confusing.

Ah, yes...

Sorry about the noise, I was somehow convinced our README were under
that directory, but you're obviously right :)

Maxime
diff mbox

Patch

diff --git a/Documentation/devicetree/bindings/arm/sunxi.txt b/Documentation/devicetree/bindings/arm/sunxi.txt
new file mode 100644
index 0000000..d7d1039
--- /dev/null
+++ b/Documentation/devicetree/bindings/arm/sunxi.txt
@@ -0,0 +1,12 @@ 
+Allwinner sunXi Platforms Device Tree Bindings
+
+Each device tree must specify which Allwinner SoC it uses,
+using one of the following compatible stirngs:
+
+  allwinner,sun4i-a10
+  allwinner,sun5i-a10s
+  allwinner,sun5i-a13
+  allwinner,sun6i-a31
+  allwinner,sun7i-a20
+  allwinner,sun8i-a23
+  allwinner,sun9i-a80