> Why do we need to do that? Leaving sector_size_supported() unset means > exactly "no to anything but 512". I'll be quite surprised if we see > PATA drives supporting 4096 or 520 byte sectors anyway. It seems like > a premium feature. Good point on the former - yes it defaults sane now so that doesn't need doing. For the sector size remember a lot of PATA chips appear with SATA bridges at times - especially promise and highpoint. -- 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