[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [FOR 0.12 PATCH] qdev: Replace device names containing
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [FOR 0.12 PATCH] qdev: Replace device names containing whitespace |
Date: |
Sat, 12 Dec 2009 19:04:58 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.1 (gnu/linux) |
Blue Swirl <address@hidden> writes:
> On Wed, Dec 9, 2009 at 11:43 AM, Markus Armbruster <address@hidden> wrote:
>> Ian Molton <address@hidden> writes:
>>
>>> Markus Armbruster wrote:
>>>
>>>> The place for verbose device names is DeviceInfo member desc. The
>>>> name should be short & sweet.
>>>
>>> Agreed, however...
>>>
>>> Why do these (maybe others) get caps in their names? they dont look
>>> right to me, compared to the others with nice names like usb-serial,
>>> piix-ide, or cirrus-vga.
>>>
>>>> - sysbus_register_dev("Uni-north main", sizeof(UNINState),
>>>> + sysbus_register_dev("Uni-north-main", sizeof(UNINState),
>>>> pci_unin_main_init_device);
>>>> pci_qdev_register(&unin_main_pci_host_info);
>>>> - sysbus_register_dev("DEC 21154", sizeof(UNINState),
>>>> + sysbus_register_dev("DEC-21154", sizeof(UNINState),
>>>> pci_dec_21154_init_device);
>>>> pci_qdev_register(&dec_21154_pci_host_info);
>>>> - sysbus_register_dev("Uni-north AGP", sizeof(UNINState),
>>>> + sysbus_register_dev("Uni-north-AGP", sizeof(UNINState),
>>>> pci_unin_agp_init_device);
>>>> pci_qdev_register(&unin_agp_pci_host_info);
>>>> - sysbus_register_dev("Uni-north internal", sizeof(UNINState),
>>>> + sysbus_register_dev("Uni-north-internal", sizeof(UNINState),
>>>> pci_unin_internal_init_device);
>>>> pci_qdev_register(&unin_internal_pci_host_info);
>>>> }
>>
>> I can downcase them. Blue Swirl, any objections?
>>
>
> I'd prefer the same names that are used by the device tree, like uni-n
> and uni-north-agp. Though the bridge is name poorly, just
> "pci-bridge".
>
> For example:
> http://penguinppc.org/historical/dev-trees-html/imac_400_1.html
If I read that document correctly, we have
old qdev name device tree name
Uni-north main uni-n
Uni-north AGP uni-north-agp
Uni-north internal pci
DEC 21154 pci-bridge
Do you want me to respin my patch? If yes, the best way to make me do
exactly what you want is to tell me exactly what you want. In this
case, tell me the names you want.