Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932665AbcDNPkn (ORCPT ); Thu, 14 Apr 2016 11:40:43 -0400 Received: from mail-yw0-f181.google.com ([209.85.161.181]:34368 "EHLO mail-yw0-f181.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755237AbcDNPkl (ORCPT ); Thu, 14 Apr 2016 11:40:41 -0400 MIME-Version: 1.0 In-Reply-To: <570C4034.20105@caviumnetworks.com> References: <1460414707-19153-1-git-send-email-jchandra@broadcom.com> <1460414707-19153-3-git-send-email-jchandra@broadcom.com> <570C4034.20105@caviumnetworks.com> From: Jayachandran C Date: Thu, 14 Apr 2016 21:10:00 +0530 Message-ID: Subject: Re: [PATCH v2 2/4] PCI: Provide common functions for ECAM mapping To: David Daney Cc: Bjorn Helgaas , Tomasz Nowicki , rafael@kernel.org, Arnd Bergmann , Will Deacon , Catalin Marinas , Hanjun Guo , Lorenzo Pieralisi , Sinan Kaya , jiang.liu@linux.intel.com, Stefano Stabellini , robert.richter@caviumnetworks.com, Marcin Wojtas , Liviu.Dudau@arm.com, Wangyijing , Suravee.Suthikulpanit@amd.com, msalter@redhat.com, linux-pci@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org, linaro-acpi@lists.linaro.org, Jon Masters Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2103 Lines: 53 On Tue, Apr 12, 2016 at 5:54 AM, David Daney wrote: > On 04/11/2016 03:45 PM, Jayachandran C wrote: >> >> Add config option PCI_GENERIC_ECAM and file drivers/pci/ecam.c to >> provide generic functions for accessing memory mapped PCI config space. >> >> The API is defined in drivers/pci/ecam.h and is written to replace the >> API in drivers/pci/host/pci-host-common.h. The file defines a new >> 'struct pci_config_window' to hold the information related to a PCI >> config area and its mapping. This structure is expected to be used as >> sysdata for controllers that have ECAM based mapping. >> >> Helper functions are provided to setup the mapping, free the mapping >> and to implement the map_bus method in 'struct pci_ops' >> >> Signed-off-by: Jayachandran C > > Tested-by: David Daney I have updated the git tree (https://github.com/jchandra-brcm/linux/) with a branch arm64-acpi-pci-v3 . The branch has a new patch to use thunder ECAM ops in case of Cavium ThunderX platform when doing generic ACPI PCI initialization. I am hoping that the controllers that have "ECAM with quirks" can use this mechanism for sharing the quirks between OF and ACPI. If you have some time to review the patch and see it works for you, then I can post it with the v3 of this patchset. >> --- >> drivers/pci/Kconfig | 3 ++ >> drivers/pci/Makefile | 2 + >> drivers/pci/ecam.c | 130 >> +++++++++++++++++++++++++++++++++++++++++++++++++++ >> drivers/pci/ecam.h | 58 +++++++++++++++++++++++ > > > I wonder if these files should go in drivers/pci/host ... I understand that > you still have to use them from drivers/pci/acpi though. > > I will let others opine on this, but could you put the contents of ecam.h > into include/linux/pci.h along with the pci_generic_config_*() > declarations? > > If you did that, the contents of ecam.c could go into > drivers/pci/access.c... Earlier discussion seems to indicated that separate ecam.c/h was preferred. But I agree that it may be small enough to be merged. Thanks, JC.