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 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id DA1C0C54E58 for ; Tue, 12 Mar 2024 10:03:52 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:In-reply-to: Date:Subject:Cc:To:From:References:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=zsJXWCYs4XFdVqMGVN1fFdPY35D+QPAT0nyq21+q+PM=; b=asMPRx4gL8v7gF ppZzOdVbSdvLZ46/KPmig66UQQRLrU+fLMJTDGhoTReVqkJAx/LCEipxli4O64l4X/RS34KudGxYj lK7eK4JiZfTFS4yP6ZW6QQLJK1jrD1c6VUdJIS1ROH5slqA0epIDBVnNOs+10+cXmSnmLTutvICkB Yb8azfwxT25A7pg0h1kWTYl7E2yoKfqa/3c9Sx0TXvZUOIWAJmWvACwbDe7i9PoaSHFwEx43FwDPT Utt23L+8d7JE+flua6aCitQPzpi8sLPR0xvJkDpStIsQDCKEVKxokpV1JegD3uQmJeNoCrJ42uuJ7 kQcdiUMrYa3TDaGekPsQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1rjyyr-00000005FqI-3H7f; Tue, 12 Mar 2024 10:03:33 +0000 Received: from mail-wm1-x32f.google.com ([2a00:1450:4864:20::32f]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1rjyyn-00000005Fny-2fvo for linux-arm-kernel@lists.infradead.org; Tue, 12 Mar 2024 10:03:32 +0000 Received: by mail-wm1-x32f.google.com with SMTP id 5b1f17b1804b1-413318a3a29so5104765e9.2 for ; Tue, 12 Mar 2024 03:03:26 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20230601.gappssmtp.com; s=20230601; t=1710237806; x=1710842606; darn=lists.infradead.org; h=mime-version:message-id:in-reply-to:date:subject:cc:to:from :user-agent:references:from:to:cc:subject:date:message-id:reply-to; bh=+6NBK9GZFZcA1mqhksqU9TzRXZJpQpDT2fBbPUtZjzY=; b=ZakBzM2Ejry28biTzGHFABxw4ETVOaBUSs+ZKJ0vnL0NgqNGC2381+8cgI1txmXkqM mnxiT8moWusfO5yYfUPp4ICWBIi8VHY/OMpC8+09UfPA8tYFQC2geub0p6eadZrvEg4e Dlac5ZhTfWCXAs/6sCUtOwR1RBMkBO5A3muUMh5z2U3GW96PqWxHzdhsLPYTXJlxI2T4 fNLJbQVbiaHCDSrNlqZ8lQ22DMTQlLoVjDCwk62fpvpfz9uf/QTHrwSYFiW/eDUwWunk Pf7tDirwIsYU7tcDQj47mAfA30mK0boZPcn33Ut96stTvDgCdirlhkpaGaRS8Qc6MSHv 45lA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1710237806; x=1710842606; h=mime-version:message-id:in-reply-to:date:subject:cc:to:from :user-agent:references:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=+6NBK9GZFZcA1mqhksqU9TzRXZJpQpDT2fBbPUtZjzY=; b=WNltnGiVdxx3SwK8LCMDyTUIknQ5wDIhEpsaLsvc6rcztByeCTQWB8nLleshY4Ffmo brtc78Yv15rsatnwvAfH1UNuqcg67dvo1WpiqS6W4oElKGMCrItQdejjlWSF0FGRZhBe vvScPgdOzza+Z+u/dYfRDJiVupcCsLa7uryhw+jKqGNHoQICRFWnitIJahd+NQxsrOS8 M19lwCH3lCKpxulLFqdiCirM2zFXn/A6VkRQe9Hny3ALqIomojo1klKcAVNBL8Xzik1u eM830hGGr3KdRPIDuOj5w+Bl+tcEKakSIwObV51oJN61IyeWIzF1nsYh7UEqBpf+DJSE Qtsg== X-Forwarded-Encrypted: i=1; AJvYcCXAtHMVEJYjfYuJziw20iYf9QG/rjy5VkGw/VJm+9CfPKX7/AOSW0APAJOQajTJJ9rVTiw8hlhUAayKFfVA9pkv6aGHFVztf9dDdgwPKP6BF9UqQyc= X-Gm-Message-State: AOJu0YzXPEmj/2obVyVnf2KA7u7KNZkxBMYzM77r5M3wHJ9Nu2JtS2Az XgF9/cGii431ABEts/AMvOniGjuPYPkBpqxFeNFrhKkp25oqdWSG5wimfUfFYu4= X-Google-Smtp-Source: AGHT+IFwmAwXZJLUTdK6OfZHgdYWyHu7gnRIhGHS72jZJ+Xk3H6cq0glt8xuBj/czyH8RrBJrrq57g== X-Received: by 2002:a05:600c:4f0f:b0:413:117e:fd4a with SMTP id l15-20020a05600c4f0f00b00413117efd4amr6520264wmq.3.1710237805350; Tue, 12 Mar 2024 03:03:25 -0700 (PDT) Received: from localhost ([2a01:e0a:3c5:5fb1:af7b:cf60:208b:83f4]) by smtp.gmail.com with ESMTPSA id h2-20020a056000000200b0033e433cb471sm8660214wrx.79.2024.03.12.03.03.24 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 12 Mar 2024 03:03:24 -0700 (PDT) References: <20240312-basic_dt-v1-0-7f11df3a0896@amlogic.com> <20240312-basic_dt-v1-3-7f11df3a0896@amlogic.com> User-agent: mu4e 1.10.8; emacs 29.2 From: Jerome Brunet To: xianwei.zhao@amlogic.com Cc: Rob Herring , Krzysztof Kozlowski , Conor Dooley , Neil Armstrong , Martin Blumenstingl , Jerome Brunet , Kevin Hilman , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-amlogic@lists.infradead.org Subject: Re: [PATCH 3/4] arm64: dts: add support for A4 based Amlogic BA400 Date: Tue, 12 Mar 2024 10:55:46 +0100 In-reply-to: <20240312-basic_dt-v1-3-7f11df3a0896@amlogic.com> Message-ID: <1jsf0vephv.fsf@starbuckisacylon.baylibre.com> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240312_030330_014910_48904449 X-CRM114-Status: GOOD ( 25.07 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Tue 12 Mar 2024 at 17:18, Xianwei Zhao via B4 Relay wrote: > From: Xianwei Zhao > > Amlogic A4 is an application processor designed for smart audio > and IoT applications. > > Add basic support for the A4 based Amlogic BA400 board, which describes > the following components: CPU, GIC, IRQ, Timer and UART. > These are capable of booting up into the serial console. > > Signed-off-by: Xianwei Zhao > --- > arch/arm64/boot/dts/amlogic/Makefile | 1 + > .../boot/dts/amlogic/amlogic-a4-a113l2-ba400.dts | 43 ++++++++++ > arch/arm64/boot/dts/amlogic/amlogic-a4.dtsi | 99 ++++++++++++++++++++++ > 3 files changed, 143 insertions(+) > > diff --git a/arch/arm64/boot/dts/amlogic/Makefile b/arch/arm64/boot/dts/amlogic/Makefile > index 1ab160bf928a..9a50ec11bb8d 100644 > --- a/arch/arm64/boot/dts/amlogic/Makefile > +++ b/arch/arm64/boot/dts/amlogic/Makefile > @@ -1,4 +1,5 @@ > # SPDX-License-Identifier: GPL-2.0 > +dtb-$(CONFIG_ARCH_MESON) += amlogic-a4-a113l2-ba400.dtb > dtb-$(CONFIG_ARCH_MESON) += amlogic-c3-c302x-aw409.dtb > dtb-$(CONFIG_ARCH_MESON) += amlogic-t7-a311d2-an400.dtb > dtb-$(CONFIG_ARCH_MESON) += amlogic-t7-a311d2-khadas-vim4.dtb > diff --git a/arch/arm64/boot/dts/amlogic/amlogic-a4-a113l2-ba400.dts b/arch/arm64/boot/dts/amlogic/amlogic-a4-a113l2-ba400.dts > new file mode 100644 > index 000000000000..60f9f23858c6 > --- /dev/null > +++ b/arch/arm64/boot/dts/amlogic/amlogic-a4-a113l2-ba400.dts > @@ -0,0 +1,43 @@ > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > +/* > + * Copyright (c) 2024 Amlogic, Inc. All rights reserved. > + */ > + > +/dts-v1/; > + > +#include "amlogic-a4.dtsi" Could you describe how the a4 and a5 differs from each other ? The description given in the commit description is the same. Beside the a53 vs a55, I'm not seeing much of a difference. Admittedly, there is not much yet but I wonder if a4 and a5 should have a common dtsi. > + > +/ { > + model = "Amlogic A113L2 ba400 Development Board"; > + compatible = "amlogic,ba400","amlogic,a4"; > + interrupt-parent = <&gic>; > + #address-cells = <2>; > + #size-cells = <2>; > + > + aliases { > + serial0 = &uart_b; > + }; > + > + memory@0 { > + device_type = "memory"; > + reg = <0x0 0x0 0x0 0x40000000>; > + }; > + > + reserved-memory { > + #address-cells = <2>; > + #size-cells = <2>; > + ranges; > + > + /* 52 MiB reserved for ARM Trusted Firmware */ That's a lot of memory to blindly reserve. Any chance we can stop doing that and have u-boot amend reserved memory zone based on the actual needs of the device ? > + secmon_reserved:linux,secmon { > + compatible = "shared-dma-pool"; > + no-map; > + alignment = <0x0 0x400000>; > + reg = <0x0 0x05000000 0x0 0x3400000>; > + }; > + }; > +}; > + > +&uart_b { > + status = "okay"; > +}; > diff --git a/arch/arm64/boot/dts/amlogic/amlogic-a4.dtsi b/arch/arm64/boot/dts/amlogic/amlogic-a4.dtsi > new file mode 100644 > index 000000000000..7e8745010b52 > --- /dev/null > +++ b/arch/arm64/boot/dts/amlogic/amlogic-a4.dtsi > @@ -0,0 +1,99 @@ > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT) > +/* > + * Copyright (c) 2024 Amlogic, Inc. All rights reserved. > + */ > + > +#include > +#include > +#include > +/ { > + cpus { > + #address-cells = <2>; > + #size-cells = <0>; > + > + cpu0: cpu@0 { > + device_type = "cpu"; > + compatible = "arm,cortex-a53"; > + reg = <0x0 0x0>; > + enable-method = "psci"; > + }; > + > + cpu1: cpu@1 { > + device_type = "cpu"; > + compatible = "arm,cortex-a53"; > + reg = <0x0 0x1>; > + enable-method = "psci"; > + }; > + > + cpu2: cpu@2 { > + device_type = "cpu"; > + compatible = "arm,cortex-a53"; > + reg = <0x0 0x2>; > + enable-method = "psci"; > + }; > + > + cpu3: cpu@3 { > + device_type = "cpu"; > + compatible = "arm,cortex-a53"; > + reg = <0x0 0x3>; > + enable-method = "psci"; > + }; > + }; > + > + timer { > + compatible = "arm,armv8-timer"; > + interrupts = , > + , > + , > + ; > + }; > + > + psci { > + compatible = "arm,psci-0.2"; Really ? still on the that old version ? > + method = "smc"; > + }; > + > + xtal: xtal-clk { > + compatible = "fixed-clock"; > + clock-frequency = <24000000>; > + clock-output-names = "xtal"; > + #clock-cells = <0>; > + }; > + > + soc { > + compatible = "simple-bus"; > + #address-cells = <2>; > + #size-cells = <2>; > + ranges; > + > + gic: interrupt-controller@fff01000 { > + compatible = "arm,gic-400"; > + #interrupt-cells = <3>; > + #address-cells = <0>; > + interrupt-controller; > + reg = <0x0 0xfff01000 0 0x1000>, > + <0x0 0xfff02000 0 0x2000>, > + <0x0 0xfff04000 0 0x2000>, > + <0x0 0xfff06000 0 0x2000>; > + interrupts = ; > + }; > + > + apb@fe000000 { > + compatible = "simple-bus"; > + reg = <0x0 0xfe000000 0x0 0x480000>; > + #address-cells = <2>; > + #size-cells = <2>; > + ranges = <0x0 0x0 0x0 0xfe000000 0x0 0x480000>; > + > + uart_b: serial@7a000 { > + compatible = "amlogic,meson-s4-uart", > + "amlogic,meson-ao-uart"; > + reg = <0x0 0x7a000 0x0 0x18>; > + interrupts = ; > + clocks = <&xtal>, <&xtal>, <&xtal>; > + clock-names = "xtal", "pclk", "baud"; > + status = "disabled"; > + }; > + }; > + }; > +}; -- Jerome _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel