Re: [PATCH] hw/sd/sdhci: Block Size Register bits [14:12] is lost

["Philippe Mathieu-Daudé" <philmd@linaro.org>]

On 22/12/22 09:54, Gao, Lu wrote:
> Hello,
> 
> Just want to check whether we can have it merged? Any other comments?

Oops sorry, I see I had this patch tagged for a merge request
but I lost track of it (likely when switching emails).

Patch queued!

> 
> 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(-)