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 BDA52C2BB85 for ; Tue, 18 Jun 2024 04:48:41 +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-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=QJH7uHrt1kV2prGHF9aTaM3a+bzU+qhdxRovtVL8RMU=; b=iUthzaEgPRYyPv vzm5gO6MASg6K2DHzitY9xgA15GrtkVqdJoPTo/U09H65qathKVeIGgTrHFDPHYCrgbH7HOPGVchS UYZ86krUrOYD3+8wxPtapZKwLfFb55V2xsnAHXGI6hH9nrMP/lUlffd5O00nN/17YgT7UAAQy5b2Y 1mFnCQkoX9nTHpqu8OfZdqxEi0t323BWX2t0DIC93la9ppD8RWkusAXLNd2MLerY7EA4dQvfuXOtd 5/bwrptit5fRIf+ab5x88p4J9Xzw6hRstM//NjN5odvx/ah1UZX8vEyE+bv6ooDpEn757tCeKs/m+ OF7chwWqbQ7SVNdUpTPQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sJQlm-0000000Daql-3AzD; Tue, 18 Jun 2024 04:48:34 +0000 Received: from sin.source.kernel.org ([145.40.73.55]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sJQli-0000000Danq-2vbu for linux-riscv@lists.infradead.org; Tue, 18 Jun 2024 04:48:33 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by sin.source.kernel.org (Postfix) with ESMTP id B58BFCE13A5; Tue, 18 Jun 2024 04:48:27 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 92685C3277B; Tue, 18 Jun 2024 04:48:24 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1718686107; bh=uQRT705hswxV4eHkecwYH5nnl/fTVa9ZyWh6LbZP3oM=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=bh6Na5MHc3JrAvMbsJpyCpZKvDy2XEoJrEmyR5EvCKzupYOrvNfZogz7BJtaibGlm fYBMJkQONTPddF47mFqhUbUTi/T832VplagecpzuDma3vWQnxjImCiAxuk4n516Zfx ueMvb1lpkIBFBTmtdXj8CSo1lf9btx8mnO4JDrofcdsCyUJN/KG9ohJ5Wa6qc3mdx7 8FhaxrQz87M3Vh+dRwj6vVA/3vmqFvZkovmoqYNCrUJYpChnLGj3fWI4g2nnqdfWtH V1FhxpfJT5gGQ4nwmXEzFsg++64f6qdQSzLAqyE9PylKgo9sAlYumN4wghb2pk6MCU LMG/+hx82la7w== Date: Tue, 18 Jun 2024 12:34:27 +0800 From: Jisheng Zhang To: Conor Dooley Cc: Yixun Lan , Yangyu Chen , linux-riscv@lists.infradead.org, Conor Dooley , Palmer Dabbelt , Paul Walmsley , Samuel Holland , Anup Patel , Rob Herring , Krzysztof Kozlowski , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Jesse Taube Subject: Re: [PATCH v1 0/9] riscv: add initial support for SpacemiT K1 Message-ID: References: <20240616-exorcism-computing-e11e26084a62@spud> <20240616224811.GC3983622@ofsar> <20240617-synapse-carmaker-0a59c7c6edb7@spud> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20240617-synapse-carmaker-0a59c7c6edb7@spud> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240617_214831_176131_15193F36 X-CRM114-Status: GOOD ( 36.20 ) X-BeenThere: linux-riscv@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-riscv" Errors-To: linux-riscv-bounces+linux-riscv=archiver.kernel.org@lists.infradead.org On Mon, Jun 17, 2024 at 04:32:59PM +0100, Conor Dooley wrote: > On Mon, Jun 17, 2024 at 10:11:17PM +0800, Jisheng Zhang wrote: > > On Sun, Jun 16, 2024 at 10:48:11PM +0000, Yixun Lan wrote: > > > Hi Conor > > > Thanks for bringing this up > > > > > > On 19:35 Sun 16 Jun , Conor Dooley wrote: > > > > On Mon, Jun 17, 2024 at 01:18:52AM +0800, Yangyu Chen wrote: > > > > > > > > No MAINTAINERS update, so I figure that means you don't want to maintain > > > > it going forwards? If there's someone out that that does care about the > > > > spacemit k1 (Jesse maybe?), then I'd be more than happy to have them > > > > look after it. > > > Yangyu kind of has limited time, too many stuff for him.. > > > > > > I'd volunteered to help on this if it can fill the gap > > > Also I'd be more than happy if anyone willing step forward to co-maintain.. > > > > Does maintainership work like this? Is willing to do enough? > > FWICT, maintainership involves active patch contributing, reviewing and > > maintaining the whole SoC. It is better to take over the maintainership > > after showing enough patch contributions and understanding of the SoC. > > I was going to reply to your other patch about providing more complete > "basic" support for the SoC, but I guess I'll reply here and address > both points. After the k230 and th1520, which were both merged with very When I saw k230 a few minutes ago, I assumed you mean k210 since I didn't found k230 support in linus tree now. After searching the maillist, I found oh there is a k230 series which is similar to this series, no pinctrl, no clk, no reset. Since the incomplete K230 initial series hasn't been merged into Linus tree now, is it possible to drop it so that we can avoid the same mistake for k230. > basic support and have made very little progress towards being a useful > platform, I'm pretty reluctant to merge another platform in a super > basic state. I was going to make this point before you brought it up, > but it's good to know I am not the only one with that view. To be clear, > I'm not pointing blame for those platforms, I'd just like to avoid a Yep previously I thought it was fine to use a fixed clock or dummy clock during the initial patches, but I changed my mind now, especially after Samuel complained the cv1800b reset dt changes. > repeat. If Yangyu doesn't have time to do any development work on the > platform, I'd like to see someone else (and as I mentioned Jesse is > interested) take on getting some of the basic driver patches written and > merge only when those are accepted. Having no in-tree clock and pinctrl > drivers is definitely a hindrance to other people doing parallel > development of drivers and I'd like to avoid that. > > Getting back to your point in this mail, whoever gets the platform to > that state is well suited to looking after it going forwards. Some other The person who can bring the platfrom support to a well-moduled state, IE, proper clk, pinctrl, reset drivers shows the passion, the code contribution and solid understanding of the SoC, sure he/she is definitely suited to maintain the SoC. I just don't think it's a good practice a person can became maintainer even w/o one LoC contrubition to the SoC, because IMHO code contribution matters for maintainership. > interested parties could also join as reviewers. I don't want to see > people joining as maintainers that are not going to have an interest > in the platform going forward, as that'll just end up with me as the > defacto maintainer. > > Thanks, > Conor. _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv