[PATCH 1/4] mmc: sandbox: Initialize the status register

Jaehoon Chung jh80.chung at samsung.com
Tue Apr 26 09:56:51 CEST 2022


Dear Sean,

On 4/22/22 21:03, Jaehoon Chung wrote:
> Hi
> 
> On 3/31/22 01:54, Sean Anderson wrote:
>> The send status command expects the status register to be returned as a
>> response. Without writing data back, whatever is on the stack will be
>> interpreted as the status register. There are a lot of fields in this
>> register, but fortunately all zeros is interpreted as "we don't support
>> anything."
>>
>> Signed-off-by: Sean Anderson <seanga2 at gmail.com>
> 
> Reviewed-by: Jaehoon Chung <jh80.chung at samsung.com>

After applied your patch, CI testing was failed.
I have been checking what's problem. So this patch set will be applied after fixing it.

Best Regards,
Jaehoon Chung

> 
> 
> Best Regards,
> Jaehoon Chung
> 
>> ---
>>
>>  drivers/mmc/sandbox_mmc.c | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/drivers/mmc/sandbox_mmc.c b/drivers/mmc/sandbox_mmc.c
>> index 451fe4a4e5..35159451e1 100644
>> --- a/drivers/mmc/sandbox_mmc.c
>> +++ b/drivers/mmc/sandbox_mmc.c
>> @@ -57,6 +57,7 @@ static int sandbox_mmc_send_cmd(struct udevice *dev, struct mmc_cmd *cmd,
>>  		break;
>>  	case MMC_CMD_SEND_STATUS:
>>  		cmd->response[0] = MMC_STATUS_RDY_FOR_DATA;
>> +		memset(data->dest, '\0', data->blocks * data->blocksize);
>>  		break;
>>  	case MMC_CMD_SELECT_CARD:
>>  		break;
> 



More information about the U-Boot mailing list