public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Frank Gevaerts <frank@gevaerts.be>
To: Jean Delvare <khali@linux-fr.org>
Cc: Frank Gevaerts <frank@gevaerts.be>, Robert Love <rlove@rlove.org>,
	linux-kernel@vger.kernel.org
Subject: Re: patch : hdaps on Thinkpad R52
Date: Sat, 1 Apr 2006 23:32:55 +0200	[thread overview]
Message-ID: <20060401213255.GA30380@gevaerts.be> (raw)
In-Reply-To: <20060401170422.cc2ff8c2.khali@linux-fr.org>

On Sat, Apr 01, 2006 at 05:04:22PM +0200, Jean Delvare wrote:
> Hi Frank,
> 
> > > I have some doubt about this. The Thinkpad R52 is already supported
> > > (with identifier string "ThinkPad R52", unsuprisingly) and "ThinkPad H"
> > > doesn't exactly sound sane. Looks like your DMI data is corrupted or
> > > something. Could you please provide the output of dmidecode and
> > > vpddecode on your laptop?
> > 
> > They are attached.
> 
> > # dmidecode 2.8
> > (...)
> > System Information
> > 	Manufacturer: IBM
> > 	Product Name: 1846AQG
> > 	Version: ThinkPad H   
> 
> OK, so as strange as it sounds, that's really the string as stored in
> the DMI table.

Yes. That's where I looked when I made the patch.

> How odd... You have to understand that I'm a bit
> reluctant to adding it officially to the hdaps driver, given that it
> clearly looks like a bogus table in your laptop.

If I am the only one with that entry, it obviously makes more sense that
I apply the patch locally.

> I guess that you only have one laptop with this string?

It's the only thinkpad I have, so yes. I'll ask around on the thinkwiki
mailing list and irc channels

> Can you please check for a BIOS update for your laptop? Maybe that will
> fix the string.

I updated the bios and embedded controller program to the latest
version, and that didn't help. The dmidecode output changed as follows:

--- dmidecode.old       2006-04-01 23:21:46.000000000 +0200
+++ dmidecode.new       2006-04-01 23:21:39.000000000 +0200
@@ -6,8 +6,8 @@
 Handle 0x0000, DMI type 0, 20 bytes
 BIOS Information
        Vendor: IBM
-       Version: 76ET58WW (1.18 )
-       Release Date: 07/19/2005
+       Version: 76ET63WW (1.23 )
+       Release Date: 02/14/2006
        Address: 0xDC000
        Runtime Size: 144 kB
        ROM Size: 1024 kB
@@ -361,7 +361,7 @@
 
 Handle 0x0029, DMI type 11, 5 bytes
 OEM Strings
-       String 1: IBM ThinkPad Embedded Controller -[76HT14WW-1.04    ]-
+       String 1: IBM ThinkPad Embedded Controller -[76HT15WW-1.05    ]-
 
 Handle 0x002A, DMI type 13, 22 bytes
 BIOS Language Information
@@ -516,7 +516,7 @@
 Handle 0x003B, DMI type 134, 13 bytes
 OEM-specific Type
        Header and Data:
-               86 0D 3B 00 0B 00 04 02 0B 00 17 02 0B
+               86 0D 3B 00 02 0B 00 09 02 0B 00 12 02
 
 Handle 0x003C, DMI type 135, 13 bytes
 OEM-specific Type

Frank

> Thanks,
> -- 
> Jean Delvare

-- 
"Debugging is twice as hard as writing the code in the first place.
Therefore, if you write the code as cleverly as possible, you are,
by definition, not smart enough to debug it." - Brian W. Kernighan

  reply	other threads:[~2006-04-01 21:33 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-03-14 20:57 patch : hdaps on Thinkpad R52 Frank Gevaerts
2006-03-25 21:09 ` Adrian Bunk
2006-03-25 23:35   ` Arkadiusz Miskiewicz
2006-03-28 16:56     ` Jean Delvare
2006-03-29 16:44   ` [PATCH 2.6.16-rc4] hdaps: add support for " Frank Gevaerts
2006-03-28 16:29 ` patch : hdaps on " Jean Delvare
2006-03-28 17:00   ` Frank Gevaerts
2006-04-01 15:04     ` Jean Delvare
2006-04-01 21:32       ` Frank Gevaerts [this message]
2006-04-03 16:35       ` Frank Gevaerts
2006-04-08 15:02         ` Jean Delvare
2006-04-08 16:53           ` Frank Gevaerts
2006-10-11 14:07           ` Jean Delvare
2006-10-11 14:38             ` Frank Gevaerts

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=20060401213255.GA30380@gevaerts.be \
    --to=frank@gevaerts.be \
    --cc=khali@linux-fr.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rlove@rlove.org \
    /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