Re: [alsa-devel] [PATCH 6/6] ARM: DTS: da850: Add McBSP0 and McBSP1

[Sekhar Nori <nsekhar-l0cyMroinI0@public.gmane.org>]

On Thursday 07 April 2016 06:15 PM, Peter Ujfalusi wrote:

> I think what the da850.dtsi does is wrong. The dtsi file should not set any
> pinmux, those need to be set by the board .dts files If one board uses McASP0,
> it will set up the pins for that and leave McBSP pins as they were, but other
> board might use McBSP1 and not use McASP0, there you will have pincontrol for
> McBSP1.

Most functions on da850 are available only on one physical pin. So every
board that needs that functionality needs to setup the pinmux the same
way. This was the idea behind adding pinmux to da850.dtsi rather than
repeating it for every board.

Even though the pinctrl entries are there in da850.dtsi, the pinmux
registers wont be touched unless the status is set to "okay" for that
device in board .dts file.

The only care that needs to be taken is that the pins listed in
da850.dtsi need to be the minimum required for achieving that
functionality. If a board needs more than what is populated in
da850.dtsi then it can define its own pinctrl entries overriding that
coming from da850.dtsi

Thanks,
Sekhar
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html