From: Jason Cooper <jason@lakedaemon.net>
To: Andrew Lunn <andrew@lunn.ch>
Cc: Chris Ball <cjb@laptop.org>,
gmbnomis@gmail.com, linux-arm-kernel@lists.infradead.org,
linux-mmc@vger.kernel.org, thomas.petazzoni@free-electrons.com,
Guennadi Liakhovetski <g.liakhovetski@gmx.de>
Subject: Re: [PATCH 1/3] mmc: mvsdio: Support inverted CD and WP GPIO lines
Date: Mon, 22 Apr 2013 18:54:56 -0400 [thread overview]
Message-ID: <20130422225456.GK25724@titan.lakedaemon.net> (raw)
In-Reply-To: <20130422214543.GJ29947@lunn.ch>
On Mon, Apr 22, 2013 at 11:45:43PM +0200, Andrew Lunn wrote:
> > > > The core function mmc_of_parse() (new in 3.9) will do all of this DT
> > > > parsing for you, and will also take care of the driver's later calls
> > > > to mmc_gpio_request_{cd,wp}(). Could you try using that function
> > > > instead, please?
> > >
> > > Hi Chris, Simon
> > >
> > > I have a vague recollection that Thomas Petazzoni already did this
> > > conversion. Take a look in linux-next, or maybe Jason can point you at
> > > a branch.
> >
> > Andrew,
> >
> > Thomas did a board conversion here (slated for v3.11 in /pcie branch):
> >
> > cd2bde5 arm: kirkwood: convert db-88f6281 to the Device Tree
> >
> > And Sebastian did some mvsdio WP and CD pin work here:
> >
> > 0d0644eARM: Kirkwood: fix unused mvsdio gpio pins
> >
> > Nothing else really jumps out though. Where either of these what you
> > were thinking of?
>
> http://lists.infradead.org/pipermail/linux-arm-kernel/2012-December/138514.html
>
> and
>
> http://lists.infradead.org/pipermail/linux-arm-kernel/2012-December/138513.html
Ahh, they were with the original submission for mvsdio:
07728b7 mmc: mvsdio: use slot-gpio for card detect gpio
3724482 mmc: mvsdio: use slot-gpio infrastructure for write protect gpio
I thought you were referring to a newer patch.
thx,
Jason.
WARNING: multiple messages have this Message-ID (diff)
From: jason@lakedaemon.net (Jason Cooper)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/3] mmc: mvsdio: Support inverted CD and WP GPIO lines
Date: Mon, 22 Apr 2013 18:54:56 -0400 [thread overview]
Message-ID: <20130422225456.GK25724@titan.lakedaemon.net> (raw)
In-Reply-To: <20130422214543.GJ29947@lunn.ch>
On Mon, Apr 22, 2013 at 11:45:43PM +0200, Andrew Lunn wrote:
> > > > The core function mmc_of_parse() (new in 3.9) will do all of this DT
> > > > parsing for you, and will also take care of the driver's later calls
> > > > to mmc_gpio_request_{cd,wp}(). Could you try using that function
> > > > instead, please?
> > >
> > > Hi Chris, Simon
> > >
> > > I have a vague recollection that Thomas Petazzoni already did this
> > > conversion. Take a look in linux-next, or maybe Jason can point you at
> > > a branch.
> >
> > Andrew,
> >
> > Thomas did a board conversion here (slated for v3.11 in /pcie branch):
> >
> > cd2bde5 arm: kirkwood: convert db-88f6281 to the Device Tree
> >
> > And Sebastian did some mvsdio WP and CD pin work here:
> >
> > 0d0644eARM: Kirkwood: fix unused mvsdio gpio pins
> >
> > Nothing else really jumps out though. Where either of these what you
> > were thinking of?
>
> http://lists.infradead.org/pipermail/linux-arm-kernel/2012-December/138514.html
>
> and
>
> http://lists.infradead.org/pipermail/linux-arm-kernel/2012-December/138513.html
Ahh, they were with the original submission for mvsdio:
07728b7 mmc: mvsdio: use slot-gpio for card detect gpio
3724482 mmc: mvsdio: use slot-gpio infrastructure for write protect gpio
I thought you were referring to a newer patch.
thx,
Jason.
next prev parent reply other threads:[~2013-04-22 22:55 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-04-22 19:54 [PATCH 0/3] ARM: Kirkwood: Add DT support for (eSATA) Sheevaplug Simon Baatz
2013-04-22 19:54 ` Simon Baatz
2013-04-22 19:54 ` [PATCH 1/3] mmc: mvsdio: Support inverted CD and WP GPIO lines Simon Baatz
2013-04-22 19:54 ` Simon Baatz
2013-04-22 20:03 ` Chris Ball
2013-04-22 20:03 ` Chris Ball
2013-04-22 20:37 ` Andrew Lunn
2013-04-22 20:37 ` Andrew Lunn
2013-04-22 21:09 ` Jason Cooper
2013-04-22 21:09 ` Jason Cooper
2013-04-22 21:45 ` Andrew Lunn
2013-04-22 21:45 ` Andrew Lunn
2013-04-22 21:50 ` Chris Ball
2013-04-22 21:50 ` Chris Ball
2013-04-23 1:48 ` Thomas Petazzoni
2013-04-23 1:48 ` Thomas Petazzoni
2013-04-23 21:22 ` Simon Baatz
2013-04-23 21:22 ` Simon Baatz
2013-04-22 22:54 ` Jason Cooper [this message]
2013-04-22 22:54 ` Jason Cooper
2013-04-22 19:54 ` [PATCH 2/3] ARM: Kirkwood: Add dts files for Sheevaplug and eSATA Sheevaplug Simon Baatz
2013-04-22 19:54 ` Simon Baatz
2013-04-22 19:55 ` [PATCH 3/3] ARM: Kirkwood: Add DT support for Sheevaplug and Sheevaplug eSATA Simon Baatz
2013-04-22 19:55 ` Simon Baatz
2013-04-23 5:00 ` [PATCH 0/3] ARM: Kirkwood: Add DT support for (eSATA) Sheevaplug Andrew Lunn
2013-04-23 5:00 ` Andrew Lunn
2013-05-13 17:30 ` Jason Cooper
2013-05-13 17:30 ` Jason Cooper
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=20130422225456.GK25724@titan.lakedaemon.net \
--to=jason@lakedaemon.net \
--cc=andrew@lunn.ch \
--cc=cjb@laptop.org \
--cc=g.liakhovetski@gmx.de \
--cc=gmbnomis@gmail.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-mmc@vger.kernel.org \
--cc=thomas.petazzoni@free-electrons.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.