[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
RE: [PATCH] hw/sd/sdhci: Block Size Register bits [14:12] is lost
From: |
Gao, Lu |
Subject: |
RE: [PATCH] hw/sd/sdhci: Block Size Register bits [14:12] is lost |
Date: |
Thu, 22 Dec 2022 08:54:39 +0000 |
Hello,
Just want to check whether we can have it merged? Any other comments?
Thanks a lot!
B.R.
-----Original Message-----
From: Philippe Mathieu-Daudé [mailto:philippe.mathieu.daude@gmail.com] On
Behalf Of Philippe Mathieu-Daudé
Sent: Tuesday, May 31, 2022 6:09 PM
To: Gao, Lu; qemu-devel@nongnu.org
Cc: Wen, Jianxian; Bin Meng; open list:SD (Secure Card); Alexander Bulekov;
Prasad J Pandit
Subject: Re: [PATCH] hw/sd/sdhci: Block Size Register bits [14:12] is lost
On 21/3/22 06:56, Lu Gao wrote:
> Block Size Register bits [14:12] is SDMA Buffer Boundary, it is missed
> in register write, but it is needed in SDMA transfer. e.g. it will be
> used in sdhci_sdma_transfer_multi_blocks to calculate boundary_ variables.
>
> Missing this field will cause wrong operation for different SDMA Buffer
> Boundary settings.
Fixes: d7dfca0807 ("hw/sdhci: introduce standard SD host controller")
Fixes: dfba99f17f ("hw/sdhci: Fix DMA Transfer Block Size field")
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> Signed-off-by: Lu Gao <lu.gao@verisilicon.com>
> Signed-off-by: Jianxian Wen <jianxian.wen@verisilicon.com>
> ---
> hw/sd/sdhci.c | 15 +++++++++++----
> 1 file changed, 11 insertions(+), 4 deletions(-)
>
> diff --git a/hw/sd/sdhci.c b/hw/sd/sdhci.c
> index e0bbc90344..350ceb487d 100644
> --- a/hw/sd/sdhci.c
> +++ b/hw/sd/sdhci.c
> @@ -321,6 +321,8 @@ static void sdhci_poweron_reset(DeviceState *dev)
>
> static void sdhci_data_transfer(void *opaque);
>
> +#define BLOCK_SIZE_MASK (4 * KiB - 1)
> +
> static void sdhci_send_command(SDHCIState *s)
> {
> SDRequest request;
> @@ -371,7 +373,8 @@ static void sdhci_send_command(SDHCIState *s)
>
> sdhci_update_irq(s);
>
> - if (!timeout && s->blksize && (s->cmdreg & SDHC_CMD_DATA_PRESENT)) {
> + if (!timeout && (s->blksize & BLOCK_SIZE_MASK) &&
> + (s->cmdreg & SDHC_CMD_DATA_PRESENT)) {
> s->data_count = 0;
> sdhci_data_transfer(s);
> }
> @@ -406,7 +409,6 @@ static void sdhci_end_transfer(SDHCIState *s)
> /*
> * Programmed i/o data transfer
> */
> -#define BLOCK_SIZE_MASK (4 * KiB - 1)
>
> /* Fill host controller's read buffer with BLKSIZE bytes of data from card
> */
> static void sdhci_read_block_from_card(SDHCIState *s)
> @@ -1137,7 +1139,8 @@ sdhci_write(void *opaque, hwaddr offset, uint64_t val,
> unsigned size)
> s->sdmasysad = (s->sdmasysad & mask) | value;
> MASKED_WRITE(s->sdmasysad, mask, value);
> /* Writing to last byte of sdmasysad might trigger transfer */
> - if (!(mask & 0xFF000000) && s->blkcnt && s->blksize &&
> + if (!(mask & 0xFF000000) && s->blkcnt &&
> + (s->blksize & BLOCK_SIZE_MASK) &&
> SDHC_DMA_TYPE(s->hostctl1) == SDHC_CTRL_SDMA) {
> if (s->trnmod & SDHC_TRNS_MULTI) {
> sdhci_sdma_transfer_multi_blocks(s);
> @@ -1151,7 +1154,11 @@ sdhci_write(void *opaque, hwaddr offset, uint64_t val,
> unsigned size)
> if (!TRANSFERRING_DATA(s->prnsts)) {
> uint16_t blksize = s->blksize;
>
> - MASKED_WRITE(s->blksize, mask, extract32(value, 0, 12));
> + /*
> + * [14:12] SDMA Buffer Boundary
> + * [11:00] Transfer Block Size
> + */
> + MASKED_WRITE(s->blksize, mask, extract32(value, 0, 15));
> MASKED_WRITE(s->blkcnt, mask >> 16, value >> 16);
>
> /* Limit block size to the maximum buffer size */
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- RE: [PATCH] hw/sd/sdhci: Block Size Register bits [14:12] is lost,
Gao, Lu <=