Skip to content
  • Markus Armbruster's avatar
    hw: Replace anti-social QOM type names (again) · c455e011
    Markus Armbruster authored and Thomas Huth's avatar Thomas Huth committed
    
    
    QOM type names containing ',' result in awful UI.  We got rid of them
    in v6.0.0 (commit e178113f hw: Replace anti-social QOM type names).
    A few have crept back since:
    
        xlnx,cframe-reg
        xlnx,efuse
        xlnx,pmc-efuse-cache
        xlnx,versal-cfu-apb
        xlnx,versal-cfu-fdro
        xlnx,versal-cfu-sfr
        xlnx,versal-crl
        xlnx,versal-efuse
        xlnx,zynqmp-efuse
    
    These are all device types.  They can't be plugged with -device /
    device_add, except for "xlnx,efuse" (I'm not sure that one is
    intentional).
    
    They *can* be used with -device / device_add to request help.
    Usability is poor, though: you have to double the comma, like this:
    
        $ qemu-system-aarch64 -device xlnx,,pmc-efuse-cache,help
    
    They can also be used with -global, where you must *not* double the
    comma:
    
        $ qemu-system-aarch64 -global xlnx,efuse.drive-index=2
    
    Trap for the unwary.
    
    "xlnx,efuse", "xlnx,versal-efuse", "xlnx,pmc-efuse-cache",
    "xlnx-zynqmp-efuse" are from v6.2.0, "xlnx,versal-crl" is from v7.1.0,
    and the remainder are new.
    
    Rename them all to "xlnx-FOO", like commit e178113f did.
    
    Reported-by: Thomas Huth's avatarThomas Huth <thuth@redhat.com>
    Signed-off-by: default avatarMarkus Armbruster <armbru@redhat.com>
    Reviewed-by: Thomas Huth's avatarThomas Huth <thuth@redhat.com>
    Reviewed-by: default avatarFrancisco Iglesias <francisco.iglesias@amd.com>
    Message-ID: <20231117114457.177308-3-thuth@redhat.com>
    Reviewed-by: default avatarAlistair Francis <alistair.francis@wdc.com>
    Signed-off-by: Thomas Huth's avatarThomas Huth <thuth@redhat.com>
    c455e011