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 04231CDB482 for ; Wed, 18 Oct 2023 08:25:02 +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:To:Subject: CC: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=jp989x7XhBcbQjr7MEHH5h0Um0HC01sNDUYKGrDWJVg=; b=dNn4tZLjhAi+wQ kCqRp8ojHKPr8/svkYMtSaMZnJK6m/RRomOuZrV4c8BTcvUc/ahVkJw4ZACn89xlRAo0kqOHoUUHG 3O7UIK7/FH8nYRr+kmdqp5TVGrY6Pc+LwJsHFhWYU6GsvRBpIh0d69RjongOfJqMZ99Mv6MCV2nln AurHZGt0RD8+pXjMH2N+9KzwVxUEap1HZ7ZI+DUFe42OBSmKdTwVuZ+vt95zv51L6MxFAtw0Hoygw 48i8UUUh8Gpf5o2Hw09yCN745RtE3ijGj6CbAylEWLZVqBToBWS49T+50/gVxLUScSt3FgbjiwBlu W6kUTcqm1yA4nXS9YxOQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qt1qy-00E8Pd-2s; Wed, 18 Oct 2023 08:24:32 +0000 Received: from fllv0015.ext.ti.com ([198.47.19.141]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qt1qv-00E8Oj-1T for linux-arm-kernel@lists.infradead.org; Wed, 18 Oct 2023 08:24:30 +0000 Received: from fllv0034.itg.ti.com ([10.64.40.246]) by fllv0015.ext.ti.com (8.15.2/8.15.2) with ESMTP id 39I8OEa1077613; Wed, 18 Oct 2023 03:24:14 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1697617454; bh=ZtCVecdF83AGjBhuAEFcCM6NSuKe7HJoHlKbAZhIm1E=; h=Date:CC:Subject:To:References:From:In-Reply-To; b=FjJyTLwkvPFOWcb37XVyA2UqptuUr5wLb/beXhPOYam/dgvFu/UlJg4GEJ/xiafnj V/ieGYIOOSCmXuqGpGVwUfy9HlWJt7EKmGyNHqt3TxIVoagJqqMiXIzTT00/i9FzC5 k9EjhaU10ZLZhlhnsNVdHgR5ExY43kUZJjJFCVdE= Received: from DLEE110.ent.ti.com (dlee110.ent.ti.com [157.170.170.21]) by fllv0034.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 39I8OEW1126017 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Wed, 18 Oct 2023 03:24:14 -0500 Received: from DLEE111.ent.ti.com (157.170.170.22) by DLEE110.ent.ti.com (157.170.170.21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23; Wed, 18 Oct 2023 03:24:13 -0500 Received: from fllv0040.itg.ti.com (10.64.41.20) by DLEE111.ent.ti.com (157.170.170.22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2507.23 via Frontend Transport; Wed, 18 Oct 2023 03:24:14 -0500 Received: from [172.24.227.9] (ileaxei01-snat2.itg.ti.com [10.180.69.6]) by fllv0040.itg.ti.com (8.15.2/8.15.2) with ESMTP id 39I8OAbx036840; Wed, 18 Oct 2023 03:24:11 -0500 Message-ID: <564eead0-9439-4bd0-8281-4539d35bc0fc@ti.com> Date: Wed, 18 Oct 2023 13:54:09 +0530 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird CC: , , , , Subject: Re: [PATCH v2] PCI: keystone: Fix ks_pcie_v3_65_add_bus() for AM654x SoC Content-Language: en-US To: Ravi Gunasekaran , , , , References: <20231018075038.2740534-1-s-vadapalli@ti.com> From: Siddharth Vadapalli In-Reply-To: X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231018_012429_577881_3512AA75 X-CRM114-Status: GOOD ( 17.94 ) 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 Ravi, On 18/10/23 13:49, Ravi Gunasekaran wrote: > Siddharth, > > > On 10/18/23 1:20 PM, Siddharth Vadapalli wrote: >> The ks_pcie_v3_65_add_bus() member of "ks_pcie_ops" was added for >> platforms using DW PCIe IP-core version 3.65a. The AM654x SoC uses >> DW PCIe IP-core version 4.90a and ks_pcie_v3_65_add_bus() is not >> applicable to it. >> >> The commit which added support for the AM654x SoC has reused majority >> of the functions with the help of the "is_am6" flag to handle AM654x >> separately where applicable. Thus, make use of the "is_am6" flag and >> change ks_pcie_v3_65_add_bus() to no-op for AM654x SoC. >> >> Fixes: 18b0415bc802 ("PCI: keystone: Add support for PCIe RC in AM654x Platforms") > > 6ab15b5e7057 (PCI: dwc: keystone: Convert .scan_bus() callback to use add_bus) > is that one that seems to have introduced this issue. > > ks_pcie_v3_65_scan_bus() was for IP version 3.65 and this was renamed and > added to "ks_pcie_ops" which is used by other IP versions as well. Thank you for reviewing the patch and pointing this out. I will update the commit message with the correct Fixes tag as well as the appropriate description and post the v3 patch if there is no further feedback from others on this patch. > > >> Signed-off-by: Siddharth Vadapalli >> --- >> Hello, >> >> This patch is based on linux-next tagged next-20231018. >> >> The v1 of this patch is at: >> https://lore.kernel.org/r/20231011123451.34827-1-s-vadapalli@ti.com/ ... >> return 0; >> >> /* Configure and set up BAR0 */ > -- Regards, Siddharth. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel