Commit 2f2debb5 authored by Jia-Ju Bai's avatar Jia-Ju Bai Committed by Alex Deucher

gpu: drm: radeon: si: Replace mdelay() with msleep() in si_pcie_gen3_enable()

si_pcie_gen3_enable() is never called in atomic context.
It calls mdelay() to busily wait, which is not necessary.
mdelay() can be replaced with msleep().

This is found by a static analysis tool named DCNS written by myself
Signed-off-by: default avatarJia-Ju Bai <baijiaju1990@gmail.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent c51cebb8
...@@ -7183,7 +7183,7 @@ static void si_pcie_gen3_enable(struct radeon_device *rdev) ...@@ -7183,7 +7183,7 @@ static void si_pcie_gen3_enable(struct radeon_device *rdev)
tmp |= LC_REDO_EQ; tmp |= LC_REDO_EQ;
WREG32_PCIE_PORT(PCIE_LC_CNTL4, tmp); WREG32_PCIE_PORT(PCIE_LC_CNTL4, tmp);
mdelay(100); msleep(100);
/* linkctl */ /* linkctl */
pci_read_config_word(root, bridge_pos + PCI_EXP_LNKCTL, &tmp16); pci_read_config_word(root, bridge_pos + PCI_EXP_LNKCTL, &tmp16);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment