[coreboot-gerrit] [S] Change in coreboot[master]: soc/amd/phoenix: Add note clarifying the eMMC controller

[Martin L Roth (Code Review)]

Martin L Roth has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/74590 )

Change subject: soc/amd/phoenix: Add note clarifying the eMMC controller
......................................................................

soc/amd/phoenix: Add note clarifying the eMMC controller

The eMMC controller is present in the silicon, and is documented in the
PPR, but is seemingly not bonded out on any of the existing sockets.
Note this, and recommend that the controller be left disabled.

Signed-off-by: Martin Roth 
Change-Id: Ie5ffd9792cbdc174d9eab7aa14d1bd6b89e1c817
---
M src/soc/amd/phoenix/chipset.cb
1 file changed, 17 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/90/74590/1

diff --git a/src/soc/amd/phoenix/chipset.cb b/src/soc/amd/phoenix/chipset.cb
index 606a7ce..6b1cced 100644
--- a/src/soc/amd/phoenix/chipset.cb
+++ b/src/soc/amd/phoenix/chipset.cb
@@ -144,5 +144,8 @@
    device mmio 0xfedce000 alias uart_2 off ops amd_uart_mmio_ops end
    device mmio 0xfedcf000 alias uart_3 off ops amd_uart_mmio_ops end
    device mmio 0xfedd1000 alias uart_4 off ops amd_uart_mmio_ops end
+	# The eMMC controller is present on Phoenix but is not currently noted
+	# as being bonded out on any socket. It's included here because it's
+	# in the PPR, but should probably not be enabled.
    device mmio 0xfedd5000 alias emmc off ops amd_emmc_mmio_ops end
 end