From: "Mark A. Greer" <mgreer@mvista.com>
To: Kumar Gala <galak@kernel.crashing.org>
Cc: "Mark A.Greer" <mgreer@mvista.com>,
Randy Vinson <rvinson@mvista.com>,
Jean Delvare <khali@linux-fr.org>,
LKML <linux-kernel@vger.kernel.org>,
Arjan van de Ven <arjan@infradead.org>,
Ingo Molnar <mingo@elte.hu>
Subject: Re: [PATCH, RFC] Stop using tasklet in ds1374 RTC driver
Date: Thu, 23 Mar 2006 15:27:23 -0700 [thread overview]
Message-ID: <20060323222723.GA3379@mag.az.mvista.com> (raw)
In-Reply-To: <C6071445-B39C-4230-92FA-E8EE5717FD05@kernel.crashing.org>
On Thu, Mar 23, 2006 at 03:52:16PM -0600, Kumar Gala wrote:
>
> On Mar 23, 2006, at 3:40 PM, Mark A. Greer wrote:
> >I'm no expert in workqueues either; however, after reading
> >http://lwn.net/Articles/23634/, I believe that its unnecessary for an
> >rtc driver to have its own workqueue since rtc writes aren't
> >particularly
> >time-critical. If I am correct, then Randy's patch uses the proper
> >wq calls.
> >
> >Agree?
>
> How does this change with the RTC subsystem that's about to be added?
> (could be irrelevant, just wanted to put that out there)
Dunno, but its broken where it is now so we need to fix it ASAP.
Once that's done, we can do whatever needs to be done to get it into
drivers/rtc.
Mark
next prev parent reply other threads:[~2006-03-23 22:26 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-03-23 19:10 Jean Delvare
2006-03-23 20:42 ` Randy Vinson
2006-03-23 21:40 ` Mark A. Greer
[not found] ` <C6071445-B39C-4230-92FA-E8EE5717FD05@kernel.crashing.org>
2006-03-23 22:27 ` Mark A. Greer [this message]
2006-03-24 20:53 ` Jean Delvare
2006-03-27 20:38 ` Mark A. Greer
2006-03-27 21:15 ` Jean Delvare
2006-03-27 22:31 ` Randy Vinson
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=20060323222723.GA3379@mag.az.mvista.com \
--to=mgreer@mvista.com \
--cc=arjan@infradead.org \
--cc=galak@kernel.crashing.org \
--cc=khali@linux-fr.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=rvinson@mvista.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
Powered by JetHome