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 D36E2C25B78 for ; Wed, 29 May 2024 00:10:54 +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-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:Message-ID:References:In-Reply-To:Subject:Cc:To:From :Date:MIME-Version:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=cBCJ94i7MTz4WNogGmskBgKPG3eFTH2yEK7SG9nFcyU=; b=a9XJW5CK1kzDa9SCKTPcaIFtT0 aa4VkadgsIfSJHDHYkln2f2PZQPSgFw+toddAZnjKNddKUKIOTBwPPeayAex/F0y8ZuX4QFzWhv85 Q7B5wsbIoRaQZ70kR68AcYJA8j+jbSJxfH9Jy2v4H5PmvFEfFzFaM1901w46GR3r9CH1gsmlGjfHL FRG7unJygN9Ou9rUr2+kkWM3Saj/5pa7DaCD7metYpe42gfhhcdzcHgEX+N3ExdYh4Qb9BNIoZ4RF Wj1yXUWoOslyf8+gw4FMQsFzGbuBjLtDNGtlC8oPvizQdXfzgPqfc5pyD1/tk/APbfTmRB0An8oOl jxXwUYSg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sC6ty-00000002Ofm-2uVL; Wed, 29 May 2024 00:10:46 +0000 Received: from mail.manjaro.org ([2a01:4f8:c0c:51f3::1]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sC6tu-00000002OfH-23Dt; Wed, 29 May 2024 00:10:44 +0000 MIME-Version: 1.0 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=manjaro.org; s=2021; t=1716941440; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=7z6F1Ao1L/n/qQAEst7AHuaT7MITPT2BokqzLZjAtkw=; b=oviVzV2CR0rzR2GZkxMTnKGG3dxWHNwxm0rcUxfymyh2sXXP+ybudxEfHhyAPcSU8fheO/ /gpNHludd2JY83m/g3BWiVhGsreWWUhgGC28vJiyQ/ntFDx6N/pkJDTF3vYLsZht87gmLz W+oWD1aJuE45p2MJL1GgT5klAmcsdkdAVD3b6rSECtBRvxf5MQahneEFmkdFLysX8YSBf4 0MrVLwy7bI6fQ7tic+/sROdvcdqlFlR7fCwG+HvX+jOEZc7DDYbInXr2NcyOrio9+AePsu xgLu0l4h5FTxgkZ7lYfkYaKRBdnGhDlveSJ3jkwRysOuFafFn6b/S23TJ+Igdw== Date: Wed, 29 May 2024 02:10:38 +0200 From: Dragan Simic To: Sebastian Kropatsch Cc: Jonas Karlman , Heiko Stuebner , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Sebastian Reichel , linux-rockchip@lists.infradead.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 2/2] arm64: dts: rockchip: Add CM3588 NAS board In-Reply-To: <29e5cf31-3d9a-469a-befa-41a5aa2fe8b3@web.de> References: <20240526214340.8459-1-seb-dev@web.de> <20240526214820.9381-1-seb-dev@web.de> <9f40c748-691b-4a03-bbd6-54870f46bf05@kwiboo.se> <29e5cf31-3d9a-469a-befa-41a5aa2fe8b3@web.de> Message-ID: X-Sender: dsimic@manjaro.org Authentication-Results: ORIGINATING; auth=pass smtp.auth=dsimic@manjaro.org smtp.mailfrom=dsimic@manjaro.org X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240528_171042_870635_8BC4F9BA X-CRM114-Status: GOOD ( 18.96 ) 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-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hello Sebastian, On 2024-05-28 19:22, Sebastian Kropatsch wrote: > Am 27.05.2024 um 21:02 schrieb Jonas Karlman: >> On 2024-05-26 23:48, Sebastian Kropatsch wrote: >>> The CM3588 NAS by FriendlyElec pairs the CM3588 compute module, based >>> on >>> the Rockchip RK3588 SoC, with the CM3588 NAS Kit carrier board. >>> >>> [...] >>> >>> PCIe bifurcation is used to handle all four M.2 sockets at PCIe 3.0 >>> x1 >>> speed. Data lane mapping in the DT is done like described in commit >>> f8020dfb311d ("phy: rockchip-snps-pcie3: fix bifurcation on rk3588"). >>> >>> This device tree includes support for eMMC, SD card, ethernet, all >>> USB2 >>> and USB3 ports, all four M.2 slots, GPU, RTC, buzzer, UART debugging >>> as >>> well as the buttons and LEDs. >>> The GPIOs are labeled according to the schematics. >>> >>> Signed-off-by: Sebastian Kropatsch >>> --- >>> arch/arm64/boot/dts/rockchip/Makefile | 1 + >>> .../boot/dts/rockchip/rk3588-cm3588-nas.dts | 1269 >>> +++++++++++++++++ >>> 2 files changed, 1270 insertions(+) >>> create mode 100644 >>> arch/arm64/boot/dts/rockchip/rk3588-cm3588-nas.dts >> >> Because the CM3588 is a SoM and the NAS is a carrier board this should >> probably be split in two, cm3588.dtsi and cm3588-nas.dts. > > I thought about this before submitting. My reason for not splitting > this > into two files for now was that as far as I know this board is the only > combination for the CM, maybe no other daughter board will ever get > released. If another carrier board compatible with the CM3588 is > released, the splitting could be done at that point in time. > > But since both you and Heiko prefer to have it split, I will figure out > a way how and which parts will have to split up to the CM so we can > have two files in the end. I guess most things will go into the NAS dts > anyway. > > I'll have a look how other Rockchip compute modules with split device > trees were done in the past and orient myself by that. I also support the DT split between the SoM and the carrier board, even if there are currently no more carrier boards available for the particular SoM. That may seem redundant, but it reflects the nature of the hardware setup, in which the SoM plugs into the carrier board. This follows the principle of the DT describing hardware. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel