Thread (20 messages) flat view 20 messages, 7 authors, 2019-01-09

Re: [PATCH] PCI: Add no-D3 quirk for Mellanox ConnectX-[45]

From: Bjorn Helgaas <bhelgaas@google.com>
Date: 2018-12-12 03:04:58
Also in: linux-pci, linux-rdma, lkml, netdev

On Tue, Dec 11, 2018 at 6:38 PM David Gibson
[off-list ref] wrote:
On Tue, Dec 11, 2018 at 08:01:43AM -0600, Bjorn Helgaas wrote:
quoted
Hi David,

I see you're still working on this, but if you do end up going this
direction eventually, would you mind splitting this into two patches:
1) rename the quirk to make it more generic (but not changing any
behavior), and 2) add the ConnectX devices to the quirk.  That way
the ConnectX change is smaller and more easily
understood/reverted/etc.
Sure.  Would it make sense to send (1) as an independent cleanup,
while I'm still working out exactly what (if anything) we need for
(2)?
You could, but I don't think there's really much benefit in doing the
first without the second, and I think there is some value in handling
both patches at the same time.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help