Tony Vroon wrote:
The first user of ata_ac_issue_prot_with_ledtrigger, the ServerWorks Frodo/
Apple K2 driver. Used by the IDE LED trigger on G5 towers.
Respin of an earlier patch, based on comments by Tejun Heo & Alan Cox.
Just two comments.
1. IMHO, ata_qc_issue_prot_ledtrigger() without 'with' is good enough.
This is just my personal preference. Feel free to ignore it.
2. Patch #1 and #2 should be merged. They're one logical change of
adding ata_qc_issue_prot_with_ledtrigger(). Patch #3 is a logically
separate change of using it, but unless it's a wide conversion,
implementing something and using something can be merged. So, please
merge #1 and #2 and possibly #3.
Thanks.
--
tejun
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/