-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA256 Jeff Garzik wrote: > [EMAIL PROTECTED] wrote: >> From: Tony Vroon <[EMAIL PROTECTED]> >> >> This adds an optional wrapper around ata_ac_issue_prot that triggers the >> LED layer. This is used for the PMU LED on G5 towers (IDE trigger). My >> test platform is a PowerMac 7,3 (Dual G5 2.0GHz, June 2004) with a K2 >> (sata_svw) controller. Now respun as a single patch, and the function >> name >> shortened as requested. >> >> Signed-off-by: Tony Vroon <[EMAIL PROTECTED]> >> Acked-by: Tejun Heo <[EMAIL PROTECTED]> >> Cc: Jeff Garzik <[EMAIL PROTECTED]> >> Signed-off-by: Andrew Morton <[EMAIL PROTECTED]> >> --- >> >> drivers/ata/libata-core.c | 21 +++++++++++++++++++++ >> drivers/ata/sata_svw.c | 2 +- >> include/linux/libata.h | 1 + >> 3 files changed, 23 insertions(+), 1 deletion(-) > > > After sitting on this for a time, I still cannot summon any interest in > taking this patch. It slows down the hot path for pretty lights.
Jeff, Would a conditional on ADB_PMU_LED_IDE (instead of the more generic LEDS_TRIGGER_IDE_DISK) alleviate your concerns? This can only be enabled on PPC platforms and would make sure that the hot path is not polluted anywhere else. Regards, Tony V. -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.4 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org iD8DBQFGVp/yp5vW4rUFj5oRCOlXAJwMpRiboqDnMcFdHxX7BU+Y1S1SxACePgeP q4yyJJzNBeJeF1YakPXrIOo= =ehsX -----END PGP SIGNATURE----- - To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html