From: Heiko Stuebner <heiko@sntech.de>
To: agajjar <Akash_Gajjar@mentor.com>,
Enric Balletbo i Serra <enric.balletbo@collabora.com>
Cc: deepak_das@mentor.com, Rob Herring <robh+dt@kernel.org>,
Mark Rutland <mark.rutland@arm.com>,
Jagan Teki <jagan@amarulasolutions.com>,
Levin Du <djw@t-chip.com.cn>,
Ezequiel Garcia <ezequiel@collabora.com>,
Klaus Goger <klaus.goger@theobroma-systems.com>,
Arnd Bergmann <arnd@arndb.de>,
Masahiro Yamada <yamada.masahiro@socionext.com>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Shohei Maruyama <cheat.sc.linux@outlook.com>,
Shawn Lin <shawn.lin@rock-chips.com>,
Heinrich Schuchardt <xypron.glpk@gmx.de>,
devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] arm64: dts: rockchip: add initial dts support for Rockpro64 board
Date: Sat, 15 Sep 2018 20:47:33 +0200 [thread overview]
Message-ID: <2506786.bBPgq83TQz@phil> (raw)
In-Reply-To: <bdb645c3-d9b7-03e8-7dcf-7d5bda91c4c8@mentor.com>
Am Samstag, 15. September 2018, 11:03:57 CEST schrieb agajjar:
> Hi Heiko,
>
> On 9/14/2018 11:23 PM, Heiko Stuebner wrote:
> > Hi Akash,
> >
> > Am Freitag, 14. September 2018, 14:09:09 CEST schrieb Akash Gajjar:
> >> Rockpro64 board is a rockchip RK3399 based board from pine64.org.
> >> This commit adds initial device tree support for Rockpro64 board.
> >>
> >> Signed-off-by: Akash Gajjar <Akash_Gajjar@mentor.com>
> > generally the patch looks great already. Hopefully we will get an Ack
> > from the dt-maintaintainers for the binding and I noticed two small
> > issues I'll describe below.
> >
> >
> >> ---
> >> Documentation/devicetree/bindings/arm/rockchip.txt | 4 +
> >> arch/arm64/boot/dts/rockchip/Makefile | 1 +
> >> arch/arm64/boot/dts/rockchip/rk3399-rockpro64.dts | 770 +++++++++++++++++++++
> >> 3 files changed, 775 insertions(+)
> >> create mode 100644 arch/arm64/boot/dts/rockchip/rk3399-rockpro64.dts
> >>
> >> diff --git a/Documentation/devicetree/bindings/arm/rockchip.txt b/Documentation/devicetree/bindings/arm/rockchip.txt
> >> index acfd3c7..ac95183 100644
> >> --- a/Documentation/devicetree/bindings/arm/rockchip.txt
> >> +++ b/Documentation/devicetree/bindings/arm/rockchip.txt
> >> @@ -160,6 +160,10 @@ Rockchip platforms device tree bindings
> >> Required root node properties:
> >> - compatible = "pine64,rock64", "rockchip,rk3328";
> >>
> >> +- Pine64 RockPro64 board:
> >> + Required root node properties:
> >> + - compatible = "pine64,rockpro64", "rockchip,rk3399";
> >> +
> >> - Rockchip PX3 Evaluation board:
> >> Required root node properties:
> >> - compatible = "rockchip,px3-evb", "rockchip,px3", "rockchip,rk3188";
> >> diff --git a/arch/arm64/boot/dts/rockchip/Makefile b/arch/arm64/boot/dts/rockchip/Makefile
> >> index b0092d9..03d523a 100644
> >> --- a/arch/arm64/boot/dts/rockchip/Makefile
> >> +++ b/arch/arm64/boot/dts/rockchip/Makefile
> >> @@ -14,5 +14,6 @@ dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399-firefly.dtb
> >> dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399-gru-bob.dtb
> >> dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399-gru-kevin.dtb
> >> dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399-puma-haikou.dtb
> >> +dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399-rockpro64.dtb
> >> dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399-sapphire.dtb
> >> dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399-sapphire-excavator.dtb
> >> diff --git a/arch/arm64/boot/dts/rockchip/rk3399-rockpro64.dts b/arch/arm64/boot/dts/rockchip/rk3399-rockpro64.dts
> >> new file mode 100644
> >> index 0000000..9e2d4d9
> >> --- /dev/null
> >> +++ b/arch/arm64/boot/dts/rockchip/rk3399-rockpro64.dts
> >> @@ -0,0 +1,770 @@
> >> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
> >> +/*
> >> + * Copyright (c) 2017 Fuzhou Rockchip Electronics Co., Ltd.
> > Rockchip copyright from 2017? Where is the original
> > dts coming from?
>
> Rockpro64 board is similar to firefly rk-3399 board. DTS is taken from
> firefly board and it is booting fine with minimal changes.
>
> Should I continue with the rockchip rights or update with the mine copy
> rights?
I guess, you could keep the old one and add a second line for 2018 and
your copyright, as you also added changes to it.
> >> + */
> >> +
> >> +/dts-v1/;
> >> +#include <dt-bindings/input/linux-event-codes.h>
> >> +#include <dt-bindings/pwm/pwm.h>
> >> +#include "rk3399.dtsi"
> >> +#include "rk3399-opp.dtsi"
> >> +
> >> +/ {
> >> + model = "Pine64 RockPro64";
> >> + compatible = "pine64,rockpro64", "rockchip,rk3399";
> >> +
> >> + chosen {
> >> + stdout-path = "serial2:1500000n8";
> >> + };
> >> +
> >> + backlight: backlight {
> >> + compatible = "pwm-backlight";
> >> + enable-gpios = <&gpio1 RK_PB5 GPIO_ACTIVE_HIGH>;
> >> + pwms = <&pwm0 0 25000 0>;
> >> + brightness-levels = <
> >> + 0 1 2 3 4 5 6 7
> >> + 8 9 10 11 12 13 14 15
> >> + 16 17 18 19 20 21 22 23
> >> + 24 25 26 27 28 29 30 31
> >> + 32 33 34 35 36 37 38 39
> >> + 40 41 42 43 44 45 46 47
> >> + 48 49 50 51 52 53 54 55
> >> + 56 57 58 59 60 61 62 63
> >> + 64 65 66 67 68 69 70 71
> >> + 72 73 74 75 76 77 78 79
> >> + 80 81 82 83 84 85 86 87
> >> + 88 89 90 91 92 93 94 95
> >> + 96 97 98 99 100 101 102 103
> >> + 104 105 106 107 108 109 110 111
> >> + 112 113 114 115 116 117 118 119
> >> + 120 121 122 123 124 125 126 127
> >> + 128 129 130 131 132 133 134 135
> >> + 136 137 138 139 140 141 142 143
> >> + 144 145 146 147 148 149 150 151
> >> + 152 153 154 155 156 157 158 159
> >> + 160 161 162 163 164 165 166 167
> >> + 168 169 170 171 172 173 174 175
> >> + 176 177 178 179 180 181 182 183
> >> + 184 185 186 187 188 189 190 191
> >> + 192 193 194 195 196 197 198 199
> >> + 200 201 202 203 204 205 206 207
> >> + 208 209 210 211 212 213 214 215
> >> + 216 217 218 219 220 221 222 223
> >> + 224 225 226 227 228 229 230 231
> >> + 232 233 234 235 236 237 238 239
> >> + 240 241 242 243 244 245 246 247
> >> + 248 249 250 251 252 253 254 255>;
> >> + default-brightness-level = <200>;
> > with recent additions to the pwm-backlight we shouldn't need the
> > long brightness-levels table and default-brightness-level anymore.
> some device support is also needs to be added in this dts. In upcoming
> patch-sets I will update those changes.
cool, thanks.
Heiko
WARNING: multiple messages have this Message-ID (diff)
From: heiko@sntech.de (Heiko Stuebner)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] arm64: dts: rockchip: add initial dts support for Rockpro64 board
Date: Sat, 15 Sep 2018 20:47:33 +0200 [thread overview]
Message-ID: <2506786.bBPgq83TQz@phil> (raw)
In-Reply-To: <bdb645c3-d9b7-03e8-7dcf-7d5bda91c4c8@mentor.com>
Am Samstag, 15. September 2018, 11:03:57 CEST schrieb agajjar:
> Hi Heiko,
>
> On 9/14/2018 11:23 PM, Heiko Stuebner wrote:
> > Hi Akash,
> >
> > Am Freitag, 14. September 2018, 14:09:09 CEST schrieb Akash Gajjar:
> >> Rockpro64 board is a rockchip RK3399 based board from pine64.org.
> >> This commit adds initial device tree support for Rockpro64 board.
> >>
> >> Signed-off-by: Akash Gajjar <Akash_Gajjar@mentor.com>
> > generally the patch looks great already. Hopefully we will get an Ack
> > from the dt-maintaintainers for the binding and I noticed two small
> > issues I'll describe below.
> >
> >
> >> ---
> >> Documentation/devicetree/bindings/arm/rockchip.txt | 4 +
> >> arch/arm64/boot/dts/rockchip/Makefile | 1 +
> >> arch/arm64/boot/dts/rockchip/rk3399-rockpro64.dts | 770 +++++++++++++++++++++
> >> 3 files changed, 775 insertions(+)
> >> create mode 100644 arch/arm64/boot/dts/rockchip/rk3399-rockpro64.dts
> >>
> >> diff --git a/Documentation/devicetree/bindings/arm/rockchip.txt b/Documentation/devicetree/bindings/arm/rockchip.txt
> >> index acfd3c7..ac95183 100644
> >> --- a/Documentation/devicetree/bindings/arm/rockchip.txt
> >> +++ b/Documentation/devicetree/bindings/arm/rockchip.txt
> >> @@ -160,6 +160,10 @@ Rockchip platforms device tree bindings
> >> Required root node properties:
> >> - compatible = "pine64,rock64", "rockchip,rk3328";
> >>
> >> +- Pine64 RockPro64 board:
> >> + Required root node properties:
> >> + - compatible = "pine64,rockpro64", "rockchip,rk3399";
> >> +
> >> - Rockchip PX3 Evaluation board:
> >> Required root node properties:
> >> - compatible = "rockchip,px3-evb", "rockchip,px3", "rockchip,rk3188";
> >> diff --git a/arch/arm64/boot/dts/rockchip/Makefile b/arch/arm64/boot/dts/rockchip/Makefile
> >> index b0092d9..03d523a 100644
> >> --- a/arch/arm64/boot/dts/rockchip/Makefile
> >> +++ b/arch/arm64/boot/dts/rockchip/Makefile
> >> @@ -14,5 +14,6 @@ dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399-firefly.dtb
> >> dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399-gru-bob.dtb
> >> dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399-gru-kevin.dtb
> >> dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399-puma-haikou.dtb
> >> +dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399-rockpro64.dtb
> >> dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399-sapphire.dtb
> >> dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3399-sapphire-excavator.dtb
> >> diff --git a/arch/arm64/boot/dts/rockchip/rk3399-rockpro64.dts b/arch/arm64/boot/dts/rockchip/rk3399-rockpro64.dts
> >> new file mode 100644
> >> index 0000000..9e2d4d9
> >> --- /dev/null
> >> +++ b/arch/arm64/boot/dts/rockchip/rk3399-rockpro64.dts
> >> @@ -0,0 +1,770 @@
> >> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
> >> +/*
> >> + * Copyright (c) 2017 Fuzhou Rockchip Electronics Co., Ltd.
> > Rockchip copyright from 2017? Where is the original
> > dts coming from?
>
> Rockpro64 board is similar to firefly rk-3399 board. DTS is taken from
> firefly board and it is booting fine with minimal changes.
>
> Should I continue with the rockchip rights or update with the mine copy
> rights?
I guess, you could keep the old one and add a second line for 2018 and
your copyright, as you also added changes to it.
> >> + */
> >> +
> >> +/dts-v1/;
> >> +#include <dt-bindings/input/linux-event-codes.h>
> >> +#include <dt-bindings/pwm/pwm.h>
> >> +#include "rk3399.dtsi"
> >> +#include "rk3399-opp.dtsi"
> >> +
> >> +/ {
> >> + model = "Pine64 RockPro64";
> >> + compatible = "pine64,rockpro64", "rockchip,rk3399";
> >> +
> >> + chosen {
> >> + stdout-path = "serial2:1500000n8";
> >> + };
> >> +
> >> + backlight: backlight {
> >> + compatible = "pwm-backlight";
> >> + enable-gpios = <&gpio1 RK_PB5 GPIO_ACTIVE_HIGH>;
> >> + pwms = <&pwm0 0 25000 0>;
> >> + brightness-levels = <
> >> + 0 1 2 3 4 5 6 7
> >> + 8 9 10 11 12 13 14 15
> >> + 16 17 18 19 20 21 22 23
> >> + 24 25 26 27 28 29 30 31
> >> + 32 33 34 35 36 37 38 39
> >> + 40 41 42 43 44 45 46 47
> >> + 48 49 50 51 52 53 54 55
> >> + 56 57 58 59 60 61 62 63
> >> + 64 65 66 67 68 69 70 71
> >> + 72 73 74 75 76 77 78 79
> >> + 80 81 82 83 84 85 86 87
> >> + 88 89 90 91 92 93 94 95
> >> + 96 97 98 99 100 101 102 103
> >> + 104 105 106 107 108 109 110 111
> >> + 112 113 114 115 116 117 118 119
> >> + 120 121 122 123 124 125 126 127
> >> + 128 129 130 131 132 133 134 135
> >> + 136 137 138 139 140 141 142 143
> >> + 144 145 146 147 148 149 150 151
> >> + 152 153 154 155 156 157 158 159
> >> + 160 161 162 163 164 165 166 167
> >> + 168 169 170 171 172 173 174 175
> >> + 176 177 178 179 180 181 182 183
> >> + 184 185 186 187 188 189 190 191
> >> + 192 193 194 195 196 197 198 199
> >> + 200 201 202 203 204 205 206 207
> >> + 208 209 210 211 212 213 214 215
> >> + 216 217 218 219 220 221 222 223
> >> + 224 225 226 227 228 229 230 231
> >> + 232 233 234 235 236 237 238 239
> >> + 240 241 242 243 244 245 246 247
> >> + 248 249 250 251 252 253 254 255>;
> >> + default-brightness-level = <200>;
> > with recent additions to the pwm-backlight we shouldn't need the
> > long brightness-levels table and default-brightness-level anymore.
> some device support is also needs to be added in this dts. In upcoming
> patch-sets I will update those changes.
cool, thanks.
Heiko
next prev parent reply other threads:[~2018-09-15 18:47 UTC|newest]
Thread overview: 55+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-09-14 12:09 [PATCH] arm64: dts: rockchip: add initial dts support for Rockpro64 board Akash Gajjar
2018-09-14 12:09 ` Akash Gajjar
2018-09-14 12:09 ` Akash Gajjar
2018-09-14 17:53 ` Heiko Stuebner
2018-09-14 17:53 ` Heiko Stuebner
2018-09-15 9:03 ` agajjar
2018-09-15 9:03 ` agajjar
2018-09-15 9:03 ` agajjar
2018-09-15 18:47 ` Heiko Stuebner [this message]
2018-09-15 18:47 ` Heiko Stuebner
2018-09-17 5:44 ` Rob Herring
2018-09-17 5:44 ` Rob Herring
-- strict thread matches above, loose matches on Subject: below --
2018-09-26 6:24 [PATCH v3] arm64: dts: rockchip: add initial dts support for Rockpro64 Akash Gajjar
2018-09-26 6:24 ` Akash Gajjar
2018-09-26 6:24 ` Akash Gajjar
2018-09-26 9:44 ` Deepak Das
2018-09-26 9:44 ` Deepak Das
2018-09-26 9:44 ` Deepak Das
2018-09-26 11:35 ` Heiko Stuebner
2018-09-26 11:35 ` Heiko Stuebner
2018-09-27 20:02 ` Rob Herring
2018-09-27 20:02 ` Rob Herring
2018-09-19 14:45 [PATCH] arm64: dts: rockchip: update pinmux setting in rk3399.dtsi Akash Gajjar
2018-09-19 14:45 ` Akash Gajjar
2018-09-19 14:45 ` Akash Gajjar
2018-09-24 12:58 ` Heiko Stuebner
2018-09-24 12:58 ` Heiko Stuebner
2018-09-19 14:43 [PATCH] arm64: dts: rockchip: add initial dts support for Rockpro64 board Akash Gajjar
2018-09-19 14:43 ` Akash Gajjar
2018-09-19 14:43 ` Akash Gajjar
2018-09-20 6:25 ` Deepak Das
2018-09-20 6:25 ` Deepak Das
2018-09-20 6:25 ` Deepak Das
2018-09-24 13:05 ` Heiko Stuebner
2018-09-24 13:05 ` Heiko Stuebner
2018-09-25 11:24 ` Rob Herring
2018-09-25 11:24 ` Rob Herring
2018-08-01 7:43 [PATCH] arm64: dts: allwinner: a64: add csi regulator to sopine dts Akash Gajjar
2018-08-01 7:43 ` Akash Gajjar
2018-08-01 7:43 ` Akash Gajjar
2018-08-07 2:52 ` Chen-Yu Tsai
2018-08-07 2:52 ` Chen-Yu Tsai
2018-07-26 6:51 [PATCH] arm64: dts: allwinner: activate spi flash on pine64 LTS board Akash Gajjar
2018-07-26 6:51 ` Akash Gajjar
2018-07-26 6:51 ` Akash Gajjar
2018-07-26 7:29 ` Maxime Ripard
2018-07-26 7:29 ` Maxime Ripard
2018-07-26 7:31 ` Maxime Ripard
2018-07-26 7:31 ` Maxime Ripard
2018-07-26 8:00 ` Emmanuel Vadot
2018-07-26 8:00 ` Emmanuel Vadot
2018-07-22 15:46 [PATCH] arm64: " Akash Gajjar
2018-07-22 15:46 ` Akash Gajjar
2018-07-24 12:51 ` Maxime Ripard
2018-07-24 12:51 ` Maxime Ripard
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=2506786.bBPgq83TQz@phil \
--to=heiko@sntech.de \
--cc=Akash_Gajjar@mentor.com \
--cc=arnd@arndb.de \
--cc=cheat.sc.linux@outlook.com \
--cc=deepak_das@mentor.com \
--cc=devicetree@vger.kernel.org \
--cc=djw@t-chip.com.cn \
--cc=enric.balletbo@collabora.com \
--cc=ezequiel@collabora.com \
--cc=gregkh@linuxfoundation.org \
--cc=jagan@amarulasolutions.com \
--cc=klaus.goger@theobroma-systems.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-rockchip@lists.infradead.org \
--cc=mark.rutland@arm.com \
--cc=robh+dt@kernel.org \
--cc=shawn.lin@rock-chips.com \
--cc=xypron.glpk@gmx.de \
--cc=yamada.masahiro@socionext.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.