On Fri, Apr 28, 2017 at 05:16:41PM +0100, Ard Biesheuvel wrote:
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?
Of course I do and I am grateful for that but I do not want to see these quirks upstream, please understand.
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.
ACPI can work on ARM64 on a standard subset, SBSA compliant HW platforms.
If HW deviates from the SBSA standard (and PCI) it can't/won't be bootstrapped via ACPI and I do not think we should try to make it work.
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.
Yes it is and I personally think that allowing MCFG quirks did more harm than good to the ARM ecosystem, that's my opinion, we enabled them but as soon as they disappear the better.
Thanks, Lorenzo