Re: [PATCH #upstream-fixes 1/4] libata: fix device iteration bugs
From: Sergei Shtylyov <hidden>
Date: 2008-10-26 10:47:29
Hello. Tejun Heo wrote:
There were several places where only enabled devices should be iterated over but device enabledness wasn't checked. * IDENTIFY data 40 wire check in cable_is_40wire() * xfer_mode/ncq_enabled saving in ata_scsi_error() * DUBIOUS_XFER handling in ata_set_mode() While at it, reformat comment in cable_is_40wire(). Signed-off-by: Tejun Heo <tj@kernel.org>
More nitpicking... :-)
quoted hunk ↗ jump to hunk
Index: work/drivers/ata/libata-core.c ===================================================================--- work.orig/drivers/ata/libata-core.c +++ work/drivers/ata/libata-core.c@@ -4169,18 +4169,16 @@ static int cable_is_40wire(struct ata_po if (ap->cbl == ATA_CBL_PATA40_SHORT) return 0; /* If the controller doesn't know we scan - - - Note: We look for all 40 wire detects at this point. - Any 80 wire detect is taken to be 80 wire cable - because - - In many setups only the one drive (slave if present) - will give a valid detect - - If you have a non detect capable drive you don't - want it to colour the choice - */ + * + * - Note: We look for all 40 wire detects at this point. Any + * 80 wire detect is taken to be 80 wire cable because - In + * many setups only the one drive (slave if present) will + * give a valid detect - If you have a non detect capable + * drive you don't want it to colour the choice + */
The comment formatting now is worse than it was... I'd drop hyphen before "Note:" and align the bullets under this note (also doing s/If/if/). MBR, Sergei