All of lore.kernel.org
 help / color / mirror / Atom feed
From: Linus Walleij <linus.walleij@linaro.org>
To: "Manjunathappa, Prakash" <prakash.pm@ti.com>
Cc: "Nori, Sekhar" <nsekhar@ti.com>,
	Grant Likely <grant.likely@linaro.org>,
	Rob Herring <rob.herring@calxeda.com>,
	Russell King - ARM Linux <linux@arm.linux.org.uk>,
	Rob Landley <rob@landley.net>,
	ext Tony Lindgren <tony@atomide.com>,
	Prabhakar Lad <prabhakar.csengg@gmail.com>,
	Peter Ujfalusi <peter.ujfalusi@ti.com>,
	"devicetree-discuss@lists.ozlabs.org" 
	<devicetree-discuss@lists.ozlabs.org>,
	"linux-doc@vger.kernel.org" <linux-doc@vger.kernel.org>,
	"linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"davinci-linux-open-source@linux.davincidsp.com" 
	<davinci-linux-open-source@linux.davincidsp.com>
Subject: Re: [PATCH 1/3] pinctrl: pinctrl-single: enhance to configure multiple pins of different modules
Date: Fri, 7 Jun 2013 11:20:39 +0200	[thread overview]
Message-ID: <CACRpkdYrmpptASxiM-YUrH-W71OxVFSh-kS_nX-Cw1Whe+fGBQ@mail.gmail.com> (raw)
In-Reply-To: <1369145282-14419-3-git-send-email-prakash.pm@ti.com>

On Tue, May 21, 2013 at 4:08 PM, Manjunathappa, Prakash
<prakash.pm@ti.com> wrote:

> Add support to configure multiple pins in each register, existing
> implementation added by [1] does not support full fledge multiple pin
> configuration in single register, reports a pin clash when different
> modules configure different bits of same register. The issue reported
> and discussed here
> http://www.spinics.net/lists/arm-kernel/msg235213.html
>
> With pinctrl-single,bits-per-mux property specified, use function-mask
> property to find out number pins to configure. Allocate and register
> pin control functions based sub mask.
>
> Tested on da850/omap-l138 EVM.
> does not support variable submask for pins.
> does not support pinconf.
>
> [1] "pinctrl: pinctrl-single: Add pinctrl-single,bits type of mux"
> (9e605cb68a21d5704839a192a46ebcf387773704),
>
> Signed-off-by: Manjunathappa, Prakash <prakash.pm@ti.com>
> Reported-by: Lad, Prabhakar <prabhakar.csengg@gmail.com>
> Tested-by: Lad, Prabhakar <prabhakar.csengg@gmail.com>

Patch applied with Haojian's and Tony's ACK.

Thanks!
Linus Walleij

WARNING: multiple messages have this Message-ID (diff)
From: linus.walleij@linaro.org (Linus Walleij)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/3] pinctrl: pinctrl-single: enhance to configure multiple pins of different modules
Date: Fri, 7 Jun 2013 11:20:39 +0200	[thread overview]
Message-ID: <CACRpkdYrmpptASxiM-YUrH-W71OxVFSh-kS_nX-Cw1Whe+fGBQ@mail.gmail.com> (raw)
In-Reply-To: <1369145282-14419-3-git-send-email-prakash.pm@ti.com>

On Tue, May 21, 2013 at 4:08 PM, Manjunathappa, Prakash
<prakash.pm@ti.com> wrote:

> Add support to configure multiple pins in each register, existing
> implementation added by [1] does not support full fledge multiple pin
> configuration in single register, reports a pin clash when different
> modules configure different bits of same register. The issue reported
> and discussed here
> http://www.spinics.net/lists/arm-kernel/msg235213.html
>
> With pinctrl-single,bits-per-mux property specified, use function-mask
> property to find out number pins to configure. Allocate and register
> pin control functions based sub mask.
>
> Tested on da850/omap-l138 EVM.
> does not support variable submask for pins.
> does not support pinconf.
>
> [1] "pinctrl: pinctrl-single: Add pinctrl-single,bits type of mux"
> (9e605cb68a21d5704839a192a46ebcf387773704),
>
> Signed-off-by: Manjunathappa, Prakash <prakash.pm@ti.com>
> Reported-by: Lad, Prabhakar <prabhakar.csengg@gmail.com>
> Tested-by: Lad, Prabhakar <prabhakar.csengg@gmail.com>

