On Sun, Nov 27, 2016 at 10:55:41PM +0100, Marcin Wojtas wrote:
Armada 70x0 SoC comprise two Sd/Mmc controllers. This commit adds platform information about the second one, which can be enabled using PciEmulation mechanism and appropriate PCD entries.
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Marcin Wojtas mw@semihalf.com
Platforms/Marvell/PciEmulation/PciEmulation.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/Platforms/Marvell/PciEmulation/PciEmulation.c b/Platforms/Marvell/PciEmulation/PciEmulation.c index f0203f8..491b886 100644 --- a/Platforms/Marvell/PciEmulation/PciEmulation.c +++ b/Platforms/Marvell/PciEmulation/PciEmulation.c @@ -70,10 +70,10 @@ STATIC PCIE_PLATFORM_DESC mA70x0PlatDescTemplate = { { 0xF2540000 }, { SIZE_8KB }, { NonDiscoverableDeviceDmaTypeCoherent },
- 1, // SDHCI
- { 0xF06E0000 },
- { SIZE_1KB },
- { NonDiscoverableDeviceDmaTypeCoherent }
- 2, // SDHCI
- { 0xF06E0000, 0xF2780000 },
- { SIZE_1KB, SIZE_1KB },
- { NonDiscoverableDeviceDmaTypeCoherent, NonDiscoverableDeviceDmaTypeCoherent }
So ... it looks somewhat counterintuitive to me to add this before the driver which provides support for it. (And yes, I should probably have tweaked on this as part of your non-discoverable set)). Is there a strong reason not to squash this single hunk into 7/7?
Regards,
Leif
}; // -- 1.8.3.1