From mboxrd@z Thu Jan 1 00:00:00 1970 From: Linus Walleij Subject: Re: [PATCH linux dev-5.1 v1] ARM: dts: aspeed: Add SGPM pinmux Date: Tue, 4 Jun 2019 01:07:41 +0200 Message-ID: References: <1559601789-27121-1-git-send-email-hongweiz@ami.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Return-path: In-Reply-To: <1559601789-27121-1-git-send-email-hongweiz@ami.com> Sender: linux-kernel-owner@vger.kernel.org To: Hongwei Zhang Cc: Joel Stanley , Andrew Jeffery , Rob Herring , Mark Rutland , "open list:GPIO SUBSYSTEM" , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , Linux ARM , linux-aspeed@lists.ozlabs.org, "linux-kernel@vger.kernel.org" List-Id: linux-gpio@vger.kernel.org Hi Hongwei, On Tue, Jun 4, 2019 at 12:44 AM Hongwei Zhang wrote: > > Add SGPM pinmux to ast2500-pinctrl function and group, to prepare for > supporting SGPIO in AST2500 SoC. > > Signed-off-by: Hongwei Zhang > --- > Documentation/devicetree/bindings/pinctrl/pinctrl-aspeed.txt | 2 +- > arch/arm/boot/dts/aspeed-g5.dtsi | 5 +++++ > drivers/pinctrl/aspeed/pinctrl-aspeed-g5.c | 4 ++++ Please try to separate out the change to arch/arm/boot/dts/aspeed-g5.dtsi into a separate patch that goes through ARM SoC. Other than that it looks fine to me. Yours, Linus Walleij