From: Janusz Krzysztofik <jkrzyszt@tis.icnet.pl>
To: Mark Brown <broonie@opensource.wolfsonmicro.com>
Cc: alsa-devel@alsa-project.org, Jonathan McDowell <noodles@earth.li>,
Tony Lindgren <tony@atomide.com>,
Peter Ujfalusi <peter.ujfalusi@nokia.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
e3-hacking@earth.li, Arun KS <arunks@mistralsolutions.com>,
"linux-serial@vger.kernel.org" <linux-serial@vger.kernel.org>,
"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>,
Alan Cox <alan@lxorguk.ukuu.org.uk>
Subject: Re: [RFC] [PATCH 3/3 v2] ASoC: add support for Amstrad E3 (Delta) machine
Date: Fri, 31 Jul 2009 13:58:41 +0200 [thread overview]
Message-ID: <4A72DC71.6020904@tis.icnet.pl> (raw)
In-Reply-To: <20090731095234.GB10844@rakim.wolfsonmicro.main>
Mark Brown wrote:
> On Wed, Jul 29, 2009 at 01:18:53PM +0200, Janusz Krzysztofik wrote:
>
>> 2) patch 2/3 form this series: TTY: Add definition of a new line
>> discipline required by Amstrad E3 (Delta) ASoC driver[2].
>
> The ASoC parts of this all look OK - what's the status of the TTY patch?
>
Alan replied off-list yet before v2 submission. Then I replied him
off-list as well... I hope he doesn't mind quoting him here.
Janusz Krzysztofik wrote:
> Alan Cox wrote:
>> Interesting use of a line discipline. The tty/ldisc bits look good to me
>>
>
> Thanks.
> Could you please have a look at today's version 2 of the patch 3/3 as
> well? The ldisc bits has been split into machine and codec related parts
> and, what's more important, updated with locking that I am not quite
> sure about its correctness.
>
> Furthermore, I am still thinking about renaming the ldisc with a more
> generic name. That way, it could be probably reused by other similiar
> implementations. As long as the ldisc is registered from a board
> specific code, it should not interfere with other implemnetations
> reusing the same ldisc number, even if compiled in side by side.
Any objections on renaming ldisc from N_AMSDLETA to N_V253?
Thnaks,
Janusz
next prev parent reply other threads:[~2009-07-31 11:58 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-07-29 11:18 [RFC] [PATCH 3/3 v2] ASoC: add support for Amstrad E3 (Delta) machine Janusz Krzysztofik
2009-07-30 9:12 ` Janusz Krzysztofik
2009-07-31 21:38 ` Mark Brown
2009-07-31 9:52 ` Mark Brown
2009-07-31 11:58 ` Janusz Krzysztofik [this message]
2009-07-31 12:16 ` Alan Cox
2009-07-31 13:27 ` Janusz Krzysztofik
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=4A72DC71.6020904@tis.icnet.pl \
--to=jkrzyszt@tis.icnet.pl \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=alsa-devel@alsa-project.org \
--cc=arunks@mistralsolutions.com \
--cc=broonie@opensource.wolfsonmicro.com \
--cc=e3-hacking@earth.li \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-omap@vger.kernel.org \
--cc=linux-serial@vger.kernel.org \
--cc=noodles@earth.li \
--cc=peter.ujfalusi@nokia.com \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox