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 2032CC3DA45 for ; Thu, 11 Jul 2024 14:22:13 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=/m545HxzhNdRfnzRSWHm5MWlcXD+mej28IX9WxEmzRw=; b=k3J2N5l8pq/PT55AnN6cc5BZhx t4Ff3lTbh29Rzs5IsrAvYNmL4o4wzqtAJYOVBSjhzJ8GehCgmLkj1N3kCnNJhLNUCFUNZptXdJ79x beEG/Xr09UTe5t5GuClblOW7YHBJS5VlfujaczfrgDm4O817Xc1Ya3Ai3ahPW5by1xwcM+BBmNe3l V6r6ykI5qbanEe13yoq/3OdzbEODvrxgNDhbvVJ/Om810wdiUBhZmzrotT91L0rOsbFloh6bCI+Gc ExTn9GSc4ATbLUR5D7owWVnDi5C54iMtcVTaqAC2Oy+aLjQxfoKlArUWZf99k9ZUT4ttw0Nam5uAk A1OppqXQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sRugK-0000000EKT1-1IXu; Thu, 11 Jul 2024 14:22:00 +0000 Received: from dfw.source.kernel.org ([2604:1380:4641:c500::1]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sRug2-0000000EKKf-2bCE for linux-arm-kernel@lists.infradead.org; Thu, 11 Jul 2024 14:21:43 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id 9A87061D2F; Thu, 11 Jul 2024 14:21:41 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id C80FBC32782; Thu, 11 Jul 2024 14:21:37 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1720707701; bh=OFfgo+TtGVgSb9VZqT8NLJ9xp+qtpitkJFL8d+tpdrs=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=kG5dHIKQy4zaAFULcBG6mIa0k+R7/V04a6hQOC7Ihi3uqvupF2HYF6k2e+PQ1haEc iW89G6csdeGntpVYH5gbI0HCtEPgQt5G/CQplddsq6IBgxq1NCAbNHJARTupI1eQ3l VEFDqz7pik/mrQn+BQ1I7pvO5/MQIp544p7HzoN5QCiI2LW2i7hAUysB3MKiB7txrO L8VdEAfYJRnJx8aL4rSEZ/mvv/L9Ik8I6czHwPwc7BUvUCmC1ck3n+BdwsI423a+kW oy3fZhacCZS7ysb/itUeEEnDkhRoRSN5D5NwWXncGmAxrNEshULSlj9q6PuFsSvwdQ 1EUTLPWPp4nRg== Date: Thu, 11 Jul 2024 16:21:34 +0200 From: Niklas Cassel To: Francesco Dolcini , Rob Herring , Krzysztof Kozlowski Cc: Nishanth Menon , Vignesh Raghavendra , Tero Kristo , Conor Dooley , Siddharth Vadapalli , Lorenzo Pieralisi , Krzysztof =?utf-8?Q?Wilczy=C5=84ski?= , Bjorn Helgaas , Francesco Dolcini , linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-omap@vger.kernel.org, linux-pci@vger.kernel.org Subject: Re: [PATCH v1 0/2] PCI: ti: k3: Fix TI J721E PERST# polarity Message-ID: References: <20240703100036.17896-1-francesco@dolcini.it> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20240703100036.17896-1-francesco@dolcini.it> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240711_072142_723168_9C909428 X-CRM114-Status: GOOD ( 14.84 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Wed, Jul 03, 2024 at 12:00:34PM +0200, Francesco Dolcini wrote: > From: Francesco Dolcini > > Fix PCIe PERST# signal polarity in TI J721E used on TI K3 machines. > > PCIe PERST# needs to be de-asserted for PCIe to work, however, the driver is > doing the opposite and the device tree files are defining the signal with the > wrong polarity to cope with that. Fix both the driver and the affected DT > files. Hello there, While I understand why you want to fix this, I'm not sure if you can actually do so without breaking device tree backwards compatibility. Imagine e.g. a board that has the DTB in ROM. The user upgrades the kernel, and the DTB that was working with the old kernel will now no longer work with the new kernel (because of your driver change). Just because you update the DTS files, you cannot assume that all DTBs out there in the wild will automatically be updated. That is what the DT maintainers told me many years ago when I wanted to do a similar change as yours, for another PCIe controller driver. Perhaps you could add a comment in the driver and the DTS files explaining that the DTS is actually wrong, but cannot be changed because of DT backwards compatibility. Kind regards, Niklas