From: Sebastien Jan <s-jan@xxxxxx> Date: Wed, 5 May 2010 20:45:55 +0200 > A more elegant alternative to ethtool for updating the ks8851 > MAC address stored on its companion eeprom. > Using this debugfs interface does not require any knowledge on the > ks8851 companion eeprom organization to update the MAC address. > > Example to write 01:23:45:67:89:AB MAC address to the companion > eeprom (assuming debugfs is mounted in /sys/kernel/debug): > $ echo "01:23:45:67:89:AB" > /sys/kernel/debug/ks8851/mac_eeprom > > Signed-off-by: Sebastien Jan <s-jan@xxxxxx> Elegant? This commit message is the biggest lie ever told. What makes your ethernet driver so god-damn special that it deserves to have a private, completely unique, and obscure interface for setting the permanent ethernet address of a network device? Tell me how damn elegant it is that users have to learn about this special, unique, and common with no other driver, interface for performing this task? Tell me how damn elegant it is when another driver wants to provide users with a way to do this too, and they (like you) come up with their own unique and different interface for doing this. No, this is the most inelegant patch ever conceived because it totally ignores the way in which we handle issues like this. There is no way in the world I'm applying this garbage patch, sorry. We have an ETHTOOL_GPERMADDR, add a new ETHTOOL_SPERMADDR operation and then any driver (not just your's) can portably provide this facility and users will have one, and only one, way of performing this task. -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
