The original patch splitting the single control up into separate ones for SPI, FC, and SAS seemed to have had the intention of doing just that, without altering defaults. Since it didn't do this correctly, a subsequent patch corrected the mistaken use of the perm macro argument, but didn't restore the default to on for the SAS case (instead it made the comment match the changed behavior). So the question is - was it indeed intended to change the default to off for the SAS case? Thanks, Jan -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html