soc/intel/fsp_broadwell_de: Add fixed VT-d MMIO range to the resources
FSP initializes the VT-d feature on Broadwell-DE and assigns an address space to the MMIO range. coreboot's resource allocator needs to be aware of this fixed resource as otherwise the address can be assigned to a different PCI device. In this case addresses are overlapped and the VT-d range is not accessible any more. To deal with it the right way add a fixed MMIO resource to the resources list if VT-d BAR is enabled. TEST=Booted into Linux and checked coreboot log for resource assignment. Change-Id: I626ac17420eadc0b49031e850f0f40b3b221a098 Signed-off-by: Werner Zeh <werner.zeh@siemens.com> Reviewed-on: https://review.coreboot.org/28672 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Nico Huber <nico.h@gmx.de> Reviewed-by: Philipp Deppenwiese <zaolin.daisuki@gmail.com>
This commit is contained in:
parent
ca0c8e75f9
commit
1a22d3bc7e
|
@ -3,7 +3,7 @@
|
||||||
*
|
*
|
||||||
* Copyright (C) 2013 Google, Inc.
|
* Copyright (C) 2013 Google, Inc.
|
||||||
* Copyright (C) 2015-2016 Intel Corp.
|
* Copyright (C) 2015-2016 Intel Corp.
|
||||||
* Copyright (C) 2017 Siemens AG
|
* Copyright (C) 2017-2018 Siemens AG
|
||||||
*
|
*
|
||||||
* This program is free software; you can redistribute it and/or modify
|
* This program is free software; you can redistribute it and/or modify
|
||||||
* it under the terms of the GNU General Public License as published by
|
* it under the terms of the GNU General Public License as published by
|
||||||
|
@ -21,6 +21,7 @@
|
||||||
#define VTBAR_OFFSET 0x180
|
#define VTBAR_OFFSET 0x180
|
||||||
#define VTBAR_MASK 0xffffe000
|
#define VTBAR_MASK 0xffffe000
|
||||||
#define VTBAR_ENABLED 0x01
|
#define VTBAR_ENABLED 0x01
|
||||||
|
#define VTBAR_SIZE 0x2000
|
||||||
|
|
||||||
#define SMM_FEATURE_CONTROL 0x58
|
#define SMM_FEATURE_CONTROL 0x58
|
||||||
#define SMM_CPU_SAVE_EN (1 << 1)
|
#define SMM_CPU_SAVE_EN (1 << 1)
|
||||||
|
|
|
@ -17,9 +17,23 @@
|
||||||
#include <device/pci_ids.h>
|
#include <device/pci_ids.h>
|
||||||
#include <soc/pci_devs.h>
|
#include <soc/pci_devs.h>
|
||||||
#include <soc/acpi.h>
|
#include <soc/acpi.h>
|
||||||
|
#include <soc/broadwell_de.h>
|
||||||
|
|
||||||
|
|
||||||
|
static void vtd_read_resources(struct device *dev)
|
||||||
|
{
|
||||||
|
uint32_t vtbar;
|
||||||
|
|
||||||
|
/* Add fixed MMIO resource for VT-d which was set up by the FSP. */
|
||||||
|
vtbar = pci_read_config32(dev, VTBAR_OFFSET);
|
||||||
|
if (vtbar & VTBAR_ENABLED) {
|
||||||
|
mmio_resource(dev, VTBAR_OFFSET,
|
||||||
|
(vtbar & VTBAR_MASK) / KiB, VTBAR_SIZE / KiB);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
static struct device_operations vtd_ops = {
|
static struct device_operations vtd_ops = {
|
||||||
|
.read_resources = vtd_read_resources,
|
||||||
.write_acpi_tables = vtd_write_acpi_tables,
|
.write_acpi_tables = vtd_write_acpi_tables,
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue