On Thu, 5 Sep 2019, at 10:48, Rashmica Gupta wrote: > The ast2600 is a new generation of SoC from ASPEED. Similarly to the > ast2400 and ast2500, it has a GPIO controller for it's 3.6V GPIO pins. > Additionally, it has a GPIO controller for 36 1.8V GPIO pins. These > voltages are fixed and cannot be configured via pinconf, so we have two > separate drivers for them. See 3/4 for discussion about the commit message. > > Signed-off-by: Rashmica Gupta <rashmica.g@xxxxxxxxx> > --- > Documentation/devicetree/bindings/gpio/gpio-aspeed.txt | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/gpio/gpio-aspeed.txt > b/Documentation/devicetree/bindings/gpio/gpio-aspeed.txt > index 7e9b586770b0..cd388797e07c 100644 > --- a/Documentation/devicetree/bindings/gpio/gpio-aspeed.txt > +++ b/Documentation/devicetree/bindings/gpio/gpio-aspeed.txt > @@ -2,7 +2,8 @@ Aspeed GPIO controller Device Tree Bindings > ------------------------------------------- > > Required properties: > -- compatible : Either "aspeed,ast2400-gpio" or "aspeed,ast2500-gpio" > +- compatible : Either "aspeed,ast2400-gpio", "aspeed,ast2500-gpio", > + "aspeed,ast2600-gpio", or "aspeed,ast2600-1-8v-gpio" See the discussion on patch 3/4 about how we might eliminate the aspeed,ast2600-1-8v-gpio compatible string. Also, this patch should be the first in the series and start the subject with "dt-bindings: gpio: aspeed: ..." Cheers, Andrew