From: "Ville Syrjälä" <syrjala@sci.fi>
To: Bjorn Helgaas <bjorn.helgaas@hp.com>
Cc: Krzysztof Helt <krzysztof.h1@wp.pl>, Adam Belay <abelay@mit.edu>,
linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org
Subject: Re: How should I handle device with two PNP-BIOS ids?
Date: Tue, 27 Jan 2009 19:17:52 +0200 [thread overview]
Message-ID: <20090127171752.GI4996@sci.fi> (raw)
In-Reply-To: <200901270852.52153.bjorn.helgaas@hp.com>
On Tue, Jan 27, 2009 at 08:52:51AM -0700, Bjorn Helgaas wrote:
> On Tuesday 27 January 2009 01:42:55 am Krzysztof Helt wrote:
> > On Mon, 26 Jan 2009 15:13:31 -0700
> > Bjorn Helgaas <bjorn.helgaas@hp.com> wrote:
> > > Systems with the old chip probably report the device with a single
> > > PNP ID. For PNPBIOS, that would be the "device product identifier."
> > > For ACPI, it would be the _HID.
> > >
> > > Systems with the new chip should report both PNP IDs. The PNPBIOS
> > > device product identifier or the ACPI _HID should be the new ID, and
> > > the older ID for backward compatibility should be reported in the
> > > PNPBIOS "compatible device identifiers" list or the ACPI _CID.
> > >
> > > The driver would list both IDs in the pnp_device_id table, and it
> > > looks like the driver probe routine gets called with the ID that
> > > matched.
> > >
> > > I don't see any drivers that actually do this, but I think the
> > > probe routine should be able to look at the ID that matched and
> > > determine whether the additional I/O range is present.
> >
> > The laptop in question is the Dell Latitude CPi and it does not
> > have ACPI only the PnP BIOS.
> > The both ids are reported as unrelated (two separate nodes in
> > the /sys/bus/pnp/).
>
> Oh. That sounds like a BIOS bug. If there's only one actual
> chip, the BIOS should not report it as two separate devices.
It's not a bug. The cs423x chips contain several logical devices.
WSS, MPU401, joystick etc. The control logical device (which is
not present in the early models) is unfortunately some kind of dumping
ground for new registers to keep the other logical devices' backwards
compatible. It contains at least some extra mixer control registers and
power management registers.
The datasheets are available at ftp.alsa-project.org.
--
Ville Syrjälä
syrjala@sci.fi
http://www.sci.fi/~syrjala/
next prev parent reply other threads:[~2009-01-27 17:18 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-01-26 9:21 How should I handle device with two PNP-BIOS ids? Krzysztof Helt
2009-01-26 22:13 ` Bjorn Helgaas
2009-01-27 8:42 ` Krzysztof Helt
2009-01-27 15:52 ` Bjorn Helgaas
2009-01-27 17:10 ` Krzysztof Helt
2009-01-27 17:17 ` Ville Syrjälä [this message]
2009-02-05 0:00 ` Bjorn Helgaas
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=20090127171752.GI4996@sci.fi \
--to=syrjala@sci.fi \
--cc=abelay@mit.edu \
--cc=bjorn.helgaas@hp.com \
--cc=krzysztof.h1@wp.pl \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-kernel@vger.kernel.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