From: Stas Sergeev <stsp@aknet.ru>
To: Sam Ravnborg <sam@ravnborg.org>
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>,
Andrew Morton <akpm@linux-foundation.org>,
Takashi Iwai <tiwai@suse.de>,
Linux kernel <linux-kernel@vger.kernel.org>,
Vojtech Pavlik <vojtech@suse.cz>,
Michael Opdenacker <michael@free-electrons.com>,
Ingo Molnar <mingo@elte.hu>, Thomas Gleixner <tglx@linutronix.de>
Subject: Re: [patch] pcspkr: fix dependancies
Date: Tue, 29 Apr 2008 23:56:32 +0400 [thread overview]
Message-ID: <48177D70.5010803@aknet.ru> (raw)
In-Reply-To: <20080428201225.GB18135@uranus.ravnborg.org>
Hello.
Sam Ravnborg wrote:
> If you google you will find a long thread about this patch.
Yes, indeed, thankyou.
http://kerneltrap.org/mailarchive/linux-kernel/2008/1/17/577151
> It is all about saving memory for embedded platforms.
OK. This wasn't clear from the log
message for some reason, but it is
said in the above thread.
> No need to have the driver if there is no speaker on the board.
Very probably, and for that case I
made the patch like this:
http://uwsg.ucs.indiana.edu/hypermail/linux/kernel/0804.3/1189.html
> Ask Michael (author) if you need more info.
Reading the thread, it actually seems
like Michael wanted to submit the patch
much like the aforementioned one of
mine:
http://kerneltrap.org/mailarchive/linux-kernel/2008/1/18/580583
but haven't done so in time, and as the
result, the wrong one stuck in.
But this is never too late to correct, I
hope.
So I see 2 options: either revert the
patch completely, or revert the dependancies
like Michael suggested initially (if I
understand his suggestion right).
Both patches are here, in this thread.
Thoughts?
next prev parent reply other threads:[~2008-04-29 19:57 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-04-26 15:16 [patch] pcspkr: fix dependancies Stas Sergeev
2008-04-26 23:18 ` Dmitry Torokhov
2008-04-26 23:39 ` Stas Sergeev
2008-04-27 4:23 ` Dmitry Torokhov
2008-04-28 19:34 ` Stas Sergeev
2008-04-28 19:58 ` Dmitry Torokhov
2008-04-28 20:12 ` Sam Ravnborg
2008-04-28 20:19 ` Dmitry Torokhov
2008-04-29 19:56 ` Stas Sergeev [this message]
2008-04-30 7:31 ` Michael Opdenacker
2008-04-30 11:23 ` Stas Sergeev
2008-05-01 7:08 ` Stas Sergeev
-- strict thread matches above, loose matches on Subject: below --
2008-05-04 4:22 [patch][resend] " Stas Sergeev
2008-05-05 3:57 ` Dmitry Torokhov
2008-05-05 4:26 ` Stas Sergeev
2008-05-05 4:35 ` Dmitry Torokhov
2008-05-05 5:02 ` [patch][resend2] " Stas Sergeev
2008-05-05 11:02 ` Takashi Iwai
2008-05-08 17:50 ` [patch] " Stas Sergeev
2008-05-08 13:58 ` Takashi Iwai
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=48177D70.5010803@aknet.ru \
--to=stsp@aknet.ru \
--cc=akpm@linux-foundation.org \
--cc=dmitry.torokhov@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=michael@free-electrons.com \
--cc=mingo@elte.hu \
--cc=sam@ravnborg.org \
--cc=tglx@linutronix.de \
--cc=tiwai@suse.de \
--cc=vojtech@suse.cz \
/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