[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 20/20] sdcard: add an enum for the SD PHY Spe
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH v4 20/20] sdcard: add an enum for the SD PHY Spec version |
Date: |
Sat, 3 Mar 2018 08:50:19 -0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 |
On 02/22/2018 11:26 AM, Peter Maydell wrote:
> On 15 February 2018 at 22:13, Philippe Mathieu-Daudé <address@hidden> wrote:
>> So far this device intends to model the Spec v1.10
>>
>> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
>> Reviewed-by: Alistair Francis <address@hidden>
>> ---
>> hw/sd/sd.c | 7 +++++++
>> 1 file changed, 7 insertions(+)
>>
>> diff --git a/hw/sd/sd.c b/hw/sd/sd.c
>> index b9429b06ca..d4565626ce 100644
>> --- a/hw/sd/sd.c
>> +++ b/hw/sd/sd.c
>> @@ -47,6 +47,11 @@
>>
>> //#define DEBUG_SD 1
>>
>> +typedef enum {
>> + SD_PHY_SPEC_VER_1_10 = 110,
>> + SD_PHY_SPEC_VER_2_00 = 200, /* not yet supported */
>> +} sd_phy_spec_ver_t;
>> +
>> typedef enum {
>> sd_r0 = 0, /* no response */
>> sd_r1, /* normal response command */
>> @@ -122,6 +127,7 @@ struct SDState {
>> qemu_irq inserted_cb;
>> QEMUTimer *ocr_power_timer;
>> const char *proto_name;
>> + int spec_version;
>> bool enable;
>> uint8_t dat_lines;
>> bool cmd_line;
>> @@ -2169,6 +2175,7 @@ static void sd_realize(DeviceState *dev, Error **errp)
>> int ret;
>>
>> sd->proto_name = sd->spi ? "SPI" : "SD";
>> + sd->spec_version = SD_PHY_SPEC_VER_1_10;
>>
>> if (sd->blk && blk_is_read_only(sd->blk)) {
>> error_setg(errp, "Cannot use read-only drive as SD card");
>
> I think I'd prefer to see this patch with the ones that actually use
> the field -- as it stands it's never used.
Indeed I see, thanks.
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH v4 20/20] sdcard: add an enum for the SD PHY Spec version,
Philippe Mathieu-Daudé <=