lkml.org 
[lkml]   [2013]   [Jun]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH] pinctrl: pinctrl-abx500: Add device tree support
From
On Fri, Jun 7, 2013 at 2:07 PM,  <patrice.chotard.st@gmail.com> wrote:

> From: Patrice Chotard <patrice.chotard@st.com>
>
> We use the same way to define pin muxing and pin
> configuration than for nomadik. So pickup code
> from pinctrl_nomadik.c to be able to implement
> pin multiplexing and pin configuration using
> the device tree.
>
> Signed-off-by: Gabriel Fernandez <gabriel.fernandez@st.com>
> Signed-off-by: Patrice Chotard <patrice.chotard@st.com>

Overall this is looking fine.

> +- ste,config: Handle of pin configuration node (e.g. ste,config = <&slpm_in_wkup_pdis>)
> +
> +- ste,input : <0/1/2>
> + 0: input with no pull
> + 1: input with pull up,
> + 2: input with pull down,
> +
> +- ste,output: <0/1/2>
> + 0: output low,
> + 1: output high,
> + 2: output (value is not specified).

As I have just discussed with Heiko, we need generic pin config bindings for
the drivers using generic pin config, such as the abx500. We have to start
unifying unifiable stuff...

i.e. this is not similar to the Nomadik pinctrl driver, because that one is not
using generic pin config.

So we're trying to agree on generic bindings with generic parsing code to
avoid code duplication.

Please check my review of Heiko's patch for my ideas and comment in
that thread.

Subject:
"[PATCH v2 5/8] pinctrl: add pinctrl driver for Rockchip SoCs"

Yours,
Linus Walleij


\
 
 \ /
  Last update: 2013-06-07 15:41    [W:0.070 / U:2.012 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site