On 13/01/17 22:54, Sebastian Hesselbarth wrote: > On 13.01.2017 10:12, Chris Packham wrote: >> From: Kalyan Kinthada <kalyan.kinthada@xxxxxxxxxxxxxxxxxxx> >> >> This pinctrl driver supports the 98DX3236, 98DX3336 and 98DX4251 SoCs >> from Marvell. >> >> Signed-off-by: Kalyan Kinthada <kalyan.kinthada@xxxxxxxxxxxxxxxxxxx> >> Signed-off-by: Chris Packham <chris.packham@xxxxxxxxxxxxxxxxxxx> >> Acked-by: Rob Herring <robh@xxxxxxxxxx> >> Acked-by: Sebastian Hesselbarth <sebastian.hesselbarth@xxxxxxxxx> >> --- >> >> Notes: >> Changes in v2: >> - include sdio support for the 98DX4251 >> Changes in v3: >> - None >> Changes in v4: >> - Correct some discrepencies between binding and driver. > > Well, unfortunately I still see differences between the "gpio" in > the binding and "gpo" in the driver. > > Please go back to that list I sent you yesterday and fix them all. > I think you may have missed my initial reply [1]. Or I have missed your response to it. Long story short "gpo" is intentional because some of those pins can't be used as inputs. But if you still want me to change it I will. [1] - https://lkml.org/lkml/2017/1/12/117 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html