[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <ede52fd2-fe20-4648-be7b-de10b14d7e3f@kernel.org>
Date: Mon, 15 Sep 2025 06:36:16 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Alexandru Chimac <alex@...mac.ro>, Alim Akhtar <alim.akhtar@...sung.com>,
Rob Herring <robh@...nel.org>, Conor Dooley <conor+dt@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>, Kees Cook <kees@...nel.org>,
Tony Luck <tony.luck@...el.com>, "Guilherme G. Piccoli" <gpiccoli@...lia.com>
Cc: linux-arm-kernel@...ts.infradead.org, linux-samsung-soc@...r.kernel.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-hardening@...r.kernel.org
Subject: Re: [PATCH 2/3] arm64: dts: exynos: Add initial support for the
Exynos9610 SoC
On 14/09/2025 22:44, Alexandru Chimac wrote:
> +
> + arm-a53-pmu {
> + compatible = "arm,cortex-a53-pmu";
> + interrupts = <0 82 4>,
> + <0 83 4>,
> + <0 84 4>,
> + <0 85 4>;
> + interrupt-affinity = <&cpu0>,
> + <&cpu1>,
> + <&cpu2>,
> + <&cpu3>;
> + };
> +
> + arm-a73-pmu {
> + compatible = "arm,cortex-a73-pmu";
> + interrupts = <0 96 4>,
> + <0 97 4>,
> + <0 98 4>,
> + <0 99 4>;
You need to use proper defines.
> + interrupt-affinity = <&cpu100>,
> + <&cpu101>,
> + <&cpu102>,
> + <&cpu103>;
> + };
> +
> + oscclk: clock-osc {
> + compatible = "fixed-clock";
> + #clock-cells = <0>;
> + clock-output-names = "oscclk";
> + clock-frequency = <26000000>;
clock-frequency is board property.
...
> +
> + timer {
> + compatible = "arm,armv8-timer";
> + interrupts = <GIC_PPI 13 (GIC_CPU_MASK_SIMPLE(8) | IRQ_TYPE_LEVEL_LOW)>,
> + <GIC_PPI 14 (GIC_CPU_MASK_SIMPLE(8) | IRQ_TYPE_LEVEL_LOW)>,
> + <GIC_PPI 11 (GIC_CPU_MASK_SIMPLE(8) | IRQ_TYPE_LEVEL_LOW)>,
> + <GIC_PPI 10 (GIC_CPU_MASK_SIMPLE(8) | IRQ_TYPE_LEVEL_LOW)>;
> +
> + /* Stock Samsung bootloader doesn't configure CNTFRQ_EL0 */
> + clock-frequency = <26000000>;
> + };
> +
> + reserved-memory {
> + #address-cells = <2>;
> + #size-cells = <1>;
> + ranges;
> +
> + abox_rmem: abox@...00000 {
What is abox?
> + compatible = "reserved-memory";
> + reg = <0x0 0xe9400000 0x2800000>;
> + no-map;
> + };
> +
> + ramoops@...10000 {
This belongs to the the board.
> + compatible = "ramoops";
> + reg = <0x0 0xf9d10000 0x200000>;
> + record-size = <0x80000>;
> + console-size = <0x80000>;
Best regards,
Krzysztof
Powered by blists - more mailing lists