<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>
</head>
<body dir="ltr">
<div style="font-family: Consolas, Courier, monospace; font-size: 12pt; color: rgb(0, 0, 0);">
<span style="color: rgb(0, 0, 0); font-family: Consolas, Courier, monospace; font-size: 12pt;">Ard: IIRC eMMC2 is a better SDHCI-like controller. It supports more high-speed modes (e.g. 1.8V ones), resulting in better I/O performance,</span><br>
</div>
<div style="font-family: Consolas, Courier, monospace; font-size: 12pt; color: rgb(0, 0, 0);">
<br>
</div>
<div style="font-family: Consolas, Courier, monospace; font-size: 12pt; color: rgb(0, 0, 0);">
A</div>
<div id="appendonsend"></div>
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>From:</b> Ard Biesheuvel <ard.biesheuvel@arm.com><br>
<b>Sent:</b> Wednesday, January 6, 2021 3:11 AM<br>
<b>To:</b> Jeremy Linton <jeremy.linton@arm.com>; devel@edk2.groups.io <devel@edk2.groups.io><br>
<b>Cc:</b> leif@nuviainc.com <leif@nuviainc.com>; pete@akeo.ie <pete@akeo.ie>; samer.el-haj-mahmoud@arm.com <samer.el-haj-mahmoud@arm.com>; Andrei Warkentin <awarkentin@vmware.com><br>
<b>Subject:</b> Re: [PATCH v4 0/7] rpi4: Enable eMMC2 controller</font>
<div> </div>
</div>
<div class="BodyFragment"><font size="2"><span style="font-size:11pt;">
<div class="PlainText">On 1/5/21 5:34 PM, Jeremy Linton wrote:<br>
> The rpi4 has a secondary SD controller at offset<br>
> 0xfe340000. This controller appears to be a mostly<br>
> compliant SDHCI controller (a newer more bugfree<br>
> Arasan?). So the existing Arasan driver should be <br>
> bound to it. This allows the rpi4 to boot<br>
> with its normal eMMC2->SD card, Arasan->wifi <br>
> configuration that is described in the Linux DT.<br>
> <br>
<br>
Could you please explain why this is an advantage? Is it related to ACPI<br>
OSes not needing to change pinmuxing etc?<br>
<br>
<br>
> To achieve this, it appears we should be tweaking<br>
> some of the expansion gpios, and probably telling<br>
> the firmware to power everything up. To do that<br>
> the vpu mailbox headers are synced with a more recent<br>
> list of the mailbox commands, then the rpi<br>
> firmware dxe is extended to support some futher<br>
> gpio/power commands. Once that is complete we tweak<br>
> the arasan driver to use an alternate register base,<br>
> add a workaround for a known clock crossing bug, and<br>
> set the card voltage.<br>
> <br>
> Of note, this set does _NOT_ change the HID/CID's<br>
> and add the additional eMMC2 controller to the<br>
> DSDT table. That remains an open item waiting<br>
> for a proper set of device ids.<br>
> <br>
> v3->v4: Commit message tweaks, variable rename.<br>
> v1->v3: Use some mailbox defines rather than opencoded constants<br>
> <br>
> Jeremy Linton (7):<br>
> Platform/RaspberryPi: Update VPU mailbox constants<br>
> Platform/RaspberryPi: Add further mailbox helpers<br>
> Platform/RaspberryPi: Split MMC register definitions<br>
> Platform/RaspberryPi/Arasan: Add write delay and voltage/clock config<br>
> Platform/RaspberryPi/Arasan: Select the correct base frequency<br>
> Platform/RaspberryPi: Power up SD, and tweak GPIOs<br>
> Platform/RaspberryPi: Correct device path removal.<br>
> <br>
> .../Drivers/ArasanMmcHostDxe/ArasanMmcHostDxe.c | 131 ++++++++---<br>
> .../Drivers/ArasanMmcHostDxe/ArasanMmcHostDxe.h | 1 +<br>
> Platform/RaspberryPi/Drivers/ConfigDxe/ConfigDxe.c | 7 +<br>
> .../Drivers/RpiFirmwareDxe/RpiFirmwareDxe.c | 240 ++++++++++++++++++++-<br>
> .../RaspberryPi/Include/IndustryStandard/RpiMbox.h | 94 +++++++-<br>
> .../RaspberryPi/Include/Protocol/RpiFirmware.h | 25 +++<br>
> .../Library/PlatformBootManagerLib/PlatformBm.c | 2 +-<br>
> .../Bcm283x/Include/IndustryStandard/Bcm2836Sdio.h | 42 ++--<br>
> 8 files changed, 484 insertions(+), 58 deletions(-)<br>
> <br>
<br>
</div>
</span></font></div>
</body>
</html>
<div width="1" style="color:white;clear:both">_._,_._,_</div> <hr> Groups.io Links:<p> You receive all messages sent to this group. <p> <a target="_blank" href="https://edk2.groups.io/g/devel/message/69813">View/Reply Online (#69813)</a> | | <a target="_blank" href="https://groups.io/mt/79453713/1813853">Mute This Topic</a> | <a href="https://edk2.groups.io/g/devel/post">New Topic</a><br> <a href="https://edk2.groups.io/g/devel/editsub/1813853">Your Subscription</a> | <a href="mailto:devel+owner@edk2.groups.io">Contact Group Owner</a> | <a href="https://edk2.groups.io/g/devel/unsub">Unsubscribe</a> [edk2-devel-archive@redhat.com]<br> <div width="1" style="color:white;clear:both">_._,_._,_</div>