提交 21461775 编写于 作者: T Tomasz Nowicki 提交者: Bjorn Helgaas

x86/PCI: Clarify AMD Fam10h config access restrictions comment

Clarify the comment about AMD Fam10h config access restrictions, fix typos,
and add a reference to the specification.

[bhelgaas: streamline]
Signed-off-by: NTomasz Nowicki <tomasz.nowicki@linaro.org>
Signed-off-by: NHanjun Guo <hanjun.guo@linaro.org>
Signed-off-by: NBjorn Helgaas <bhelgaas@google.com>
Tested-by: NSuravee Suthikulpanit <Suravee.Suthikulpanit@amd.com>
上级 4066df63
...@@ -151,11 +151,11 @@ extern struct list_head pci_mmcfg_list; ...@@ -151,11 +151,11 @@ extern struct list_head pci_mmcfg_list;
#define PCI_MMCFG_BUS_OFFSET(bus) ((bus) << 20) #define PCI_MMCFG_BUS_OFFSET(bus) ((bus) << 20)
/* /*
* AMD Fam10h CPUs are buggy, and cannot access MMIO config space * On AMD Fam10h CPUs, all PCI MMIO configuration space accesses must use
* on their northbrige except through the * %eax register. As such, you MUST * %eax. No other source or target registers may be used. The following
* NOT use normal IOMEM accesses, you need to only use the magic mmio-config * mmio_config_* accessors enforce this. See "BIOS and Kernel Developer's
* accessor functions. * Guide (BKDG) For AMD Family 10h Processors", rev. 3.48, sec 2.11.1,
* In fact just use pci_config_*, nothing else please. * "MMIO Configuration Coding Requirements".
*/ */
static inline unsigned char mmio_config_readb(void __iomem *pos) static inline unsigned char mmio_config_readb(void __iomem *pos)
{ {
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册