[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V4 11/12] SD card: introduce "spi" property for
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [PATCH V4 11/12] SD card: introduce "spi" property for SD card objects |
Date: |
Tue, 31 Jul 2012 14:19:44 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:14.0) Gecko/20120713 Thunderbird/14.0 |
Am 31.07.2012 11:54, schrieb Markus Armbruster:
> Igor Mitsyanko <address@hidden> writes:
>
>> And drop passing is_spi argument to SDCardClass::init function.
>> "spi" property could be set only while SD card object is not
>> attached to any BlockDriverState.
>> It defaults to "false".
>>
>> Signed-off-by: Igor Mitsyanko <address@hidden>
>> Cc: Paul Brook <address@hidden>
>> ---
>> hw/sd.c | 33 +++++++++++++++++++++++++++++++--
>> hw/sd.h | 8 ++++++--
>> 2 files changed, 37 insertions(+), 4 deletions(-)
>>
>> diff --git a/hw/sd.c b/hw/sd.c
>> index fe2c138..611e1f3 100644
>> --- a/hw/sd.c
>> +++ b/hw/sd.c
>> @@ -491,10 +491,9 @@ static const VMStateDescription sd_vmstate = {
>> whether card should be in SSI or MMC/SD mode. It is also up to the
>> board to ensure that ssi transfers only occur when the chip select
>> is asserted. */
>> -static void sd_init_card(SDState *sd, BlockDriverState *bs, bool is_spi)
>> +static void sd_init_card(SDState *sd, BlockDriverState *bs)
>> {
>> sd->buf = qemu_blockalign(bs, 512);
>> - sd->spi = is_spi;
>> sd->enable = true;
>> sd_reset(sd, bs);
>> if (sd->bdrv) {
>> @@ -1766,10 +1765,40 @@ static void sd_class_init(ObjectClass *klass, void
>> *data)
>> k->enable = sd_enable_card;
>> }
>>
>> +static void sd_is_spi(Object *obj, Visitor *v, void *opaque,
>> + const char *name, Error **errp)
>> +{
>> + SDState *sd = SD_CARD(obj);
>> +
>> + visit_type_bool(v, &sd->spi, name, errp);
>> +}
>> +
>> +static void sd_set_spimode(Object *obj, Visitor *v, void *opaque,
>> + const char *name, Error **errp)
>> +{
>> + SDState *sd = SD_CARD(obj);
>> +
>> + if (sd->bdrv) {
>> + error_set(errp, QERR_DEVICE_IN_USE, bdrv_get_device_name(sd->bdrv));
>> + } else {
>> + visit_type_bool(v, &sd->spi, name, errp);
>> + }
>> +}
>> +
>> +static void sd_initfn(Object *obj)
>> +{
>> + SDState *sd = SD_CARD(obj);
>> +
>> + sd->spi = false;
>> + object_property_add(obj, "spi", "boolean", sd_is_spi, sd_set_spimode,
>> + NULL, NULL, NULL);
>> +}
>> +
>> static const TypeInfo sd_type_info = {
>> .name = TYPE_SD_CARD,
>> .parent = TYPE_OBJECT,
>> .instance_size = sizeof(SDState),
>> + .instance_init = sd_initfn,
>> .class_init = sd_class_init,
>> .class_size = sizeof(SDClass)
>> };
>
> I suspect this would be much simpler the declarative way qdevs normally
> use. For an example, check out scsi_hd_properties[] and its use in
> hw/scsi-disk.c.
[snip]
For static properties bool support was missing some time ago...
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
- [Qemu-devel] [PATCH V4 09/12] hw/sd.c: convert SD state to QOM object, (continued)
- [Qemu-devel] [PATCH V4 09/12] hw/sd.c: convert SD state to QOM object, Igor Mitsyanko, 2012/07/27
- Re: [Qemu-devel] [PATCH V4 09/12] hw/sd.c: convert SD state to QOM object, Markus Armbruster, 2012/07/31
- Re: [Qemu-devel] [PATCH V4 09/12] hw/sd.c: convert SD state to QOM object, Peter Maydell, 2012/07/31
- Re: [Qemu-devel] [PATCH V4 09/12] hw/sd.c: convert SD state to QOM object, Igor Mitsyanko, 2012/07/31
- Re: [Qemu-devel] [PATCH V4 09/12] hw/sd.c: convert SD state to QOM object, Markus Armbruster, 2012/07/31
- Re: [Qemu-devel] [PATCH V4 09/12] hw/sd.c: convert SD state to QOM object, Peter Maydell, 2012/07/31
- Re: [Qemu-devel] [PATCH V4 09/12] hw/sd.c: convert SD state to QOM object, Igor Mitsyanko, 2012/07/31
[Qemu-devel] [PATCH V4 06/12] hw/sd.c: make sd_dataready() return bool, Igor Mitsyanko, 2012/07/27
[Qemu-devel] [PATCH V4 11/12] SD card: introduce "spi" property for SD card objects, Igor Mitsyanko, 2012/07/27
[Qemu-devel] [PATCH V4 08/12] hw/sd.c: add SD card save/load support, Igor Mitsyanko, 2012/07/27
[Qemu-devel] [PATCH V4 10/12] SD card users: optimize access to SDClass methods, Igor Mitsyanko, 2012/07/27