hw/sd/milkymist-memcard: expose a SDBus and connect the SDCard to it
using the sdbus_*() API. Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org> Reviewed-by: Alistair Francis <alistair.francis@xilinx.com> Acked-by: Michael Walle <michael@walle.cc> Message-id: 20180216022933.10945-4-f4bug@amsat.org Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
This commit is contained in:
parent
85fd6e5db1
commit
3d0369ba49
@ -69,7 +69,7 @@ struct MilkymistMemcardState {
|
||||
SysBusDevice parent_obj;
|
||||
|
||||
MemoryRegion regs_region;
|
||||
SDState *card;
|
||||
SDBus sdbus;
|
||||
|
||||
int command_write_ptr;
|
||||
int response_read_ptr;
|
||||
@ -105,7 +105,7 @@ static void memcard_sd_command(MilkymistMemcardState *s)
|
||||
req.crc = s->command[5];
|
||||
|
||||
s->response[0] = req.cmd;
|
||||
s->response_len = sd_do_command(s->card, &req, s->response+1);
|
||||
s->response_len = sdbus_do_command(&s->sdbus, &req, s->response + 1);
|
||||
s->response_read_ptr = 0;
|
||||
|
||||
if (s->response_len == 16) {
|
||||
@ -150,10 +150,10 @@ static uint64_t memcard_read(void *opaque, hwaddr addr,
|
||||
r = 0xffffffff;
|
||||
} else {
|
||||
r = 0;
|
||||
r |= sd_read_data(s->card) << 24;
|
||||
r |= sd_read_data(s->card) << 16;
|
||||
r |= sd_read_data(s->card) << 8;
|
||||
r |= sd_read_data(s->card);
|
||||
r |= sdbus_read_data(&s->sdbus) << 24;
|
||||
r |= sdbus_read_data(&s->sdbus) << 16;
|
||||
r |= sdbus_read_data(&s->sdbus) << 8;
|
||||
r |= sdbus_read_data(&s->sdbus);
|
||||
}
|
||||
break;
|
||||
case R_CLK2XDIV:
|
||||
@ -207,10 +207,10 @@ static void memcard_write(void *opaque, hwaddr addr, uint64_t value,
|
||||
if (!s->enabled) {
|
||||
break;
|
||||
}
|
||||
sd_write_data(s->card, (value >> 24) & 0xff);
|
||||
sd_write_data(s->card, (value >> 16) & 0xff);
|
||||
sd_write_data(s->card, (value >> 8) & 0xff);
|
||||
sd_write_data(s->card, value & 0xff);
|
||||
sdbus_write_data(&s->sdbus, (value >> 24) & 0xff);
|
||||
sdbus_write_data(&s->sdbus, (value >> 16) & 0xff);
|
||||
sdbus_write_data(&s->sdbus, (value >> 8) & 0xff);
|
||||
sdbus_write_data(&s->sdbus, value & 0xff);
|
||||
break;
|
||||
case R_ENABLE:
|
||||
s->regs[addr] = value;
|
||||
@ -251,10 +251,6 @@ static void milkymist_memcard_reset(DeviceState *d)
|
||||
for (i = 0; i < R_MAX; i++) {
|
||||
s->regs[i] = 0;
|
||||
}
|
||||
/* Since we're still using the legacy SD API the card is not plugged
|
||||
* into any bus, and we must reset it manually.
|
||||
*/
|
||||
device_reset(DEVICE(s->card));
|
||||
}
|
||||
|
||||
static void milkymist_memcard_init(Object *obj)
|
||||
@ -270,15 +266,23 @@ static void milkymist_memcard_init(Object *obj)
|
||||
static void milkymist_memcard_realize(DeviceState *dev, Error **errp)
|
||||
{
|
||||
MilkymistMemcardState *s = MILKYMIST_MEMCARD(dev);
|
||||
DeviceState *carddev;
|
||||
BlockBackend *blk;
|
||||
DriveInfo *dinfo;
|
||||
Error *err = NULL;
|
||||
|
||||
qbus_create_inplace(&s->sdbus, sizeof(s->sdbus), TYPE_SD_BUS,
|
||||
dev, "sd-bus");
|
||||
|
||||
/* Create and plug in the sd card */
|
||||
/* FIXME use a qdev drive property instead of drive_get_next() */
|
||||
dinfo = drive_get_next(IF_SD);
|
||||
blk = dinfo ? blk_by_legacy_dinfo(dinfo) : NULL;
|
||||
s->card = sd_init(blk, false);
|
||||
if (s->card == NULL) {
|
||||
error_setg(errp, "failed to init SD card");
|
||||
carddev = qdev_create(&s->sdbus.qbus, TYPE_SD_CARD);
|
||||
qdev_prop_set_drive(carddev, "drive", blk, &err);
|
||||
object_property_set_bool(OBJECT(carddev), true, "realized", &err);
|
||||
if (err) {
|
||||
error_setg(errp, "failed to init SD card: %s", error_get_pretty(err));
|
||||
return;
|
||||
}
|
||||
s->enabled = blk && blk_is_inserted(blk);
|
||||
|
Loading…
Reference in New Issue
Block a user