diff mbox series

[2/4] dt-bindings: arm: rockchip: Add binding for Rock960 board

Message ID 20180910151356.25946-3-manivannan.sadhasivam@linaro.org (mailing list archive)
State New, archived
Headers show
Series Add 96Boards Rock960 CE board support | expand

Commit Message

Manivannan Sadhasivam Sept. 10, 2018, 3:13 p.m. UTC
Add devicetree binding for Rock960 board from Vamrs Limited.

Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
---
 Documentation/devicetree/bindings/arm/rockchip.txt | 4 ++++
 1 file changed, 4 insertions(+)

Comments

Heiko Stuebner Sept. 10, 2018, 5:22 p.m. UTC | #1
Am Montag, 10. September 2018, 17:13:54 CEST schrieb Manivannan Sadhasivam:
> Add devicetree binding for Rock960 board from Vamrs Limited.
> 
> Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> ---
>  Documentation/devicetree/bindings/arm/rockchip.txt | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/arm/rockchip.txt b/Documentation/devicetree/bindings/arm/rockchip.txt
> index acfd3c773dd0..aaad9e2ff9a3 100644
> --- a/Documentation/devicetree/bindings/arm/rockchip.txt
> +++ b/Documentation/devicetree/bindings/arm/rockchip.txt
> @@ -5,6 +5,10 @@ Rockchip platforms device tree bindings
>      Required root node properties:
>        - compatible = "vamrs,ficus", "rockchip,rk3399";
>  
> +- 96boards RK3399 Rock960 (ROCK960 Consumer Edition)
> +    Required root node properties:
> +      - compatible = "vamrs,rk3399-rock960", "rockchip,rk3399";

hmm, are there more boards labeled rock960 from Vamrs?
Because I'm trying to determine what makes "vamrs,rock960" unsuitable.


Heiko
Manivannan Sadhasivam Sept. 10, 2018, 5:26 p.m. UTC | #2
Hi Heiko,

On Mon, Sep 10, 2018 at 07:22:26PM +0200, Heiko Stuebner wrote:
> Am Montag, 10. September 2018, 17:13:54 CEST schrieb Manivannan Sadhasivam:
> > Add devicetree binding for Rock960 board from Vamrs Limited.
> > 
> > Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> > ---
> >  Documentation/devicetree/bindings/arm/rockchip.txt | 4 ++++
> >  1 file changed, 4 insertions(+)
> > 
> > diff --git a/Documentation/devicetree/bindings/arm/rockchip.txt b/Documentation/devicetree/bindings/arm/rockchip.txt
> > index acfd3c773dd0..aaad9e2ff9a3 100644
> > --- a/Documentation/devicetree/bindings/arm/rockchip.txt
> > +++ b/Documentation/devicetree/bindings/arm/rockchip.txt
> > @@ -5,6 +5,10 @@ Rockchip platforms device tree bindings
> >      Required root node properties:
> >        - compatible = "vamrs,ficus", "rockchip,rk3399";
> >  
> > +- 96boards RK3399 Rock960 (ROCK960 Consumer Edition)
> > +    Required root node properties:
> > +      - compatible = "vamrs,rk3399-rock960", "rockchip,rk3399";
> 
> hmm, are there more boards labeled rock960 from Vamrs?
> Because I'm trying to determine what makes "vamrs,rock960" unsuitable.
> 

Not that I'm aware of but the intention was to follow other RK3399 based
boards. Anyhow, I have no issues with "vamrs,rock960" if you prefer it!

Thanks,
Mani

> 
> Heiko
> 
>
Heiko Stuebner Sept. 10, 2018, 5:53 p.m. UTC | #3
Am Montag, 10. September 2018, 19:26:55 CEST schrieb Manivannan Sadhasivam:
> Hi Heiko,
> 
> On Mon, Sep 10, 2018 at 07:22:26PM +0200, Heiko Stuebner wrote:
> > Am Montag, 10. September 2018, 17:13:54 CEST schrieb Manivannan Sadhasivam:
> > > Add devicetree binding for Rock960 board from Vamrs Limited.
> > > 
> > > Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
> > > ---
> > >  Documentation/devicetree/bindings/arm/rockchip.txt | 4 ++++
> > >  1 file changed, 4 insertions(+)
> > > 
> > > diff --git a/Documentation/devicetree/bindings/arm/rockchip.txt b/Documentation/devicetree/bindings/arm/rockchip.txt
> > > index acfd3c773dd0..aaad9e2ff9a3 100644
> > > --- a/Documentation/devicetree/bindings/arm/rockchip.txt
> > > +++ b/Documentation/devicetree/bindings/arm/rockchip.txt
> > > @@ -5,6 +5,10 @@ Rockchip platforms device tree bindings
> > >      Required root node properties:
> > >        - compatible = "vamrs,ficus", "rockchip,rk3399";
> > >  
> > > +- 96boards RK3399 Rock960 (ROCK960 Consumer Edition)
> > > +    Required root node properties:
> > > +      - compatible = "vamrs,rk3399-rock960", "rockchip,rk3399";
> > 
> > hmm, are there more boards labeled rock960 from Vamrs?
> > Because I'm trying to determine what makes "vamrs,rock960" unsuitable.
> > 
> 
> Not that I'm aware of but the intention was to follow other RK3399 based
> boards. Anyhow, I have no issues with "vamrs,rock960" if you prefer it!

yep, I think I would prefer it :-)

Looking at the bindings we have, it seems we really have some sort of mix,
but I guess most boards with a real name, just use this one, like all
the google boards or also the ficus itself.

Most of the others are eval boards and fireflys, where the board is actually
unnamed and just uses company name + soc :-)


Heiko
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/arm/rockchip.txt b/Documentation/devicetree/bindings/arm/rockchip.txt
index acfd3c773dd0..aaad9e2ff9a3 100644
--- a/Documentation/devicetree/bindings/arm/rockchip.txt
+++ b/Documentation/devicetree/bindings/arm/rockchip.txt
@@ -5,6 +5,10 @@  Rockchip platforms device tree bindings
     Required root node properties:
       - compatible = "vamrs,ficus", "rockchip,rk3399";
 
+- 96boards RK3399 Rock960 (ROCK960 Consumer Edition)
+    Required root node properties:
+      - compatible = "vamrs,rk3399-rock960", "rockchip,rk3399";
+
 - Amarula Vyasa RK3288 board
     Required root node properties:
       - compatible = "amarula,vyasa-rk3288", "rockchip,rk3288";