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