Commit 7df5218d authored by Lendacky, Thomas's avatar Lendacky, Thomas Committed by Herbert Xu

crypto: ccp - Update driver messages to remove some confusion

The current content of some of the driver messages and the way that they
are issued results in some confusion, especially in the area of the PSP
as it relates to SEV support. If SEV is not supported, a message is issued
that says "psp initialization failed." This makes it seem like there was
a problem, when in fact, the PSP support is just disabled if SEV is not
supported.

Update the driver to check SEV support a bit earlier and issue a debug-
level message if SEV is not supported, followed by a debug-level message
that the PSP is disabled. This way you will only see PSP messages if SEV
is supported or if debug information is desired. Also, remove the overall
"enabled" and "disabled" messages for the driver and rely on the CCP and
PSP support to issue component-specific messages.
Signed-off-by: default avatarTom Lendacky <thomas.lendacky@amd.com>
Signed-off-by: default avatarHerbert Xu <herbert@gondor.apana.org.au>
parent 333e6647
...@@ -858,15 +858,15 @@ static int sev_misc_init(struct psp_device *psp) ...@@ -858,15 +858,15 @@ static int sev_misc_init(struct psp_device *psp)
return 0; return 0;
} }
static int sev_init(struct psp_device *psp) static int psp_check_sev_support(struct psp_device *psp)
{ {
/* Check if device supports SEV feature */ /* Check if device supports SEV feature */
if (!(ioread32(psp->io_regs + psp->vdata->feature_reg) & 1)) { if (!(ioread32(psp->io_regs + psp->vdata->feature_reg) & 1)) {
dev_dbg(psp->dev, "device does not support SEV\n"); dev_dbg(psp->dev, "psp does not support SEV\n");
return 1; return -ENODEV;
} }
return sev_misc_init(psp); return 0;
} }
int psp_dev_init(struct sp_device *sp) int psp_dev_init(struct sp_device *sp)
...@@ -891,6 +891,10 @@ int psp_dev_init(struct sp_device *sp) ...@@ -891,6 +891,10 @@ int psp_dev_init(struct sp_device *sp)
psp->io_regs = sp->io_map; psp->io_regs = sp->io_map;
ret = psp_check_sev_support(psp);
if (ret)
goto e_disable;
/* Disable and clear interrupts until ready */ /* Disable and clear interrupts until ready */
iowrite32(0, psp->io_regs + psp->vdata->inten_reg); iowrite32(0, psp->io_regs + psp->vdata->inten_reg);
iowrite32(-1, psp->io_regs + psp->vdata->intsts_reg); iowrite32(-1, psp->io_regs + psp->vdata->intsts_reg);
...@@ -902,7 +906,7 @@ int psp_dev_init(struct sp_device *sp) ...@@ -902,7 +906,7 @@ int psp_dev_init(struct sp_device *sp)
goto e_err; goto e_err;
} }
ret = sev_init(psp); ret = sev_misc_init(psp);
if (ret) if (ret)
goto e_irq; goto e_irq;
...@@ -923,6 +927,11 @@ int psp_dev_init(struct sp_device *sp) ...@@ -923,6 +927,11 @@ int psp_dev_init(struct sp_device *sp)
dev_notice(dev, "psp initialization failed\n"); dev_notice(dev, "psp initialization failed\n");
return ret;
e_disable:
sp->psp_data = NULL;
return ret; return ret;
} }
......
...@@ -226,8 +226,6 @@ static int sp_pci_probe(struct pci_dev *pdev, const struct pci_device_id *id) ...@@ -226,8 +226,6 @@ static int sp_pci_probe(struct pci_dev *pdev, const struct pci_device_id *id)
if (ret) if (ret)
goto e_err; goto e_err;
dev_notice(dev, "enabled\n");
return 0; return 0;
e_err: e_err:
...@@ -246,8 +244,6 @@ static void sp_pci_remove(struct pci_dev *pdev) ...@@ -246,8 +244,6 @@ static void sp_pci_remove(struct pci_dev *pdev)
sp_destroy(sp); sp_destroy(sp);
sp_free_irqs(sp); sp_free_irqs(sp);
dev_notice(dev, "disabled\n");
} }
#ifdef CONFIG_PM #ifdef CONFIG_PM
......
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