[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 0/7] include/hw/pci include/hw/cxl: Clean up includes
From: |
Markus Armbruster |
Subject: |
Re: [PATCH v2 0/7] include/hw/pci include/hw/cxl: Clean up includes |
Date: |
Thu, 22 Dec 2022 11:48:25 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) |
Bernhard Beschow <shentey@gmail.com> writes:
> Am 22. Dezember 2022 10:03:23 UTC schrieb Markus Armbruster
> <armbru@redhat.com>:
>>Back in 2016, we discussed[1] rules for headers, and these were
>>generally liked:
>>
>>1. Have a carefully curated header that's included everywhere first. We
>> got that already thanks to Peter: osdep.h.
>>
>>2. Headers should normally include everything they need beyond osdep.h.
>> If exceptions are needed for some reason, they must be documented in
>> the header. If all that's needed from a header is typedefs, put
>> those into qemu/typedefs.h instead of including the header.
>>
>>3. Cyclic inclusion is forbidden.
>
> Sounds like these -- useful and sane -- rules belong in QEMU's coding style.
> What about putting them there for easy reference?
Makes sense. I'll see what I can do. Thanks!
- [PATCH v2 0/7] include/hw/pci include/hw/cxl: Clean up includes, Markus Armbruster, 2022/12/22
- [PATCH v2 6/7] include/hw/pci: Include hw/pci/pci.h where needed, Markus Armbruster, 2022/12/22
- [PATCH v2 3/7] include/hw/cxl: Include hw/cxl/*.h where needed, Markus Armbruster, 2022/12/22
- [PATCH v2 5/7] include/hw/pci: Split pci_device.h off pci.h, Markus Armbruster, 2022/12/22
- [PATCH v2 4/7] include/hw/pci: Clean up a few things checkpatch.pl would flag, Markus Armbruster, 2022/12/22
- [PATCH v2 1/7] include/hw/pci: Break inclusion loop pci_bridge.h and cxl.h, Markus Armbruster, 2022/12/22
- [PATCH v2 2/7] include/hw/cxl: Move typedef PXBDev to cxl.h, and put it to use, Markus Armbruster, 2022/12/22
- [PATCH v2 7/7] include/hw/cxl: Break inclusion loop cxl_pci.h and cxl_cdat_h, Markus Armbruster, 2022/12/22
- Re: [PATCH v2 0/7] include/hw/pci include/hw/cxl: Clean up includes, Bernhard Beschow, 2022/12/22
- Re: [PATCH v2 0/7] include/hw/pci include/hw/cxl: Clean up includes,
Markus Armbruster <=