(Nit: I think the $subject is typically 'PCI: rockchip: ...'.)
Hi Rajat,
On Thu, Sep 22, 2016 at 02:00:27PM -0700, Rajat Jain wrote:
> The register value gets lost on a Link speed/width change, and the
> ideal fix should reprogram this on that event (refer "Link Bandwidth
> Management Interrupt Enable" & "Link Autonomous Bandwidth Interrupt Enable"
> in link control reg?).
Oh, that that reminds me (sorry for not noticing this earlier): Rockchip
did add handling of those two interrupt bits. See:
} else if (reg & PCIE_CLIENT_INT_PHY) {
dev_dbg(dev, "phy link changes\n");
rockchip_pcie_clr_bw_int(rockchip);
}
Do you ever see such interrupts? In any case, it's possible we could do
the re-programming there, just to be sure, though I'm not sure we can
test it well.
Brian
--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
