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 2F4D3C74A5B for ; Wed, 29 Mar 2023 11:46:49 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 8006085CF4; Wed, 29 Mar 2023 13:46:44 +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 B0C4C85C95; Wed, 29 Mar 2023 13:46:40 +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 EC44F8580C for ; Wed, 29 Mar 2023 13:46:36 +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 22AE968D06; Wed, 29 Mar 2023 13:46:35 +0200 (CEST) Received: from blackhole.lan (p5b2ece7e.dip0.t-ipconnect.de [91.46.206.126]) by verein.lst.de (Postfix) with ESMTPSA id 083926732A; Wed, 29 Mar 2023 13:46:24 +0200 (CEST) Date: Wed, 29 Mar 2023 13:46:25 +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: <20230329134625.44be3d18@blackhole.lan> In-Reply-To: References: <20230329034224.26545-1-yanhong.wang@starfivetech.com> <20230329114138.3458974f@blackhole.lan> 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 18:16:20 +0800 yanhong wang wrote: > > > On 2023/3/29 17:41, Torsten Duwe wrote: > > 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? > > > > There is only one defconfig in V4, and dts is separate for versions 1.2a and 1.3b. > Andreas Schwab suggested that defconfig is also defined separately, so the definition > of defconfig in V5 is also separated. > > The discussion process as follows: > > https://patchwork.ozlabs.org/project/uboot/patch/20230316025332.3297-18-yanhong.wang@starfivetech.com/ Very well. I'll talk to Andreas. Sorry for the confusion. > Do you have any better suggestion on whether defconfig is defined separately? See above. Roughly, define a superset, like a node YT8512C attached to GMAC1, and another node with a YT8531C attached to GMAC1, both "disabled", and at runtime mark the detected one "okay", maybe patching the board name or not. But I'll discuss this with Andreas first. Torsten