[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] hw/sd.c: Add missing state change for SD_STATUS
From: |
Aurelien Jarno |
Subject: |
Re: [Qemu-devel] [PATCH] hw/sd.c: Add missing state change for SD_STATUS, SEND_NUM_WR_BLOCKS |
Date: |
Thu, 24 Feb 2011 09:03:53 +0100 |
User-agent: |
Mutt/1.5.20 (2009-06-14) |
On Fri, Feb 18, 2011 at 01:39:00PM +0000, Peter Maydell wrote:
> The SD_STATUS and SEND_NUM_WR_BLOCKS commands are supposed to cause
> the card to send data back to the host. However sd.c was missing the
> state change to sd_sendingdata_state for these commands, with the effect
> that the Linux driver would either hang indefinitely waiting for
> nonexistent data (pl181) or read zeroes and provoke a qemu warning
> message (omap).
>
> Signed-off-by: Peter Maydell <address@hidden>
> ---
> hw/sd.c | 2 ++
> 1 files changed, 2 insertions(+), 0 deletions(-)
Thanks, applied.
> diff --git a/hw/sd.c b/hw/sd.c
> index 789ca84..5e29752 100644
> --- a/hw/sd.c
> +++ b/hw/sd.c
> @@ -1168,6 +1168,7 @@ static sd_rsp_type_t sd_app_command(SDState *sd,
> case 13: /* ACMD13: SD_STATUS */
> switch (sd->state) {
> case sd_transfer_state:
> + sd->state = sd_sendingdata_state;
> sd->data_start = 0;
> sd->data_offset = 0;
> return sd_r1;
> @@ -1182,6 +1183,7 @@ static sd_rsp_type_t sd_app_command(SDState *sd,
> case sd_transfer_state:
> *(uint32_t *) sd->data = sd->blk_written;
>
> + sd->state = sd_sendingdata_state;
> sd->data_start = 0;
> sd->data_offset = 0;
> return sd_r1;
> --
> 1.7.1
>
>
>
--
Aurelien Jarno GPG: 1024D/F1BCDB73
address@hidden http://www.aurel32.net