Digging back through the archives, this came up once in 2007, Alan Cox did not like the idea of globally adding this call into the critical paths of libata at the time. (I've seen it done via external patches in ata_qc_issue and ata_qc_complete for example.) If that concern is no longer in place, I can redo with it within libata so it'll work for all libata driven devices similar to how it's handled for legacy IDE drivers now. Josh C On Wed, Jan 9, 2013 at 11:56 AM, Jeff Garzik <jgarzik@xxxxxxxxx> wrote: > On 01/08/2013 01:16 PM, Joshua Coombs wrote: >> >> Add a call to the IDE LED Trigger within the Marvell SATA driver to allow >> Marvell SoC devices to show SATA activity via GPIO connected LEDs. >> >> Signed-off-by: Joshua Coombs <josh.coombs@xxxxxxxxx> >> --- >> drivers/ata/sata_mv.c | 3 +++ >> drivers/leds/Kconfig | 3 +-- >> 2 files changed, 4 insertions(+), 2 deletions(-) > > > NAK. This is not appropriate at the specific driver level. This would be > better implemented at a higher level. > > > -- 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