Hi,
2016-11-30 13:19 GMT+01:00 Ard Biesheuvel ard.biesheuvel@linaro.org:
On 30 Nov 2016, at 12:01, Haojian Zhuang haojian.zhuang@linaro.org wrote:
On 30 November 2016 at 16:24, Marcin Wojtas mw@semihalf.com wrote: Hi,
A tiny remark - how about moving this driver to SdMmc?
Hi Marcin,
I tried it a few months ago. But Intel guys claimed SdMmc only supports SDHC. They don't want to support any vendor's IP that isn't compatible with SDHC.
There is a new UEFI pass through protocol for SD/MMC, and there is a driver inplementation producing this protocol based on PCI controllers that adhere to the SDHCI spec.
I guess Marcin's comment was about the protocol, and not the driver. The legacy Mmc host protocol predates the new UEFI one, and may be useful going forward for mobile platforms where the UEFI varstore needs to be hosted on MMC
For new generic inplementations, SD/MMC controller drivers should a) be based on the UEFI driver model, and b) implement the new SD/MMC pass through protocol, not the legacy Mmc host protocol
Correct. I also meant to place driver in Drivers/SdMmc directory, regardless implemetation details.
Best regards, Marcin