Commit 6dd3a164 authored by Philippe Mathieu-Daudé's avatar Philippe Mathieu-Daudé
Browse files

hw/sd/sdcard: Simplify realize() a bit 



We don't need to check if sd->blk is set twice.

Reviewed-by: default avatarPeter Maydell <peter.maydell@linaro.org>
Signed-off-by: default avatarPhilippe Mathieu-Daudé <f4bug@amsat.org>
Reviewed-by: default avatarAlistair Francis <alistair.francis@wdc.com>
Message-Id: <20200630133912.9428-18-f4bug@amsat.org>
parent 9157dd59

hw/sd/sd.c

+5 −5
Original line number Diff line number Diff line
@@ -2105,12 +2105,12 @@ static void sd_realize(DeviceState *dev, Error **errp) 
        return;

    }



    if (sd->blk && blk_is_read_only(sd->blk)) {

    if (sd->blk) {

        if (blk_is_read_only(sd->blk)) {

            error_setg(errp, "Cannot use read-only drive as SD card");

            return;

        }



    if (sd->blk) {

        ret = blk_set_perm(sd->blk, BLK_PERM_CONSISTENT_READ | BLK_PERM_WRITE,

                           BLK_PERM_ALL, errp);

        if (ret < 0) {