Message ID | 20231201121410.95298-3-jeeheng.sia@starfivetech.com |
---|---|
State | New |
Headers |
Return-Path: <linux-kernel-owner@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:bcd1:0:b0:403:3b70:6f57 with SMTP id r17csp1072786vqy; Fri, 1 Dec 2023 04:15:06 -0800 (PST) X-Google-Smtp-Source: AGHT+IGMMwojg3PwxGgOyr3Zn8iwwGVY6HLpc6mu+pMKJADOGFk/jTDEvUAYxlNzzYHHInISPLAG X-Received: by 2002:a05:6358:c31f:b0:16e:27d6:b9cb with SMTP id fk31-20020a056358c31f00b0016e27d6b9cbmr21166857rwb.10.1701432906573; Fri, 01 Dec 2023 04:15:06 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1701432906; cv=none; d=google.com; s=arc-20160816; b=kHeU3UiviI0W+NRvDmwcyYH9u7VkyTkb+iJc4VGuNJdF2OA5K97bM3n+ZTcS6p+LJ/ ZkEq3oXsxuFUSS/QY0PSt5WcmWJXkjGwTks7aUxXVWvDR5/qv2WKw8K7uDRjFCtGBsaJ s94o+R76+L28VZKo1mMSi1QB2193UpNGJ4ZKhE1I5Kpe6tbSDRw9x8e+8ANDsdnn5a0e 8ZmAFg0Bd+Ld2H3uM61P/XGKBfEdvKLBkOMhXxyzBI9ukaNZvcfPsyLYl4eiuwjE6Fq2 +5uSYXT28UxC2zPanFIiVIVji7NeF9ligd6fY6s0wlT6FroZN9JTo+SSzXgeUcCYNx+k +JEA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from; bh=iVpNFGmewzm37+T5gEicU4XhgzgpyKEUQqDbbHbDkbc=; fh=sN4+/BjIy2DV9ynVxPCz6esahHdQ/8sI7w0nO1fhIxw=; b=O0Z5CwrEOwzfRB9deYGWbS0zwlxW21q9T/IY5F0y5F46CZQeDr8seZIUdPWJ1EfuYA T7uDZ0SAn7bQ9PvdArW835XIk8vX4Z+h7pwIEQQQ2T3hnfebmfkC1/rGRGjE/9A0o6Bf bn7cZEq56NJcQabZzMfx7keEieT12h0MpwnxWQvBY5z5olZNyAEXrnb4Ao0SMQijPEW0 t5aIX/uotuPiPHgk31rFuV9WgJyyixGxwAVJVZ+umWlPn2yJ9w8qZODlKzviUhsL7Nbs Oz0VoBDEMMTgKDi96msxmUC/Anhm6NnxWG9b/Ap7TGSidAA273ZApOA8+wGaiDADYi6h 3kxQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.32 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from agentk.vger.email (agentk.vger.email. [23.128.96.32]) by mx.google.com with ESMTPS id e18-20020a639a52000000b00573f94e8b83si3123101pgo.265.2023.12.01.04.15.05 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 01 Dec 2023 04:15:06 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.32 as permitted sender) client-ip=23.128.96.32; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.32 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by agentk.vger.email (Postfix) with ESMTP id ED1C080FCE3B; Fri, 1 Dec 2023 04:15:02 -0800 (PST) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.11 at agentk.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1378775AbjLAMOx convert rfc822-to-8bit (ORCPT <rfc822;ruipengqi7@gmail.com> + 99 others); Fri, 1 Dec 2023 07:14:53 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:36480 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1378770AbjLAMOv (ORCPT <rfc822;linux-kernel@vger.kernel.org>); Fri, 1 Dec 2023 07:14:51 -0500 Received: from fd01.gateway.ufhost.com (fd01.gateway.ufhost.com [61.152.239.71]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 0B3501724; Fri, 1 Dec 2023 04:14:57 -0800 (PST) Received: from EXMBX165.cuchost.com (unknown [175.102.18.54]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client CN "EXMBX165", Issuer "EXMBX165" (not verified)) by fd01.gateway.ufhost.com (Postfix) with ESMTP id 8A9318178; Fri, 1 Dec 2023 20:14:55 +0800 (CST) Received: from EXMBX066.cuchost.com (172.16.7.66) by EXMBX165.cuchost.com (172.16.6.75) with Microsoft SMTP Server (TLS) id 15.0.1497.42; Fri, 1 Dec 2023 20:14:55 +0800 Received: from jsia-virtual-machine.localdomain (60.54.3.230) by EXMBX066.cuchost.com (172.16.6.66) with Microsoft SMTP Server (TLS) id 15.0.1497.42; Fri, 1 Dec 2023 20:14:43 +0800 From: Sia Jee Heng <jeeheng.sia@starfivetech.com> To: <kernel@esmil.dk>, <robh+dt@kernel.org>, <krzysztof.kozlowski+dt@linaro.org>, <krzk@kernel.org>, <conor+dt@kernel.org>, <paul.walmsley@sifive.com>, <palmer@dabbelt.com>, <aou@eecs.berkeley.edu>, <daniel.lezcano@linaro.org>, <tglx@linutronix.de>, <conor@kernel.org>, <anup@brainfault.org>, <gregkh@linuxfoundation.org>, <jirislaby@kernel.org>, <michal.simek@amd.com>, <michael.zhu@starfivetech.com>, <drew@beagleboard.org> CC: <devicetree@vger.kernel.org>, <linux-riscv@lists.infradead.org>, <linux-kernel@vger.kernel.org>, <jeeheng.sia@starfivetech.com>, <leyfoon.tan@starfivetech.com>, Conor Dooley <conor.dooley@microchip.com> Subject: [PATCH v3 2/6] dt-bindings: riscv: Add StarFive JH8100 SoC Date: Fri, 1 Dec 2023 20:14:06 +0800 Message-ID: <20231201121410.95298-3-jeeheng.sia@starfivetech.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20231201121410.95298-1-jeeheng.sia@starfivetech.com> References: <20231201121410.95298-1-jeeheng.sia@starfivetech.com> MIME-Version: 1.0 Content-Type: text/plain X-Originating-IP: [60.54.3.230] X-ClientProxiedBy: EXCAS062.cuchost.com (172.16.6.22) To EXMBX066.cuchost.com (172.16.6.66) X-YovoleRuleAgent: yovoleflag Content-Transfer-Encoding: 8BIT X-Spam-Status: No, score=-0.8 required=5.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on agentk.vger.email Precedence: bulk List-ID: <linux-kernel.vger.kernel.org> X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (agentk.vger.email [0.0.0.0]); Fri, 01 Dec 2023 04:15:03 -0800 (PST) X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1784081711359664546 X-GMAIL-MSGID: 1784081711359664546 |
Series |
Initial device tree support for StarFive JH8100 SoC
|
|
Commit Message
JeeHeng Sia
Dec. 1, 2023, 12:14 p.m. UTC
Add device tree bindings for the StarFive JH8100 RISC-V SoC. Signed-off-by: Sia Jee Heng <jeeheng.sia@starfivetech.com> Reviewed-by: Ley Foon Tan <leyfoon.tan@starfivetech.com> Acked-by: Conor Dooley <conor.dooley@microchip.com> --- Documentation/devicetree/bindings/riscv/starfive.yaml | 4 ++++ 1 file changed, 4 insertions(+)
Comments
On Fri, Dec 01, 2023 at 08:14:06PM +0800, Sia Jee Heng wrote: > Add device tree bindings for the StarFive JH8100 RISC-V SoC. > > Signed-off-by: Sia Jee Heng <jeeheng.sia@starfivetech.com> > Reviewed-by: Ley Foon Tan <leyfoon.tan@starfivetech.com> > Acked-by: Conor Dooley <conor.dooley@microchip.com> > --- > Documentation/devicetree/bindings/riscv/starfive.yaml | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/Documentation/devicetree/bindings/riscv/starfive.yaml b/Documentation/devicetree/bindings/riscv/starfive.yaml > index cc4d92f0a1bf..12d7844232b8 100644 > --- a/Documentation/devicetree/bindings/riscv/starfive.yaml > +++ b/Documentation/devicetree/bindings/riscv/starfive.yaml > @@ -30,6 +30,10 @@ properties: > - starfive,visionfive-2-v1.3b > - const: starfive,jh7110 > > + - items: > + - enum: > + - starfive,jh8100-evb Hmm, reading some of the other threads it appears that the evaluation platform that you guys have is actually just an FPGA? Could you please provide more information as to what this "evb" actually is? If it is just an FPGA-based evaluation platform I don't think that we want to merge patches for the platform. I'm fine with patches adding peripheral support, but the soc/board dts files and things like pinctrl or clock drivers I am not keen on. Perhaps Emil also has an opinion on this. Thanks, Conor. > + - const: starfive,jh8100 > additionalProperties: true > > ... > -- > 2.34.1 > >
> -----Original Message----- > From: Conor Dooley <conor@kernel.org> > Sent: Wednesday, December 13, 2023 8:43 PM > To: JeeHeng Sia <jeeheng.sia@starfivetech.com> > Cc: kernel@esmil.dk; robh+dt@kernel.org; krzysztof.kozlowski+dt@linaro.org; > krzk@kernel.org; conor+dt@kernel.org; paul.walmsley@sifive.com; > palmer@dabbelt.com; aou@eecs.berkeley.edu; daniel.lezcano@linaro.org; > tglx@linutronix.de; anup@brainfault.org; gregkh@linuxfoundation.org; > jirislaby@kernel.org; michal.simek@amd.com; Michael Zhu > <michael.zhu@starfivetech.com>; drew@beagleboard.org; > devicetree@vger.kernel.org; linux-riscv@lists.infradead.org; linux- > kernel@vger.kernel.org; Leyfoon Tan <leyfoon.tan@starfivetech.com>; Conor > Dooley <conor.dooley@microchip.com> > Subject: Re: [PATCH v3 2/6] dt-bindings: riscv: Add StarFive JH8100 SoC > > On Fri, Dec 01, 2023 at 08:14:06PM +0800, Sia Jee Heng wrote: > > Add device tree bindings for the StarFive JH8100 RISC-V SoC. > > > > Signed-off-by: Sia Jee Heng <jeeheng.sia@starfivetech.com> > > Reviewed-by: Ley Foon Tan <leyfoon.tan@starfivetech.com> > > Acked-by: Conor Dooley <conor.dooley@microchip.com> > > --- > > Documentation/devicetree/bindings/riscv/starfive.yaml | 4 ++++ > > 1 file changed, 4 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/riscv/starfive.yaml > > b/Documentation/devicetree/bindings/riscv/starfive.yaml > > index cc4d92f0a1bf..12d7844232b8 100644 > > --- a/Documentation/devicetree/bindings/riscv/starfive.yaml > > +++ b/Documentation/devicetree/bindings/riscv/starfive.yaml > > @@ -30,6 +30,10 @@ properties: > > - starfive,visionfive-2-v1.3b > > - const: starfive,jh7110 > > > > + - items: > > + - enum: > > + - starfive,jh8100-evb > > Hmm, reading some of the other threads it appears that the evaluation > platform that you guys have is actually just an FPGA? Could you please provide > more information as to what this "evb" actually is? > > If it is just an FPGA-based evaluation platform I don't think that we want to > merge patches for the platform. I'm fine with patches adding peripheral > support, but the soc/board dts files and things like pinctrl or clock drivers I am > not keen on. > Perhaps Emil also has an opinion on this. > > Thanks, > Conor. We have been testing on the FPGA/emulator for pre-silicon validation. It will have real silicon SoC next year. Regards Ley Foon
> -----Original Message----- > From: Conor Dooley <conor@kernel.org> > Sent: Wednesday, December 13, 2023 8:43 PM > To: JeeHeng Sia <jeeheng.sia@starfivetech.com> > Cc: kernel@esmil.dk; robh+dt@kernel.org; krzysztof.kozlowski+dt@linaro.org; krzk@kernel.org; conor+dt@kernel.org; > paul.walmsley@sifive.com; palmer@dabbelt.com; aou@eecs.berkeley.edu; daniel.lezcano@linaro.org; tglx@linutronix.de; > anup@brainfault.org; gregkh@linuxfoundation.org; jirislaby@kernel.org; michal.simek@amd.com; Michael Zhu > <michael.zhu@starfivetech.com>; drew@beagleboard.org; devicetree@vger.kernel.org; linux-riscv@lists.infradead.org; linux- > kernel@vger.kernel.org; Leyfoon Tan <leyfoon.tan@starfivetech.com>; Conor Dooley <conor.dooley@microchip.com> > Subject: Re: [PATCH v3 2/6] dt-bindings: riscv: Add StarFive JH8100 SoC > > On Fri, Dec 01, 2023 at 08:14:06PM +0800, Sia Jee Heng wrote: > > Add device tree bindings for the StarFive JH8100 RISC-V SoC. > > > > Signed-off-by: Sia Jee Heng <jeeheng.sia@starfivetech.com> > > Reviewed-by: Ley Foon Tan <leyfoon.tan@starfivetech.com> > > Acked-by: Conor Dooley <conor.dooley@microchip.com> > > --- > > Documentation/devicetree/bindings/riscv/starfive.yaml | 4 ++++ > > 1 file changed, 4 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/riscv/starfive.yaml b/Documentation/devicetree/bindings/riscv/starfive.yaml > > index cc4d92f0a1bf..12d7844232b8 100644 > > --- a/Documentation/devicetree/bindings/riscv/starfive.yaml > > +++ b/Documentation/devicetree/bindings/riscv/starfive.yaml > > @@ -30,6 +30,10 @@ properties: > > - starfive,visionfive-2-v1.3b > > - const: starfive,jh7110 > > > > + - items: > > + - enum: > > + - starfive,jh8100-evb > > Hmm, reading some of the other threads it appears that the evaluation > platform that you guys have is actually just an FPGA? Could you please > provide more information as to what this "evb" actually is? > > If it is just an FPGA-based evaluation platform I don't think that we > want to merge patches for the platform. I'm fine with patches adding > peripheral support, but the soc/board dts files and things like pinctrl > or clock drivers I am not keen on. > Perhaps Emil also has an opinion on this. Eco the same reply here. I am not sure what you mean. We verified on FPGA & Emulator, and the logic is pretty much close to the real silicon. I did mention that in the cover letter as well. I am new to Linux, so I am wondering if there is a Linux upstream guideline mentioning that pre-silicon software is not allowed to upstream? Hope there is an updated Linux upstream guideline that benefit other vendors. > > Thanks, > Conor. > > > + - const: starfive,jh8100 > > additionalProperties: true > > > > ... > > -- > > 2.34.1 > > > >
On Thu, Dec 14, 2023 at 12:36:57AM +0000, JeeHeng Sia wrote: > > > > -----Original Message----- > > From: Conor Dooley <conor@kernel.org> > > Sent: Wednesday, December 13, 2023 8:43 PM > > To: JeeHeng Sia <jeeheng.sia@starfivetech.com> > > Cc: kernel@esmil.dk; robh+dt@kernel.org; krzysztof.kozlowski+dt@linaro.org; krzk@kernel.org; conor+dt@kernel.org; > > paul.walmsley@sifive.com; palmer@dabbelt.com; aou@eecs.berkeley.edu; daniel.lezcano@linaro.org; tglx@linutronix.de; > > anup@brainfault.org; gregkh@linuxfoundation.org; jirislaby@kernel.org; michal.simek@amd.com; Michael Zhu > > <michael.zhu@starfivetech.com>; drew@beagleboard.org; devicetree@vger.kernel.org; linux-riscv@lists.infradead.org; linux- > > kernel@vger.kernel.org; Leyfoon Tan <leyfoon.tan@starfivetech.com>; Conor Dooley <conor.dooley@microchip.com> > > Subject: Re: [PATCH v3 2/6] dt-bindings: riscv: Add StarFive JH8100 SoC > > > > On Fri, Dec 01, 2023 at 08:14:06PM +0800, Sia Jee Heng wrote: > > > Add device tree bindings for the StarFive JH8100 RISC-V SoC. > > > > > > Signed-off-by: Sia Jee Heng <jeeheng.sia@starfivetech.com> > > > Reviewed-by: Ley Foon Tan <leyfoon.tan@starfivetech.com> > > > Acked-by: Conor Dooley <conor.dooley@microchip.com> > > > --- > > > Documentation/devicetree/bindings/riscv/starfive.yaml | 4 ++++ > > > 1 file changed, 4 insertions(+) > > > > > > diff --git a/Documentation/devicetree/bindings/riscv/starfive.yaml b/Documentation/devicetree/bindings/riscv/starfive.yaml > > > index cc4d92f0a1bf..12d7844232b8 100644 > > > --- a/Documentation/devicetree/bindings/riscv/starfive.yaml > > > +++ b/Documentation/devicetree/bindings/riscv/starfive.yaml > > > @@ -30,6 +30,10 @@ properties: > > > - starfive,visionfive-2-v1.3b > > > - const: starfive,jh7110 > > > > > > + - items: > > > + - enum: > > > + - starfive,jh8100-evb > > > > Hmm, reading some of the other threads it appears that the evaluation > > platform that you guys have is actually just an FPGA? Could you please > > provide more information as to what this "evb" actually is? > > > > If it is just an FPGA-based evaluation platform I don't think that we > > want to merge patches for the platform. I'm fine with patches adding > > peripheral support, but the soc/board dts files and things like pinctrl > > or clock drivers I am not keen on. > > Perhaps Emil also has an opinion on this. > Eco the same reply here. I am not sure what you mean. We verified on FPGA & Emulator, > and the logic is pretty much close to the real silicon. "Pretty much close" That doesn't give me confidence. The compatible should uniquely identify an SoC, but if it is used for both the actual SoC and for something "pretty much close" to the actual SoC then that does not hold. > I did mention that in the cover letter as well. Ah apologies for missing that. I try to read cover letters but the volume of mail gets to me at times. > I am new to Linux, so I am wondering if there is a Linux upstream guideline mentioning > that pre-silicon software is not allowed to upstream? I wouldn't say that this is the case, but things like clock and pinctrl drivers are the sort of things that are likely to vary in your "pretty much close" as that is the kind of thing that change for your final integration, versus a more "standalone" peripheral. For dts stuff, in RISC-V at least, we've been operating so far on the basis that systems implemented entirely on an FPGA are not suitable for inclusion in mainline. I would say that this can probably be relaxed to allow systems where there are publicly available, versioned, designs or bitstreams that are widely used that these devicetrees correspond to. This would suit something like if AMD published a bitstream using one of their new MicroblazeV cpu cores as a sort of "reference design". > Hope there is an updated Linux > upstream guideline that benefit other vendors. I have no idea if there is one or not. I think it generally varies on individual maintainers etc, and for something like a dts it comes down to the platform maintainer (Emil) I suppose. Sending stuff out before your SoC has been produced is really great though, so it is a fine line to avoid discouraging something we really like to see. Cheers, Conor.
On Thu, 14 Dec 2023 08:22:29 PST (-0800), Conor Dooley wrote: > On Thu, Dec 14, 2023 at 12:36:57AM +0000, JeeHeng Sia wrote: >> >> >> > -----Original Message----- >> > From: Conor Dooley <conor@kernel.org> >> > Sent: Wednesday, December 13, 2023 8:43 PM >> > To: JeeHeng Sia <jeeheng.sia@starfivetech.com> >> > Cc: kernel@esmil.dk; robh+dt@kernel.org; krzysztof.kozlowski+dt@linaro.org; krzk@kernel.org; conor+dt@kernel.org; >> > paul.walmsley@sifive.com; palmer@dabbelt.com; aou@eecs.berkeley.edu; daniel.lezcano@linaro.org; tglx@linutronix.de; >> > anup@brainfault.org; gregkh@linuxfoundation.org; jirislaby@kernel.org; michal.simek@amd.com; Michael Zhu >> > <michael.zhu@starfivetech.com>; drew@beagleboard.org; devicetree@vger.kernel.org; linux-riscv@lists.infradead.org; linux- >> > kernel@vger.kernel.org; Leyfoon Tan <leyfoon.tan@starfivetech.com>; Conor Dooley <conor.dooley@microchip.com> >> > Subject: Re: [PATCH v3 2/6] dt-bindings: riscv: Add StarFive JH8100 SoC >> > >> > On Fri, Dec 01, 2023 at 08:14:06PM +0800, Sia Jee Heng wrote: >> > > Add device tree bindings for the StarFive JH8100 RISC-V SoC. >> > > >> > > Signed-off-by: Sia Jee Heng <jeeheng.sia@starfivetech.com> >> > > Reviewed-by: Ley Foon Tan <leyfoon.tan@starfivetech.com> >> > > Acked-by: Conor Dooley <conor.dooley@microchip.com> >> > > --- >> > > Documentation/devicetree/bindings/riscv/starfive.yaml | 4 ++++ >> > > 1 file changed, 4 insertions(+) >> > > >> > > diff --git a/Documentation/devicetree/bindings/riscv/starfive.yaml b/Documentation/devicetree/bindings/riscv/starfive.yaml >> > > index cc4d92f0a1bf..12d7844232b8 100644 >> > > --- a/Documentation/devicetree/bindings/riscv/starfive.yaml >> > > +++ b/Documentation/devicetree/bindings/riscv/starfive.yaml >> > > @@ -30,6 +30,10 @@ properties: >> > > - starfive,visionfive-2-v1.3b >> > > - const: starfive,jh7110 >> > > >> > > + - items: >> > > + - enum: >> > > + - starfive,jh8100-evb >> > >> > Hmm, reading some of the other threads it appears that the evaluation >> > platform that you guys have is actually just an FPGA? Could you please >> > provide more information as to what this "evb" actually is? >> > >> > If it is just an FPGA-based evaluation platform I don't think that we >> > want to merge patches for the platform. I'm fine with patches adding >> > peripheral support, but the soc/board dts files and things like pinctrl >> > or clock drivers I am not keen on. >> > Perhaps Emil also has an opinion on this. >> Eco the same reply here. I am not sure what you mean. We verified on FPGA & Emulator, >> and the logic is pretty much close to the real silicon. > > "Pretty much close" That doesn't give me confidence. The compatible > should uniquely identify an SoC, but if it is used for both the actual > SoC and for something "pretty much close" to the actual SoC then that > does not hold. Ya, trying to have some pre-silicon FPGA-based platform alias with the real chip is a repice for disaster. >> I did mention that in the cover letter as well. > > Ah apologies for missing that. I try to read cover letters but the > volume of mail gets to me at times. > >> I am new to Linux, so I am wondering if there is a Linux upstream guideline mentioning >> that pre-silicon software is not allowed to upstream? > > I wouldn't say that this is the case, but things like clock and pinctrl > drivers are the sort of things that are likely to vary in your "pretty > much close" as that is the kind of thing that change for your final > integration, versus a more "standalone" peripheral. Yep, and since integration issues in the ASIC blocks can end up manifesting as SW-visible behavior in nearby blocks it's hard to just pull out the peripherals -- we sort of try by getting the DT topology to match the SOC, but there's always some mismatches. > For dts stuff, in RISC-V at least, we've been operating so far on the > basis that systems implemented entirely on an FPGA are not suitable for > inclusion in mainline. I would say that this can probably be relaxed to > allow systems where there are publicly available, versioned, designs or > bitstreams that are widely used that these devicetrees correspond to. > This would suit something like if AMD published a bitstream using one > of their new MicroblazeV cpu cores as a sort of "reference design". FPGAs are definately in a grey area, but that's been my mindset as well. For me it's less about FPGA vs ASIC (or any other manufacturing technology in between) and more about whether something is being used publicly. Specifically: is the FPGA used for internal pre-silicon work or is it some publicly availiable system? The versioning stuff is also important, but we need that for ASICs as well since they can be re-spun. >> Hope there is an updated Linux >> upstream guideline that benefit other vendors. > > I have no idea if there is one or not. I think it generally varies on > individual maintainers etc, and for something like a dts it comes down > to the platform maintainer (Emil) I suppose. Sending stuff out before > your SoC has been produced is really great though, so it is a fine line > to avoid discouraging something we really like to see. IIRC we've got some stuff written for arch/riscv somewhere in Documentation, but the hardest part here is that each subsystem is going to have different policies so it's kind of hard to try and come up with a general rule. > Cheers, > Conor.
> -----Original Message----- > From: Palmer Dabbelt <palmer@dabbelt.com> > Sent: Friday, December 15, 2023 1:21 AM > To: Conor Dooley <conor@kernel.org> > Cc: JeeHeng Sia <jeeheng.sia@starfivetech.com>; kernel@esmil.dk; robh+dt@kernel.org; krzysztof.kozlowski+dt@linaro.org; > krzk@kernel.org; conor+dt@kernel.org; Paul Walmsley <paul.walmsley@sifive.com>; aou@eecs.berkeley.edu; > daniel.lezcano@linaro.org; tglx@linutronix.de; anup@brainfault.org; Greg KH <gregkh@linuxfoundation.org>; jirislaby@kernel.org; > michal.simek@amd.com; Michael Zhu <michael.zhu@starfivetech.com>; drew@beagleboard.org; devicetree@vger.kernel.org; linux- > riscv@lists.infradead.org; linux-kernel@vger.kernel.org; Leyfoon Tan <leyfoon.tan@starfivetech.com>; Conor Dooley > <conor.dooley@microchip.com> > Subject: Re: [PATCH v3 2/6] dt-bindings: riscv: Add StarFive JH8100 SoC > > On Thu, 14 Dec 2023 08:22:29 PST (-0800), Conor Dooley wrote: > > On Thu, Dec 14, 2023 at 12:36:57AM +0000, JeeHeng Sia wrote: > >> > >> > >> > -----Original Message----- > >> > From: Conor Dooley <conor@kernel.org> > >> > Sent: Wednesday, December 13, 2023 8:43 PM > >> > To: JeeHeng Sia <jeeheng.sia@starfivetech.com> > >> > Cc: kernel@esmil.dk; robh+dt@kernel.org; krzysztof.kozlowski+dt@linaro.org; krzk@kernel.org; conor+dt@kernel.org; > >> > paul.walmsley@sifive.com; palmer@dabbelt.com; aou@eecs.berkeley.edu; daniel.lezcano@linaro.org; tglx@linutronix.de; > >> > anup@brainfault.org; gregkh@linuxfoundation.org; jirislaby@kernel.org; michal.simek@amd.com; Michael Zhu > >> > <michael.zhu@starfivetech.com>; drew@beagleboard.org; devicetree@vger.kernel.org; linux-riscv@lists.infradead.org; linux- > >> > kernel@vger.kernel.org; Leyfoon Tan <leyfoon.tan@starfivetech.com>; Conor Dooley <conor.dooley@microchip.com> > >> > Subject: Re: [PATCH v3 2/6] dt-bindings: riscv: Add StarFive JH8100 SoC > >> > > >> > On Fri, Dec 01, 2023 at 08:14:06PM +0800, Sia Jee Heng wrote: > >> > > Add device tree bindings for the StarFive JH8100 RISC-V SoC. > >> > > > >> > > Signed-off-by: Sia Jee Heng <jeeheng.sia@starfivetech.com> > >> > > Reviewed-by: Ley Foon Tan <leyfoon.tan@starfivetech.com> > >> > > Acked-by: Conor Dooley <conor.dooley@microchip.com> > >> > > --- > >> > > Documentation/devicetree/bindings/riscv/starfive.yaml | 4 ++++ > >> > > 1 file changed, 4 insertions(+) > >> > > > >> > > diff --git a/Documentation/devicetree/bindings/riscv/starfive.yaml b/Documentation/devicetree/bindings/riscv/starfive.yaml > >> > > index cc4d92f0a1bf..12d7844232b8 100644 > >> > > --- a/Documentation/devicetree/bindings/riscv/starfive.yaml > >> > > +++ b/Documentation/devicetree/bindings/riscv/starfive.yaml > >> > > @@ -30,6 +30,10 @@ properties: > >> > > - starfive,visionfive-2-v1.3b > >> > > - const: starfive,jh7110 > >> > > > >> > > + - items: > >> > > + - enum: > >> > > + - starfive,jh8100-evb > >> > > >> > Hmm, reading some of the other threads it appears that the evaluation > >> > platform that you guys have is actually just an FPGA? Could you please > >> > provide more information as to what this "evb" actually is? > >> > > >> > If it is just an FPGA-based evaluation platform I don't think that we > >> > want to merge patches for the platform. I'm fine with patches adding > >> > peripheral support, but the soc/board dts files and things like pinctrl > >> > or clock drivers I am not keen on. > >> > Perhaps Emil also has an opinion on this. > >> Eco the same reply here. I am not sure what you mean. We verified on FPGA & Emulator, > >> and the logic is pretty much close to the real silicon. > > > > "Pretty much close" That doesn't give me confidence. The compatible > > should uniquely identify an SoC, but if it is used for both the actual > > SoC and for something "pretty much close" to the actual SoC then that > > does not hold. > > Ya, trying to have some pre-silicon FPGA-based platform alias with the > real chip is a repice for disaster. > > >> I did mention that in the cover letter as well. > > > > Ah apologies for missing that. I try to read cover letters but the > > volume of mail gets to me at times. > > > >> I am new to Linux, so I am wondering if there is a Linux upstream guideline mentioning > >> that pre-silicon software is not allowed to upstream? > > > > I wouldn't say that this is the case, but things like clock and pinctrl > > drivers are the sort of things that are likely to vary in your "pretty > > much close" as that is the kind of thing that change for your final > > integration, versus a more "standalone" peripheral. > > Yep, and since integration issues in the ASIC blocks can end up > manifesting as SW-visible behavior in nearby blocks it's hard to just > pull out the peripherals -- we sort of try by getting the DT topology to > match the SOC, but there's always some mismatches. Thank you everyone. I think I get your point. Is it possible to send "RFC" patches for things like DT, clk&reset, and pinctrl? Please note that these have been tested on FPGA/Emulator. > > > For dts stuff, in RISC-V at least, we've been operating so far on the > > basis that systems implemented entirely on an FPGA are not suitable for > > inclusion in mainline. I would say that this can probably be relaxed to > > allow systems where there are publicly available, versioned, designs or > > bitstreams that are widely used that these devicetrees correspond to. > > This would suit something like if AMD published a bitstream using one > > of their new MicroblazeV cpu cores as a sort of "reference design". > > FPGAs are definately in a grey area, but that's been my mindset as well. > For me it's less about FPGA vs ASIC (or any other manufacturing > technology in between) and more about whether something is being used > publicly. Specifically: is the FPGA used for internal pre-silicon work > or is it some publicly availiable system? It is internal. > > The versioning stuff is also important, but we need that for ASICs as > well since they can be re-spun. > > >> Hope there is an updated Linux > >> upstream guideline that benefit other vendors. > > > > I have no idea if there is one or not. I think it generally varies on > > individual maintainers etc, and for something like a dts it comes down > > to the platform maintainer (Emil) I suppose. Sending stuff out before > > your SoC has been produced is really great though, so it is a fine line > > to avoid discouraging something we really like to see. > > IIRC we've got some stuff written for arch/riscv somewhere in > Documentation, but the hardest part here is that each subsystem is going > to have different policies so it's kind of hard to try and come up with > a general rule. > > > Cheers, > > Conor.
On Fri, Dec 15, 2023 at 01:49:02AM +0000, JeeHeng Sia wrote: > Thank you everyone. I think I get your point. Is it possible to send "RFC" > patches for things like DT, clk&reset, and pinctrl? Please note that > these have been tested on FPGA/Emulator. For sure you can send them as RFC, yes.
diff --git a/Documentation/devicetree/bindings/riscv/starfive.yaml b/Documentation/devicetree/bindings/riscv/starfive.yaml index cc4d92f0a1bf..12d7844232b8 100644 --- a/Documentation/devicetree/bindings/riscv/starfive.yaml +++ b/Documentation/devicetree/bindings/riscv/starfive.yaml @@ -30,6 +30,10 @@ properties: - starfive,visionfive-2-v1.3b - const: starfive,jh7110 + - items: + - enum: + - starfive,jh8100-evb + - const: starfive,jh8100 additionalProperties: true ...