From: Bin Meng Date: Tue, 16 Feb 2021 03:46:52 +0000 (+0800) Subject: hw/sd: sdhci: Simplify updating s->prnsts in sdhci_sdma_transfer_multi_blocks() X-Git-Tag: v6.0.0-rc0~73^2~1 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=8bc1f1aa51d32c3184e7b19d5b94c35ecc06f056;p=thirdparty%2Fqemu.git hw/sd: sdhci: Simplify updating s->prnsts in sdhci_sdma_transfer_multi_blocks() s->prnsts is updated in both branches of the if () else () statement. Move the common bits outside so that it is cleaner. Signed-off-by: Bin Meng Tested-by: Alexander Bulekov Reviewed-by: Alexander Bulekov Reviewed-by: Philippe Mathieu-Daudé Message-Id: <1613447214-81951-5-git-send-email-bmeng.cn@gmail.com> Signed-off-by: Philippe Mathieu-Daudé --- diff --git a/hw/sd/sdhci.c b/hw/sd/sdhci.c index 8ffa53999d8..9acf4467a32 100644 --- a/hw/sd/sdhci.c +++ b/hw/sd/sdhci.c @@ -596,9 +596,9 @@ static void sdhci_sdma_transfer_multi_blocks(SDHCIState *s) page_aligned = true; } + s->prnsts |= SDHC_DATA_INHIBIT | SDHC_DAT_LINE_ACTIVE; if (s->trnmod & SDHC_TRNS_READ) { - s->prnsts |= SDHC_DOING_READ | SDHC_DATA_INHIBIT | - SDHC_DAT_LINE_ACTIVE; + s->prnsts |= SDHC_DOING_READ; while (s->blkcnt) { if (s->data_count == 0) { sdbus_read_data(&s->sdbus, s->fifo_buffer, block_size); @@ -625,8 +625,7 @@ static void sdhci_sdma_transfer_multi_blocks(SDHCIState *s) } } } else { - s->prnsts |= SDHC_DOING_WRITE | SDHC_DATA_INHIBIT | - SDHC_DAT_LINE_ACTIVE; + s->prnsts |= SDHC_DOING_WRITE; while (s->blkcnt) { begin = s->data_count; if (((boundary_count + begin) < block_size) && page_aligned) {