diff options
| author | Hans Zhang <[email protected]> | 2025-05-10 16:07:08 +0000 |
|---|---|---|
| committer | Manivannan Sadhasivam <[email protected]> | 2025-05-13 09:12:59 +0000 |
| commit | f46bfb1d3c6a601caad90eb3c11a1e1e17cccb1a (patch) | |
| tree | c4e059220b319d991a827ef4791dbfa7ed1d89f4 /drivers/pci/controller/dwc/pcie-visconti.c | |
| parent | PCI: dw-rockchip: Fix PHY function call sequence in rockchip_pcie_phy_deinit() (diff) | |
| download | kernel-f46bfb1d3c6a601caad90eb3c11a1e1e17cccb1a.tar.gz kernel-f46bfb1d3c6a601caad90eb3c11a1e1e17cccb1a.zip | |
PCI: dwc: Return bool from link up check
PCIe link status check is supposed to return a boolean to indicate whether
the link is up or not. So, modify the link_up callbacks and
dw_pcie_link_up() function to return bool instead of int.
Signed-off-by: Hans Zhang <[email protected]>
[mani: commit message reword]
Signed-off-by: Manivannan Sadhasivam <[email protected]>
Reviewed-by: Manivannan Sadhasivam <[email protected]>
Reviewed-by: Niklas Cassel <[email protected]>
Link: https://patch.msgid.link/[email protected]
Diffstat (limited to 'drivers/pci/controller/dwc/pcie-visconti.c')
| -rw-r--r-- | drivers/pci/controller/dwc/pcie-visconti.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/pci/controller/dwc/pcie-visconti.c b/drivers/pci/controller/dwc/pcie-visconti.c index 318c278e65c8..cdeac6177143 100644 --- a/drivers/pci/controller/dwc/pcie-visconti.c +++ b/drivers/pci/controller/dwc/pcie-visconti.c @@ -121,13 +121,13 @@ static u32 visconti_mpu_readl(struct visconti_pcie *pcie, u32 reg) return readl_relaxed(pcie->mpu_base + reg); } -static int visconti_pcie_link_up(struct dw_pcie *pci) +static bool visconti_pcie_link_up(struct dw_pcie *pci) { struct visconti_pcie *pcie = dev_get_drvdata(pci->dev); void __iomem *addr = pcie->ulreg_base; u32 val = readl_relaxed(addr + PCIE_UL_REG_V_PHY_ST_02); - return !!(val & PCIE_UL_S_L0); + return val & PCIE_UL_S_L0; } static int visconti_pcie_start_link(struct dw_pcie *pci) |