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 0F5B3C433F5 for ; Sun, 20 Feb 2022 11:05:37 +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=CkiLvbU15cQyTEB5u/vkNUnrjTtAWYZ322M1/SmkzCI=; b=xjJngQx/1vthzGhejnEM7SSXwz 8kcc/Mia8fz6bJ5SgEA6HMI5QJRiTNrc5X3lJTbtZ/HwMFcTHHwGz2idef0oIrnfQtcmc2McwsDqz 5YO8ISfjI6sjAlL6HBLBcC3CrcbHAjO+JJeOxOiaJf/UEtxncEL6tluC5PiuWUjvjowOKX1nQFwh1 bYd0RU0g8UId6bhTzmDY4fVZtEr3PoaslB+4jCoj9lhNFvFU/xYonbHoeUCYN8MkUCka6ASMjUOhe O6nNT7+5VPDdujz/f8dqApsDD4BOtk47Ru20NThOrJwfyEA+ys3np/FBl9k3fBM4mkQBkbPoURb5x 8RA7EvEA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nLk1q-0016W4-Ud; Sun, 20 Feb 2022 11:05:22 +0000 Received: from dfw.source.kernel.org ([139.178.84.217]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nLk1o-0016VW-FV for linux-riscv@lists.infradead.org; Sun, 20 Feb 2022 11:05:21 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id AD10360FDF; Sun, 20 Feb 2022 11:05:17 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 1D878C340E8; Sun, 20 Feb 2022 11:05:17 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1645355117; bh=x9J+ZOe2edxPaBXq7MDasjcZNR3NhXOu1vA+wgVP93U=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=NYE1ADu8MGRD9Rpyc9dccMIA2QJqdOKiJc0511rAQeBxW0NhWeRdD+UJIWhwhpYeJ 57XcLgYRkxJyCDjeyycJzYl2gpY/gFDpZ23yhfgfIQb71r4Fd+lcyiSsE5DTsWNRuS rZz0DR5zJ8dkdX0sCox4UejhTFVSCW+yrPmt/JvvJ7iZBEBiLkcBOmNG7YwPhfhK7V rnxmSJz5wxje8fUq36yRLVjUPb10XAqTaFW0wmOrTRnKcyIFYqGRA2K7pbZpbwqAB/ pT0fEbFabrQM5GigdLgeSGP+aIjZINCogcliCt4G73ajobWvCqC7PeXaM7CXdUpJcw ZoqSWaVOucVQw== Received: from disco-boy.misterjones.org ([51.254.78.96] helo=www.loen.fr) by disco-boy.misterjones.org with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1nLk1i-0093l2-OP; Sun, 20 Feb 2022 11:05:14 +0000 MIME-Version: 1.0 Date: Sun, 20 Feb 2022 11:05:14 +0000 From: Marc Zyngier To: Pavel Machek Cc: Sasha Levin , linux-kernel@vger.kernel.org, stable@vger.kernel.org, Guo Ren , Anup Patel , Palmer Dabbelt , Samuel Holland , Thomas Gleixner , paul.walmsley@sifive.com, aou@eecs.berkeley.edu, linux-riscv@lists.infradead.org Subject: Re: [PATCH AUTOSEL 4.19 03/11] irqchip/sifive-plic: Add missing thead,c900-plic match string In-Reply-To: <20220220095431.GA5251@amd> References: <20220215153104.581786-1-sashal@kernel.org> <20220215153104.581786-3-sashal@kernel.org> <20220220095431.GA5251@amd> User-Agent: Roundcube Webmail/1.4.13 Message-ID: X-Sender: maz@kernel.org X-SA-Exim-Connect-IP: 51.254.78.96 X-SA-Exim-Rcpt-To: pavel@ucw.cz, sashal@kernel.org, linux-kernel@vger.kernel.org, stable@vger.kernel.org, guoren@linux.alibaba.com, anup@brainfault.org, palmer@dabbelt.com, samuel@sholland.org, tglx@linutronix.de, paul.walmsley@sifive.com, aou@eecs.berkeley.edu, linux-riscv@lists.infradead.org X-SA-Exim-Mail-From: maz@kernel.org X-SA-Exim-Scanned: No (on disco-boy.misterjones.org); SAEximRunCond expanded to false X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220220_030520_652772_A4B2D2DD X-CRM114-Status: GOOD ( 16.70 ) 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-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-riscv" Errors-To: linux-riscv-bounces+linux-riscv=archiver.kernel.org@lists.infradead.org On 2022-02-20 09:54, Pavel Machek wrote: > Hi! > >> [ Upstream commit 1d4df649cbb4b26d19bea38ecff4b65b10a1bbca ] >> >> The thead,c900-plic has been used in opensbi to distinguish >> PLIC [1]. Although PLICs have the same behaviors in Linux, >> they are different hardware with some custom initializing in >> firmware(opensbi). >> >> Qute opensbi patch commit-msg by Samuel: >> >> The T-HEAD PLIC implementation requires setting a delegation bit >> to allow access from S-mode. Now that the T-HEAD PLIC has its own >> compatible string, set this bit automatically from the PLIC driver, >> instead of reaching into the PLIC's MMIO space from another driver. >> >> [1]: >> https://github.com/riscv-software-src/opensbi/commit/78c2b19218bd62653b9fb31623a42ced45f38ea6 >> > > The "thead,c900-plic" string is added into single place in the > kernel. This means that a) it will probably not do anything useful in > -stable kernels and b) it is certainly missing documentation etc. > > In mainline, string is documented in > Documentation/devicetree/bindings/interrupt-controller/sifive,plic-1.0.0.yaml (b) is certainly true. And to make the above comment useful, the missing patch is 321a8be37e1a ("dt-bindings: update riscv plic compatible string"). Regarding (a), the DT is provided by the firmware (as it should be on any reasonable platform). As such, no need for this string to be mentioned anywhere else but in the documentation. Now, the real question is where there is any point in backporting this to such an old kernel, as this HW is unlikely to ever run it. M. -- Jazz is not dead. It just smells funny... _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv