[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <29453556.BCB7AZ5P6c@wuerfel>
Date: Wed, 23 Sep 2015 21:36:42 +0200
From: Arnd Bergmann <arnd@...db.de>
To: linux-arm-kernel@...ts.infradead.org
Cc: David Daney <ddaney@...iumnetworks.com>,
David Daney <david.daney@...ium.com>,
Marc Zyngier <marc.zyngier@....com>,
Catalin Marinas <catalin.marinas@....com>,
Will Deacon <will.deacon@....com>,
linux-kernel@...r.kernel.org, David Daney <ddaney.cavm@...il.com>,
linux-pci@...r.kernel.org, Bjorn Helgaas <bhelgaas@...gle.com>
Subject: Re: [PATCH] arm64/pci: Add quirks for Cavium Thunder PCI bridges.
On Wednesday 23 September 2015 09:00:36 David Daney wrote:
> On 09/23/2015 12:51 AM, Arnd Bergmann wrote:
> > On Tuesday 22 September 2015 17:09:56 David Daney wrote:
> >> From: David Daney <david.daney@...ium.com>
> >>
> >> The Cavium ThunderX SoC needs a PCI quirk for its on-chip bridges.
> >> Since it is arm64, create a new quirks.c file there to contain arm64
> >> related quirks. Add the ThunderX bridge quirk, gated by a new config
> >> variable, so that it can be disabled for kernels that aren't expected
> >> to be used on ThunderX.
> >>
> >> Signed-off-by: David Daney <david.daney@...ium.com>
> >> ---
> >> arch/arm64/Kconfig | 11 +++++++++++
> >> arch/arm64/kernel/Makefile | 2 +-
> >> arch/arm64/kernel/quirks.c | 36 ++++++++++++++++++++++++++++++++++++
> >> 3 files changed, 48 insertions(+), 1 deletion(-)
> >> create mode 100644 arch/arm64/kernel/quirks.c
> >>
> >
> > Looks reasonable to me. Just one question: Is the same bridge used
> > on MIPS machines?
>
> No. The MIPS64 based OCTEON family of SoCs does not contain
> PCI-buses/config-space/bridges for on-chip hardware blocks. The on-chip
> blocks in OCTEON are all platform devices. So, ...
>
> > If so, maybe it should be moved to drivers/pci/quirks.c
> > instead for better reuse.
>
> The quirk is specific to some arm64 based SoCs, thus my idea to have
> arch specific quirks.
Ok.
Arnd
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists