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 C7994C433F5 for ; Sun, 29 May 2022 17:38:29 +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:From:References:Cc:To: Subject:MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=6IUqDbhvunWjftZ1jGqlbEwRJoAoNl9LFnggmto9iFc=; b=sjZsCDvbn29NzD uiIqGkwPqBxMLJdiFWAXPodjjEJmBHriEfD9ZD0gTKClgbAjigKb2uvtnb+yx8StbsoXaTeUEOfOO WE3eOIeXrbx6/SvOKa+Ni9Mk1c7d6Ezda84Mw4RfEiHIW0j4vk0NMnVLJgAkOM/ghh8X3j91msL3/ lE22ecesn/NiR1u5enBFCMnayrs+zD27zhUhkYHSqZX3dREWXExW7QrGmuOMEOOODAK2FazpH6Xa/ i5v0zlaPDamwqhM1bPzyoUwRh9g/GNCGQu977Jsi++St6sAn+8n02YsbWZii4wS1IlJ68rwg0BWQO U1DMJZFInBcyAqrzuDkA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nvMqG-004MeE-E9; Sun, 29 May 2022 17:36:40 +0000 Received: from wp530.webpack.hosteurope.de ([2a01:488:42:1000:50ed:8234::]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nvMqC-004McQ-Ai; Sun, 29 May 2022 17:36:38 +0000 Received: from [2a02:8108:963f:de38:eca4:7d19:f9a2:22c5]; authenticated by wp530.webpack.hosteurope.de running ExIM with esmtpsa (TLS1.3:ECDHE_RSA_AES_128_GCM_SHA256:128) id 1nvMq1-0005v5-9X; Sun, 29 May 2022 19:36:25 +0200 Message-ID: Date: Sun, 29 May 2022 19:36:24 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.9.1 Subject: Re: [PATCH v2 1/1] PCI: brcmstb: Fix regression regarding missing PCIe linkup Content-Language: en-US To: Jim Quinlan , linux-pci , Nicolas Saenz Julienne , Bjorn Helgaas , James Dutton , Cyril Brulebois , bcm-kernel-feedback-list , Jim Quinlan Cc: Florian Fainelli , Lorenzo Pieralisi , Rob Herring , =?UTF-8?Q?Krzysztof_Wilczy=c5=84ski?= , "moderated list:BROADCOM BCM2711/BCM2835 ARM ARCHITECTURE" , "moderated list:BROADCOM BCM2711/BCM2835 ARM ARCHITECTURE" , open list References: <20220528224423.7017-1-jim2101024@gmail.com> <20220528224423.7017-2-jim2101024@gmail.com> From: Thorsten Leemhuis In-Reply-To: X-bounce-key: webpack.hosteurope.de;regressions@leemhuis.info;1653845796;5f6da462; X-HE-SMSGID: 1nvMq1-0005v5-9X X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220529_103636_586914_C9680637 X-CRM114-Status: GOOD ( 14.46 ) 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: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 29.05.22 18:52, Jim Quinlan wrote: > On Sat, May 28, 2022 at 6:44 PM Jim Quinlan wrote: >> >> commit 93e41f3fca3d ("PCI: brcmstb: Add control of subdevice voltage regulators") > >> Signed-off-by: Jim Quinlan >> introduced a regression on the PCIe RPi4 Compute Module. If the PCIe >> root port DT node described in [2] was missing, no linkup would be attempted, >> and subsequent accesses would cause a panic because this particular PCIe HW >> causes a CPU abort on illegal accesses (instead of returning 0xffffffff). >> >> We fix this by allowing the DT root port node to be missing, as it behaved >> before the original patchset messed things up. >> >> In addition, two small changes are made: >> >> 1. Having pci_subdev_regulators_remove_bus() call >> regulator_bulk_free() in addtion to regulator_bulk_disable(). >> 2. Having brcm_pcie_add_bus() return 0 if there is an >> error in calling pci_subdev_regulators_add_bus(). >> Instead, we dev_err() and turn on our refusal mode instead. >> >> It would be best if this commit were tested by someone with a Rpi CM4 >> platform, as that is how the regression was found. I have only emulated >> the problem and fix on different platform. >> >> Note that a bisection identified >> >> commit 830aa6f29f07 ("PCI: brcmstb: Split brcm_pcie_setup() into two funcs") >> >> as the first failing commit. This commit is a regression, but is unrelated >> and was fixed by a subsequent commit in the original patchset. >> >> [1] https://bugzilla.kernel.org/show_bug.cgi?id=215925 >> [2] Documentation/devicetree/bindings/pci/brcm,stb-pcie.yaml >> >> Fixes: 93e41f3fca3d ("PCI: brcmstb: Add control of subdevice voltage regulators") >> Fixes: 830aa6f29f07 ("PCI: brcmstb: Split brcm_pcie_setup() into two funcs") >> Link: https://bugzilla.kernel.org/show_bug.cgi?id=215925 > Thorston -- I forgot to replace the bugzilla link; I'll get it on V3. -- Jim Don't worry to much about these details that might matter for regzbot normally (the automatic handling of links to bugzilla ticket is sadly dysfunctional currently anyway). Just getting the issue fixed in the not-to-distant future is what I mainly care about. :-D Ciao Thorsten _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel