From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934072AbcBDQIN (ORCPT ); Thu, 4 Feb 2016 11:08:13 -0500 Received: from mout.kundenserver.de ([212.227.126.133]:54923 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933784AbcBDQIL (ORCPT ); Thu, 4 Feb 2016 11:08:11 -0500 From: Arnd Bergmann To: Gabriele Paoloni Cc: "Guohanjun (Hanjun Guo)" , "Wangzhou (B)" , "liudongdong (C)" , Linuxarm , qiujiang , "bhelgaas@google.com" , "Lorenzo.Pieralisi@arm.com" , "tn@semihalf.com" , zhangjukuo , "xuwei (O)" , "Liguozhu (Kenneth)" , "linux-pci@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-acpi@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "jcm@redhat.com" Subject: Re: [RFC PATCH 0/4] Add ACPI support for HiSilicon PCIe Host Controllers Date: Thu, 04 Feb 2016 17:07:17 +0100 Message-ID: <76564455.dlc1WHiVY2@wuerfel> User-Agent: KMail/4.11.5 (Linux/3.16.0-10-generic; KDE/4.11.5; x86_64; ; ) In-Reply-To: References: <1454585860-60709-1-git-send-email-gabriele.paoloni@huawei.com> <2387127.DMZaDnZvEW@wuerfel> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Provags-ID: V03:K0:OXR+6NTJB6SwsNP/pgQOKRzBRhX5SpFLhu5isEB0sToLM7vc7Op yHbRhQ7SizbCzVlbep6kCi7Rx5a6AKFwFyBKlM6Ua5amXCs30SY2BpQVCNP35jqmX5ZCffv rj2xHSEUpA+2VhGMFKz1RFN1D+gw9DblwC5o+lf7cNg8EaZaoS0VaTY1m74skk2ka5nW2C+ RR8KMRxrFWt7GiDfRyZDg== X-UI-Out-Filterresults: notjunk:1;V01:K0:qN3CIjYsRgE=:nxlLKtc/bjoTHuKJ6O+KyW 0Zh84CQK7oPrGKE0Yq3RDOOP3rBcQhVqfHXfWlBie+o4xtDvBVs9ca3k8byHKbBhUzN5hgBJ/ gSoB6yi/Ieu7u9dsPbn4vL+eBr9XYP4KiU6yHYEuRZ61nlrGmXTJbK+MGw6gx97uwa13IxwSi fP2lBpkHtaxCCyx8sdsb4wn9/13hsP4DoRJTMYY7ZEX7f0e/7yt7JqhXYb5Nhv2K1P+K8MzDj SL4cafQ2g+ll+x2tPggv1kFN9+3jJo+tbQE9h/7Tpvb3TBHTDcOX9wWrDs1darFWssBQiqvHZ 5U2QW2fT+0m5NtNKqjN8UOhik/liNh8XOTfSAulZxDTiiWNxRYk5f+J2W3r8FKHLZPtOnwhdy zpii2GeKKxMpzeHv+Ej0PP09wl9DxrZAdQMD0QV3MUHFTFUBj1it2M+xX4QJgaxtjqqq3e9zm Okx1scIIxAmVZfcys/Dl+Y/HL0CmGjUWBBuTnPOJEZi77pNxk+ZTOEx37OpzC4pwrFXDC/MQa 8g/aoPv++SEeTc4NN7o0h2NR8MpPqSZFaGPxIYP+klfiNi8GBFGmWYhsh75l9ZXUKHb7mnKXk LQe/e7S6U9kmCeDm2hVGynWy/eFyGmii2ukVQTDsyHL4Aporw3Q2On+LkoE3aPnQ55RSjOvrS l1E08tv1s8KzT1II6Upq8PJdBAX0luqeS2REs3L9Un0fNjOQFTYzJEWgzwcp03yPrc5tJV+Bp ou+0DHC3uP1lkvAw Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thursday 04 February 2016 15:11:18 Gabriele Paoloni wrote: > > > > ACPI has its own PCI support, and should not need drivers for host > > bridges. I don't think we can really mix the two things, as ACPI > > needs to have access to things like PCI config space way before > > we are probing normal device drivers. > > > > Please put this in drivers/acpi/pci*.c. > > I can put pcie-hisi-acpi.c under drivers/acpi/ > > However if you look at the driver it is made up of three parts: > pcie-hisi.c --> the DT based driver > pcie-hisi-acpi.c --> the ACPI based hook and ACPI specific init callback > pcie-hisi-common.c --> common functions shared between DT and ACPI versions > of the driver > > Now I think that moving pcie-hisi-acpi.c under drivers/acpi/ > would make it hard to read as you need to jump across directories > and it seems a bit unnatural... > > However it is not a big issue to me... That's not really what I meant though: the pcie-hisi driver uses the pcie-designware.c library, most of which makes no sense in an environment where you have ACPI, e.g. link training, custom MSI support, initial register setup, platform driver hooks, etc. You should add a very minimal set of hacks for the parts in this driver that diverge from a standard SBSA compliant PCIe host that ACPI expects. Arnd