Skip to content

pinctrl-rockchip: Change wrong initial assumptions

Max Schwarz linux-arm-kernel linux-devicetree 2 versions – New

Get series with b4
b4 am 4611044.dLc2GHPYxV@xq-nb

Versions newest first · linked by author and title

  1. v2 Latest0 of 8 patches 0 repliesNew
  2. v11 of 8 patches 5 repliesSuperseded

Patches in v2 No applies recorded yet

None of the 8 patches the cover letter announces are in this archive.

Reviews trailers per patch

PatchMax Schwarz
SeriesA-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v2 thread

Cover letter [PATCH v2 0/8] pinctrl-rockchip: Change wrong initial assumptions

Am Montag, 5. Mai 2014, 13:57:31 schrieb Heiko St?bner:

Quoted 36 lines

Initially due to lack of documentation and (personal) understanding I assumed that the area holding the iomux settings would be separate from everything else, while in fact the grf registers contain not only pinctrl stuff but also dma, usb-phy and general soc-status settings. Also things like drive-strength we do not support currently are intermixed.

The same is true for the pmu, which does not only contain power domains but also the system reset as well as well as general registers surviving system-resets. Additionally the rk3188 moved parts of the pull-setting registers into the pmu space.

While this wasn't a problem until now, the upcoming rk3288 introduces additional changes to both the grf and pmu areas. On it even part of the pinmux registers move into the pmu space.

The code in it's current form supports both the old as well as the changed bindings.

As stated in the v1 thread, this is also a first step to deprecate the wrongly included gpio-controllers which now duplicate the recently added gpio-dwapb, that should be used instead.

changes since v1:
- add select MFD_SYSCON as suggested by Max

Heiko Stuebner (8):
pinctrl: rockchip: do not require 2nd register area
pinctrl: rockchip: use regmaps instead of raw mappings
pinctrl: rockchip: rockchip_pinctrl in rockchip_get_bank_data
pinctrl: rockchip: let pmu registers be supplied by a syscon
pinctrl: rockchip: only map bank0-pull-region when pmu regmap missing
pinctrl: rockchip: base regmap supplied by a syscon
dt-bindings: adapt rockchip-pinctrl doc to changed bindings
ARM: dts: rockchip: convert pinctrl nodes to new bindings

Quoted 7 lines
 .../bindings/pinctrl/rockchip,pinctrl.txt          |  28 +++-
 arch/arm/boot/dts/rk3066a.dtsi                     |   2 +-
 arch/arm/boot/dts/rk3188.dtsi                      |   9 +-
 arch/arm/boot/dts/rk3xxx.dtsi                      |   9 +-
 drivers/pinctrl/Kconfig                            |   1 +
 drivers/pinctrl/pinctrl-rockchip.c                 | 178

+++++++++++++++------ 6 files changed, 165 insertions(+), 62 deletions(-)

This series

Cheers,
Max