Received: by 2002:a05:6a10:f3d0:0:0:0:0 with SMTP id a16csp3615887pxv; Mon, 5 Jul 2021 01:25:37 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzKhOkLMGh3tlg/mprslrGHPu15xjJm2pP947HCKFmtuaA4gwGdC5a9j7BjV1ra0znRrv+5 X-Received: by 2002:a17:907:97d6:: with SMTP id js22mr4257882ejc.345.1625473537082; Mon, 05 Jul 2021 01:25:37 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1625473537; cv=none; d=google.com; s=arc-20160816; b=Xj1BFGGpLu9ku+jU6hpo/pgWmLFh4pc9e7rOz5eyV5YyvdZNd/62fJVKc6tB7kx49X XIDXcwOtErHJyfbWXtkpEJm9HtMn7knTyPGDvZNSZmZW+JaNd5z36XD3wv+P2q83k3Wy u7kNCSLdW4jMORFrOIg7Fyg3q+YIThovRVz0TS1hUQrdmGgwoo3LdcRpDLI+yRQsjNtA Tctj6k8KPSmNcYDD30YPGfJJbA2CVH7jkvek2m6rDe/IqWCfqdBV3wJRDsuvhOcOLGP+ 3htaJO8KtCfg0zPZTsCOYl4FpjESQqlklqt7GFodwvU8KMEO8J5+Rs0uEKtamfG/tVRk g3FA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:content-language :in-reply-to:mime-version:user-agent:date:message-id:references:cc :to:subject:from:dkim-signature; bh=Y5AlfMUgJJYX/2dmO0aHXkikhRLjjtuU8z3wKxricB0=; b=v8F5Ay/e50DbpGLaCo1C9x6EgE/qnccxvkOPeSgGP655QDCrILqBcs+w7CSRT6YPYw oRTNoTflBM5Ka4C2LRZSda9saa16MnvhP/wLPvSdRXcWsRH45Abcbp8CK8F/qObf2koR HQI950fHMhR8UO4GZ3tLgFCi5wc7p1XFjD1nykle7tzsFBOaTU3CSqaUavq+GY2Wc9p/ lzBKuNw91WprSP9iYnnf8ydbYu3cg9ePhsFuEUOjOCh3vaR8hzKFU7ctxX2R7Cr8yx7g rs2UrUuqSMTg7Q4v6/A22kdG9ULj730m+O32zAkDZK7mFVndnaY0I+5M48nsKRWBkGvu mqvg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@ti.com header.s=ti-com-17Q1 header.b=AaI85ThQ; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=NONE dis=NONE) header.from=ti.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id w14si8793299edq.169.2021.07.05.01.25.14; Mon, 05 Jul 2021 01:25:37 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@ti.com header.s=ti-com-17Q1 header.b=AaI85ThQ; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=NONE dis=NONE) header.from=ti.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230121AbhGEIY2 (ORCPT + 99 others); Mon, 5 Jul 2021 04:24:28 -0400 Received: from fllv0016.ext.ti.com ([198.47.19.142]:43966 "EHLO fllv0016.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230081AbhGEIYY (ORCPT ); Mon, 5 Jul 2021 04:24:24 -0400 Received: from lelv0266.itg.ti.com ([10.180.67.225]) by fllv0016.ext.ti.com (8.15.2/8.15.2) with ESMTP id 1658LTJ0005911; Mon, 5 Jul 2021 03:21:29 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1625473289; bh=Y5AlfMUgJJYX/2dmO0aHXkikhRLjjtuU8z3wKxricB0=; h=From:Subject:To:CC:References:Date:In-Reply-To; b=AaI85ThQ6sn8gb1DpFaqvtAe9iPcadMGWGata0NI9XKZ3m1sX8+UyeZt8Nm4GvGia m6/BYDrsUoYJ0+4fW2O7HDwekOiteOnirynRY2s7B5m7PjzNCEoKIp6uNvMo6uIsQ+ PISwMnGtSwdxUbEli4ipDfWO5CjjQ3luGPrn8k6Y= Received: from DLEE100.ent.ti.com (dlee100.ent.ti.com [157.170.170.30]) by lelv0266.itg.ti.com (8.15.2/8.15.2) with ESMTPS id 1658LTVF129783 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=FAIL); Mon, 5 Jul 2021 03:21:29 -0500 Received: from DLEE109.ent.ti.com (157.170.170.41) by DLEE100.ent.ti.com (157.170.170.30) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2176.2; Mon, 5 Jul 2021 03:21:29 -0500 Received: from fllv0040.itg.ti.com (10.64.41.20) by DLEE109.ent.ti.com (157.170.170.41) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2176.2 via Frontend Transport; Mon, 5 Jul 2021 03:21:28 -0500 Received: from [10.250.232.207] (ileax41-snat.itg.ti.com [10.172.224.153]) by fllv0040.itg.ti.com (8.15.2/8.15.2) with ESMTP id 1658LNtD100793; Mon, 5 Jul 2021 03:21:24 -0500 From: Kishon Vijay Abraham I Subject: Re: [RFC PATCH 08/40] PCI: keystone: Cleanup MSI/legacy interrupt configuration and handling To: =?UTF-8?Q?Krzysztof_Wilczy=c5=84ski?= CC: Jingoo Han , Joao Pinto , Bjorn Helgaas , Rob Herring , Lorenzo Pieralisi , , Mark Rutland , Santosh Shilimkar , Nishanth Menon , , , , References: <20180921102155.22839-1-kishon@ti.com> <20180921102155.22839-9-kishon@ti.com> <20210703210152.GA16176@rocinante> Message-ID: <56160f1d-ec91-3b99-312c-aef66eb1a7c2@ti.com> Date: Mon, 5 Jul 2021 13:51:23 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 MIME-Version: 1.0 In-Reply-To: <20210703210152.GA16176@rocinante> Content-Type: text/plain; charset="utf-8" Content-Language: en-US Content-Transfer-Encoding: 8bit X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Krzysztof, On 04/07/21 2:31 am, Krzysztof WilczyƄski wrote: > Hi Kishon, > >> Now that all PCI keystone functionality has been moved to pci-keystone.c, >> cleanup MSI/legacy interrupt configuration and handling. >> *) Cleanup macros >> *) Remove unnecessary structure variables (required when 2 files are >> used) >> *) Remove ks_dw_pcie_legacy_irq_chip and use dummy_irq_chip >> *) Move request_irq of error irq from ks_add_pcie_port to ks_pcie_probe >> as error_irq is common to both host mode and device mode > [...] > > While looking at some small clean-ups for Bjorn, I stumbled upon this > series, and it seems a lot of your work here cover what Bjorn wanted to > do, thus I need to ask - do you recall, and I appreciate it's been > a while (three years actually), what happened and/or if you ever had the > time to work on this series? > > Would it be possible to resurrect this? Do you need any help? A lot of patches in this series should already be merged (after splitting into smaller ones) http://patchwork.ozlabs.org/project/linux-pci/list/?series=71185 https://patchwork.kernel.org/project/linux-arm-kernel/cover/20190321095927.7058-1-kishon@ti.com/ The following series is still pending and is in my TODO list https://lore.kernel.org/r/20210325090026.8843-1-kishon@ti.com Are there any other clean-ups you are looking into? Thanks and Regards Kishon