summaryrefslogtreecommitdiffstats
path: root/drivers/ata/pata_serverworks.c
diff options
context:
space:
mode:
authorMikael Pettersson <mikpe@it.uu.se>2008-05-17 18:48:15 +0200
committerJeff Garzik <jgarzik@redhat.com>2008-05-19 23:31:26 +0200
commit821d22cdcd3c2944b93ac5f217ec0b6593ae6f48 (patch)
tree32940fdc3924b2b02de10ed68fd3c22bd15fb23a /drivers/ata/pata_serverworks.c
parentsata_promise: fix irq clearing buglets (diff)
downloadlinux-821d22cdcd3c2944b93ac5f217ec0b6593ae6f48.tar.xz
linux-821d22cdcd3c2944b93ac5f217ec0b6593ae6f48.zip
sata_promise: mmio access cleanups
This patch cleans up sata_promise's mmio accesses. In sata_promise there are three distinct mmio address spaces: 1. global registers, offsets from host->iomap[PDC_MMIO_BAR] 2. per-port ATA registers, offsets from ap->ioaddr.cmd_addr 3. per-port SATA registers, offsets from ap->ioaddr.scr_addr The driver currently often fails to indicate which address space a given mmio base pointer refers to, which is a source of bugs and confusion (see recent pdc_thaw() irq clearing bug; it's also been an obstacle for the pending NCQ extensions). To reduce these problems, adopt a coding style where the name of a base pointer always indicates which address space it refers to: 1. global registers: host_mmio 2. per-port ATA registers: ata_mmio 3. per-port SATA registers: sata_mmio Also rearrange register offset definitions to clearly indicate which address space they belong to, and add a symbolic definition for the previously hard-coded PHYMODE4 register. Signed-off-by: Mikael Pettersson <mikpe@it.uu.se> Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
Diffstat (limited to 'drivers/ata/pata_serverworks.c')
0 files changed, 0 insertions, 0 deletions