diff options
author | Alan Cox <alan@redhat.com> | 2007-07-19 19:11:54 -0400 |
---|---|---|
committer | Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> | 2007-07-19 19:11:54 -0400 |
commit | 9702b5d5c26be12d376e1baf2a8e5b82e6c06b66 (patch) | |
tree | 58e37c85d121fd01c845e2c121d662c8df0a0135 /drivers/ide/pci | |
parent | c82d72f5c521434c36b9550b3ac6601eb8167687 (diff) |
ide: Stop mapping ROMs
Various old IDE drivers go mapping ROM devices for no apparent reason and
without using the ROM mapping API we now have. They don't actually use
the ROM they map and the new libata drivers are happy without it being
mapped so rather than port them lets just junk it for the next -rc1.
Signed-off-by: Alan Cox <alan@redhat.com>
Acked-by: Sergei Shtylyov <sshtylyov@ru.mvista.com>
Cc: Andrew Morton <akpm@osdl.org>
Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Diffstat (limited to 'drivers/ide/pci')
-rw-r--r-- | drivers/ide/pci/aec62xx.c | 6 | ||||
-rw-r--r-- | drivers/ide/pci/hpt34x.c | 11 | ||||
-rw-r--r-- | drivers/ide/pci/hpt366.c | 8 | ||||
-rw-r--r-- | drivers/ide/pci/pdc202xx_new.c | 7 | ||||
-rw-r--r-- | drivers/ide/pci/pdc202xx_old.c | 8 |
5 files changed, 2 insertions, 38 deletions
diff --git a/drivers/ide/pci/aec62xx.c b/drivers/ide/pci/aec62xx.c index e5d09367627e..6d18fef558bd 100644 --- a/drivers/ide/pci/aec62xx.c +++ b/drivers/ide/pci/aec62xx.c | |||
@@ -174,12 +174,6 @@ static unsigned int __devinit init_chipset_aec62xx(struct pci_dev *dev, const ch | |||
174 | { | 174 | { |
175 | int bus_speed = system_bus_clock(); | 175 | int bus_speed = system_bus_clock(); |
176 | 176 | ||
177 | if (dev->resource[PCI_ROM_RESOURCE].start) { | ||
178 | pci_write_config_dword(dev, PCI_ROM_ADDRESS, dev->resource[PCI_ROM_RESOURCE].start | PCI_ROM_ADDRESS_ENABLE); | ||
179 | printk(KERN_INFO "%s: ROM enabled at 0x%08lx\n", name, | ||
180 | (unsigned long)dev->resource[PCI_ROM_RESOURCE].start); | ||
181 | } | ||
182 | |||
183 | if (bus_speed <= 33) | 177 | if (bus_speed <= 33) |
184 | pci_set_drvdata(dev, (void *) aec6xxx_33_base); | 178 | pci_set_drvdata(dev, (void *) aec6xxx_33_base); |
185 | else | 179 | else |
diff --git a/drivers/ide/pci/hpt34x.c b/drivers/ide/pci/hpt34x.c index 2c24c3de8846..64f19743b127 100644 --- a/drivers/ide/pci/hpt34x.c +++ b/drivers/ide/pci/hpt34x.c | |||
@@ -120,17 +120,10 @@ static unsigned int __devinit init_chipset_hpt34x(struct pci_dev *dev, const cha | |||
120 | pci_write_config_byte(dev, HPT34X_PCI_INIT_REG, 0x00); | 120 | pci_write_config_byte(dev, HPT34X_PCI_INIT_REG, 0x00); |
121 | pci_read_config_word(dev, PCI_COMMAND, &cmd); | 121 | pci_read_config_word(dev, PCI_COMMAND, &cmd); |
122 | 122 | ||
123 | if (cmd & PCI_COMMAND_MEMORY) { | 123 | if (cmd & PCI_COMMAND_MEMORY) |
124 | if (pci_resource_start(dev, PCI_ROM_RESOURCE)) { | ||
125 | pci_write_config_dword(dev, PCI_ROM_ADDRESS, | ||
126 | dev->resource[PCI_ROM_RESOURCE].start | PCI_ROM_ADDRESS_ENABLE); | ||
127 | printk(KERN_INFO "HPT345: ROM enabled at 0x%08lx\n", | ||
128 | (unsigned long)dev->resource[PCI_ROM_RESOURCE].start); | ||
129 | } | ||
130 | pci_write_config_byte(dev, PCI_LATENCY_TIMER, 0xF0); | 124 | pci_write_config_byte(dev, PCI_LATENCY_TIMER, 0xF0); |
131 | } else { | 125 | else |
132 | pci_write_config_byte(dev, PCI_LATENCY_TIMER, 0x20); | 126 | pci_write_config_byte(dev, PCI_LATENCY_TIMER, 0x20); |
133 | } | ||
134 | 127 | ||
135 | /* | 128 | /* |
136 | * Since 20-23 can be assigned and are R/W, we correct them. | 129 | * Since 20-23 can be assigned and are R/W, we correct them. |
diff --git a/drivers/ide/pci/hpt366.c b/drivers/ide/pci/hpt366.c index e9b07a97c340..81853d740aee 100644 --- a/drivers/ide/pci/hpt366.c +++ b/drivers/ide/pci/hpt366.c | |||
@@ -994,14 +994,6 @@ static unsigned int __devinit init_chipset_hpt366(struct pci_dev *dev, const cha | |||
994 | */ | 994 | */ |
995 | *info = *(struct hpt_info *)pci_get_drvdata(dev); | 995 | *info = *(struct hpt_info *)pci_get_drvdata(dev); |
996 | 996 | ||
997 | /* | ||
998 | * FIXME: Not portable. Also, why do we enable the ROM in the first place? | ||
999 | * We don't seem to be using it. | ||
1000 | */ | ||
1001 | if (dev->resource[PCI_ROM_RESOURCE].start) | ||
1002 | pci_write_config_dword(dev, PCI_ROM_ADDRESS, | ||
1003 | dev->resource[PCI_ROM_RESOURCE].start | PCI_ROM_ADDRESS_ENABLE); | ||
1004 | |||
1005 | pci_write_config_byte(dev, PCI_CACHE_LINE_SIZE, (L1_CACHE_BYTES / 4)); | 997 | pci_write_config_byte(dev, PCI_CACHE_LINE_SIZE, (L1_CACHE_BYTES / 4)); |
1006 | pci_write_config_byte(dev, PCI_LATENCY_TIMER, 0x78); | 998 | pci_write_config_byte(dev, PCI_LATENCY_TIMER, 0x78); |
1007 | pci_write_config_byte(dev, PCI_MIN_GNT, 0x08); | 999 | pci_write_config_byte(dev, PCI_MIN_GNT, 0x08); |
diff --git a/drivers/ide/pci/pdc202xx_new.c b/drivers/ide/pci/pdc202xx_new.c index ee5020df005d..06c5e93d37a2 100644 --- a/drivers/ide/pci/pdc202xx_new.c +++ b/drivers/ide/pci/pdc202xx_new.c | |||
@@ -378,13 +378,6 @@ static unsigned int __devinit init_chipset_pdcnew(struct pci_dev *dev, const cha | |||
378 | int f, r; | 378 | int f, r; |
379 | u8 pll_ctl0, pll_ctl1; | 379 | u8 pll_ctl0, pll_ctl1; |
380 | 380 | ||
381 | if (dev->resource[PCI_ROM_RESOURCE].start) { | ||
382 | pci_write_config_dword(dev, PCI_ROM_ADDRESS, | ||
383 | dev->resource[PCI_ROM_RESOURCE].start | PCI_ROM_ADDRESS_ENABLE); | ||
384 | printk(KERN_INFO "%s: ROM enabled at 0x%08lx\n", name, | ||
385 | (unsigned long)dev->resource[PCI_ROM_RESOURCE].start); | ||
386 | } | ||
387 | |||
388 | #ifdef CONFIG_PPC_PMAC | 381 | #ifdef CONFIG_PPC_PMAC |
389 | apple_kiwi_init(dev); | 382 | apple_kiwi_init(dev); |
390 | #endif | 383 | #endif |
diff --git a/drivers/ide/pci/pdc202xx_old.c b/drivers/ide/pci/pdc202xx_old.c index 41ac4a94959f..25cebfe18ddd 100644 --- a/drivers/ide/pci/pdc202xx_old.c +++ b/drivers/ide/pci/pdc202xx_old.c | |||
@@ -316,14 +316,6 @@ static void pdc202xx_reset (ide_drive_t *drive) | |||
316 | static unsigned int __devinit init_chipset_pdc202xx(struct pci_dev *dev, | 316 | static unsigned int __devinit init_chipset_pdc202xx(struct pci_dev *dev, |
317 | const char *name) | 317 | const char *name) |
318 | { | 318 | { |
319 | /* This doesn't appear needed */ | ||
320 | if (dev->resource[PCI_ROM_RESOURCE].start) { | ||
321 | pci_write_config_dword(dev, PCI_ROM_ADDRESS, | ||
322 | dev->resource[PCI_ROM_RESOURCE].start | PCI_ROM_ADDRESS_ENABLE); | ||
323 | printk(KERN_INFO "%s: ROM enabled at 0x%08lx\n", name, | ||
324 | (unsigned long)dev->resource[PCI_ROM_RESOURCE].start); | ||
325 | } | ||
326 | |||
327 | return dev->irq; | 319 | return dev->irq; |
328 | } | 320 | } |
329 | 321 | ||