Re: [PATCH v3 12/32] PCI/ARM: use PCIe capabilities access functions to simplify implementation |
|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- Subject: Re: [PATCH v3 12/32] PCI/ARM: use PCIe capabilities access functions to simplify implementation
- From: Thierry Reding <thierry.reding@xxxxxxxxxxxxxxxxx>
- Date: Thu, 2 Aug 2012 07:58:39 +0200
- Cc: Jiang Liu <liuj97@xxxxxxxxx>, Bjorn Helgaas <bhelgaas@xxxxxxxxxx>, Don Dutile <ddutile@xxxxxxxxxx>, Colin Cross <ccross@xxxxxxxxxxx>, Olof Johansson <olof@xxxxxxxxx>, Jiang Liu <jiang.liu@xxxxxxxxxx>, Yinghai Lu <yinghai@xxxxxxxxxx>, Taku Izumi <izumi.taku@xxxxxxxxxxxxxx>, "Rafael J . Wysocki" <rjw@xxxxxxx>, Kenji Kaneshige <kaneshige.kenji@xxxxxxxxxxxxxx>, Yijing Wang <wangyijing@xxxxxxxxxx>, Russell King <linux@xxxxxxxxxxxxxxxx>, linux-tegra@xxxxxxxxxxxxxxx, linux-arm-kernel@xxxxxxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, linux-pci@xxxxxxxxxxxxxxx
- In-reply-to: <5019654D.2000408@wwwdotorg.org>
- User-agent: Mutt/1.5.21 (2010-09-15)
On Wed, Aug 01, 2012 at 11:20:13AM -0600, Stephen Warren wrote:
> On 08/01/2012 09:54 AM, Jiang Liu wrote:
> > From: Jiang Liu <jiang.liu@xxxxxxxxxx>
> >
> > Use PCIe capabilities access functions to simplify PCIe ARM implementation.
> >
> > Signed-off-by: Jiang Liu <liuj97@xxxxxxxxx>
>
> This is probably fine from my perspective assuming the underlying PCI
> API implementation is OKd by relevant people.
>
> Thierry, can you comment on whether this will conflict with your Tegra
> PCIe driver series? If it doesn't, this patch can go through any
> relevant PCI tree. If it does, we may have to take this through the
> Tegra tree after merging the dependencies, or defer it to later.
I haven't touched those lines. tegra_pcie_relax_enable() only uses
generic PCI functions so I don't expect any conflicts. git blame
confirms that those lines haven't been changed in about 2 years.
Thierry
Attachment:
pgpCniXOTVn5v.pgp
Description: PGP signature
[ARM Kernel]
[Linux USB Devel]
[Video for Linux]
[Linux Audio Users]
[Photo]
[Yosemite News]
[Yosemite Photos]
[Free Online Dating]
[Linux Kernel]
[Linux SCSI]
[XFree86]