Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1645742

Re: [PATCH 5/6] gpio: mvebu: Add support for the Armada 7K/8K SoCs

Show all headers | View raw

On Fri, May 19, 2017 at 06:09:24PM +0200, Gregory CLEMENT wrote:
> The Armada 7K and 8K SoCs use the same gpio controller as most of the
> other mvebu SoCs. However, the main difference is that the GPIO
> controller is part of a bigger system controller, and a syscon is used to
> control the overall system controller. Therefore, the driver needs to be
> adjusted to retrieve the regmap of the syscon to access registers, and
> account for the fact that registers are located at a certain offset
> within the regmap.
> 
> This commit add the support of the syscon and introduce a new variant for
> this case.

Do you really need all this complexity of merging what seems to be two
GPIO devices into one for DT purposes?  The device itself appears to be
two instances of the orion-gpio controller, one at 0x100 into the
system controller and a second one at 0x140.

If you instantiate two orion-gpio controllers, then you don't need to
add the "mvchip->offset" stuff.

I'd also say that it was the more correct approach, rather than adding
more complexity to GPIO drivers to support whatever incarnation comes
next.

-- 
RMK's Patch system: http://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up
according to speedtest.net.

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 5/6] gpio: mvebu: Add support for the Armada 7K/8K SoCs Gregory CLEMENT <gregory.clement@free-electrons.com> - 2017-05-19 18:10 +0200
  Re: [PATCH 5/6] gpio: mvebu: Add support for the Armada 7K/8K SoCs Russell King - ARM Linux <linux@armlinux.org.uk> - 2017-05-19 18:30 +0200
    Re: [PATCH 5/6] gpio: mvebu: Add support for the Armada 7K/8K SoCs Russell King - ARM Linux <linux@armlinux.org.uk> - 2017-05-19 18:30 +0200
      Re: [PATCH 5/6] gpio: mvebu: Add support for the Armada 7K/8K SoCs Thomas Petazzoni <thomas.petazzoni@free-electrons.com> - 2017-05-19 19:10 +0200

csiph-web