Patch applied with Haojian's and Tony's ACK.

Thanks!
Linus Walleij

  parent reply	other threads:[~2013-06-07  9:20 UTC|newest]

Thread overview: 41+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-21 14:07 [PATCH 0/3] pinctrl: pinctrl-single: Add full fledge support to configure multiple pins of different modules Manjunathappa, Prakash
2013-05-21 14:07 ` Manjunathappa, Prakash
2013-05-21 14:07 ` Manjunathappa, Prakash
2013-05-21 14:08 ` [PATCH 1/3] pinctrl: pinctrl-single: enhance " Manjunathappa, Prakash
2013-05-21 14:08   ` Manjunathappa, Prakash
2013-05-21 14:08   ` Manjunathappa, Prakash
2013-06-05  5:24   ` Haojian Zhuang
2013-06-05  5:24     ` Haojian Zhuang
2013-06-05  5:24     ` Haojian Zhuang
2013-06-05 22:01   ` Tony Lindgren
2013-06-05 22:01     ` Tony Lindgren
2013-06-07  9:20   ` Linus Walleij [this message]
2013-06-07  9:20     ` Linus Walleij
2013-06-07  9:20     ` Linus Walleij
2013-05-21 14:08 ` [PATCH 2/3] pinctrl: pinctrl-single: pin names for pinctrl-single.bits Manjunathappa, Prakash
2013-05-21 14:08   ` Manjunathappa, Prakash
2013-05-21 14:08   ` Manjunathappa, Prakash
2013-06-05  5:25   ` Haojian Zhuang
2013-06-05  5:25     ` Haojian Zhuang
2013-06-05  5:25     ` Haojian Zhuang
2013-06-05 22:02     ` Tony Lindgren
2013-06-05 22:02       ` Tony Lindgren
2013-06-05 22:02       ` Tony Lindgren
2013-06-07 11:14   ` Linus Walleij
2013-06-07 11:14     ` Linus Walleij
2013-06-07 11:14     ` Linus Walleij
2013-05-21 14:08 ` [PATCH 3/3] ARM: davinci: da850: adopt to pinctrl-single driver to configure multiple pins Manjunathappa, Prakash
2013-05-21 14:08   ` Manjunathappa, Prakash
2013-05-21 14:08   ` Manjunathappa, Prakash
2013-06-24 11:48   ` Manjunathappa, Prakash
2013-06-24 11:48     ` Manjunathappa, Prakash
2013-06-24 11:48     ` Manjunathappa, Prakash
2013-06-24 14:52     ` Sekhar Nori
2013-06-24 14:52       ` Sekhar Nori
2013-06-24 14:52       ` Sekhar Nori
2013-05-24  9:03 ` [PATCH 0/3] pinctrl: pinctrl-single: Add full fledge support to configure multiple pins of different modules Linus Walleij
2013-05-24  9:03   ` Linus Walleij
2013-05-24  9:03   ` Linus Walleij
2013-05-26 16:11   ` Haojian Zhuang
2013-05-26 16:11     ` Haojian Zhuang
2013-05-26 16:11     ` Haojian Zhuang

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CACRpkdYrmpptASxiM-YUrH-W71OxVFSh-kS_nX-Cw1Whe+fGBQ@mail.gmail.com \
    --to=linus.walleij@linaro.org \
    --cc=davinci-linux-open-source@linux.davincidsp.com \
    --cc=devicetree-discuss@lists.ozlabs.org \
    --cc=grant.likely@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --cc=nsekhar@ti.com \
    --cc=peter.ujfalusi@ti.com \
    --cc=prabhakar.csengg@gmail.com \
    --cc=prakash.pm@ti.com \
    --cc=rob.herring@calxeda.com \
    --cc=rob@landley.net \
    --cc=tony@atomide.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.