Re: [RFC PATCH] arm64: dts: rockchip: add core devicetree for rk3318

From: Heiko Stuebner
Date: Fri Apr 17 2020 - 07:05:36 EST


Hi Johan,

Am Freitag, 17. April 2020, 12:57:39 CEST schrieb Johan Jonker:
> The rk3318 is basically a rk3328 with improved gpu,
> so add a dtsi based on that.
>
> Signed-off-by: Johan Jonker <jbx6244@xxxxxxxxx>

Please don't add dangling dtsi files.
I'd expect at least a board dts to actually use that and
also the updated gpu node.

>From a cursory glance it looks like it gets upgraded from
450-MP2 to 450MP3 or so, so that would at least mean
another set of interrupts.

Heiko

> ---
> arch/arm64/boot/dts/rockchip/rk3318.dtsi | 3 +++
> 1 file changed, 3 insertions(+)
> create mode 100644 arch/arm64/boot/dts/rockchip/rk3318.dtsi
>
> diff --git a/arch/arm64/boot/dts/rockchip/rk3318.dtsi b/arch/arm64/boot/dts/rockchip/rk3318.dtsi
> new file mode 100644
> index 000000000..a32f771bc
> --- /dev/null
> +++ b/arch/arm64/boot/dts/rockchip/rk3318.dtsi
> @@ -0,0 +1,3 @@
> +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
> +
> +#include "rk3328.dtsi"
>