Frans Pop wrote:
On Thursday 12 March 2009, Mark Lord wrote:
And after you answer that, then does this revised patch work?
If not, then perhaps you could try and figure out why,
since I just don't have your hardware available to me here.
I've just tested (on top of .29-rc8) the new development version of the
driver you sent me privately, which essentially contains this patch, and
it worked perfectly!
In the final patch, please add my:
Reported-by: Frans Pop <elendil@xxxxxxxxx>
Tested-by: Frans Pop <elendil@xxxxxxxxx>
Thanks Mark.
..
Super. Could do one more test, perhaps:
Does it also still work (turn off, turn on) with this
slight change to the copy you were testing there?
Thanks!
--- old/drivers/ata/sata_mv.c 2009-03-12 10:23:41.000000000 -0400
+++ new/drivers/ata/sata_mv.c 2009-03-13 09:03:47.000000000 -0400
@@ -1455,9 +1455,8 @@
struct ata_port *this_ap = host->ports[port];
struct mv_port_priv *pp = this_ap->private_data;
- if (pp->pp_flags & MV_PP_FLAG_EDMA_EN)
- if (pp->pp_flags & MV_PP_FLAG_NCQ_EN)
- return;
+ if (pp->pp_flags & MV_PP_FLAG_NCQ_EN)
+ return;
}
hpriv->hp_flags &= ~MV_HP_QUIRK_LED_BLINK_EN;
--
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html