[PATCH v42 09/98] hw/sd/sdcard: Use READY_FOR_DATA definition instead of magic value

Philippe Mathieu-Daudé posted 98 patches 6 months ago
There is a newer version of this series
[PATCH v42 09/98] hw/sd/sdcard: Use READY_FOR_DATA definition instead of magic value
Posted by Philippe Mathieu-Daudé 6 months ago
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Tested-by: Cédric Le Goater <clg@redhat.com>
---
 hw/sd/sd.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/hw/sd/sd.c b/hw/sd/sd.c
index 1f37d9c93a..135b7d2e23 100644
--- a/hw/sd/sd.c
+++ b/hw/sd/sd.c
@@ -561,7 +561,7 @@ FIELD(CSR, OUT_OF_RANGE,               31,  1)
 
 static void sd_set_cardstatus(SDState *sd)
 {
-    sd->card_status = 0x00000100;
+    sd->card_status = READY_FOR_DATA;
 }
 
 static void sd_set_sdstatus(SDState *sd)
-- 
2.41.0


Re: [PATCH v42 09/98] hw/sd/sdcard: Use READY_FOR_DATA definition instead of magic value
Posted by Cédric Le Goater 6 months ago
On 6/28/24 9:00 AM, Philippe Mathieu-Daudé wrote:
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> Tested-by: Cédric Le Goater <clg@redhat.com>


Reviewed-by: Cédric Le Goater <clg@redhat.com>

Thanks,

C.


> ---
>   hw/sd/sd.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/hw/sd/sd.c b/hw/sd/sd.c
> index 1f37d9c93a..135b7d2e23 100644
> --- a/hw/sd/sd.c
> +++ b/hw/sd/sd.c
> @@ -561,7 +561,7 @@ FIELD(CSR, OUT_OF_RANGE,               31,  1)
>   
>   static void sd_set_cardstatus(SDState *sd)
>   {
> -    sd->card_status = 0x00000100;
> +    sd->card_status = READY_FOR_DATA;
>   }
>   
>   static void sd_set_sdstatus(SDState *sd)