[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <be1bcd99-8a05-4b7f-8546-23cd96071e05@gmail.com>
Date: Fri, 6 Dec 2024 18:29:08 +0200
From: Ivaylo Ivanov <ivo.ivanov.ivanov1@...il.com>
To: Umer Uddin <umer.uddin@...tallysanemainliners.org>,
Krzysztof Kozlowski <krzk@...nel.org>, Alim Akhtar
<alim.akhtar@...sung.com>, Rob Herring <robh@...nel.org>,
Conor Dooley <conor+dt@...nel.org>
Cc: linux-arm-kernel@...ts.infradead.org, linux-samsung-soc@...r.kernel.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
igor.belwon@...tallysanemainliners.org
Subject: Re: [PATCH v5 3/4] arm64: dts: exynos: Add initial support for
Samsung Galaxy S20 5G (x1s)
On 12/6/24 17:41, Umer Uddin wrote:
> Add initial support for the Samsung Galaxy S20 5G (x1s/SM-G981B)
> phone. It was launched in 2020, and it's based on the Exynos 990 SoC. It
> has only one configuration with 12GB of RAM and 128GB of UFS 3.0 storage.
>
> This device tree adds support for the following:
>
> - SimpleFB
> - 12GB RAM
> - Buttons
>
> Signed-off-by: Umer Uddin <umer.uddin@...tallysanemainliners.org>
> ---
> arch/arm64/boot/dts/exynos/Makefile | 1 +
> arch/arm64/boot/dts/exynos/exynos990-x1s.dts | 28 ++++++++++++++++++++
> 2 files changed, 29 insertions(+)
> create mode 100644 arch/arm64/boot/dts/exynos/exynos990-x1s.dts
>
> diff --git a/arch/arm64/boot/dts/exynos/Makefile b/arch/arm64/boot/dts/exynos/Makefile
> index 948a2c6cb..7dbecd367 100644
> --- a/arch/arm64/boot/dts/exynos/Makefile
> +++ b/arch/arm64/boot/dts/exynos/Makefile
> @@ -10,5 +10,6 @@ dtb-$(CONFIG_ARCH_EXYNOS) += \
> exynos8895-dreamlte.dtb \
> exynos990-c1s.dtb \
> exynos990-r8s.dtb \
> + exynos990-x1s.dtb \
I noticed that there's spaces instead of tabs to indent the '\' after
exynos990-r8s.dtb, unlike the rest of the entries. This patch also
uses spaces to indent the whole entry. If maintainers are fine with
it, then it's alright, but if there's a v6 you could change it to use
tabs for consistancy's sake.
Rest looks fine to me.
Best regards, Ivo.
> exynosautov9-sadk.dtb \
> exynosautov920-sadk.dtb
> diff --git a/arch/arm64/boot/dts/exynos/exynos990-x1s.dts b/arch/arm64/boot/dts/exynos/exynos990-x1s.dts
> new file mode 100644
> index 000000000..1ae881015
> --- /dev/null
> +++ b/arch/arm64/boot/dts/exynos/exynos990-x1s.dts
> @@ -0,0 +1,28 @@
> +// SPDX-License-Identifier: GPL-2.0 OR BSD-3-Clause
> +/*
> + * Samsung Galaxy S20 5G (x1s/SM-G981B) device tree source
> + *
> + * Copyright (c) 2024, Umer Uddin <umer.uddin@...tallysanemainliners.org>
> + */
> +
> +/dts-v1/;
> +#include "exynos990-x1s-common.dtsi"
> +
> +/ {
> + #address-cells = <2>;
> + #size-cells = <2>;
> +
> + model = "Samsung Galaxy S20 5G";
> + compatible = "samsung,x1s", "samsung,exynos990";
> +
> + memory@...00000 {
> + device_type = "memory";
> + reg = <0x0 0x80000000 0x0 0x3ab00000>,
> + /* Memory hole */
> + <0x0 0xc1200000 0x0 0x1ee00000>,
> + /* Memory hole */
> + <0x0 0xe1900000 0x0 0x1e700000>,
> + /* Memory hole */
> + <0x8 0x80000000 0x2 0x7e800000>;
> + };
> +};
Powered by blists - more mailing lists