From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from foss.arm.com (foss.arm.com [217.140.110.172])
 by mx.groups.io with SMTP id smtpd.web11.7107.1591881579514823609
 for <devel@edk2.groups.io>;
 Thu, 11 Jun 2020 06:19:39 -0700
Authentication-Results: mx.groups.io;
 dkim=missing; spf=pass (domain: arm.com, ip: 217.140.110.172, mailfrom: ard.biesheuvel@arm.com)
Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14])
	by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 15A671FB;
	Thu, 11 Jun 2020 06:19:39 -0700 (PDT)
Received: from [192.168.1.69] (unknown [10.37.8.184])
	by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 2BB673F66F;
	Thu, 11 Jun 2020 06:19:37 -0700 (PDT)
Subject: Re: [edk2-devel] Additional configuration options on Armada/Cn913x
To: devel@edk2.groups.io, mw@semihalf.com
Cc: Leif Lindholm <leif@nuviainc.com>, Greg V <greg@unrelenting.technology>
References: <CAPv3WKfnhQE3v4TAkQXh+PcUpxAgAuq7SsDfejVQjP1PZSmD9w@mail.gmail.com>
From: "Ard Biesheuvel" <ard.biesheuvel@arm.com>
Message-ID: <05412f93-d4ca-3ae8-744f-6ba7f1ce7e8e@arm.com>
Date: Thu, 11 Jun 2020 15:19:36 +0200
User-Agent: Mozilla/5.0 (X11; Linux aarch64; rv:68.0) Gecko/20100101
 Thunderbird/68.9.0
MIME-Version: 1.0
In-Reply-To: <CAPv3WKfnhQE3v4TAkQXh+PcUpxAgAuq7SsDfejVQjP1PZSmD9w@mail.gmail.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: quoted-printable

On 6/5/20 5:19 PM, Marcin Wojtas via groups.io wrote:
> Hi,
>=20
> I'd like to ask for comments before I develop the actual code -=20
> currently we have 2 workarounds done specifically for Linux:
> a. ECAM shift in PCIE
> b. SPCR address space definition

What does this mean?

>=20
> Both above are not needed e.g. in FreeBSD and I was requested to add=20
> their optional disabling.

Disabling ECAM shift is just a matter of exposing the iATU controls to=20
the OS, right? Why do you need to disable it?

> The idea is to add dedicated variables that=20
> would optionally allow to disable the quirks, accessible via=20
> BootManager. Questions:
>=20
> - Would above be acceptable or is there a better way to handle=C2=A0suc=
h cases?
> - In case it's fine, is there a dedicated place in the BootManager menu=
=20
> to add custom switches?

Typically in a 'platform' submenu under the Device Manager

> - Are you aware of good examples for adding custom options?
>=20

You could look at SynQuacer or Raspberry Pi for inspiration on how to=20
create HII pages in the UiApp menu system.