From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.1 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 27A6DC433E0 for ; Mon, 29 Jun 2020 21:22:39 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 0648E21D7F for ; Mon, 29 Jun 2020 21:22:39 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=cerno.tech header.i=@cerno.tech header.b="VEtU5b+E"; dkim=pass (2048-bit key) header.d=messagingengine.com header.i=@messagingengine.com header.b="L6DQtBRb" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728040AbgF2Svq (ORCPT ); Mon, 29 Jun 2020 14:51:46 -0400 Received: from new4-smtp.messagingengine.com ([66.111.4.230]:35733 "EHLO new4-smtp.messagingengine.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726831AbgF2Sva (ORCPT ); Mon, 29 Jun 2020 14:51:30 -0400 Received: from compute4.internal (compute4.nyi.internal [10.202.2.44]) by mailnew.nyi.internal (Postfix) with ESMTP id 14CE6580597; Mon, 29 Jun 2020 11:21:17 -0400 (EDT) Received: from mailfrontend1 ([10.202.2.162]) by compute4.internal (MEProxy); Mon, 29 Jun 2020 11:21:17 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cerno.tech; h= date:from:to:cc:subject:message-id:references:mime-version :content-type:in-reply-to; s=fm3; bh=krUOveEdR6vT0SBK3qtfa5n706o ryNuDtBLGwOel0Fo=; b=VEtU5b+EqJvRVoGcecDwOgK9INc1igaqGpNTrbyWjp7 t4GknrXv7tKctUi5JWrH2p6O6uYYIDpxpq2pM/8mTaxmakYjDGs2EQutbHreQYwd fJv+kZaW3ewSRuY8FVsI1HEbFNUwl4erX42gfLwEeTg1ns00ICau45mAoNseOhbC 8kD0h8BGjs8fPp3YjhpxkYlwLEC/2n2D77Szii/1UwuSkmfTTqEGQnnsoyY3GpIl EHBhoY9KmtyUnCsveKv0MW5JG2xy2qCB6TaqQxf0mcFwx0IB64WHhkgiQN6O0lpn DT8q0Ta5pesXagJSN5DYMMpMFpbUCjfwZIr1JA4Z/0w== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-proxy :x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s=fm3; bh=krUOve EdR6vT0SBK3qtfa5n706oryNuDtBLGwOel0Fo=; b=L6DQtBRbaLn4l3ffIzF0Bw bJfxa6fVOnQh9qO0p0mJrmSfuq9FDhIP89CuSO4gemX0ANlp8rukEIP6+d25PuP6 a3QSGPxF/k+Xcx7uqFlMQ2P4OF4kTh9ELsXnA5re3hD2MOlpZgMYXre6ehH5kS4a cCxOeIagn8x5pSEn1VwUNGpbyjjxxHPsAy6Hdca09MtGrRJpAf6loosOhXjVBlR3 Ny+o5D6hqOsFi534Rc4VwRjOE/iQBXI0yjAY/wmsvZwTjr3/tXfURRfYZhIh+Mk8 hd14z5Ml3RtHtEw0mRB/nAAy5xb04b4c9LOIvEKkmyqZj/VrfLGJuhd6ky+jhXhA == X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeduhedrudelledgudehucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepfffhvffukfhfgggtuggjsehgtderredttddvnecuhfhrohhmpeforgigihhm vgcutfhiphgrrhguuceomhgrgihimhgvsegtvghrnhhordhtvggthheqnecuggftrfgrth htvghrnhepleekgeehhfdutdeljefgleejffehfffgieejhffgueefhfdtveetgeehieeh gedunecukfhppeeltddrkeelrdeikedrjeeinecuvehluhhsthgvrhfuihiivgeptdenuc frrghrrghmpehmrghilhhfrhhomhepmhgrgihimhgvsegtvghrnhhordhtvggthh X-ME-Proxy: Received: from localhost (lfbn-tou-1-1502-76.w90-89.abo.wanadoo.fr [90.89.68.76]) by mail.messagingengine.com (Postfix) with ESMTPA id 83B73328005E; Mon, 29 Jun 2020 11:21:13 -0400 (EDT) Date: Mon, 29 Jun 2020 17:21:12 +0200 From: Maxime Ripard To: Frank Lee Cc: robh+dt@kernel.org, wens@csie.org, mturquette@baylibre.com, sboyd@kernel.org, srinivas.kandagatla@linaro.org, linus.walleij@linaro.org, anarsoul@gmail.com, tiny.windzz@gmail.com, rui.zhang@intel.com, daniel.lezcano@linaro.org, amit.kucheria@verdurent.com, p.zabel@pengutronix.de, clabbe@baylibre.com, icenowy@aosc.io, megous@megous.com, karlp@tweak.net.au, bage@linutronix.de, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-clk@vger.kernel.org, linux-gpio@vger.kernel.org, linux-pm@vger.kernel.org, huangshuosheng@allwinnertech.com, liyong@allwinnertech.com Subject: Re: [PATCH v2 09/11] arm64: allwinner: A100: add the basical Allwinner A100 DTSI file Message-ID: <20200629152112.fo3kgcztlutj66hc@gilmour.lan> References: <20200622025907.32574-1-frank@allwinnertech.com> <20200622025907.32574-10-frank@allwinnertech.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="lbejoaecuizw4km4" Content-Disposition: inline In-Reply-To: <20200622025907.32574-10-frank@allwinnertech.com> Sender: linux-gpio-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-gpio@vger.kernel.org --lbejoaecuizw4km4 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi, On Mon, Jun 22, 2020 at 10:59:05AM +0800, Frank Lee wrote: > Allwinner A100 is a new SoC with Cortex-A53 cores, this commit adds > the basical DTSI file of it, including the clock, i2c, pins, sid, ths, > and UART support. >=20 > Signed-off-by: Frank Lee > --- > arch/arm64/boot/dts/allwinner/sun50i-a100.dtsi | 337 +++++++++++++++++++= ++++++ > 1 file changed, 337 insertions(+) > create mode 100644 arch/arm64/boot/dts/allwinner/sun50i-a100.dtsi >=20 > diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a100.dtsi b/arch/arm64/= boot/dts/allwinner/sun50i-a100.dtsi > new file mode 100644 > index 0000000..5133897 > --- /dev/null > +++ b/arch/arm64/boot/dts/allwinner/sun50i-a100.dtsi > @@ -0,0 +1,337 @@ > +// SPDX-License-Identifier: (GPL-2.0+ or MIT) > +/* > + * Copyright (c) 2020 Frank Lee > + */ > + > +#include > +#include > +#include > +#include > +#include > + > +/ { > + interrupt-parent =3D <&gic>; > + #address-cells =3D <2>; > + #size-cells =3D <2>; > + > + cpus { > + #address-cells =3D <1>; > + #size-cells =3D <0>; > + > + cpu0: cpu@0 { > + compatible =3D "arm,armv8"; > + device_type =3D "cpu"; > + reg =3D <0x0>; > + enable-method =3D "psci"; > + }; > + > + cpu@1 { > + compatible =3D "arm,armv8"; > + device_type =3D "cpu"; > + reg =3D <0x1>; > + enable-method =3D "psci"; > + }; > + > + cpu@2 { > + compatible =3D "arm,armv8"; > + device_type =3D "cpu"; > + reg =3D <0x2>; > + enable-method =3D "psci"; > + }; > + > + cpu@3 { > + compatible =3D "arm,armv8"; > + device_type =3D "cpu"; > + reg =3D <0x3>; > + enable-method =3D "psci"; > + }; > + }; > + > + psci { > + compatible =3D "arm,psci-1.0"; > + method =3D "smc"; > + }; > + > + iosc: internal-osc-clk { > + compatible =3D "fixed-clock"; > + clock-frequency =3D <16000000>; > + clock-accuracy =3D <300000000>; > + clock-output-names =3D "iosc"; > + #clock-cells =3D <0>; > + }; > + > + dcxo24M: dcxo24M_clk { You shouldn't have underscores in the node names. > + compatible =3D "fixed-clock"; > + clock-frequency =3D <24000000>; > + clock-output-names =3D "dcxo24M"; > + #clock-cells =3D <0>; > + }; > + > + osc32k: osc32k_clk { Same thing here Also, ordering by node name here would be nice > + compatible =3D "fixed-clock"; > + clock-frequency =3D <32768>; > + clock-output-names =3D "osc32k"; > + #clock-cells =3D <0>; > + }; > + > + timer { > + compatible =3D "arm,armv8-timer"; > + interrupts =3D + (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_HIGH)>, > + + (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_HIGH)>, > + + (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_HIGH)>, > + + (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_HIGH)>; > + }; Does it suffer from the same time instability than the one in the A64? If so, you probably want to enable the workaround too. > + soc: soc { Do you really need a label for that node? > + compatible =3D "simple-bus"; > + #address-cells =3D <2>; > + #size-cells =3D <2>; Why do you need cells with 2 items here? You don't seem to be using them. > + r_i2c0: i2c@7081400 { > + compatible =3D "allwinner,sun6i-a31-i2c"; > + reg =3D <0x0 0x07081400 0x0 0x400>; > + interrupts =3D ; > + clocks =3D <&ccu CLK_BUS_I2C0>; > + resets =3D <&ccu RST_BUS_I2C0>; > + pinctrl-names =3D "default"; > + pinctrl-0 =3D <&r_i2c0_pins>; > + status =3D "disabled"; > + #address-cells =3D <1>; > + #size-cells =3D <0>; > + }; > + > + r_i2c1: i2c@7081800 { > + compatible =3D "allwinner,sun6i-a31-i2c"; > + reg =3D <0x0 0x07081800 0x0 0x400>; > + interrupts =3D ; > + clocks =3D <&ccu CLK_BUS_I2C1>; > + resets =3D <&ccu RST_BUS_I2C1>; > + pinctrl-names =3D "default"; > + pinctrl-0 =3D <&r_i2c1_pins>; > + status =3D "disabled"; > + #address-cells =3D <1>; > + #size-cells =3D <0>; > + }; > + }; The clocks and resets phandles and IDs don't look right > + thermal-zones { > + cpu_thermal_zone { > + polling-delay-passive =3D <0>; > + polling-delay =3D <0>; > + thermal-sensors =3D <&ths 0>; > + }; Please add a new line here > + gpu_thermal_zone{ You should have a space here ^=20 > + polling-delay-passive =3D <0>; > + polling-delay =3D <0>; > + thermal-sensors =3D <&ths 1>; > + }; newline > + ddr_thermal_zone{ space ^=20 Thanks! Maxime --lbejoaecuizw4km4 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRcEzekXsqa64kGDp7j7w1vZxhRxQUCXvoG6AAKCRDj7w1vZxhR xRRCAQCSyQ9nHql6TEykxi9l3Cj8dRiNVK0CnZJ3fTQSc1l2AAD/fJV1R7VkbxZi OFU2QxjJ1CZU81YE8OkmeFr/wMlCsw8= =kgim -----END PGP SIGNATURE----- --lbejoaecuizw4km4-- From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.0 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id CCB11C433E0 for ; Mon, 29 Jun 2020 16:45:31 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 9B4BA2559C for ; Mon, 29 Jun 2020 16:45:31 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="m0qIwX3S"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=cerno.tech header.i=@cerno.tech header.b="VEtU5b+E"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=messagingengine.com header.i=@messagingengine.com header.b="L6DQtBRb" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 9B4BA2559C Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=cerno.tech Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Type:Cc: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=Sh9IeJWSeO9mIz+AMbJ2FfuqChZrESRVkYpCO8/VHz0=; b=m0qIwX3SS2J5EL78ZlmxkespC 7Hqb8pw+i/0IBYcBosCUrI2bWNpdxy8RFRmI6mqOadGiX6aF+aF+TFgOPPgt83Y26ChVRjco4FG8d kCSrDBfdbzu8mqd4jQt9873j+fAXNOUh0PVVAdQ28L57oYsEaLdPfTDLaPLHAkTKNX1AcbF7H//w6 0j0CrSrnZTihKmfsYf1FPvIQY7VFe1Za86u3DDZZPeZ3smcO928HuEUclAo5QDjtq3oNqzBsAPeVs yUbEWsgILpIYOw4sN4WHKXCA+P6uK1JfqH9tZ4N6ivIuZPiGANkzUzzaOdCTDtvx4V2dSCu7e7DRq J1bwZnOfw==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1jpwtX-00065I-B2; Mon, 29 Jun 2020 16:44:35 +0000 Received: from new2-smtp.messagingengine.com ([66.111.4.224]) by merlin.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1jpvax-0008Cn-0y for linux-arm-kernel@lists.infradead.org; Mon, 29 Jun 2020 15:21:25 +0000 Received: from compute4.internal (compute4.nyi.internal [10.202.2.44]) by mailnew.nyi.internal (Postfix) with ESMTP id 14CE6580597; Mon, 29 Jun 2020 11:21:17 -0400 (EDT) Received: from mailfrontend1 ([10.202.2.162]) by compute4.internal (MEProxy); Mon, 29 Jun 2020 11:21:17 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cerno.tech; h= date:from:to:cc:subject:message-id:references:mime-version :content-type:in-reply-to; s=fm3; bh=krUOveEdR6vT0SBK3qtfa5n706o ryNuDtBLGwOel0Fo=; b=VEtU5b+EqJvRVoGcecDwOgK9INc1igaqGpNTrbyWjp7 t4GknrXv7tKctUi5JWrH2p6O6uYYIDpxpq2pM/8mTaxmakYjDGs2EQutbHreQYwd fJv+kZaW3ewSRuY8FVsI1HEbFNUwl4erX42gfLwEeTg1ns00ICau45mAoNseOhbC 8kD0h8BGjs8fPp3YjhpxkYlwLEC/2n2D77Szii/1UwuSkmfTTqEGQnnsoyY3GpIl EHBhoY9KmtyUnCsveKv0MW5JG2xy2qCB6TaqQxf0mcFwx0IB64WHhkgiQN6O0lpn DT8q0Ta5pesXagJSN5DYMMpMFpbUCjfwZIr1JA4Z/0w== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-proxy :x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s=fm3; bh=krUOve EdR6vT0SBK3qtfa5n706oryNuDtBLGwOel0Fo=; b=L6DQtBRbaLn4l3ffIzF0Bw bJfxa6fVOnQh9qO0p0mJrmSfuq9FDhIP89CuSO4gemX0ANlp8rukEIP6+d25PuP6 a3QSGPxF/k+Xcx7uqFlMQ2P4OF4kTh9ELsXnA5re3hD2MOlpZgMYXre6ehH5kS4a cCxOeIagn8x5pSEn1VwUNGpbyjjxxHPsAy6Hdca09MtGrRJpAf6loosOhXjVBlR3 Ny+o5D6hqOsFi534Rc4VwRjOE/iQBXI0yjAY/wmsvZwTjr3/tXfURRfYZhIh+Mk8 hd14z5Ml3RtHtEw0mRB/nAAy5xb04b4c9LOIvEKkmyqZj/VrfLGJuhd6ky+jhXhA == X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeduhedrudelledgudehucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepfffhvffukfhfgggtuggjsehgtderredttddvnecuhfhrohhmpeforgigihhm vgcutfhiphgrrhguuceomhgrgihimhgvsegtvghrnhhordhtvggthheqnecuggftrfgrth htvghrnhepleekgeehhfdutdeljefgleejffehfffgieejhffgueefhfdtveetgeehieeh gedunecukfhppeeltddrkeelrdeikedrjeeinecuvehluhhsthgvrhfuihiivgeptdenuc frrghrrghmpehmrghilhhfrhhomhepmhgrgihimhgvsegtvghrnhhordhtvggthh X-ME-Proxy: Received: from localhost (lfbn-tou-1-1502-76.w90-89.abo.wanadoo.fr [90.89.68.76]) by mail.messagingengine.com (Postfix) with ESMTPA id 83B73328005E; Mon, 29 Jun 2020 11:21:13 -0400 (EDT) Date: Mon, 29 Jun 2020 17:21:12 +0200 From: Maxime Ripard To: Frank Lee Subject: Re: [PATCH v2 09/11] arm64: allwinner: A100: add the basical Allwinner A100 DTSI file Message-ID: <20200629152112.fo3kgcztlutj66hc@gilmour.lan> References: <20200622025907.32574-1-frank@allwinnertech.com> <20200622025907.32574-10-frank@allwinnertech.com> MIME-Version: 1.0 In-Reply-To: <20200622025907.32574-10-frank@allwinnertech.com> X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: megous@megous.com, liyong@allwinnertech.com, tiny.windzz@gmail.com, mturquette@baylibre.com, srinivas.kandagatla@linaro.org, linux-clk@vger.kernel.org, amit.kucheria@verdurent.com, daniel.lezcano@linaro.org, anarsoul@gmail.com, wens@csie.org, huangshuosheng@allwinnertech.com, clabbe@baylibre.com, rui.zhang@intel.com, karlp@tweak.net.au, linus.walleij@linaro.org, icenowy@aosc.io, devicetree@vger.kernel.org, bage@linutronix.de, linux-pm@vger.kernel.org, linux-gpio@vger.kernel.org, robh+dt@kernel.org, linux-arm-kernel@lists.infradead.org, sboyd@kernel.org, linux-kernel@vger.kernel.org, p.zabel@pengutronix.de Content-Type: multipart/mixed; boundary="===============6279051369370171109==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============6279051369370171109== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="lbejoaecuizw4km4" Content-Disposition: inline --lbejoaecuizw4km4 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi, On Mon, Jun 22, 2020 at 10:59:05AM +0800, Frank Lee wrote: > Allwinner A100 is a new SoC with Cortex-A53 cores, this commit adds > the basical DTSI file of it, including the clock, i2c, pins, sid, ths, > and UART support. >=20 > Signed-off-by: Frank Lee > --- > arch/arm64/boot/dts/allwinner/sun50i-a100.dtsi | 337 +++++++++++++++++++= ++++++ > 1 file changed, 337 insertions(+) > create mode 100644 arch/arm64/boot/dts/allwinner/sun50i-a100.dtsi >=20 > diff --git a/arch/arm64/boot/dts/allwinner/sun50i-a100.dtsi b/arch/arm64/= boot/dts/allwinner/sun50i-a100.dtsi > new file mode 100644 > index 0000000..5133897 > --- /dev/null > +++ b/arch/arm64/boot/dts/allwinner/sun50i-a100.dtsi > @@ -0,0 +1,337 @@ > +// SPDX-License-Identifier: (GPL-2.0+ or MIT) > +/* > + * Copyright (c) 2020 Frank Lee > + */ > + > +#include > +#include > +#include > +#include > +#include > + > +/ { > + interrupt-parent =3D <&gic>; > + #address-cells =3D <2>; > + #size-cells =3D <2>; > + > + cpus { > + #address-cells =3D <1>; > + #size-cells =3D <0>; > + > + cpu0: cpu@0 { > + compatible =3D "arm,armv8"; > + device_type =3D "cpu"; > + reg =3D <0x0>; > + enable-method =3D "psci"; > + }; > + > + cpu@1 { > + compatible =3D "arm,armv8"; > + device_type =3D "cpu"; > + reg =3D <0x1>; > + enable-method =3D "psci"; > + }; > + > + cpu@2 { > + compatible =3D "arm,armv8"; > + device_type =3D "cpu"; > + reg =3D <0x2>; > + enable-method =3D "psci"; > + }; > + > + cpu@3 { > + compatible =3D "arm,armv8"; > + device_type =3D "cpu"; > + reg =3D <0x3>; > + enable-method =3D "psci"; > + }; > + }; > + > + psci { > + compatible =3D "arm,psci-1.0"; > + method =3D "smc"; > + }; > + > + iosc: internal-osc-clk { > + compatible =3D "fixed-clock"; > + clock-frequency =3D <16000000>; > + clock-accuracy =3D <300000000>; > + clock-output-names =3D "iosc"; > + #clock-cells =3D <0>; > + }; > + > + dcxo24M: dcxo24M_clk { You shouldn't have underscores in the node names. > + compatible =3D "fixed-clock"; > + clock-frequency =3D <24000000>; > + clock-output-names =3D "dcxo24M"; > + #clock-cells =3D <0>; > + }; > + > + osc32k: osc32k_clk { Same thing here Also, ordering by node name here would be nice > + compatible =3D "fixed-clock"; > + clock-frequency =3D <32768>; > + clock-output-names =3D "osc32k"; > + #clock-cells =3D <0>; > + }; > + > + timer { > + compatible =3D "arm,armv8-timer"; > + interrupts =3D + (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_HIGH)>, > + + (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_HIGH)>, > + + (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_HIGH)>, > + + (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_HIGH)>; > + }; Does it suffer from the same time instability than the one in the A64? If so, you probably want to enable the workaround too. > + soc: soc { Do you really need a label for that node? > + compatible =3D "simple-bus"; > + #address-cells =3D <2>; > + #size-cells =3D <2>; Why do you need cells with 2 items here? You don't seem to be using them. > + r_i2c0: i2c@7081400 { > + compatible =3D "allwinner,sun6i-a31-i2c"; > + reg =3D <0x0 0x07081400 0x0 0x400>; > + interrupts =3D ; > + clocks =3D <&ccu CLK_BUS_I2C0>; > + resets =3D <&ccu RST_BUS_I2C0>; > + pinctrl-names =3D "default"; > + pinctrl-0 =3D <&r_i2c0_pins>; > + status =3D "disabled"; > + #address-cells =3D <1>; > + #size-cells =3D <0>; > + }; > + > + r_i2c1: i2c@7081800 { > + compatible =3D "allwinner,sun6i-a31-i2c"; > + reg =3D <0x0 0x07081800 0x0 0x400>; > + interrupts =3D ; > + clocks =3D <&ccu CLK_BUS_I2C1>; > + resets =3D <&ccu RST_BUS_I2C1>; > + pinctrl-names =3D "default"; > + pinctrl-0 =3D <&r_i2c1_pins>; > + status =3D "disabled"; > + #address-cells =3D <1>; > + #size-cells =3D <0>; > + }; > + }; The clocks and resets phandles and IDs don't look right > + thermal-zones { > + cpu_thermal_zone { > + polling-delay-passive =3D <0>; > + polling-delay =3D <0>; > + thermal-sensors =3D <&ths 0>; > + }; Please add a new line here > + gpu_thermal_zone{ You should have a space here ^=20 > + polling-delay-passive =3D <0>; > + polling-delay =3D <0>; > + thermal-sensors =3D <&ths 1>; > + }; newline > + ddr_thermal_zone{ space ^=20 Thanks! Maxime --lbejoaecuizw4km4 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRcEzekXsqa64kGDp7j7w1vZxhRxQUCXvoG6AAKCRDj7w1vZxhR xRRCAQCSyQ9nHql6TEykxi9l3Cj8dRiNVK0CnZJ3fTQSc1l2AAD/fJV1R7VkbxZi OFU2QxjJ1CZU81YE8OkmeFr/wMlCsw8= =kgim -----END PGP SIGNATURE----- --lbejoaecuizw4km4-- --===============6279051369370171109== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============6279051369370171109==--