pci: Configure expansion ROM during auto config process
Currently PCI expansion ROM address is assigned by a call to pciauto_setup_rom() outside of the pci auto config process. This does not work when expansion ROM is on a device behind PCI bridge where bridge's memory limit register was already programmed to a value that does not cover the newly assigned expansion ROM address. To fix this, we should configure the ROM address during the auto config process. Signed-off-by: Bin Meng <bmeng.cn@gmail.com> Reviewed-by: Simon Glass <sjg@chromium.org> Tested-by: Simon Glass <sjg@chromium.org> Acked-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
parent
e1783b5ba3
commit
6c89663cb1
|
@ -87,6 +87,8 @@ void pciauto_setup_device(struct pci_controller *hose,
|
|||
pci_size_t bar_size;
|
||||
u16 cmdstat = 0;
|
||||
int bar, bar_nr = 0;
|
||||
u8 header_type;
|
||||
int rom_addr;
|
||||
#ifndef CONFIG_PCI_ENUM_ONLY
|
||||
pci_addr_t bar_value;
|
||||
struct pci_region *bar_res;
|
||||
|
@ -182,38 +184,32 @@ void pciauto_setup_device(struct pci_controller *hose,
|
|||
bar_nr++;
|
||||
}
|
||||
|
||||
/* Configure the expansion ROM address */
|
||||
pci_hose_read_config_byte(hose, dev, PCI_HEADER_TYPE, &header_type);
|
||||
if (header_type != PCI_HEADER_TYPE_CARDBUS) {
|
||||
rom_addr = (header_type == PCI_HEADER_TYPE_NORMAL) ?
|
||||
PCI_ROM_ADDRESS : PCI_ROM_ADDRESS1;
|
||||
pci_hose_write_config_dword(hose, dev, rom_addr, 0xfffffffe);
|
||||
pci_hose_read_config_dword(hose, dev, rom_addr, &bar_response);
|
||||
if (bar_response) {
|
||||
bar_size = -(bar_response & ~1);
|
||||
DEBUGF("PCI Autoconfig: ROM, size=%#x, ", bar_size);
|
||||
if (pciauto_region_allocate(mem, bar_size,
|
||||
&bar_value) == 0) {
|
||||
pci_hose_write_config_dword(hose, dev, rom_addr,
|
||||
bar_value);
|
||||
}
|
||||
cmdstat |= PCI_COMMAND_MEMORY;
|
||||
DEBUGF("\n");
|
||||
}
|
||||
}
|
||||
|
||||
pci_hose_write_config_word(hose, dev, PCI_COMMAND, cmdstat);
|
||||
pci_hose_write_config_byte(hose, dev, PCI_CACHE_LINE_SIZE,
|
||||
CONFIG_SYS_PCI_CACHE_LINE_SIZE);
|
||||
pci_hose_write_config_byte(hose, dev, PCI_LATENCY_TIMER, 0x80);
|
||||
}
|
||||
|
||||
int pciauto_setup_rom(struct pci_controller *hose, pci_dev_t dev)
|
||||
{
|
||||
pci_addr_t bar_value;
|
||||
pci_size_t bar_size;
|
||||
u32 bar_response;
|
||||
u16 cmdstat = 0;
|
||||
|
||||
pci_hose_write_config_dword(hose, dev, PCI_ROM_ADDRESS, 0xfffffffe);
|
||||
pci_hose_read_config_dword(hose, dev, PCI_ROM_ADDRESS, &bar_response);
|
||||
if (!bar_response)
|
||||
return -ENOENT;
|
||||
|
||||
bar_size = -(bar_response & ~1);
|
||||
DEBUGF("PCI Autoconfig: ROM, size=%#x, ", bar_size);
|
||||
if (pciauto_region_allocate(hose->pci_mem, bar_size, &bar_value) == 0) {
|
||||
pci_hose_write_config_dword(hose, dev, PCI_ROM_ADDRESS,
|
||||
bar_value);
|
||||
}
|
||||
DEBUGF("\n");
|
||||
pci_hose_read_config_word(hose, dev, PCI_COMMAND, &cmdstat);
|
||||
cmdstat |= PCI_COMMAND_IO | PCI_COMMAND_MEMORY | PCI_COMMAND_MASTER;
|
||||
pci_hose_write_config_word(hose, dev, PCI_COMMAND, cmdstat);
|
||||
|
||||
return 0;
|
||||
}
|
||||
|
||||
void pciauto_prescan_setup_bridge(struct pci_controller *hose,
|
||||
pci_dev_t dev, int sub_bus)
|
||||
{
|
||||
|
|
|
@ -84,11 +84,6 @@ static int pci_rom_probe(pci_dev_t dev, uint class,
|
|||
rom_address = CONFIG_VGA_BIOS_ADDR;
|
||||
#else
|
||||
|
||||
if (pciauto_setup_rom(pci_bus_to_hose(PCI_BUS(dev)), dev)) {
|
||||
debug("Cannot find option ROM\n");
|
||||
return -ENOENT;
|
||||
}
|
||||
|
||||
pci_read_config_dword(dev, PCI_ROM_ADDRESS, &rom_address);
|
||||
if (rom_address == 0x00000000 || rom_address == 0xffffffff) {
|
||||
debug("%s: rom_address=%x\n", __func__, rom_address);
|
||||
|
|
|
@ -721,15 +721,6 @@ void pci_write_bar32(struct pci_controller *hose, pci_dev_t dev, int barnum,
|
|||
* */
|
||||
u32 pci_read_bar32(struct pci_controller *hose, pci_dev_t dev, int barnum);
|
||||
|
||||
/**
|
||||
* pciauto_setup_rom() - Set up access to a device ROM
|
||||
*
|
||||
* @hose: PCI hose to use
|
||||
* @dev: PCI device to adjust
|
||||
* @return 0 if done, -ve on error
|
||||
*/
|
||||
int pciauto_setup_rom(struct pci_controller *hose, pci_dev_t dev);
|
||||
|
||||
/**
|
||||
* pci_hose_find_devices() - Find devices by vendor/device ID
|
||||
*
|
||||
|
|
Loading…
Reference in New Issue