[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <201112212314.39844.jkrzyszt@tis.icnet.pl>
Date: Wed, 21 Dec 2011 23:14:39 +0100
From: Janusz Krzysztofik <jkrzyszt@....icnet.pl>
To: Liam Girdwood <lrg@...com>
Cc: Tony Lindgren <tony@...mide.com>,
Jarkko Nikula <jarkko.nikula@...mer.com>,
linux-omap@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
linux-kernel@...r.kernel.org, alsa-devel@...a-project.org,
Mark Brown <broonie@...nsource.wolfsonmicro.com>
Subject: Re: [PATCH 07/10] ASoC: OMAP: ams-delta: Drive modem/codec pins over GPIO API
On Wednesday 21 of December 2011 at 19:33:44, Liam Girdwood wrote:
> On Sun, 2011-12-11 at 21:12 +0100, Janusz Krzysztofik wrote:
> > Don't use Amstrad Delta custom I/O functions any longer, replace them
> > with GPIO. Old pin definitions, no longer used by the modem bits either,
> > can be dropped.
> >
> > Depends on patch 2/10 "ARM: OMAP1: Convert Amstrad E3 latches to
> > basic_mmio_gpio".
> >
> > Signed-off-by: Janusz Krzysztofik <jkrzyszt@....icnet.pl>
>
> Applied.
Hi,
I'm sorry, but I've probably missed to infrom you explicitly, i.e. other
than in a cover letter to v2 of this series, that this particular patch
has been dropped from the series. I've already started working on an
alternative solution, which I intend to submit as a separate set. It
will be free of potentail problems with two devices, a modem and the
sound card, accessing the same GPIO pins concurrently. For the
MODEM_NRESET pin I've already set up a virtual regulator on top of it,
and for the MODEM_CODEC pin I'll probably choose a virtual clock (pinmux
would match the hardware more closely here, but it looks too complicated
to me). Moreover, I've already started to move all those GPIO pin
related functions, i.e. bias control, digital mute and line discipline
interaction, from the board file to the codec driver, where all those
seem to belong.
Having this patch applied will conflict with the regultor and the clock
solutions, taking control over those two GPIO pins. Please revert this
patch, or reset it if still possible.
Thanks,
Janusz
--
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/
Powered by blists - more mailing lists