On Mon, Sep 14, 2020 at 3:44 AM Drew Fustini <drew@xxxxxxxxxxxxxxx> wrote: > > + > +When #pinctrl-cells = 2, then setting a pin for a device could be done with: > + > + pinctrl-single,pins = <0xdc 0x30 0x07>; > + > +Where 0x30 is the pin configuration value and 0x07 is the pin mux mode value. > +See the device example and static board pins example below for more information. Pin configuration and mux mode don't mean anything in pinctrl-single. On another machine, mux mode might not be programmed this way or even exist. Or the location of bits would probably be different, and this would seem to imply the 0x07 would get shifted to the correct location for where the pin mux setting was on that machine's pinctrl registers. It seems like it would be better to explain the values are ORed together. What is the purpose of this change anyway? It seems like in the end it just does what it did before. The data is now split into two cells in the device tree, but why?
