PCI: pciehp: Stop disabling notifications during init
authorYinghai Lu <yinghai@kernel.org>
Sat, 23 Aug 2014 01:15:12 +0000 (18:15 -0700)
committerBjorn Helgaas <bhelgaas@google.com>
Tue, 23 Sep 2014 16:03:59 +0000 (10:03 -0600)
commit31ff2a5e42f459f36a88afa82244dd3d3285572e
tree61970da1588c32c538b4f1de6edcbad0f8e2eaa5
parentcf8d7b589c53f17e10e9f1ef91dd9e2ba3ca9a7c
PCI: pciehp: Stop disabling notifications during init

During pciehp initialization, we previously wrote two hotplug commands:

  pciehp_probe
    pcie_init
      pcie_disable_notification
        pcie_write_cmd           # command 1
    pcie_init_notification
      pcie_enable_notification
        pcie_write_cmd           # command 2

For controllers with errata like Intel CF118, we previously waited for a
timeout before issuing the second hotplug command because the first command
only updates interrupt enable bits and is not a "real" hotplug command, so
the controller doesn't report Command Completed for it.

But there's no need to disable notifications in the first place.  If BIOS
left them enabled, we could easily take an interrupt before disabling them,
so there's no benefit in disabling them for the tiny window before we
enable them.

Drop the unnecessary pcie_disable_notification() call.

[bhelgaas: changelog]
Link: http://www.intel.com/content/www/us/en/processors/xeon/xeon-e7-v2-spec-update.html
Signed-off-by: Yinghai Lu <yinghai@kernel.org>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
drivers/pci/hotplug/pciehp_hpc.c