Commit 4d6186ca authored by Gavin Shan's avatar Gavin Shan Committed by Michael Ellerman

powerpc/powernv: Remove pnv_eeh_cap_start()

This moves the logic of pnv_eeh_cap_start() to pnv_eeh_find_cap()
as the function is only called by pnv_eeh_find_cap(). The logic
of both functions are pretty simple. No need to have separate
functions.
Signed-off-by: default avatarGavin Shan <gwshan@linux.vnet.ibm.com>
Reviewed-by: default avatarAndrew Donnellan <andrew.donnellan@au1.ibm.com>
Signed-off-by: default avatarMichael Ellerman <mpe@ellerman.id.au>
parent 608fb9c2
...@@ -281,29 +281,20 @@ static int pnv_eeh_post_init(void) ...@@ -281,29 +281,20 @@ static int pnv_eeh_post_init(void)
return ret; return ret;
} }
static int pnv_eeh_cap_start(struct pci_dn *pdn) static int pnv_eeh_find_cap(struct pci_dn *pdn, int cap)
{ {
u32 status; int pos = PCI_CAPABILITY_LIST;
int cnt = 48; /* Maximal number of capabilities */
u32 status, id;
if (!pdn) if (!pdn)
return 0; return 0;
/* Check if the device supports capabilities */
pnv_pci_cfg_read(pdn, PCI_STATUS, 2, &status); pnv_pci_cfg_read(pdn, PCI_STATUS, 2, &status);
if (!(status & PCI_STATUS_CAP_LIST)) if (!(status & PCI_STATUS_CAP_LIST))
return 0; return 0;
return PCI_CAPABILITY_LIST;
}
static int pnv_eeh_find_cap(struct pci_dn *pdn, int cap)
{
int pos = pnv_eeh_cap_start(pdn);
int cnt = 48; /* Maximal number of capabilities */
u32 id;
if (!pos)
return 0;
while (cnt--) { while (cnt--) {
pnv_pci_cfg_read(pdn, pos, 1, &pos); pnv_pci_cfg_read(pdn, pos, 1, &pos);
if (pos < 0x40) if (pos < 0x40)
......
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