Commit d23f33de authored by Bartlomiej Zolnierkiewicz's avatar Bartlomiej Zolnierkiewicz Committed by David S. Miller

alim15x3: cleanup ali_cable_detect()

Remove leftover local_irq_[save,restore]() and FIXME note.
Signed-off-by: default avatarBartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 5f6bd68e
...@@ -345,19 +345,13 @@ static int ali_cable_override(struct pci_dev *pdev) ...@@ -345,19 +345,13 @@ static int ali_cable_override(struct pci_dev *pdev)
* *
* This checks if the controller and the cable are capable * This checks if the controller and the cable are capable
* of UDMA66 transfers. It doesn't check the drives. * of UDMA66 transfers. It doesn't check the drives.
* But see note 2 below!
*
* FIXME: frobs bits that are not defined on newer ALi devicea
*/ */
static u8 ali_cable_detect(ide_hwif_t *hwif) static u8 ali_cable_detect(ide_hwif_t *hwif)
{ {
struct pci_dev *dev = to_pci_dev(hwif->dev); struct pci_dev *dev = to_pci_dev(hwif->dev);
unsigned long flags;
u8 cbl = ATA_CBL_PATA40, tmpbyte; u8 cbl = ATA_CBL_PATA40, tmpbyte;
local_irq_save(flags);
if (m5229_revision >= 0xC2) { if (m5229_revision >= 0xC2) {
/* /*
* m5229 80-pin cable detection (from Host View) * m5229 80-pin cable detection (from Host View)
...@@ -377,8 +371,6 @@ static u8 ali_cable_detect(ide_hwif_t *hwif) ...@@ -377,8 +371,6 @@ static u8 ali_cable_detect(ide_hwif_t *hwif)
} }
} }
local_irq_restore(flags);
return cbl; return cbl;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment