[U-Boot] [PATCH v2 06/19] tegra: Add display support to funcmux

[Stephen Warren]

On 06/13/2012 10:19 AM, Simon Glass wrote:
> Add support for a default pin mapping for display1.

> +static const struct pingroup_config disp1_default[] = {
> +	PINMUX(LM0,   RSVD4,      NORMAL,    NORMAL),
> +	PINMUX(LPW1,  RSVD4,      NORMAL,    TRISTATE),

Do you really need to explicitly program these pingroups that aren't
used by display? It seems like it'll probably still work fine
irrespective of whether those are actually configured to output display
controller signals or not. However, if those pingroups are used by
something else, then changing their configuration here may stop the
other functionality from working.

> +	PINMUX(SLXD,  SPDIF,      NORMAL,    NORMAL),

Similarly here, yet SLXD doesn't ever support any display functionality,
so there should definitely be no need for this entry.

> @@ -185,6 +210,20 @@ int funcmux_select(enum periph_id id, int config)

> +	case PERIPH_ID_DISP1:
> +		if (config == FUNCMUX_DEFAULT) {

Shouldn't this option have some kind of name in funcmux.h; there are
many many many possible configurations for display, so just assuming
this as some kind of default seems a little presumptuous.

But, given the number of possibilities, I wonder if it isn't time to
replace funcmux with the device tree pinctrl bindings?