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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 41F4BE95A61 for ; Sat, 7 Oct 2023 11:57:16 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1343867AbjJGL5P (ORCPT ); Sat, 7 Oct 2023 07:57:15 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:46640 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1343680AbjJGL5O (ORCPT ); Sat, 7 Oct 2023 07:57:14 -0400 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D1047B9 for ; Sat, 7 Oct 2023 04:57:13 -0700 (PDT) Received: by smtp.kernel.org (Postfix) with ESMTPSA id D7471C433C9; Sat, 7 Oct 2023 11:57:12 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1696679833; bh=0yt73kgAPYECXihzzq92WeV8qT2NdU3GEyERzOSit4o=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=bBXwYLyCuthuf8wsye76/C/pOhMzc9i5GERBM0kM5HdbHS5h4JNN6YH7yaYQXFh7O XR2NzYF+Csj8TMjq5yJjmCFuVOpC2yX/UT1aMh609tNvtGl6wSr1RdQ4tImFfSYDdA cqx78vorYwmh/V7VKf50hTwYRh115cvoLHjMAD+E= Date: Sat, 7 Oct 2023 13:57:10 +0200 From: Greg Kroah-Hartman To: Ben Hutchings Cc: stable@vger.kernel.org, patches@lists.linux.dev, Manivannan Sadhasivam , Lorenzo Pieralisi Subject: Re: [PATCH 5.10 397/509] PCI: qcom: Disable write access to read only registers for IP v2.3.3 Message-ID: <2023100736-enlarged-return-6dc0@gregkh> References: <20230725104553.588743331@linuxfoundation.org> <20230725104611.936185910@linuxfoundation.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: stable@vger.kernel.org On Sun, Sep 24, 2023 at 11:15:35PM +0200, Ben Hutchings wrote: > On Tue, 2023-07-25 at 12:45 +0200, Greg Kroah-Hartman wrote: > > From: Manivannan Sadhasivam > > > > commit a33d700e8eea76c62120cb3dbf5e01328f18319a upstream. > > > > In the post init sequence of v2.9.0, write access to read only registers > > are not disabled after updating the registers. Fix it by disabling the > > access after register update. > > > > Link: https://lore.kernel.org/r/20230619150408.8468-2-manivannan.sadhasivam@linaro.org > > Fixes: 5d76117f070d ("PCI: qcom: Add support for IPQ8074 PCIe controller") > > Signed-off-by: Manivannan Sadhasivam > > Signed-off-by: Lorenzo Pieralisi > > Cc: > > Signed-off-by: Greg Kroah-Hartman > > --- > > drivers/pci/controller/dwc/pcie-qcom.c | 2 ++ > > 1 file changed, 2 insertions(+) > > > > --- a/drivers/pci/controller/dwc/pcie-qcom.c > > +++ b/drivers/pci/controller/dwc/pcie-qcom.c > > @@ -771,6 +771,8 @@ static int qcom_pcie_get_resources_2_4_0 > > This fix was supposed to be for v2.3.3 of the hardware and originally > changed the function qcom_pcie_get_resources_2_3_3(). > > However, the backports to 4.19, 5.4, and 5.10 applied this change to > the similar function qcom_pcie_get_resources_2_4_0(). > > Please move the added function call into the correct function. That function is not in those older kernels, which is why patch tried it's best and moved to the other function. I'll just go revert the offending commit from all of these branches, thanks for noticing! greg k-h