[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-arm] [PATCH for-2.12 2/2] hw/sd/bcm2835_sdhost: Don't raise sp
From: |
Peter Maydell |
Subject: |
Re: [Qemu-arm] [PATCH for-2.12 2/2] hw/sd/bcm2835_sdhost: Don't raise spurious interrupts |
Date: |
Mon, 19 Mar 2018 16:34:41 +0000 |
On 19 March 2018 at 16:15, Peter Maydell <address@hidden> wrote:
> The Linux bcm2835_sdhost driver doesn't work on QEMU, because our
> model raises spurious data interrupts. Our function
> bcm2835_sdhost_fifo_run() will flag an interrupt any time it is
> called with s->datacnt == 0, even if the host hasn't actually issued
> a data read or write command yet. This means that the driver gets a
> spurious data interrupt as soon as it enables IRQs and then does
> something else that causes us to call the fifo_run routine, like
> writing to SDHCFG, and before it does the write to SDCMD to issue the
> read. The driver's IRQ handler then spins forever complaining that
> there's no data and the SD controller isn't in a state where there's
> going to be any data:
>
> [ 41.040738] sdhost-bcm2835 3f202000.mmc: fsm 1, hsts 00000000
> [ 41.042059] sdhost-bcm2835 3f202000.mmc: fsm 1, hsts 00000000
> (continues forever).
>
> Move the interrupt flag setting to more plausible places:
> * for BUSY, raise this as soon as a BUSYWAIT command has executed
> * for DATA, raise this when the FIFO has any space free (for a write)
> or any data in it (for a read)
> * for BLOCK, raise this when the data count is 0 and we've
> actually done some reading or writing
>
> This is pure guesswork since the documentation for this hardware is
> not public, but it is sufficient to get the Linux bcm2835_sdhost
> driver to work.
>
> Signed-off-by: Peter Maydell <address@hidden>
> ---
> hw/sd/bcm2835_sdhost.c | 43 +++++++++++++++++++++++--------------------
> 1 file changed, 23 insertions(+), 20 deletions(-)
>
> diff --git a/hw/sd/bcm2835_sdhost.c b/hw/sd/bcm2835_sdhost.c
> index 79f3c5ceeb..0fd0853fa3 100644
> --- a/hw/sd/bcm2835_sdhost.c
> +++ b/hw/sd/bcm2835_sdhost.c
> @@ -137,6 +137,9 @@ static void
> bcm2835_sdhost_send_command(BCM2835SDHostState *s)
> }
> #undef RWORD
> }
> + if ((s->cmd & SDCMD_BUSYWAIT) && (s->config & SDHCFG_BUSY_IRPT_EN)) {
> + s->status |= SDHSTS_BUSY_IRPT;
> + }
> return;
>
> error:
Oops, I had a comment here that should go above this if() that I failed
to refresh into the patch before sending it:
+ /* We never really delay commands, so if this was a 'busywait' command
+ * then we've completed it now and can raise the interrupt.
+ */
thanks
-- PMM