-----Original Message----- From: Linus Walleij <linus.wall...@linaro.org> Sent: Monday, June 3, 2019 7:08 PM To: Hongwei Zhang <hongw...@ami.com> Cc: Joel Stanley <j...@jms.id.au>; Andrew Jeffery <and...@aj.id.au>; Rob Herring <robh...@kernel.org>; Mark Rutland <mark.rutl...@arm.com>; open list:GPIO SUBSYSTEM <linux-g...@vger.kernel.org>; open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS <devicet...@vger.kernel.org>; Linux ARM <linux-arm-ker...@lists.infradead.org>; linux-asp...@lists.ozlabs.org; linux-kernel@vger.kernel.org Subject: Re: [PATCH linux dev-5.1 v1] ARM: dts: aspeed: Add SGPM pinmux
Hi Hongwei, On Tue, Jun 4, 2019 at 12:44 AM Hongwei Zhang <hongw...@ami.com> wrote: > > Add SGPM pinmux to ast2500-pinctrl function and group, to prepare for > supporting SGPIO in AST2500 SoC. > > Signed-off-by: Hongwei Zhang <hongw...@ami.com> > --- > 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. Just committed the DT patch to _https://git.kernel.org/pub/scm/linux/kernel/git/arm/arm-soc.git , please help to review. Thanks! Other than that it looks fine to me. Yours, Linus Walleij
