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 phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id E34A7C74A5B for ; Wed, 29 Mar 2023 09:42:07 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id B317F85AAA; Wed, 29 Mar 2023 11:42:04 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=lst.de Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Received: by phobos.denx.de (Postfix, from userid 109) id B17A785AAA; Wed, 29 Mar 2023 11:42:01 +0200 (CEST) Received: from verein.lst.de (verein.lst.de [213.95.11.211]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id 29FBA8591B for ; Wed, 29 Mar 2023 11:41:59 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=none (p=none dis=none) header.from=lst.de Authentication-Results: phobos.denx.de; spf=none smtp.mailfrom=duwe@lst.de Received: by verein.lst.de (Postfix, from userid 107) id 6AEF568C4E; Wed, 29 Mar 2023 11:41:57 +0200 (CEST) Received: from blackhole.lan (p5b2ece7e.dip0.t-ipconnect.de [91.46.206.126]) by verein.lst.de (Postfix) with ESMTPSA id 8D17E6732D; Wed, 29 Mar 2023 11:41:43 +0200 (CEST) Date: Wed, 29 Mar 2023 11:41:38 +0200 From: Torsten Duwe To: Yanhong Wang Cc: , Rick Chen , Leo , Lukasz Majewski , Sean Anderson , Lee Kuan Lim , Jianlong Huang , Emil Renner Berthing Subject: Re: [PATCH v5 00/17] Basic StarFive JH7110 RISC-V SoC support Message-ID: <20230329114138.3458974f@blackhole.lan> In-Reply-To: <20230329034224.26545-1-yanhong.wang@starfivetech.com> References: <20230329034224.26545-1-yanhong.wang@starfivetech.com> Organization: LST e.V. X-Mailer: Claws Mail 4.0.0 (GTK+ 3.24.34; x86_64-suse-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.39 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.103.8 at phobos.denx.de X-Virus-Status: Clean On Wed, 29 Mar 2023 11:42:07 +0800 Yanhong Wang wrote: > v5: [...] > - Splitted starfive_visionfive2_defconfig into starfive_visionfive2_12a_defconfig > and starfive_visionfive2_13b_defconfig. Is this really necessary? It puts another burden on people building U-Boot, distribution networks, and last but not least users, who will need to pick the correct binary blob, after trying to find out which board they actually have. Even past versions can detect the installed RAM correctly and will modify the DT accordingly, I assume? Why not make an inquiry on GMAC1_MDIO to tell whether it's a YT8512C (->v1.2A) or another YT8531C (->v1.3B), in the ethernet patch set, and likewise update the device tree dynamically then? Torsten