From: Stelian Pop <stelian@popies.net>
To: dtor_core@ameritech.net
Cc: Mattia Dongili <malattia@linux.it>, LKML <linux-kernel@vger.kernel.org>
Subject: Re: sonypi searching new maintainer (Was: Re: [RFT] Sonypi: convert to the new platform device interface)
Date: Wed, 14 Dec 2005 07:46:40 +0100 [thread overview]
Message-ID: <1134542801.9642.39.camel@deep-space-9.dsnet> (raw)
In-Reply-To: <d120d5000512131406t2cff3c19ic309c5cb766b8d01@mail.gmail.com>
Le mardi 13 décembre 2005 à 17:06 -0500, Dmitry Torokhov a écrit :
> On 12/13/05, Stelian Pop <stelian@popies.net> wrote:
> > Le mardi 13 décembre 2005 à 15:48 -0500, Dmitry Torokhov a écrit :
> > > On 12/13/05, Stelian Pop <stelian@popies.net> wrote:
> > > > Le mardi 13 décembre 2005 à 14:04 -0500, Dmitry Torokhov a >
> > > > > > sonypi: unknown event port1=0x0f,port2=0x05
> > > > [...]
> > > > > > Oh, there seems to be a spurious interrupt happening at modules
> > > > > > insertion (I suspect sonypi_enable triggering and ignoring it), but this
> > > > > > happens with the old module too and I never noticed it before. Wouldn't
> > > > > > make more sense to print the warning even if verbose=0 to be able to
> > > > > > catch it timely? I mean it's since 2.4 times I don't enable verbose mode
> > > > > > in sonypi...
> > > > > >
> > > > > Probably, let's see what Stelian will say.
> > > >
> > > > This is the "ok I'm loaded" event. I am not sure this event is available
> > > > on all the sonypi supported platforms, that's why it hasn't been defined
> > > > as a known event. And it doesn't make much sense to forward it anyways.
> > > >
> > > > I would leave it like it is now.
> > > >
> > >
> > > But when it is reported is it the same event?
> >
> > I don't follow you here...
> >
>
> I was talking about that "ok, i'm loaded" event. When it is reported
> is it the same v1,v2 pair all the time?
I have no idea. That's one of the problems with the sonypi events:
different laptops behave differently, and sometimes Sony even changes
the meaning of some events...
> If it is the same we could
> suppress "unknown" message in verbose mode.
Why would you want to suppress a message in verbose mode ? The verbose
option is there exactly for that: show the user all the data coming from
the hardware, and depending on the verbose value only show the data
which hasn't been decoded, or all of it.
I don't think that adding a special hack for this event in an already
hacky driver has some added value.
Stelian.
--
Stelian Pop <stelian@popies.net>
next prev parent reply other threads:[~2005-12-14 6:46 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-12-13 7:19 [RFT] Sonypi: convert to the new platform device interface Dmitry Torokhov
2005-12-13 18:32 ` Mattia Dongili
2005-12-13 19:04 ` Dmitry Torokhov
2005-12-13 20:30 ` sonypi searching new maintainer (Was: Re: [RFT] Sonypi: convert to the new platform device interface) Stelian Pop
2005-12-13 20:48 ` Dmitry Torokhov
2005-12-13 21:55 ` Stelian Pop
2005-12-13 22:06 ` Dmitry Torokhov
2005-12-14 6:46 ` Stelian Pop [this message]
2005-12-25 21:09 ` [RFT] Sonypi: convert to the new platform device interface Jan Engelhardt
2005-12-25 21:17 ` Dmitry Torokhov
2005-12-27 18:02 ` Jan Engelhardt
2005-12-27 22:18 ` Dmitry Torokhov
2005-12-27 22:53 ` Stelian Pop
2006-01-15 20:48 ` Jan Engelhardt
2006-01-16 4:48 ` Dmitry Torokhov
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=1134542801.9642.39.camel@deep-space-9.dsnet \
--to=stelian@popies.net \
--cc=dtor_core@ameritech.net \
--cc=linux-kernel@vger.kernel.org \
--cc=malattia@linux.it \
/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
all inboxes | Powered by JetHome®