Commit 7b3a24c5 authored by Maxime Bizon's avatar Maxime Bizon Committed by Jeff Garzik

ahci: don't enable port irq before handler is registered

The ahci_pmp_attach() & ahci_pmp_detach() unmask port irqs, but they
are also called during port initialization, before ahci host irq
handler is registered. On ce4100 platform, this sometimes triggers
"irq 4: nobody cared" message when loading driver.

Fixed this by not touching the register if the port is in frozen
state, and mark all uninitialized port as frozen.
Signed-off-by: default avatarMaxime Bizon <mbizon@freebox.fr>
Acked-by: default avatarTejun Heo <tj@kernel.org>
Cc: stable@kernel.org
Signed-off-by: default avatarJeff Garzik <jgarzik@pobox.com>
parent ae01b249
...@@ -1919,7 +1919,17 @@ static void ahci_pmp_attach(struct ata_port *ap) ...@@ -1919,7 +1919,17 @@ static void ahci_pmp_attach(struct ata_port *ap)
ahci_enable_fbs(ap); ahci_enable_fbs(ap);
pp->intr_mask |= PORT_IRQ_BAD_PMP; pp->intr_mask |= PORT_IRQ_BAD_PMP;
writel(pp->intr_mask, port_mmio + PORT_IRQ_MASK);
/*
* We must not change the port interrupt mask register if the
* port is marked frozen, the value in pp->intr_mask will be
* restored later when the port is thawed.
*
* Note that during initialization, the port is marked as
* frozen since the irq handler is not yet registered.
*/
if (!(ap->pflags & ATA_PFLAG_FROZEN))
writel(pp->intr_mask, port_mmio + PORT_IRQ_MASK);
} }
static void ahci_pmp_detach(struct ata_port *ap) static void ahci_pmp_detach(struct ata_port *ap)
...@@ -1935,7 +1945,10 @@ static void ahci_pmp_detach(struct ata_port *ap) ...@@ -1935,7 +1945,10 @@ static void ahci_pmp_detach(struct ata_port *ap)
writel(cmd, port_mmio + PORT_CMD); writel(cmd, port_mmio + PORT_CMD);
pp->intr_mask &= ~PORT_IRQ_BAD_PMP; pp->intr_mask &= ~PORT_IRQ_BAD_PMP;
writel(pp->intr_mask, port_mmio + PORT_IRQ_MASK);
/* see comment above in ahci_pmp_attach() */
if (!(ap->pflags & ATA_PFLAG_FROZEN))
writel(pp->intr_mask, port_mmio + PORT_IRQ_MASK);
} }
int ahci_port_resume(struct ata_port *ap) int ahci_port_resume(struct ata_port *ap)
......
...@@ -5480,7 +5480,7 @@ struct ata_port *ata_port_alloc(struct ata_host *host) ...@@ -5480,7 +5480,7 @@ struct ata_port *ata_port_alloc(struct ata_host *host)
if (!ap) if (!ap)
return NULL; return NULL;
ap->pflags |= ATA_PFLAG_INITIALIZING; ap->pflags |= ATA_PFLAG_INITIALIZING | ATA_PFLAG_FROZEN;
ap->lock = &host->lock; ap->lock = &host->lock;
ap->print_id = -1; ap->print_id = -1;
ap->host = host; ap->host = host;
......
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