From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f46.google.com (mail-wm1-f46.google.com [209.85.128.46]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9E8732D7DF1 for ; Tue, 23 Jun 2026 15:35:40 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.46 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1782228943; cv=none; b=QvB8RlqvlqIm06oiJQBr3ikpgIqkrP+7e3MS2cJ75WAkUPK+d7HSbKK4KxyhBwiNuQKgfJbDYOS/Pywhzdtr3lJY2fsLn8XK3ipsKEp0Vw4Ri16D67X6FFjxPpNH3P2KkzXlQCWv+SoVzlt9INXZxQ7uWo+atFJ785jWDDnXPNA= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1782228943; c=relaxed/simple; bh=Mo5yQHzaURvGYHb7CZmhngVSC2hCJb50rxMxvMmHtdU=; h=From:To:Cc:Subject:In-Reply-To:References:Date:Message-ID: MIME-Version:Content-Type; b=JxISrK3V0qnaSRgqTdGmK23Xri2wz9BdSjSWAQy26/BD/JvfnMLpYsb8CYpilYY8hvbMCcycnaAls5xGXI+blS++hZSo0CKxxgFD8+DAUxekCc1cru8ov6YqmZhZXKoVYQKAr7PnYoca1n1bUvoGy1wqa9giBDApykXfvojBGHY= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com; spf=pass smtp.mailfrom=baylibre.com; dkim=pass (2048-bit key) header.d=baylibre.com header.i=@baylibre.com header.b=Th9sr2mR; arc=none smtp.client-ip=209.85.128.46 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=baylibre.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=baylibre.com header.i=@baylibre.com header.b="Th9sr2mR" Received: by mail-wm1-f46.google.com with SMTP id 5b1f17b1804b1-490cdae130cso219225e9.0 for ; Tue, 23 Jun 2026 08:35:40 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre.com; s=google; t=1782228939; x=1782833739; darn=vger.kernel.org; h=mime-version:message-id:date:user-agent:references:in-reply-to :subject:cc:to:from:from:to:cc:subject:date:message-id:reply-to; bh=Tv+HFheUg03EfuoN4T+QaHYdh9A1PWG93ECLbXxMf3g=; b=Th9sr2mRNwwdIKBsCMjHWRKy4CD7VG31vQFJrE8XtRne61mbBFNYa/uBkkACiE8aVK XSSsekfyCSxHZZ1GKPVTfCFtDpXWkHH8vpxUD9IXShZDoIQvHpsjaQRJ9xsl+mmOKzTd iMyG4Y3Wlb6h3FBFprYBuAdmzMRYgJHSl8rlQPl6JOFUqQ2faiOIDw8TlBFtVbFbarEC ZNDRL6UwOzYA57SjPYxDmEAeldAnaMQbseeQugL6+ZZxGm+eBJAyXJ2MQZiKYZTt+zTQ 2+Z0b3PmR5nYFZXPCPGxYX7iwcT92JwaKe6RQvHdV4XZ2DKwMeg4qBMpvmZ+X3UbFkHj BFgQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1782228939; x=1782833739; h=mime-version:message-id:date:user-agent:references:in-reply-to :subject:cc:to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject :date:message-id:reply-to; bh=Tv+HFheUg03EfuoN4T+QaHYdh9A1PWG93ECLbXxMf3g=; b=EqtR1I4RNZXZw5/m4G/Jt+AiIDlOJ/mZ1K1rpfmno4J/BMrdsRWATI3oSKObTCoFEn xsVlyCjpkwrUfjzoQVgeDEKItx4OhLERoG8v+sMHIs3c2wBK7hmGalBSp3j6l8puUOTl 3u3FT3QU/bhHj/1pWjE26kg+aaW43blc/h0YfzDtD3S8QJN/3v57DN4s3vFKbjL16S2H mSlyKrEUZI1KK9HOX9YRgQl4ntLveHW8ulZcVQk+fv/y4O8nHTYxGi/sf4AGonRG5C8Z 7qCQSBWSYMca4AuxazB3D4woldlw1wAwX80kQDmvf76ykcFHcm+y8N9KlwYRPkzCy3uT Ns3Q== X-Forwarded-Encrypted: i=1; AFNElJ8J5pKkEgsjJ5ZQM3sul/4lCBm/hqKzZ+MBKklY2/dslLVG9v9sMHrCb4GoL8foNKE6WUMPzVPsFfiBYdU=@vger.kernel.org X-Gm-Message-State: AOJu0Yxe215lqbEPkRoHK8pHYE1r2yGdOaiFNxMOrChLT1z5AxEKXugr RNuR6y/uPfhJamy2S8y3jo/IlQFTccWLOAN9LE9sBqLPmt+vKvgPFaPkjpfof5ul1js= X-Gm-Gg: AfdE7cki2l6o/QG41EKbevGT/K2WUxG7cOKwpBozGwGzPXSnUsXwKBjJqaT2nDqxGDt 7BO/iV6lcIuRPoI2fTKlvMwoNjUs0Z/avwKYN/7WhBrr8sQxdyBl1GLwSWVOJh2cNduVukdaSp/ Wojkz3ZqVTLkHcGTYYRam2J3qCB/MonrncHZOnONgSmQucug5GeVKS+tqx8jdtHTLOIplECpRpA NEBxB0LmrIxR5Ws3CUSjuwNAh1niaz4ztZ7phhpctDz91kkLOWSLUKW/Awqz44/Wyy89Pa8Uk/B WljPM8XUemVh0xJtkiRE4qcWILAHHgbqUiubhBzV2bWroPcEQ/LqsuJww2UAO4VB6dxV7r0fVIt YZ9dsU9HkISw+PSy+UYeSrYZR6yHyH51I2rSPESnGHHTqhxjfAlxpK7AQEPW4d97cPbzQgOyusq pGyefreG6+xA== X-Received: by 2002:a05:600c:8b18:b0:490:e5c1:b8bf with SMTP id 5b1f17b1804b1-4925b38cfa9mr49681285e9.13.1782228939052; Tue, 23 Jun 2026 08:35:39 -0700 (PDT) Received: from localhost ([2a01:e0a:3c5:5fb1:20f1:5c9:8a6d:48a1]) by smtp.gmail.com with UTF8SMTPSA id 5b1f17b1804b1-4923fe7b9e5sm370653045e9.10.2026.06.23.08.35.38 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 23 Jun 2026 08:35:38 -0700 (PDT) From: Jerome Brunet To: Enzo Adriano via B4 Relay Cc: Rob Herring , Krzysztof Kozlowski , Conor Dooley , Chen-Yu Tsai , Jernej Skrabec , Samuel Holland , Maxime Ripard , Ulf Hansson , enzo.adriano.code@gmail.com, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-sunxi@lists.linux.dev, linux-kernel@vger.kernel.org, linux-mmc@vger.kernel.org Subject: Re: [PATCH 3/4] arm64: dts: allwinner: add Allwinner A733 SoC In-Reply-To: <20260613-a733-dts-v1-public-ready-v1-3-7787c94681db@gmail.com> (Enzo Adriano via's message of "Sat, 13 Jun 2026 05:42:15 -0400") References: <20260613-a733-dts-v1-public-ready-v1-0-7787c94681db@gmail.com> <20260613-a733-dts-v1-public-ready-v1-3-7787c94681db@gmail.com> User-Agent: mu4e 1.12.9; emacs 30.1 Date: Tue, 23 Jun 2026 17:35:36 +0200 Message-ID: <1j33ydb77b.fsf@starbuckisacylon.baylibre.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain On sam. 13 juin 2026 at 05:42, Enzo Adriano via B4 Relay wrote: > From: Enzo Adriano > > Add the initial A733 SoC description with CPUs, timers, interrupt > controller, clocks, pinctrl, UART0, and MMC0. > > Keep peripherals disabled by default. Board DTS files can enable only the > devices that are proven on their hardware. > > Signed-off-by: Enzo Adriano > --- > arch/arm64/boot/dts/allwinner/sun60i-a733.dtsi | 198 +++++++++++++++++++++++++ > 1 file changed, 198 insertions(+) > > diff --git a/arch/arm64/boot/dts/allwinner/sun60i-a733.dtsi b/arch/arm64/boot/dts/allwinner/sun60i-a733.dtsi > new file mode 100644 > index 000000000000..3721aa9e8573 > --- /dev/null > +++ b/arch/arm64/boot/dts/allwinner/sun60i-a733.dtsi > @@ -0,0 +1,198 @@ > +// SPDX-License-Identifier: (GPL-2.0-only OR MIT) > + > +#include > +#include > +#include > + > +/ { > + interrupt-parent = <&gic>; > + #address-cells = <2>; > + #size-cells = <2>; > + > + cpus { > + #address-cells = <1>; > + #size-cells = <0>; > + > + cpu0: cpu@0 { > + compatible = "arm,cortex-a55"; > + device_type = "cpu"; > + reg = <0x000>; > + enable-method = "psci"; > + capacity-dmips-mhz = <530>; > + }; > + > + cpu1: cpu@100 { > + compatible = "arm,cortex-a55"; > + device_type = "cpu"; > + reg = <0x100>; > + enable-method = "psci"; > + capacity-dmips-mhz = <530>; > + }; > + > + cpu2: cpu@200 { > + compatible = "arm,cortex-a55"; > + device_type = "cpu"; > + reg = <0x200>; > + enable-method = "psci"; > + capacity-dmips-mhz = <530>; > + }; > + > + cpu3: cpu@300 { > + compatible = "arm,cortex-a55"; > + device_type = "cpu"; > + reg = <0x300>; > + enable-method = "psci"; > + capacity-dmips-mhz = <530>; > + }; > + > + cpu4: cpu@400 { > + compatible = "arm,cortex-a55"; > + device_type = "cpu"; > + reg = <0x400>; > + enable-method = "psci"; > + capacity-dmips-mhz = <530>; > + }; > + > + cpu5: cpu@500 { > + compatible = "arm,cortex-a55"; > + device_type = "cpu"; > + reg = <0x500>; > + enable-method = "psci"; > + capacity-dmips-mhz = <530>; > + }; > + > + cpu6: cpu@600 { > + compatible = "arm,cortex-a76"; > + device_type = "cpu"; > + reg = <0x600>; > + enable-method = "psci"; > + capacity-dmips-mhz = <1024>; > + }; > + > + cpu7: cpu@700 { > + compatible = "arm,cortex-a76"; > + device_type = "cpu"; > + reg = <0x700>; > + enable-method = "psci"; > + capacity-dmips-mhz = <1024>; > + }; > + }; > + > + osc24M: osc24M-clk { Note A733 supports 19.2MHz, 24MHz and 26MHz xtals apparently. The A7S and A7A do have a 26MHz xtal according to the schematics. While this might be fine in the SoC dtsi, your are missing something in your board dts to change the xtal rate, at least. Also the node and clock name are a bit misleading now. > + #clock-cells = <0>; > + compatible = "fixed-clock"; > + clock-frequency = <24000000>; > + clock-output-names = "osc24M"; > + }; > + > + osc32k: osc32k-clk { > + #clock-cells = <0>; > + compatible = "fixed-clock"; > + clock-frequency = <32768>; > + clock-output-names = "osc32k"; I think this is the ext32k supposed to feed the rtc ccu ... > + }; > + > + iosc: internal-osc-clk { > + #clock-cells = <0>; > + compatible = "fixed-clock"; > + clock-frequency = <16000000>; > + clock-output-names = "iosc"; > + }; > + > + psci { > + compatible = "arm,psci-1.0", "arm,psci-0.2"; > + method = "smc"; > + }; > + > + timer { > + compatible = "arm,armv8-timer"; > + arm,no-tick-in-suspend; > + interrupts = , > + , > + , > + ; > + }; > + > + soc { > + compatible = "simple-bus"; > + #address-cells = <1>; > + #size-cells = <1>; > + ranges = <0x0 0x0 0x0 0x40000000>; > + > + pio: pinctrl@2000000 { > + compatible = "allwinner,sun60i-a733-pinctrl"; > + reg = <0x02000000 0x600>; > + interrupts = , > + , > + , > + , > + , > + , > + , > + , > + , > + , > + ; > + clocks = <&ccu CLK_APB1>, <&osc24M>, <&osc32k>; > + clock-names = "apb", "hosc", "losc"; > + gpio-controller; > + #gpio-cells = <3>; > + interrupt-controller; > + #interrupt-cells = <3>; > + > + mmc0_pins: mmc0-pins { > + pins = "PF0", "PF1", "PF2", > + "PF3", "PF4", "PF5"; > + function = "mmc0"; > + drive-strength = <30>; > + bias-pull-up; > + }; > + }; > + > + ccu: clock-controller@2002000 { > + compatible = "allwinner,sun60i-a733-ccu"; > + reg = <0x02002000 0x2000>; > + clocks = <&osc24M>, <&osc32k>, <&iosc>; ^ ... not directly the main CCU. > + clock-names = "hosc", "losc", "iosc"; > + #clock-cells = <1>; > + #reset-cells = <1>; > + }; > + > + uart0: serial@2500000 { > + compatible = "snps,dw-apb-uart"; > + reg = <0x02500000 0x400>; > + interrupts = ; > + reg-shift = <2>; > + reg-io-width = <4>; > + clocks = <&ccu CLK_BUS_UART0>; > + resets = <&ccu RST_BUS_UART0>; > + status = "disabled"; > + }; > + > + gic: interrupt-controller@3400000 { > + compatible = "arm,gic-v3"; > + #interrupt-cells = <3>; > + interrupt-controller; > + reg = <0x03400000 0x10000>, > + <0x03460000 0x100000>; > + }; > + > + mmc0: mmc@4020000 { > + compatible = "allwinner,sun60i-a733-mmc", > + "allwinner,sun20i-d1-mmc"; > + reg = <0x04020000 0x1000>; > + interrupts = ; > + clocks = <&ccu CLK_BUS_MMC0>, <&ccu CLK_MMC0>; > + clock-names = "ahb", "mmc"; > + resets = <&ccu RST_BUS_MMC0>; > + reset-names = "ahb"; > + pinctrl-names = "default"; > + pinctrl-0 = <&mmc0_pins>; > + max-frequency = <200000000>; > + cap-sd-highspeed; > + status = "disabled"; > + #address-cells = <1>; > + #size-cells = <0>; > + }; > + }; > +}; -- Jerome