mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
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>


  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®