Re: [PATCH v3 0/3] dp83640: Get pin and master/slave configuration from DT

From: David Miller
Date: Thu Feb 13 2014 - 17:40:12 EST


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 linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/