On Mon, Jun 20, 2016 at 5:57 PM, Rob Herring [off-list ref] wrote:

On Fri, Jun 17, 2016 at 11:51:03AM +0200, Neil Armstrong wrote:

quoted

+Optional Properties:
+- oscio-is-gpo: Boolean, Indicated the oscio pin can be used as additional
+             output gpo port.
+

quoted

+- pull-up-ports: Array of port numbers which must have pull-up enabled.
+- pull-down-ports: Array of port numbers which must have pull-down enabled.
+- open-drain-ports: Array of port numbers which must be configured as open-drain,
+                     Push-Pull mode is default.
+- polarity-invert-ports: Array of port numbers whih port polarity must be inverted.

Seems like these should be done in a common way.

If not, they all need a vendor prefix.

Actually on second look, this takes the sx150 to pin control territory.

I am starting to feel like a move to drivers/pinctrl/* might be warranted.

Neil you worked on other pin controllers IIRC, do you think it would
be much work to make a combined GPIO+pinctrl driver and move
this over to drivers/pinctrl?

I know refactoring across subsystems can be a pain, but at least
I'm maintaining both and happy to help out.

Yours,
Linus Walleij