All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jean Delvare <khali@linux-fr.org>
To: djwong@us.ibm.com
Cc: "D. Can Celasun" <dcelasun@gmail.com>,
	Huaxu Wan <huaxu.wan@intel.com>,
	lm-sensors@lm-sensors.org, linux-acpi@vger.kernel.org,
	lenb@kernel.org
Subject: Re: [lm-sensors] Core i3 support?
Date: Fri, 12 Feb 2010 20:09:55 +0100	[thread overview]
Message-ID: <20100212200955.05730d69@hyperion.delvare> (raw)
In-Reply-To: <20100212185241.GJ29604@tux1.beaverton.ibm.com>

On Fri, 12 Feb 2010 10:52:41 -0800, Darrick J. Wong wrote:
> On Thu, Feb 11, 2010 at 06:42:28PM +0200, D. Can Celasun wrote:
> > On 11 February 2010 10:17, Jean Delvare <khali@linux-fr.org> wrote:
> > > Darrick, any chance we could get the i2c-scmi driver to load
> > > automatically on systems which need it?
> 
> I think all we need to do is add:
> 
> MODULE_DEVICE_TABLE(acpi, fan_device_ids);

I guess you mean:

MODULE_DEVICE_TABLE(acpi, acpi_smbus_cmi_ids);

> 
> to i2c-scmi.c, which will add the appropriate aliases to the module info.

OK. Care to send a patch?

> That reminds me, I don't think that patch to add ACPI_SMBUS_IBM_HID to the ACPI
> header files ever went in, did it?  It doesn't seem to be in 2.6.33-rc7, but
> maybe it's queued in someone else's ACPI tree for .34?

I have pinged Bjorn and Len twice already (December 17th, January 5th)
but never heard back.

> If it's not, then I'd
> imagine it's still blocking the patchset to i2c-scmi that works around broken
> IBM BIOSes, so I'm hoping someone from the ACPI part of the world can comment?

Yes it is blocking the i2c-scmi patch. And yes, please ACPI people take
care of it ASAP.

-- 
Jean Delvare

WARNING: multiple messages have this Message-ID (diff)
From: Jean Delvare <khali@linux-fr.org>
To: djwong@us.ibm.com
Cc: "D. Can Celasun" <dcelasun@gmail.com>,
	Huaxu Wan <huaxu.wan@intel.com>,
	lm-sensors@lm-sensors.org, linux-acpi@vger.kernel.org,
	lenb@kernel.org
Subject: Re: [lm-sensors] Core i3 support?
Date: Fri, 12 Feb 2010 19:09:55 +0000	[thread overview]
Message-ID: <20100212200955.05730d69@hyperion.delvare> (raw)
In-Reply-To: <20100212185241.GJ29604@tux1.beaverton.ibm.com>

On Fri, 12 Feb 2010 10:52:41 -0800, Darrick J. Wong wrote:
> On Thu, Feb 11, 2010 at 06:42:28PM +0200, D. Can Celasun wrote:
> > On 11 February 2010 10:17, Jean Delvare <khali@linux-fr.org> wrote:
> > > Darrick, any chance we could get the i2c-scmi driver to load
> > > automatically on systems which need it?
> 
> I think all we need to do is add:
> 
> MODULE_DEVICE_TABLE(acpi, fan_device_ids);

I guess you mean:

MODULE_DEVICE_TABLE(acpi, acpi_smbus_cmi_ids);

> 
> to i2c-scmi.c, which will add the appropriate aliases to the module info.

OK. Care to send a patch?

> That reminds me, I don't think that patch to add ACPI_SMBUS_IBM_HID to the ACPI
> header files ever went in, did it?  It doesn't seem to be in 2.6.33-rc7, but
> maybe it's queued in someone else's ACPI tree for .34?

I have pinged Bjorn and Len twice already (December 17th, January 5th)
but never heard back.

> If it's not, then I'd
> imagine it's still blocking the patchset to i2c-scmi that works around broken
> IBM BIOSes, so I'm hoping someone from the ACPI part of the world can comment?

Yes it is blocking the i2c-scmi patch. And yes, please ACPI people take
care of it ASAP.

-- 
Jean Delvare

_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

  reply	other threads:[~2010-02-12 19:09 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-02-10 11:00 [lm-sensors] Core i3 support? D. Can Celasun
2010-02-10 12:25 ` Jean Delvare
2010-02-10 12:28 ` D. Can Celasun
2010-02-10 13:21 ` Jean Delvare
2010-02-10 13:42 ` D. Can Celasun
2010-02-10 14:44 ` Jean Delvare
2010-02-10 15:03 ` D. Can Celasun
2010-02-10 18:01 ` Darrick J. Wong
2010-02-10 18:07 ` D. Can Celasun
2010-02-10 19:40 ` Jean Delvare
2010-02-10 20:23 ` D. Can Celasun
2010-02-11  1:21 ` Darrick J. Wong
2010-02-11  7:33 ` D. Can Celasun
2010-02-11  8:17 ` Jean Delvare
2010-02-11 16:42 ` D. Can Celasun
2010-02-12 18:52   ` Darrick J. Wong
2010-02-12 18:52     ` Darrick J. Wong
2010-02-12 19:09     ` Jean Delvare [this message]
2010-02-12 19:09       ` Jean Delvare
2010-02-12 21:51       ` [PATCH] i2c-scmi: Provide module aliases for automatic loading Darrick J. Wong
2010-02-12 21:51         ` [lm-sensors] [PATCH] i2c-scmi: Provide module aliases for automatic Darrick J. Wong
2010-02-13 13:59         ` [PATCH] i2c-scmi: Provide module aliases for automatic loading Jean Delvare
2010-02-13 13:59           ` [lm-sensors] [PATCH] i2c-scmi: Provide module aliases for Jean Delvare
2010-02-19 21:56       ` Core i3 support? D. Can Celasun
2010-02-19 21:56         ` [lm-sensors] " D. Can Celasun

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=20100212200955.05730d69@hyperion.delvare \
    --to=khali@linux-fr.org \
    --cc=dcelasun@gmail.com \
    --cc=djwong@us.ibm.com \
    --cc=huaxu.wan@intel.com \
    --cc=lenb@kernel.org \
    --cc=linux-acpi@vger.kernel.org \
    --cc=lm-sensors@lm-sensors.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.