Hello, On 08/17/2010 08:23 PM, Alan Cox wrote: >> I don't really think it would be dangerous to grab intel IDE >> controllers with ata_generic. Again, that's what windows would do. >> And for sysfs unbind case, if the user specifically unbinds the >> controller from ata_piix, what is broken? > > If the user changes the PCI id list what happens ? Then that's upto the user, right? That's what the (root) user wants. You can add any random device ID to any random driver and if you do it wrong something will break. That's not different from any other driver. >> Maybe it's okay now but who's gonna remember what's going on there >> after five years and nobody guarantees the above would continue to > > Add a comment ? Yeah, definitely. Fat amount of them. >> hold in the future. > > It did occur to me to check this would be true in the future. Either way > - an id table would go out of date more often. Sure, the thing is that your patch doesn't mean we don't have to keep ata_piix device table up-to-date. We would still need to be maintaining that table whether ata_generic can detect IDE-R by itself or not. The device ID table in ata_piix is given, and ata_generic picking up the rest of intel IDE's wouldn't miss anything. So, unless IDE-R devices need some special treatment, I don't really see how the detection code would be useful. Thanks. -- tejun -- 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
