[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 11/11] xilinx_axidma: changed device name
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [PATCH v2 11/11] xilinx_axidma: changed device name |
Date: |
Thu, 28 Jun 2012 15:05:42 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:12.0) Gecko/20120421 Thunderbird/12.0 |
Am 28.06.2012 03:08, schrieb Peter Crosthwaite:
> On Thu, Jun 28, 2012 at 9:06 AM, Alexander Graf <address@hidden> wrote:
>>
[...]
>> [...] Why did commas in names work before, but now don't? Or put
>> differently: Was this change on purpose?
[...]
>>>>> ---
>>>>> hw/xilinx.h | 2 +-
>>>>> hw/xilinx_axidma.c | 4 ++--
>>>>> 2 files changed, 3 insertions(+), 3 deletions(-)
>>>>>
>>>>> diff --git a/hw/xilinx.h b/hw/xilinx.h
>>>>> index 8f915b4..7df21eb 100644
>>>>> --- a/hw/xilinx.h
>>>>> +++ b/hw/xilinx.h
>>>>> @@ -75,7 +75,7 @@ xilinx_axiethernetdma_create(void *dmach,
>>>>> {
>>>>> DeviceState *dev = NULL;
>>>>>
>>>>> - dev = qdev_create(NULL, "xilinx,axidma");
>>>>> + dev = qdev_create(NULL, "xlnx.axi-dma");
>>
>> comma -> dot
>>
>>>>> qdev_prop_set_uint32(dev, "freqhz", freqhz);
>>>>> qdev_prop_set_ptr(dev, "dmach", dmach);
>>>>> qdev_init_nofail(dev);
>>>>> diff --git a/hw/xilinx_axidma.c b/hw/xilinx_axidma.c
>>>>> index 59373b5..f4bec37 100644
>>>>> --- a/hw/xilinx_axidma.c
>>>>> +++ b/hw/xilinx_axidma.c
>>>>> @@ -473,7 +473,7 @@ static int xilinx_axidma_init(SysBusDevice *dev)
>>>>> xlx_dma_connect_dma(s->dmach, s, axidma_push);
>>>>>
>>>>> memory_region_init_io(&s->iomem, &axidma_ops, s,
>>>>> - "axidma", R_MAX * 4 * 2);
>>>>> + "xlnx.axi-dma", R_MAX * 4 * 2);
>>>>> sysbus_init_mmio(dev, &s->iomem);
>>>>>
>>>>> for (i = 0; i < 2; i++) {
>>>>> @@ -502,7 +502,7 @@ static void axidma_class_init(ObjectClass *klass,
>>>>> void *data)
>>>>> }
>>>>>
>>>>> static TypeInfo axidma_info = {
>>>>> - .name = "xilinx,axidma",
>>>>> + .name = "xlnx.axi-dma",
>>
>> comma -> dot
>>
>
> The -device command line arg. E.G. qemu-system-microblaze -device
> xilinx.axidma,foo=bar, ...
>
> If I have ,'s in the device name i need to escape them as they are
> syntax in -device.
QOM is perfectly capable of handling commas and SPARC uses "SUNW,", too.
Using QMP in the future (Markus' RFC) should not be a problem either.
Do you really need to construct board-level devices using -device?
Maybe you have a suggestion to fix the syntax escaping issue for
Anthony's suggested -object? Because a comma is rather common in OF/FDT.
Adjusting device names to match what your Xilinx tool set uses sounds
very reasonable. But translating from "," to "." in two places does not
sound superior to translating from "," to ",," in one place to me? Maybe
I'm misunderstanding something? It might help to introduce QOM-style
TYPE_ constants, then the name is in a single location only.
Cheers,
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 v2 06/11] xilinx_ethlite: tweaked naming of ping-pong props, (continued)
- [Qemu-devel] [PATCH v2 06/11] xilinx_ethlite: tweaked naming of ping-pong props, Peter A. G. Crosthwaite, 2012/06/13
- [Qemu-devel] [PATCH v2 07/11] xilinx_ethlite: changed device name, Peter A. G. Crosthwaite, 2012/06/13
- [Qemu-devel] [PATCH v2 09/11] xilinx_axienet: changed device name, Peter A. G. Crosthwaite, 2012/06/13
- [Qemu-devel] [PATCH v2 11/11] xilinx_axidma: changed device name, Peter A. G. Crosthwaite, 2012/06/13
- Re: [Qemu-devel] [PATCH v2 11/11] xilinx_axidma: changed device name, Andreas Färber, 2012/06/15
- Re: [Qemu-devel] [PATCH v2 11/11] xilinx_axidma: changed device name, Edgar E. Iglesias, 2012/06/15
- Re: [Qemu-devel] [PATCH v2 11/11] xilinx_axidma: changed device name, Alexander Graf, 2012/06/27
- Re: [Qemu-devel] [PATCH v2 11/11] xilinx_axidma: changed device name, Peter Crosthwaite, 2012/06/27
- Re: [Qemu-devel] [PATCH v2 11/11] xilinx_axidma: changed device name,
Andreas Färber <=
- Re: [Qemu-devel] [PATCH v2 11/11] xilinx_axidma: changed device name, Peter Crosthwaite, 2012/06/28
- Re: [Qemu-devel] [PATCH v2 11/11] xilinx_axidma: changed device name, Andreas Färber, 2012/06/29
- Re: [Qemu-devel] [PATCH v2 11/11] xilinx_axidma: changed device name, Andreas Färber, 2012/06/28
- Re: [Qemu-devel] [PATCH v2 11/11] xilinx_axidma: changed device name, Edgar E. Iglesias, 2012/06/28
[Qemu-devel] [PATCH v2 08/11] xilinx_axienet: droped the c_ on parameters, Peter A. G. Crosthwaite, 2012/06/13
[Qemu-devel] [PATCH v2 10/11] xilinx_axidma: (un)reversed irq initialisation, Peter A. G. Crosthwaite, 2012/06/13
Re: [Qemu-devel] [PATCH v2 00/11] Xilinx Devs: refactored device and property names, Edgar E. Iglesias, 2012/06/15