sata_nv: don't diddle with nIEN on mcp55

On mcp55, nIEN gets stuck once set and liteon blueray rom iHOS104-08
violates ATA specification and fails to set I on D2H Reg FIS if nIEN
is set when the command was issued. When the other party is following
the spec, both devices can work fine but when the two flaws are put
together, they can't talk to each other.

mcp55 has its own IRQ masking mechanism and there's no reason to mess
with nIEN in the first place. Fix it by dropping nIEN diddling from
nv_mcp55_freeze/thaw().

This was originally reported by Cengiz. Although Cengiz hasn't
verified the fix yet, I could reproduce this problem and verfiy the
fix. Even if Cengiz is experiencing different or additional problems,
this patch is needed.

Signed-off-by: Tejun Heo <tj@kernel.org>
Reported-by: Cengiz Günay <cgunay@emory.edu>
Cc: stable@kernel.org
Signed-off-by: Jeff Garzik <jgarzik@redhat.com>

authored by Tejun Heo and committed by Jeff Garzik f3faf8fc 8b27ff4c

-2
-2
drivers/ata/sata_nv.c
··· 1669 1669 mask = readl(mmio_base + NV_INT_ENABLE_MCP55); 1670 1670 mask &= ~(NV_INT_ALL_MCP55 << shift); 1671 1671 writel(mask, mmio_base + NV_INT_ENABLE_MCP55); 1672 - ata_sff_freeze(ap); 1673 1672 } 1674 1673 1675 1674 static void nv_mcp55_thaw(struct ata_port *ap) ··· 1682 1683 mask = readl(mmio_base + NV_INT_ENABLE_MCP55); 1683 1684 mask |= (NV_INT_MASK_MCP55 << shift); 1684 1685 writel(mask, mmio_base + NV_INT_ENABLE_MCP55); 1685 - ata_sff_thaw(ap); 1686 1686 } 1687 1687 1688 1688 static void nv_adma_error_handler(struct ata_port *ap)