From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S1751320AbcDPHoR (ORCPT <rfc822;w@1wt.eu>);
	Sat, 16 Apr 2016 03:44:17 -0400
Received: from mout.kundenserver.de ([212.227.17.10]:57572 "EHLO
	mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1751024AbcDPHoP (ORCPT
	<rfc822;linux-kernel@vger.kernel.org>);
	Sat, 16 Apr 2016 03:44:15 -0400
From: Arnd Bergmann <arnd@arndb.de>
To: linux-arm-kernel@lists.infradead.org
Cc: Duc Dang <dhdang@apm.com>, Feng Kan <fkan@apm.com>,
        Tomasz Nowicki <tn@semihalf.com>, Mark Salter <msalter@redhat.com>,
        patches <patches@apm.com>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Loc Ho <lho@apm.com>, linux-pci@vger.kernel.org,
        Bjorn Helgaas <bhelgaas@google.com>, Tanmay Inamdar <tinamdar@apm.com>
Subject: Re: [PATCH] pci: xgene: Add ECAM fixups
Date: Sat, 16 Apr 2016 09:43:40 +0200
Message-ID: <6597458.plyKBtcpp4@wuerfel>
User-Agent: KMail/4.11.5 (Linux/3.16.0-10-generic; KDE/4.11.5; x86_64; ; )
In-Reply-To: <CADaLNDkCtUgFyX5tVgeM9grdHSQvpTHaq7qXk2WZ1tTbrP7OGA@mail.gmail.com>
References: <1455068984-22210-1-git-send-email-dhdang@apm.com> <2994764.UGKJE3zfLx@wuerfel> <CADaLNDkCtUgFyX5tVgeM9grdHSQvpTHaq7qXk2WZ1tTbrP7OGA@mail.gmail.com>
MIME-Version: 1.0
Content-Transfer-Encoding: 7Bit
Content-Type: text/plain; charset="us-ascii"
X-Provags-ID: V03:K0:T9xuZMLeVHSc4FqgFC5Kepo1SVU7A+l/VMXsjEZ6S3587LkZipH
 FCGwlUSlMR5SGlyhOesRIjVv/3AZE+WDUMeFJQVJyDMTIOe3Xs/VwYPC2gK4ywwlid7vgd/
 CMG+6yk7wMSvux451RhGGvX8ZYAexEV8hhV++5BoqGE2zEzPQtHBFhIxXWsqHCIdpVoEkkC
 c3g3FWDRzGGIiEwAje+mw==
X-UI-Out-Filterresults: notjunk:1;V01:K0:WWlt+xxgx7M=:JGqjKKU3ZaA7i1Y6FR/rwN
 FdFCwVQQ7uNaJOniAluMVfed+6K3K6GPkDD0Hgx4LLiFZut4zLFxUA/4M2Oi5IZ047oJKN956
 e6hSP5givHJtec9XqLqa9wLhrAebjz799jG4AKSy66zbpFm1ei6txdYKnikm9d7jcS+ifdtXq
 H1DkcJEq0XvNHQKGchdHtHMPO/F+HPi9JR6nRHwsEhN8pYD/5GKK+DpyoNeWBXGDCFCzWmAGo
 psaGiZvQchp5sJhdX9ithbGOjMjoJ2OTGKVnJqmTIrHbFtcthNpj94PpwRCBCg+z4Pd5cCjQy
 mT6ChpGejwkuqKB3iH8tuAX+VYXFkGkEFuY/tTOtqZ/iE7+iZh2wcYLs6Egit8fk7OyzSdtHA
 OAo41ZJoOegxopKbFpse256lL38vtc6X+KU91Mo/nGZGxZXUGcNMiZMLA4k6tpJszVp34DnD+
 zA7hsw/4vafni46YKChY6smzxf1WdsulY2RRZOhWBUmAC1bmImipJA/vxBt7MkBEqUoicFYhP
 sTLBO+AiWSEbkwTvTmtUFMg9obrgygP9CAbONNZzP4wtRVFQ6tfWwdWKUebmelU845aaRiGd6
 sYON0gJPxhrcefPSqwjooncpz74YSk4V4s4KpOMwyw8eSJbRqf8e4FCPvVwUeaG4ZnzEtMl5R
 SnbeRc3pfapZ7E/1q+I3xjKGjg0UPLq7fzXHfd3uDcGSndpE2G24ceyLkdcz8iiTeij/6jixW
 cJSbbmpu4+YcwEx+
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Friday 15 April 2016 14:30:56 Duc Dang wrote:
> On Wed, Feb 10, 2016 at 6:28 AM, Arnd Bergmann <arnd@arndb.de> wrote:
> > On Tuesday 09 February 2016 17:49:44 Duc Dang wrote:
> >> X-Gene PCIe controller does not fully support ECAM.
> >> This patch adds required ECAM fixup to allow X-Gene
> >> PCIe controller to be functional in ACPI boot mode.
> >>
> >> This patch is based on the original work of
> >> Mark Salter <msalter@redhat.com> and depends on
> >> Tomasz's PCIe ACPI series:
> >> https://lkml.org/lkml/2016/2/4/646
> >>
> >> Signed-off-by: Duc Dang <dhdang@apm.com>
> >> ---
> >
> > This really has nothing to do with the PCI host device driver, please
> > keep it in a separate file in drivers/acpi/.
> 
> Hi Arnd,
> 
> With Tomasz latest PCIe ACPI series
> (https://lkml.org/lkml/2016/4/15/594), do you think it still better to
> keep the ACPI ECAM fixup code in a separate file in drivers/acpi
> directory?

Yes, I made the same comment for the cavium PCI workaround in that
series.

	Arnd