[U-Boot] [PATCH 1/7] mmc: Add some usefull macro definition

[Zhang Haijun]

Hi, Jaehoon


? 2013/11/8 9:15, Jaehoon Chung ??:
> Dear Haijun,
>
> On 11/05/2013 03:23 PM, Haijun Zhang wrote:
>> Add command class define.
>> Add mmc erase and secure erase define.
>> Add secure erase and trim support bit define.
>>
>> Signed-off-by: Haijun Zhang <haijun.zhang at freescale.com>
>> ---
>>   include/mmc.h | 49 +++++++++++++++++++++++++++++++++++++++++++++++++
>>   1 file changed, 49 insertions(+)
>>
>> diff --git a/include/mmc.h b/include/mmc.h
>> index cb558da..26fab07 100644
>> --- a/include/mmc.h
>> +++ b/include/mmc.h
>> @@ -53,6 +53,7 @@
>>   #define COMM_ERR		-18 /* Communications Error */
>>   #define TIMEOUT			-19
>>   #define IN_PROGRESS		-20 /* operation is in progress */
>> +#define NOT_SUPPORT		-21 /* Operation is not support */
>>   
>>   #define MMC_CMD_GO_IDLE_STATE		0
>>   #define MMC_CMD_SEND_OP_COND		1
>> @@ -105,6 +106,39 @@
>>   #define OCR_VOLTAGE_MASK	0x007FFF80
>>   #define OCR_ACCESS_MODE		0x60000000
>>   
>> +/*
>> + * Card Command Classes (CCC)
>> + *
>> + * (0) Basic protocol functions (CMD0,1,2,3,4,7,9,10,12,13,15)
>> + * (and for SPI, CMD58,59)
>> + * (1) Stream read commands (CMD11)
>> + * (2) Block read commands (CMD16,17,18)
>> + * (3) Stream write commands (CMD20)
>> + * (4) Block write commands (CMD16,24,25,26,27)
>> + * (5) Ability to erase blocks (CMD32,33,34,35,36,37,38,39)
>> + * (6) Able to write protect blocks (CMD28,29,30)
>> + * (7) Able to lock down card (CMD16,CMD42)
>> + * (8) Application specific (CMD55,56,57,ACMD*)
>> + * (9) I/O mode (CMD5,39,40,52,53)
>> + * (10) High speed switch (CMD6,34,35,36,37,50)
>> + */
>> +#define CCC_BASIC		(1<<0)
>> +#define CCC_STREAM_READ		(1<<1)
>> +#define CCC_BLOCK_READ		(1<<2)
>> +#define CCC_STREAM_WRITE	(1<<3)
>> +#define CCC_BLOCK_WRITE		(1<<4)
>> +#define CCC_ERASE		(1<<5)
>> +#define CCC_WRITE_PROT		(1<<6)
>> +#define CCC_LOCK_CARD		(1<<7)
>> +#define CCC_APP_SPEC		(1<<8)
>> +#define CCC_IO_MODE		(1<<9)
>> +#define CCC_SWITCH		(1<<10)
>> +
>> +#define MMC_ERASE_ARG           0x00000000
>> +#define MMC_SECURE_ERASE_ARG    0x80000000
>> +#define MMC_TRIM_ARG            0x00000001
>> +#define MMC_DISCARD_ARG         0x00000003
>> +
>>   #define SECURE_ERASE		0x80000000
>>   
>>   #define MMC_STATUS_MASK		(~0x0206BF7F)
>> @@ -160,8 +194,12 @@
>>   #define EXT_CSD_CARD_TYPE		196	/* RO */
>>   #define EXT_CSD_SEC_CNT			212	/* RO, 4 bytes */
>>   #define EXT_CSD_HC_WP_GRP_SIZE		221	/* RO */
>> +#define EXT_CSD_REL_WR_SEC_C		222	/* RO */
>> +#define EXT_CSD_ERASE_TIMEOUT_MULT	223	/* RO */
>>   #define EXT_CSD_HC_ERASE_GRP_SIZE	224	/* RO */
>>   #define EXT_CSD_BOOT_MULT		226	/* RO */
>> +#define EXT_CSD_SEC_ERASE_MULT		230	/* RO */
>> +#define EXT_CSD_SEC_FEATURE_SUPPORT	231	/* RO */
>>   
>>   /*
>>    * EXT_CSD field definitions
>> @@ -178,6 +216,12 @@
>>   #define EXT_CSD_BUS_WIDTH_4	1	/* Card is in 4 bit mode */
>>   #define EXT_CSD_BUS_WIDTH_8	2	/* Card is in 8 bit mode */
>>   
>> +#define EXT_CSD_SEC_ER_EN	(1<<0)
> Where is this bit defined?
EXT_CSD[231] bit 0 (eMMC 4.41 and eMMC4.5  or later?)
>
>> +#define EXT_CSD_SEC_BD_BLK_EN	(1<<2)
>> +#define EXT_CSD_SEC_GB_CL_EN	(1<<4)
>> +#define EXT_CSD_SEC_SANITIZE	(1<<6)	/* v4.5 only */
> Which version do you refer?
> SEC_SANITIZE is only supported v4.5? then v4.51 or v5.0?
> I think that Comment need to modify.
> /* Since v4.5 */
Thanks, I change it.
>
>> +
>> +
> remove the empty line.
Thanks.
>
> Best Regards,
> Jaehoon Chung
>>   #define EXT_CSD_BOOT_ACK_ENABLE			(1 << 6)
>>   #define EXT_CSD_BOOT_PARTITION_ENABLE		(1 << 3)
>>   #define EXT_CSD_PARTITION_ACCESS_ENABLE		(1 << 0)
>> @@ -268,10 +312,15 @@ struct mmc {
>>   	ushort rca;
>>   	char part_config;
>>   	char part_num;
>> +	ushort cmdclass;
>>   	uint tran_speed;
>>   	uint read_bl_len;
>>   	uint write_bl_len;
>>   	uint erase_grp_size;
>> +	uint erase_timeout_mult;
>> +	char sec_feature_support;
>> +	uint sec_erase_mult;
>> +	uint sec_erase_timeout;
>>   	u64 capacity;
>>   	u64 capacity_user;
>>   	u64 capacity_boot;
>>
>

-- 
Thanks & Regards
Haijun.