On Wednesday 10 of October 2012 09:43:25 Linus Walleij wrote: > On Mon, Oct 8, 2012 at 10:39 AM, Tomasz Figa <t.figa@xxxxxxxxxxx> wrote: > > This patch modifies the pinctrl-samsung driver to register one GPIO > > chip > > per pin bank, instead of a single chip for all pin banks of the > > controller. > > > > It simplifies GPIO accesses a lot (constant time instead of looping > > through the list of banks to find the right one) and should have a good > > effect on performance of any bit-banging driver. > > > > In addition it allows to reference GPIO pins by a phandle to the bank > > node and a local pin offset inside of the bank (similar to previous > > gpiolib driver), which is more clear and readable than using indices > > relative to the whole pin controller. > > > > Example: > > device { > > > > /* ... */ > > gpios = <&gpk0 4 0>; > > /* ... */ > > > > }; > > > > Signed-off-by: Tomasz Figa <t.figa@xxxxxxxxxxx> > > This also looks good (and I think it has been discussed before) > so needs to be applied in isolation from the regs-to-DT stuff. Please see my reply for patch 9. Best regards, -- Tomasz Figa Samsung Poland R&D Center -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html