archive mirror
 help / color / mirror / Atom feed
From: Jean Delvare <>
Subject: Re: [PATCH 2.4] dmi_ident made public
Date: Fri, 25 Apr 2003 12:15:17 +0200	[thread overview]
Message-ID: <> (raw)
In-Reply-To: <>

> > If this patch is accepted and applied, I'll work together with Peter
> > to get the three above-mentioned modules simplified, as well as any
> > other I may have missed. Also, I'll take care of porting this patch
> > to the 2.5 series, since it also belongs there.
> i2c-piix4 in the 2.5 kernel tree does not need this patch, as
> everything it needs to detect IBM laptops is already made public.  See
> the current 2.5 releases to verify this.

Thanks for pointing this out. I took a look and could actually verify
it. This made me think again about the whole problem. I wondered wether
all other modules could be fixed that way (in which case my patch
wouldn't make sense), but it turns out that neither i8k nor omke can use
the same trick as the one you used for i2c-piix4. This is due to the
fact that i2c-piix4 only needs a flag (does the system match a given DMI
"mask"? yes/no) whereas the other modules need the actual data. Peter,
correct me if I'm wrong.

What's more, I plan to write another module that exports the DMI data,
as scanned at boot time, to userland (via sysfs), and such a module
definitely requires that the DMI data is made public in dmi_scan.

The good thing is that I think I now understand a bit better what Alan
meant yesterday by "register multiple DMI tables for boot time scanning
to set further flags in the dmi properties post scan". It must be what
you did for i2c-piix4, and isn't what I need there if my analysis is

So, I still think my patch makes sense and should be applied.

Jean Delvare

  reply	other threads:[~2003-04-25 10:03 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-04-24 16:47 [PATCH 2.4] dmi_ident made public Jean Delvare
2003-04-24 18:31 ` Alan Cox
2003-04-24 22:43   ` Jean Delvare
2003-04-24 23:10 ` Greg KH
2003-04-25 10:15   ` Jean Delvare [this message]
2003-04-25 10:11     ` Alan Cox
2003-04-26 12:20       ` Jean Delvare

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:

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \ \ \ \ \

* 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;
as well as URLs for NNTP newsgroup(s).