Skip to content
  • Alexander Graf's avatar
    checkpatch: don't error out on },{ lines · 9fbe4784
    Alexander Graf authored
    
    
    When having code like this:
    
        static PCIDeviceInfo piix_ide_info[] = {
            {
                .qdev.name    = "piix3-ide",
                .qdev.size    = sizeof(PCIIDEState),
                .qdev.no_user = 1,
                .no_hotplug   = 1,
                .init         = pci_piix_ide_initfn,
                .vendor_id    = PCI_VENDOR_ID_INTEL,
                .device_id    = PCI_DEVICE_ID_INTEL_82371SB_1,
                .class_id     = PCI_CLASS_STORAGE_IDE,
            },{
                .qdev.name    = "piix4-ide",
                .qdev.size    = sizeof(PCIIDEState),
                .qdev.no_user = 1,
                .no_hotplug   = 1,
                .init         = pci_piix_ide_initfn,
                .vendor_id    = PCI_VENDOR_ID_INTEL,
                .device_id    = PCI_DEVICE_ID_INTEL_82371AB,
                .class_id     = PCI_CLASS_STORAGE_IDE,
            },{
                /* end of list */
            }
        };
    
    checkpatch currently errors out, claiming that spaces need to follow
    commas. However, this particular style of defining structs is pretty
    common in qemu code and very readable. So let's declare it as supported
    for the above case.
    
    Reported-by: default avatarKevin Wolf <kwolf@redhat.com>
    Signed-off-by: default avatarAlexander Graf <agraf@suse.de>
    9fbe4784