Support for generic ACPI based PCI host controller
Get series with b4
b4 am [email protected]Versions newest first
- v6 Latest0 of 13 patches 0 repliesNew
Patches in v6 No applies recorded yet
None of the 13 patches the cover letter announces are in this archive.
Reviews trailers per patch
No Reviewed-by, Acked-by or Tested-by trailers for v6 yet.
Cover letter [PATCH V6 00/13] Support for generic ACPI based PCI host controller
Quoted 17 lines · Tomasz Nowicki
Quoted 10 lines · Arnd Bergmann
Quoted 8 lines · On Monday 18 April 2016 15:33:24 Tomasz Nowicki wrote:
Of course we can split discussion into the two topics:
1. ECAM based ACPI host controller - patches [1-10]
2. Quirks handling and examples.IMO, it is very helpful for reviewers to go with one unified patch set and see the whole picture. Also, as you can see, quirks handling allows people to test it easily with their servers (not only QEMU but real HW).
Quoted 5 lines
I think splitting the two would help tremendously. The regular PCI support should just get merged (it should have been completed years ago when ACPI for ARM64 was first implemented), while the quirks handling contains all ugly nonstandard hacks we have to be careful about.
Quoted 3 lines
OK, so for those who want to review just "ECAM based ACPI host controller" lets consider only patches [1-10]. Patches 11-13 are well isolated and do not affect previous one. Is that ok for this series?
As much as I appreciate the idea of splitting the patches and getting something in sooner rather than later, I'm concerned because without quirk handling, this code is effectively useless on 60% or more of available server hardware platforms.
If we're going to split the review into two threads, it can't be an excuse to drag out the quirk handling for months.
--Mark Langsdorf