On 28 April 2017 at 17:14, Lorenzo Pieralisi lorenzo.pieralisi@arm.com wrote:
On Thu, Apr 27, 2017 at 02:17:32PM +0100, Ard Biesheuvel wrote:
This implements a quirk for the Marvell Armada80x0 running in ACPI mode, in which case the config space configuration is not 100% ECAM compatible.
Too bad, I am not keen on merging any more ECAM quirks, we have been very clear about this and it is in our best interest to keep this code out of tree - we will bootstrap ACPI PCI systems on ECAM HW/FW compliant systems, that's it (I understand it is painful but it is necessary).
I agree, and I was one of the vocal ones, remember?
The only difference is that this particular board may be very useful for development purposes, i.e., the firmware is completely open (and easily updated) and the board is cheap compared to other development hardware.
Graeme has volunteered to carry this patch in the Linaro ERP release, and if this is all we can wish for, I am not going to debate it any further (and I deliberately kept it off the linux-acpi list for now). But I do think it is useful to have the discussion.