ata: pata_artop: use *switch* in atp8xx_fixup()

This driver uses a string of the *if* statements in atp8xx_fixup() where
a *switch* statement would fit better...

Signed-off-by: Sergey Shtylyov <s.shtylyov@omp.ru>
Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
This commit is contained in:
Sergey Shtylyov 2022-02-09 23:25:35 +03:00 committed by Damien Le Moal
parent 183a4bfbd7
commit 7ad3128efe
1 changed files with 9 additions and 5 deletions

View File

@ -28,7 +28,7 @@
#include <linux/ata.h> #include <linux/ata.h>
#define DRV_NAME "pata_artop" #define DRV_NAME "pata_artop"
#define DRV_VERSION "0.4.7" #define DRV_VERSION "0.4.8"
/* /*
* The ARTOP has 33 Mhz and "over clocked" timing tables. Until we * The ARTOP has 33 Mhz and "over clocked" timing tables. Until we
@ -315,12 +315,15 @@ static struct ata_port_operations artop6260_ops = {
static void atp8xx_fixup(struct pci_dev *pdev) static void atp8xx_fixup(struct pci_dev *pdev)
{ {
if (pdev->device == 0x0005) u8 reg;
switch (pdev->device) {
case 0x0005:
/* BIOS may have left us in UDMA, clear it before libata probe */ /* BIOS may have left us in UDMA, clear it before libata probe */
pci_write_config_byte(pdev, 0x54, 0); pci_write_config_byte(pdev, 0x54, 0);
else if (pdev->device == 0x0008 || pdev->device == 0x0009) { break;
u8 reg; case 0x0008:
case 0x0009:
/* Mac systems come up with some registers not set as we /* Mac systems come up with some registers not set as we
will need them */ will need them */
@ -338,6 +341,7 @@ static void atp8xx_fixup(struct pci_dev *pdev)
/* Enable IRQ output and burst mode */ /* Enable IRQ output and burst mode */
pci_read_config_byte(pdev, 0x4a, &reg); pci_read_config_byte(pdev, 0x4a, &reg);
pci_write_config_byte(pdev, 0x4a, (reg & ~0x01) | 0x80); pci_write_config_byte(pdev, 0x4a, (reg & ~0x01) | 0x80);
break;
} }
} }