| Message ID | 20200722190949.215656-3-jagan@amarulasolutions.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [v2,1/6] dt-bindings: arm: rockchip: Update ROCKPi 4 with 4A binding | expand |
Hi Jagan, Am Mittwoch, 22. Juli 2020, 21:09:46 CEST schrieb Jagan Teki: > Add dt-bindings for ROCKPi 4B which is similar to 4A with > additional AP6256 Wifi/BT, PoE. > > Signed-off-by: Jagan Teki <jagan@amarulasolutions.com> > --- > Changes for v2: > - new patch > > Documentation/devicetree/bindings/arm/rockchip.yaml | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/Documentation/devicetree/bindings/arm/rockchip.yaml b/Documentation/devicetree/bindings/arm/rockchip.yaml > index 36057c9e4b83..7250adb43d24 100644 > --- a/Documentation/devicetree/bindings/arm/rockchip.yaml > +++ b/Documentation/devicetree/bindings/arm/rockchip.yaml > @@ -448,6 +448,12 @@ properties: > - const: radxa,rockpi4a > - const: rockchip,rk3399 > > + - description: Radxa ROCK Pi 4B > + items: > + - const: radxa,rockpi4 > + - const: radxa,rockpi4b > + - const: rockchip,rk3399 > + Please do all RockPi4 variants into one entry, so we want something like: - description: Radxa ROCK Pi 4 items: - enum: - radxa,rockpi4a - radxa,rockpi4b - radxa,rockpi4c - const: radxa,rockpi4 - const: rockchip,rk3399 And as the {a,b,c} variant is the most specific it also should be the first entry, not the second. Heiko > - description: Radxa ROCK Pi N8 > items: > - const: radxa,rockpi-n8 >
Hi Heiko, On Thu, Jul 23, 2020 at 4:43 AM Heiko Stuebner <heiko@sntech.de> wrote: > > Hi Jagan, > > Am Mittwoch, 22. Juli 2020, 21:09:46 CEST schrieb Jagan Teki: > > Add dt-bindings for ROCKPi 4B which is similar to 4A with > > additional AP6256 Wifi/BT, PoE. > > > > Signed-off-by: Jagan Teki <jagan@amarulasolutions.com> > > --- > > Changes for v2: > > - new patch > > > > Documentation/devicetree/bindings/arm/rockchip.yaml | 6 ++++++ > > 1 file changed, 6 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/arm/rockchip.yaml b/Documentation/devicetree/bindings/arm/rockchip.yaml > > index 36057c9e4b83..7250adb43d24 100644 > > --- a/Documentation/devicetree/bindings/arm/rockchip.yaml > > +++ b/Documentation/devicetree/bindings/arm/rockchip.yaml > > @@ -448,6 +448,12 @@ properties: > > - const: radxa,rockpi4a > > - const: rockchip,rk3399 > > > > + - description: Radxa ROCK Pi 4B > > + items: > > + - const: radxa,rockpi4 > > + - const: radxa,rockpi4b > > + - const: rockchip,rk3399 > > + > > Please do all RockPi4 variants into one entry, so we want something like: > > - description: Radxa ROCK Pi 4 What if the description has something like below. - description: Radxa ROCK Pi 4A/B/C Jagan.
Am Donnerstag, 23. Juli 2020, 08:34:12 CEST schrieb Jagan Teki: > Hi Heiko, > > On Thu, Jul 23, 2020 at 4:43 AM Heiko Stuebner <heiko@sntech.de> wrote: > > > > Hi Jagan, > > > > Am Mittwoch, 22. Juli 2020, 21:09:46 CEST schrieb Jagan Teki: > > > Add dt-bindings for ROCKPi 4B which is similar to 4A with > > > additional AP6256 Wifi/BT, PoE. > > > > > > Signed-off-by: Jagan Teki <jagan@amarulasolutions.com> > > > --- > > > Changes for v2: > > > - new patch > > > > > > Documentation/devicetree/bindings/arm/rockchip.yaml | 6 ++++++ > > > 1 file changed, 6 insertions(+) > > > > > > diff --git a/Documentation/devicetree/bindings/arm/rockchip.yaml b/Documentation/devicetree/bindings/arm/rockchip.yaml > > > index 36057c9e4b83..7250adb43d24 100644 > > > --- a/Documentation/devicetree/bindings/arm/rockchip.yaml > > > +++ b/Documentation/devicetree/bindings/arm/rockchip.yaml > > > @@ -448,6 +448,12 @@ properties: > > > - const: radxa,rockpi4a > > > - const: rockchip,rk3399 > > > > > > + - description: Radxa ROCK Pi 4B > > > + items: > > > + - const: radxa,rockpi4 > > > + - const: radxa,rockpi4b > > > + - const: rockchip,rk3399 > > > + > > > > Please do all RockPi4 variants into one entry, so we want something like: > > > > - description: Radxa ROCK Pi 4 > > What if the description has something like below. > > - description: Radxa ROCK Pi 4A/B/C That's also ok :-)
diff --git a/Documentation/devicetree/bindings/arm/rockchip.yaml b/Documentation/devicetree/bindings/arm/rockchip.yaml index 36057c9e4b83..7250adb43d24 100644 --- a/Documentation/devicetree/bindings/arm/rockchip.yaml +++ b/Documentation/devicetree/bindings/arm/rockchip.yaml @@ -448,6 +448,12 @@ properties: - const: radxa,rockpi4a - const: rockchip,rk3399 + - description: Radxa ROCK Pi 4B + items: + - const: radxa,rockpi4 + - const: radxa,rockpi4b + - const: rockchip,rk3399 + - description: Radxa ROCK Pi N8 items: - const: radxa,rockpi-n8
Add dt-bindings for ROCKPi 4B which is similar to 4A with additional AP6256 Wifi/BT, PoE. Signed-off-by: Jagan Teki <jagan@amarulasolutions.com> --- Changes for v2: - new patch Documentation/devicetree/bindings/arm/rockchip.yaml | 6 ++++++ 1 file changed, 6 insertions(+)