Attention is currently required from: Wenbin Mei. Hung-Te Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/51966 )
Change subject: soc/mediatek: add new driver 'msdc' for eMMC ......................................................................
Patch Set 5:
(1 comment)
File src/soc/mediatek/common/include/soc/msdc.h:
https://review.coreboot.org/c/coreboot/+/51966/comment/1d54285a_92571e8d PS5, Line 18: #define MSDC_CFG 0x0 : #define MSDC_IOCON 0x04 : #define MSDC_PS 0x08 : #define MSDC_INT 0x0c : #define MSDC_INTEN 0x10 : #define MSDC_FIFOCS 0x14 : #define MSDC_TXDATA 0x18 : #define MSDC_RXDATA 0x1c : #define SDC_CFG 0x30 : #define SDC_CMD 0x34 : #define SDC_ARG 0x38 : #define SDC_STS 0x3c : #define SDC_RESP0 0x40 : #define SDC_RESP1 0x44 : #define SDC_RESP2 0x48 : #define SDC_RESP3 0x4c : #define SDC_BLK_NUM 0x50 : #define SDC_ADV_CFG0 0x64 : #define EMMC_IOCON 0x7c : #define SDC_ACMD_RESP 0x80 : #define DMA_SA_H4BIT 0x8c : #define MSDC_DMA_SA 0x90 : #define MSDC_DMA_CTRL 0x98 : #define MSDC_DMA_CFG 0x9c : #define MSDC_PATCH_BIT 0xb0 : #define MSDC_PATCH_BIT1 0xb4 : #define MSDC_PATCH_BIT2 0xb8 : #define MSDC_PAD_TUNE 0xec : #define MSDC_PAD_TUNE0 0xf0 : #define PAD_DS_TUNE 0x188 : #define PAD_CMD_TUNE 0x18c : #define EMMC51_CFG0 0x204 : #define EMMC50_CFG0 0x208 : #define EMMC50_CFG1 0x20c : #define EMMC50_CFG3 0x220 : #define SDC_FIFO_CFG 0x228 : Will you consider changing this to a struct?
struct msdc_regs { u32 cfg; u32 iocon; ... }