On Mon, Dec 18, 2017 at 03:25:19PM +0000, Bounine, Alexandre wrote: > Acked-by: Alexandre Bounine <alexandre.bounine@xxxxxxx> > > Hi Andrew, > > Please apply the patch below. Applied with Alexandre's ack to pci/misc for v4.17. > -----Original Message----- > From: Bjorn Helgaas [mailto:helgaas@xxxxxxxxxx] > Sent: Friday, December 15, 2017 6:00 PM > To: Bounine, Alexandre <Alexandre.Bounine@xxxxxxx>; Matt Porter <mporter@xxxxxxxxxxxxxxxxxxx> > Cc: linux-pci@xxxxxxxxxxxxxxx > Subject: [PATCH] rapidio/tsi721: use PCI_EXP_DEVCTL2_COMP_TIMEOUT macro > > From: Bjorn Helgaas <bhelgaas@xxxxxxxxxx> > > Use the existing PCI_EXP_DEVCTL2_COMP_TIMEOUT macro instead of hard-coding > the PCIe Completion Timeout Value mask. No functional change intended. > > Signed-off-by: Bjorn Helgaas <bhelgaas@xxxxxxxxxx> > --- > drivers/rapidio/devices/tsi721.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/drivers/rapidio/devices/tsi721.c b/drivers/rapidio/devices/tsi721.c > index 9a68914100ad..bb655854713d 100644 > --- a/drivers/rapidio/devices/tsi721.c > +++ b/drivers/rapidio/devices/tsi721.c > @@ -2880,8 +2880,9 @@ static int tsi721_probe(struct pci_dev *pdev, > "Invalid MRRS override value %d", pcie_mrrs); > } > > - /* Adjust PCIe completion timeout. */ > - pcie_capability_clear_and_set_word(pdev, PCI_EXP_DEVCTL2, 0xf, 0x2); > + /* Set PCIe completion timeout to 1-10ms */ > + pcie_capability_clear_and_set_word(pdev, PCI_EXP_DEVCTL2, > + PCI_EXP_DEVCTL2_COMP_TIMEOUT, 0x2); > > /* > * FIXUP: correct offsets of MSI-X tables in the MSI-X Capability Block >