PCI: cadence: Simplify J721e link status check
authorHans Zhang <18255117159@163.com>
Sat, 10 May 2025 16:07:10 +0000 (00:07 +0800)
committerManivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Tue, 13 May 2025 09:16:48 +0000 (10:16 +0100)
Replace explicit if-else condition with direct return statement in
j721e_pcie_link_up(). This reduces code verbosity while maintaining
the same logic for detecting PCIe link completion.

Signed-off-by: Hans Zhang <18255117159@163.com>
Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Reviewed-by: Niklas Cassel <cassel@kernel.org>
Link: https://patch.msgid.link/20250510160710.392122-4-18255117159@163.com
drivers/pci/controller/cadence/pci-j721e.c

index ef1cfdae33bb40f21dd5c74cafcd46dec6b5d1a5..bea1944a7eb2248cc96a15a52e463f9c61f971e8 100644 (file)
@@ -153,11 +153,7 @@ static bool j721e_pcie_link_up(struct cdns_pcie *cdns_pcie)
        u32 reg;
 
        reg = j721e_pcie_user_readl(pcie, J721E_PCIE_USER_LINKSTATUS);
-       reg &= LINK_STATUS;
-       if (reg == LINK_UP_DL_COMPLETED)
-               return true;
-
-       return false;
+       return (reg & LINK_STATUS) == LINK_UP_DL_COMPLETED;
 }
 
 static const struct cdns_pcie_ops j721e_pcie_ops = {