linux-kernel - Re: [PATCH] pinctrl: pinctrl-abx500: Add device tree support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CACRpkdbixv21db-_TcPApRj7=BHCQ52TDXnHqMpn+kaXcBBrhA@mail.gmail.com>
Date:	Fri, 7 Jun 2013 15:00:38 +0200
From:	Linus Walleij <linus.walleij@...aro.org>
To:	Patrice Chotard <patrice.chotard.st@...il.com>,
	Heiko Stübner <heiko@...ech.de>
Cc:	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	Olivier Clergeaud <olivier.clergeaud@...com>,
	Lee Jones <lee.jones@...aro.org>,
	Fabio Baltieri <fabio.baltieri@...aro.org>,
	Patrice Chotard <patrice.chotard@...com>,
	Gabriel Fernandez <gabriel.fernandez@...com>
Subject: Re: [PATCH] pinctrl: pinctrl-abx500: Add device tree support

On Fri, Jun 7, 2013 at 2:07 PM,  <patrice.chotard.st@...il.com> wrote:

> From: Patrice Chotard <patrice.chotard@...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@...com>
> Signed-off-by: Patrice Chotard <patrice.chotard@...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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/