From: Stefan Sørensen <stefan.sorensen@xxxxxxxxxxxxxxx> Date: Thu, 13 Feb 2014 15:35:22 +0100 > This patch series add DT configuration to the DP83640 PHY driver and makes > the configuration of periodic output pins dynamic. > > Changes since v2: > - Add patch to properly configure perout triggers 0+1 > - Keep extts and perout numbers separate > - Shorten pr_err lines > > Changes since v1: > - Add bindings documentation > - Keep module parameters > - Rename gpio->pin > - Split patch into DT part and dynamic periodic output support > > Stefan Sørensen (3): > net:phy:dp83640: Program pulsewidth2 values of perout triggers 0 and 1 > net:phy:dp83640: Support a configurable number of periodic outputs > net:phy:dp83640: Get pin and master/slave configuration from DT I'm not applying this. This is a great example of why I hate module parameters. Whoever added these module parameters in the first place took the easy road and didn't have to think about whether this was a good way or not to specify these parameters. Do you really think we can, like magic, just remove this module parameter without having to think about the consequences of that action at all? The entities currently specifying this parameter, do they just lose? Sorry, that's unacceptable. You'll have to code this in a way such that people who currently specify this parameter keep working. If you don't like it, be angry at whoever added this module parameter in the first place, not me. Module parameters are 99 times out of 100 not the right way to configure something, really. Thanks. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html