Hi Heiko, On 17 July 2016 at 09:47, Heiko Stübner <he...@sntech.de> wrote: > Hi Simon, > > Am Sonntag, 17. Juli 2016, 08:13:51 schrieb Simon Glass: >> On 15 July 2016 at 16:17, Heiko Stuebner <he...@sntech.de> wrote: >> > Add a driver which supports pin multiplexing setup for the most commonly >> > used peripherals. >> > >> > Signed-off-by: Heiko Stuebner <he...@sntech.de> >> > --- >> > >> > drivers/pinctrl/Kconfig | 9 + >> > drivers/pinctrl/rockchip/Makefile | 1 + >> > drivers/pinctrl/rockchip/pinctrl_rk3188.c | 614 >> > ++++++++++++++++++++++++++++++ 3 files changed, 624 insertions(+) >> > create mode 100644 drivers/pinctrl/rockchip/pinctrl_rk3188.c >> >> Acked-by: Simon Glass <s...@chromium.org> >> >> But is it possible to use some common code with the rk3288 driver? >> Things like rk3288_pinctrl_set_pins()? > > I don't really think that will fly. > > The iomux area always sees a lot of creativity when the GRF gets designed. > > For example on the rk3288 you have gpio0 iomux + pull + etc in the pmu and all > other banks in the grf. On the rk3188 the iomux is contained completely in the > grf but the pull setting of the first _12_ pins are living in the pmu - and > even their ordering is inverted - see the whole if(flags) part in the rk3188 > pinctrl driver. > > The rk3399 introduces even more funny things. So I don't see where we would > save without adding all the indirection the linux driver currently has.
That's fine, thanks for the info. Regards, Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot