6 messages in org.kernel.vger.linux-ide[PATCH,RFC] sata_mv: don't avoid clea...
FromSent OnAttachments
Lennert BuytenhekAug 23, 2008 8:04 pm 
Mark LordAug 26, 2008 6:50 am 
saeed bisharaAug 28, 2008 3:28 am 
Lennert BuytenhekAug 28, 2008 1:04 pm 
Martin MichlmayrJan 10, 2009 10:41 am 
Grant GrundlerJan 12, 2009 1:27 pm 
Actions with this message:
Paste this link in email or IM:
Paste this link in email or IM:
Atom feed for this thread
Paste this URL into your reader:
Subject:[PATCH,RFC] sata_mv: don't avoid clearing interrupt status on SoC host adaptersActions
From:Lennert Buytenhek (buyt@wantstofly.org)
Date:Aug 23, 2008 8:04:06 pm
List:org.kernel.vger.linux-ide

For some reason, sata_mv doesn't clear interrupt status during init when it's running on an SoC host adapter. If the bootloader has touched the SATA controller before starting Linux, Linux can end up enabling the SATA interrupt with events pending, which will cause the interrupt to be marked as spurious and then be disabled, which then breaks all further accesses to the controller.

This patch makes the SoC path clear interrupt status on init like in the non-SoC case.

Signed-off-by: Lennert Buytenhek <buyt@marvell.com>

--- drivers/ata/sata_mv.c | 21 ++++++++++----------- 1 files changed, 10 insertions(+), 11 deletions(-)

diff --git a/drivers/ata/sata_mv.c b/drivers/ata/sata_mv.c index ad169ff..e829a3a 100644 --- a/drivers/ata/sata_mv.c +++ b/drivers/ata/sata_mv.c @@ -3131,19 +3131,18 @@ static int mv_init_host(struct ata_host *host, unsigned
int board_idx) writelfl(0, hc_mmio + HC_IRQ_CAUSE_OFS); }

- if (!IS_SOC(hpriv)) { - /* Clear any currently outstanding host interrupt conditions */ - writelfl(0, mmio + hpriv->irq_cause_ofs); + /* Clear any currently outstanding host interrupt conditions */ + writelfl(0, mmio + hpriv->irq_cause_ofs);

- /* and unmask interrupt generation for host regs */ - writelfl(hpriv->unmask_all_irqs, mmio + hpriv->irq_mask_ofs); + /* and unmask interrupt generation for host regs */ + writelfl(hpriv->unmask_all_irqs, mmio + hpriv->irq_mask_ofs); + + /* + * enable only global host interrupts for now. + * The per-port interrupts get done later as ports are set up. + */ + mv_set_main_irq_mask(host, 0, PCI_ERR);

- /* - * enable only global host interrupts for now. - * The per-port interrupts get done later as ports are set up. - */ - mv_set_main_irq_mask(host, 0, PCI_ERR); - } done: return rc; }