[Skiboot] [PATCH 2/6] core/pci-quirk: Remove broken comment in quirk_astbmc_vga()
Andrew Jeffery
andrew at aj.id.au
Tue Jul 17 12:02:50 AEST 2018
The comment talks about one mechanism to handle the quirk whilst the
code uses another. Avoid confusion by removing the comment.
Signed-off-by: Andrew Jeffery <andrew at aj.id.au>
---
core/pci-quirk.c | 15 ---------------
1 file changed, 15 deletions(-)
diff --git a/core/pci-quirk.c b/core/pci-quirk.c
index 1b60e921db72..983240734c16 100644
--- a/core/pci-quirk.c
+++ b/core/pci-quirk.c
@@ -25,21 +25,6 @@ static void quirk_astbmc_vga(struct phb *phb __unused,
struct dt_node *np = pd->dn;
uint32_t revision, mcr_configuration, mcr_scu_mpll, mcr_scu_strap;
- /*
- * These accesses will only work if the BMC address 0x1E6E2180 is set
- * to 0x7B, which is its default state on current systems. In future,
- * for security purposes it is proposed to configure this register to
- * disallow accesses from the host, and provide the properties that
- * the Linux ast VGA driver used through the device tree instead.
- * Here we set those properties so we can test how things would work
- * if the window into BMC memory was closed.
- *
- * If both the petitboot kernel and the host kernel have an ast driver
- * that reads properties from the device tree, setting 0x1E6E2180 to
- * 0x79 will disable the backdoor into BMC memory and the only way the
- * ast driver can operate is using the device tree properties.
- */
-
revision = ast_ahb_readl(SCU_REVISION_ID);
mcr_configuration = ast_ahb_readl(MCR_CONFIGURATION);
mcr_scu_mpll = ast_ahb_readl(MCR_SCU_MPLL);
--
2.17.1
More information about the Skiboot
mailing list