From: "Philippe Mathieu-Daudé" <philmd@redhat.com>
To: devel@edk2.groups.io, leif@nuviainc.com,
Ard Biesheuvel <ard.biesheuvel@arm.com>
Subject: Re: [edk2-devel] [PATCH edk2-platforms v3 4/8] Silicon/Synopsys/DesignWare: import eMMC DXE driver from EmbeddedPkg
Date: Mon, 4 May 2020 13:40:18 +0200 [thread overview]
Message-ID: <dc9994e4-a7a6-2755-6d26-b33737e1d188@redhat.com> (raw)
In-Reply-To: <20200504105818.GD21486@vanye>
On 5/4/20 12:58 PM, Leif Lindholm wrote:
> On Thu, Apr 30, 2020 at 19:16:45 +0200, Ard Biesheuvel wrote:
>> Incorporate the driver for the DesignWare eMMC host controller that is
>> based on the obsolete MMC host controller protocol that is defined in
>> EmbeddedPkg.
>>
>> This driver does not follow the UEFI driver model, and is only kept
>> around for its only users, which is the HiKey platform, which is
>> rapidly reaching obsolescence itself, at which point this driver may
>> be removed again.
>>
>> To prevent inadvertent use in new platforms, add a PCD that needs to
>> be changed from its default value in order for the driver to be
>> functional.
>>
>> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@arm.com>
>> ---
>> Silicon/Synopsys/DesignWare/DesignWare.dec | 9 +
>> Silicon/Synopsys/DesignWare/DesignWare.dsc | 2 +
>> Silicon/Synopsys/DesignWare/Drivers/DwEmmcDxe/DwEmmc.h | 132 ++++
>> Silicon/Synopsys/DesignWare/Drivers/DwEmmcDxe/DwEmmcDxe.c | 693 ++++++++++++++++++++
>> Silicon/Synopsys/DesignWare/Drivers/DwEmmcDxe/DwEmmcDxe.inf | 56 ++
>> 5 files changed, 892 insertions(+)
>>
>> diff --git a/Silicon/Synopsys/DesignWare/DesignWare.dec b/Silicon/Synopsys/DesignWare/DesignWare.dec
>> index 71ddd24b7404..f7ec7927543c 100755
>> --- a/Silicon/Synopsys/DesignWare/DesignWare.dec
>> +++ b/Silicon/Synopsys/DesignWare/DesignWare.dec
>> @@ -21,4 +21,13 @@ [Guids.common]
>> gDesignWareTokenSpaceGuid = { 0x89cb1241, 0xd283, 0x4543, { 0x88, 0x9c, 0x6b, 0x62, 0x36, 0x1a, 0x95, 0x7a } }
>> gDwEmacNetNonDiscoverableDeviceGuid = { 0x401950CD, 0xF9CD, 0x4A65, { 0xAD, 0x8E, 0x84, 0x9F, 0x3B, 0xAF, 0x23, 0x04 } }
>>
>> +[PcdsFixedAtBuild.common]
>> + #
>> + # Permit the use of obsolete drivers in this package
>> + #
>> + gDesignWareTokenSpaceGuid.PcdDwPermitObsoleteDrivers|FALSE|BOOLEAN|0x00000001
>>
>> + gDesignWareTokenSpaceGuid.PcdDwEmmcDxeBaseAddress|0x0|UINT32|0x00000002
>> + gDesignWareTokenSpaceGuid.PcdDwEmmcDxeClockFrequencyInHz|0x0|UINT32|0x00000003
>> + gDesignWareTokenSpaceGuid.PcdDwEmmcDxeMaxClockFreqInHz|0x0|UINT32|0x00000004
>> + gDesignWareTokenSpaceGuid.PcdDwEmmcDxeFifoDepth|0x0|UINT32|0x00000005
>> diff --git a/Silicon/Synopsys/DesignWare/DesignWare.dsc b/Silicon/Synopsys/DesignWare/DesignWare.dsc
>> index ad6a5ede4ae0..098bba3f7d68 100755
>> --- a/Silicon/Synopsys/DesignWare/DesignWare.dsc
>> +++ b/Silicon/Synopsys/DesignWare/DesignWare.dsc
>> @@ -20,6 +20,7 @@ [LibraryClasses]
>> ArmLib|ArmPkg/Library/ArmLib/ArmBaseLib.inf
>> BaseLib|MdePkg/Library/BaseLib/BaseLib.inf
>> BaseMemoryLib|MdePkg/Library/BaseMemoryLib/BaseMemoryLib.inf
>> + CacheMaintenanceLib|MdePkg/Library/BaseCacheMaintenanceLib/BaseCacheMaintenanceLib.inf
>> DebugLib|MdePkg/Library/BaseDebugLibNull/BaseDebugLibNull.inf
>> DevicePathLib|MdePkg/Library/UefiDevicePathLib/UefiDevicePathLib.inf
>> DmaLib|EmbeddedPkg/Library/NonCoherentDmaLib/NonCoherentDmaLib.inf
>> @@ -39,3 +40,4 @@ [LibraryClasses]
>>
>> [Components]
>> Silicon/Synopsys/DesignWare/Drivers/DwEmacSnpDxe/DwEmacSnpDxe.inf
>> + Silicon/Synopsys/DesignWare/Drivers/DwEmmcDxe/DwEmmcDxe.inf
>> diff --git a/Silicon/Synopsys/DesignWare/Drivers/DwEmmcDxe/DwEmmc.h b/Silicon/Synopsys/DesignWare/Drivers/DwEmmcDxe/DwEmmc.h
>> new file mode 100644
>> index 000000000000..09ad9b8428c4
>> --- /dev/null
>> +++ b/Silicon/Synopsys/DesignWare/Drivers/DwEmmcDxe/DwEmmc.h
>> @@ -0,0 +1,132 @@
>> +/** @file
>> +*
>> +* WARNING:
>> +* This driver fails to follow the UEFI driver model without a good
>> +* reason, and only remains in the tree because it is still used by
>> +* a small number of platforms. It will be removed when no longer used.
>> +*
>> +* Copyright (c) 2014-2017, Linaro Limited. All rights reserved.
>> +*
>> +* SPDX-License-Identifier: BSD-2-Clause-Patent
>> +*
>> +**/
>> +
>> +
>> +#ifndef __DWEMMC_H__
>> +#define __DWEMMC_H__
>> +
>> +#include <Protocol/EmbeddedGpio.h>
>> +
>> +// DW MMC Registers
>> +#define DWEMMC_CTRL ((UINT32)PcdGet32 (PcdDwEmmcDxeBaseAddress) + 0x000)
>> +#define DWEMMC_PWREN ((UINT32)PcdGet32 (PcdDwEmmcDxeBaseAddress) + 0x004)
>> +#define DWEMMC_CLKDIV ((UINT32)PcdGet32 (PcdDwEmmcDxeBaseAddress) + 0x008)
>> +#define DWEMMC_CLKSRC ((UINT32)PcdGet32 (PcdDwEmmcDxeBaseAddress) + 0x00c)
>> +#define DWEMMC_CLKENA ((UINT32)PcdGet32 (PcdDwEmmcDxeBaseAddress) + 0x010)
>> +#define DWEMMC_TMOUT ((UINT32)PcdGet32 (PcdDwEmmcDxeBaseAddress) + 0x014)
>> +#define DWEMMC_CTYPE ((UINT32)PcdGet32 (PcdDwEmmcDxeBaseAddress) + 0x018)
>> +#define DWEMMC_BLKSIZ ((UINT32)PcdGet32 (PcdDwEmmcDxeBaseAddress) + 0x01c)
>> +#define DWEMMC_BYTCNT ((UINT32)PcdGet32 (PcdDwEmmcDxeBaseAddress) + 0x020)
>> +#define DWEMMC_INTMASK ((UINT32)PcdGet32 (PcdDwEmmcDxeBaseAddress) + 0x024)
>> +#define DWEMMC_CMDARG ((UINT32)PcdGet32 (PcdDwEmmcDxeBaseAddress) + 0x028)
>> +#define DWEMMC_CMD ((UINT32)PcdGet32 (PcdDwEmmcDxeBaseAddress) + 0x02c)
>> +#define DWEMMC_RESP0 ((UINT32)PcdGet32 (PcdDwEmmcDxeBaseAddress) + 0x030)
>> +#define DWEMMC_RESP1 ((UINT32)PcdGet32 (PcdDwEmmcDxeBaseAddress) + 0x034)
>> +#define DWEMMC_RESP2 ((UINT32)PcdGet32 (PcdDwEmmcDxeBaseAddress) + 0x038)
>> +#define DWEMMC_RESP3 ((UINT32)PcdGet32 (PcdDwEmmcDxeBaseAddress) + 0x03c)
>> +#define DWEMMC_RINTSTS ((UINT32)PcdGet32 (PcdDwEmmcDxeBaseAddress) + 0x044)
>> +#define DWEMMC_STATUS ((UINT32)PcdGet32 (PcdDwEmmcDxeBaseAddress) + 0x048)
>> +#define DWEMMC_FIFOTH ((UINT32)PcdGet32 (PcdDwEmmcDxeBaseAddress) + 0x04c)
>> +#define DWEMMC_TCBCNT ((UINT32)PcdGet32 (PcdDwEmmcDxeBaseAddress) + 0x05c)
>> +#define DWEMMC_TBBCNT ((UINT32)PcdGet32 (PcdDwEmmcDxeBaseAddress) + 0x060)
>> +#define DWEMMC_DEBNCE ((UINT32)PcdGet32 (PcdDwEmmcDxeBaseAddress) + 0x064)
>> +#define DWEMMC_HCON ((UINT32)PcdGet32 (PcdDwEmmcDxeBaseAddress) + 0x070)
>> +#define DWEMMC_UHSREG ((UINT32)PcdGet32 (PcdDwEmmcDxeBaseAddress) + 0x074)
>> +#define DWEMMC_BMOD ((UINT32)PcdGet32 (PcdDwEmmcDxeBaseAddress) + 0x080)
>> +#define DWEMMC_DBADDR ((UINT32)PcdGet32 (PcdDwEmmcDxeBaseAddress) + 0x088)
>> +#define DWEMMC_IDSTS ((UINT32)PcdGet32 (PcdDwEmmcDxeBaseAddress) + 0x08c)
>> +#define DWEMMC_IDINTEN ((UINT32)PcdGet32 (PcdDwEmmcDxeBaseAddress) + 0x090)
>> +#define DWEMMC_DSCADDR ((UINT32)PcdGet32 (PcdDwEmmcDxeBaseAddress) + 0x094)
>> +#define DWEMMC_BUFADDR ((UINT32)PcdGet32 (PcdDwEmmcDxeBaseAddress) + 0x098)
>> +#define DWEMMC_CARDTHRCTL ((UINT32)PcdGet32 (PcdDwEmmcDxeBaseAddress) + 0X100)
>> +#define DWEMMC_DATA ((UINT32)PcdGet32 (PcdDwEmmcDxeBaseAddress) + 0X200)
>> +
>> +#define CMD_UPDATE_CLK 0x80202000
>> +#define CMD_START_BIT (1 << 31)
>> +
>> +#define MMC_8BIT_MODE (1 << 16)
>> +
>> +#define BIT_CMD_RESPONSE_EXPECT (1 << 6)
>> +#define BIT_CMD_LONG_RESPONSE (1 << 7)
>> +#define BIT_CMD_CHECK_RESPONSE_CRC (1 << 8)
>> +#define BIT_CMD_DATA_EXPECTED (1 << 9)
>> +#define BIT_CMD_READ (0 << 10)
>> +#define BIT_CMD_WRITE (1 << 10)
>> +#define BIT_CMD_BLOCK_TRANSFER (0 << 11)
>> +#define BIT_CMD_STREAM_TRANSFER (1 << 11)
>> +#define BIT_CMD_SEND_AUTO_STOP (1 << 12)
>> +#define BIT_CMD_WAIT_PRVDATA_COMPLETE (1 << 13)
>> +#define BIT_CMD_STOP_ABORT_CMD (1 << 14)
>> +#define BIT_CMD_SEND_INIT (1 << 15)
>> +#define BIT_CMD_UPDATE_CLOCK_ONLY (1 << 21)
>> +#define BIT_CMD_READ_CEATA_DEVICE (1 << 22)
>> +#define BIT_CMD_CCS_EXPECTED (1 << 23)
>> +#define BIT_CMD_ENABLE_BOOT (1 << 24)
>> +#define BIT_CMD_EXPECT_BOOT_ACK (1 << 25)
>> +#define BIT_CMD_DISABLE_BOOT (1 << 26)
>> +#define BIT_CMD_MANDATORY_BOOT (0 << 27)
>> +#define BIT_CMD_ALTERNATE_BOOT (1 << 27)
>> +#define BIT_CMD_VOLT_SWITCH (1 << 28)
>> +#define BIT_CMD_USE_HOLD_REG (1 << 29)
>> +#define BIT_CMD_START (1 << 31)
>> +
>> +#define DWEMMC_INT_EBE (1 << 15) /* End-bit Err */
>> +#define DWEMMC_INT_SBE (1 << 13) /* Start-bit Err */
>> +#define DWEMMC_INT_HLE (1 << 12) /* Hardware-lock Err */
>> +#define DWEMMC_INT_FRUN (1 << 11) /* FIFO UN/OV RUN */
>> +#define DWEMMC_INT_DRT (1 << 9) /* Data timeout */
>> +#define DWEMMC_INT_RTO (1 << 8) /* Response timeout */
>> +#define DWEMMC_INT_DCRC (1 << 7) /* Data CRC err */
>> +#define DWEMMC_INT_RCRC (1 << 6) /* Response CRC err */
>> +#define DWEMMC_INT_RXDR (1 << 5)
>> +#define DWEMMC_INT_TXDR (1 << 4)
>> +#define DWEMMC_INT_DTO (1 << 3) /* Data trans over */
>> +#define DWEMMC_INT_CMD_DONE (1 << 2)
>> +#define DWEMMC_INT_RE (1 << 1)
>> +
>> +#define DWEMMC_IDMAC_DES0_DIC (1 << 1)
>> +#define DWEMMC_IDMAC_DES0_LD (1 << 2)
>> +#define DWEMMC_IDMAC_DES0_FS (1 << 3)
>> +#define DWEMMC_IDMAC_DES0_CH (1 << 4)
>> +#define DWEMMC_IDMAC_DES0_ER (1 << 5)
>> +#define DWEMMC_IDMAC_DES0_CES (1 << 30)
>> +#define DWEMMC_IDMAC_DES0_OWN (1 << 31)
>> +#define DWEMMC_IDMAC_DES1_BS1(x) ((x) & 0x1fff)
>> +#define DWEMMC_IDMAC_DES2_BS2(x) (((x) & 0x1fff) << 13)
>> +#define DWEMMC_IDMAC_SWRESET (1 << 0)
>> +#define DWEMMC_IDMAC_FB (1 << 1)
>> +#define DWEMMC_IDMAC_ENABLE (1 << 7)
>> +
>> +#define EMMC_FIX_RCA 6
>> +
>> +/* bits in MMC0_CTRL */
>> +#define DWEMMC_CTRL_RESET (1 << 0)
>> +#define DWEMMC_CTRL_FIFO_RESET (1 << 1)
>> +#define DWEMMC_CTRL_DMA_RESET (1 << 2)
>> +#define DWEMMC_CTRL_INT_EN (1 << 4)
>> +#define DWEMMC_CTRL_DMA_EN (1 << 5)
>> +#define DWEMMC_CTRL_IDMAC_EN (1 << 25)
>> +#define DWEMMC_CTRL_RESET_ALL (DWEMMC_CTRL_RESET | DWEMMC_CTRL_FIFO_RESET | DWEMMC_CTRL_DMA_RESET)
>> +
>> +#define DWEMMC_STS_DATA_BUSY (1 << 9)
>> +
>> +#define DWEMMC_FIFO_TWMARK(x) (x & 0xfff)
>> +#define DWEMMC_FIFO_RWMARK(x) ((x & 0x1ff) << 16)
>> +#define DWEMMC_DMA_BURST_SIZE(x) ((x & 0x7) << 28)
>> +
>> +#define DWEMMC_CARD_RD_THR(x) ((x & 0xfff) << 16)
>> +#define DWEMMC_CARD_RD_THR_EN (1 << 0)
>> +
>> +#define DWEMMC_GET_HDATA_WIDTH(x) (((x) >> 7) & 0x7)
>> +
>> +#endif // __DWEMMC_H__
>> diff --git a/Silicon/Synopsys/DesignWare/Drivers/DwEmmcDxe/DwEmmcDxe.c b/Silicon/Synopsys/DesignWare/Drivers/DwEmmcDxe/DwEmmcDxe.c
>> new file mode 100644
>> index 000000000000..eed5fc57fc22
>> --- /dev/null
>> +++ b/Silicon/Synopsys/DesignWare/Drivers/DwEmmcDxe/DwEmmcDxe.c
>> @@ -0,0 +1,693 @@
>> +/** @file
>> + This file implement the MMC Host Protocol for the DesignWare eMMC.
>> +
>> + Copyright (c) 2014-2017, Linaro Limited. All rights reserved.
>> +
>> + SPDX-License-Identifier: BSD-2-Clause-Patent
>
> Comment only:
> This file, unlike some other .c files in this series, does not include
> the WARNING header. Feel free to fold one in before pushing.
Good catch.
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
>
> /
> Leif
>
[...]
next prev parent reply other threads:[~2020-05-04 11:40 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-04-30 17:16 [PATCH edk2-platforms v3 0/8] move obsolete platform drivers out of core edk2 Ard Biesheuvel
2020-04-30 17:16 ` [PATCH edk2-platforms v3 1/8] Omap35xxPkg/LcdGraphicsOutputDxe: add missing protocol reference Ard Biesheuvel
2020-04-30 17:16 ` [PATCH edk2-platforms v3 2/8] Platform/ARM/VExpressPkg: incorporate PL180 driver Ard Biesheuvel
2020-04-30 17:16 ` [PATCH edk2-platforms v3 3/8] Platform/ARM/JunoPkg: incorporate SiI3132 SATA controller driver Ard Biesheuvel
2020-05-04 10:57 ` Leif Lindholm
2020-04-30 17:16 ` [PATCH edk2-platforms v3 4/8] Silicon/Synopsys/DesignWare: import eMMC DXE driver from EmbeddedPkg Ard Biesheuvel
2020-05-04 10:58 ` Leif Lindholm
2020-05-04 11:40 ` Philippe Mathieu-Daudé [this message]
2020-05-04 11:44 ` [edk2-devel] " Philippe Mathieu-Daudé
2020-04-30 17:16 ` [PATCH edk2-platforms v3 5/8] Platform/HiKey: switch to relocated version of eMMC driver Ard Biesheuvel
2020-04-30 17:16 ` [PATCH edk2-platforms v3 6/8] Platform/ARM/VExpressPkg: incorporate Lan91x driver Ard Biesheuvel
2020-05-04 11:41 ` [edk2-devel] " Philippe Mathieu-Daudé
2020-05-04 11:44 ` Philippe Mathieu-Daudé
2020-04-30 17:16 ` [PATCH edk2-platforms v3 7/8] Platform/ARM/VExpressPkg: incorporate Lan9118 driver Ard Biesheuvel
2020-05-04 11:42 ` [edk2-devel] " Philippe Mathieu-Daudé
2020-05-04 11:44 ` Philippe Mathieu-Daudé
2020-04-30 17:16 ` [PATCH edk2-platforms v3 8/8] Platform/ARM/VExpressPkg: incorporate ISP 1761 USB host driver Ard Biesheuvel
2020-05-04 11:00 ` Leif Lindholm
2020-05-04 11:00 ` [PATCH edk2-platforms v3 0/8] move obsolete platform drivers out of core edk2 Leif Lindholm
2020-05-04 13:19 ` Ard Biesheuvel
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=dc9994e4-a7a6-2755-6d26-b33737e1d188@redhat.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox