Message-ID: <3669d2de-5324-4f48-a55c-942ce61d19bb@kwiboo.se>
Date: Tue, 22 Oct 2024 20:43:53 +0200
From: Jonas Karlman <jonas@...boo.se>
To: Tianling Shen <cnsztl@...il.com>
Cc: Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
 Conor Dooley <conor+dt@...nel.org>, Heiko Stuebner <heiko@...ech.de>,
 Chris Morgan <macromorgan@...mail.com>, Andy Yan <andyshrk@....com>,
 Jagan Teki <jagan@...eble.ai>, devicetree@...r.kernel.org,
 linux-arm-kernel@...ts.infradead.org, linux-rockchip@...ts.infradead.org,
 linux-kernel@...r.kernel.org
Subject: Re: [PATCH 2/2] arm64: dts: rockchip: Add FriendlyARM NanoPi R3S
 board

Hi Tianling,

On 2024-10-22 20:18, Tianling Shen wrote:
> Hi Jonas,
> 
> On Wed, Oct 23, 2024 at 1:56 AM Jonas Karlman <jonas@...boo.se> wrote:
>>
>> Hi Tianling,
>>
>> On 2024-10-22 19:38, Tianling Shen wrote:
>>> Hi Jonas,
>>>
>>> On Wed, Oct 23, 2024 at 12:49 AM Jonas Karlman <jonas@...boo.se> wrote:
>>>>
>>>> Hi Tianling,
>>>>
>>>> On 2024-10-20 19:39, Tianling Shen wrote:
>>>>> The NanoPi R3S(as "R3S") is an open source platform with dual-Gbps
>>>>> Ethernet ports designed and developed by FriendlyElec for IoT
>>>>> applications.
>>>>>
>>>>> Specification:
>>>>> - Rockchip RK3566
>>>>> - 2GB LPDDR4X RAM
>>>>> - optional 32GB eMMC module
>>>>> - SD card slot
>>>>> - 2x 1000 Base-T
>>>>> - 3x LEDs (POWER, LAN, WAN)
>>>>> - 2x Buttons (Reset, MaskROM)
>>>>> - 1x USB 3.0 Port
>>>>> - Type-C 5V 2A Power
>>>>>
>>>>> Signed-off-by: Tianling Shen <cnsztl@...il.com>
>>>>> ---
>>>>>  arch/arm64/boot/dts/rockchip/Makefile   Â  Â  Â  |   1 +
>>>>>  .../boot/dts/rockchip/rk3566-nanopi-r3s.dts   | 554 ++++++++++++++++++
>>>>> Â 2 files changed, 555 insertions(+)
>>>>> Â create mode 100644 arch/arm64/boot/dts/rockchip/rk3566-nanopi-r3s.dts
>>>>>
>>>>> diff --git a/arch/arm64/boot/dts/rockchip/Makefile b/arch/arm64/boot/dts/rockchip/Makefile
>>>>> index 8e1025cb5d20..e641033d23d1 100644
>>>>> --- a/arch/arm64/boot/dts/rockchip/Makefile
>>>>> +++ b/arch/arm64/boot/dts/rockchip/Makefile
>>>>> @@ -109,6 +109,7 @@ dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-soquartz-model-a.dtb
>>>>> Â dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-box-demo.dtb
>>>>> Â dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-lckfb-tspi.dtb
>>>>> Â dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-lubancat-1.dtb
>>>>> +dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3566-nanopi-r3s.dtb
>>>>> Â dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3568-bpi-r2-pro.dtb
>>>>> Â dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3568-evb1-v10.dtb
>>>>> Â dtb-$(CONFIG_ARCH_ROCKCHIP) += rk3568-fastrhino-r66s.dtb
>>>>> diff --git a/arch/arm64/boot/dts/rockchip/rk3566-nanopi-r3s.dts b/arch/arm64/boot/dts/rockchip/rk3566-nanopi-r3s.dts
>>>>> new file mode 100644
>>>>> index 000000000000..951dd9bcb4b0
>>>>> --- /dev/null
>>>>> +++ b/arch/arm64/boot/dts/rockchip/rk3566-nanopi-r3s.dts
>>>>> @@ -0,0 +1,554 @@
>>>>> +// SPDX-License-Identifier: GPL-2.0-or-later OR MIT
>>>>> +/*
>>>>> + * Copyright (c) 2020 Rockchip Electronics Co., Ltd.
>>>>> + *
>>>>> + * Copyright (c) 2024 FriendlyElec Computer Tech. Co., Ltd.
>>>>> + * (http://www.friendlyarm.com/
>>
>> This could probably be updated to use https://www.friendlyelec.com/
> 
> Yes!
> 
>>
>>>>> + *
>>>>> + * Copyright (c) 2024 Tianling Shen <cnsztl@...il.com>
>>>>> + */
>>>>> +
>>>>> +/dts-v1/;
>>>>> +#include <dt-bindings/gpio/gpio.h>
>>>>> +#include <dt-bindings/input/input.h>
>>>>> +#include <dt-bindings/leds/common.h>
>>>>> +#include <dt-bindings/pinctrl/rockchip.h>
>>>>> +#include <dt-bindings/soc/rockchip,vop2.h>
>>>>> +#include "rk3566.dtsi"
>>>>> +
>>>>> +/ {
>>>>> + Â  Â  model = "FriendlyARM NanoPi R3S";
>>>>
>>>> Please use the marketing name, and the name added in binding:
>>>>
>>>> Â  Â FriendlyElec NanoPi R3S
>>>
>>> Sorry for the copy & paste error, will fix it in new patch.
>>
>> The commit subject could probably also be updated :-)
>>
>>>
>>>>
>>>>> + Â  Â  compatible = "friendlyarm,nanopi-r3s", "rockchip,rk3566";
>>>>> +
>>>>> + Â  Â  aliases {
>>>>> + Â  Â  Â  Â  Â  Â  ethernet0 = &gmac1;
>>>>> + Â  Â  Â  Â  Â  Â  mmc0 = &sdmmc0;
>>>>> + Â  Â  Â  Â  Â  Â  mmc1 = &sdhci;
>>>>
>>>> This board comes with on-board eMMC this please use mmc0 as alias for
>>>> &sdhci; and mmc1 for sd-card.
>>>
>>> Is using mmc0 for sdhci a common practice for eMMC boards?
>>> The eMMC module on R3S is optional and users may get boards without it.
>>
>> Typically any non-removable storage (emmc) is listed before removable
>> storage (sd-card) options. Also U-Boot will try to override and use
>> mmc0=sdhci and mmc1=sdmmc0 for all rk356x boards, so for EFI booing
>> using control FDT the alias may be different.
>>
> 
> Thank you for the details! May I have your Suggested-by tag for all
> of these fixes?

Sure,

Suggested-by: Jonas Karlman <jonas@...boo.se>

Regards,
Jonas

> 
> Thanks,
> Tianling.
> 
>> Regards,
>> Jonas
>>
>> [snip]

Powered by blists - more mailing lists