From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756652AbYGMVLg (ORCPT ); Sun, 13 Jul 2008 17:11:36 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1755496AbYGMVL2 (ORCPT ); Sun, 13 Jul 2008 17:11:28 -0400 Received: from py-out-1112.google.com ([64.233.166.176]:29799 "EHLO py-out-1112.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755479AbYGMVL1 (ORCPT ); Sun, 13 Jul 2008 17:11:27 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:to:subject:cc:in-reply-to:mime-version :content-type:content-transfer-encoding:content-disposition :references; b=U3tbpVPHrDoJhxnrqDTJZ0atE7bvDFu6WZHl3EirA5q0F3imtjQ3R9LN3oJAqEp602 dBHcFOzZUaWrSRWCJcstms3meWXtcr/JVQpS94fpLp4q1ARtBmUe9Byi4G4HyQoFGCne /vK0wl6Km1Loxti5kmcxjbmBAnAZZXmie4vLg= Message-ID: <4dfa50520807131411ied883cgcb20eb6bd94f761@mail.gmail.com> Date: Sun, 13 Jul 2008 15:11:24 -0600 From: "David Hubbard" To: "Hans de Goede" Subject: Re: [lm-sensors] [RFC] (almost) booting allyesconfig -- please don't poke super-io without request_region Cc: "Jean Delvare" , linuxppc-dev@ozlabs.org, "Samuel Ortiz" , linux-kernel@vger.kernel.org, "Milton Miller" , lm-sensors@lm-sensors.org In-Reply-To: <4879A144.8060203@hhs.nl> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <48768018.2070704@hhs.nl> <20080711085246.1ead773b@hyperion.delvare> <48770B5E.7000308@hhs.nl> <20080711093650.4b98e3b7@hyperion.delvare> <4879A144.8060203@hhs.nl> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Hans, Milton, Jean, On Sun, Jul 13, 2008 at 12:31 AM, Hans de Goede wrote: > Jean Delvare wrote: >> On Fri, 11 Jul 2008 09:27:26 +0200, Hans de Goede wrote: >>> Jean Delvare wrote: >>>> Hi Hans, hi Milton, >>>> >>> >>> >>>>> One could make a superio driver, and create sub-devices for the IR, >>>>> I2C, floppy, parallel, etc >>>>> nodes. >>>> There have been proposals to do this, and this would indeed be a very >>>> good idea, but unfortunately nobody took the time to implement this >>>> properly, push it upstream and volunteer to maintain it. The problem is >>>> that you don't need just a "driver", but a new subsystem, that needs to >>>> be designed and maintained. >>> Well, I believe there have been some lightweight superio locking coordinator >>> patches been floating around on the lm_sensors list, and I have reviewed them >>> and then a new version was done with my issues fixed. >>> >>> I kinda liked the proposed solution there, it was quite simple, moved all the >>> generic superio stuff into generic superio code, and added locking for super io >>> access from multiple drivers, what ever happened to those patches? >> >> As far as I know, nothing, and this is the problem. Somebody needs to >> step up and call him/herself the maintainer of the new code, and push >> it upstream and convert all the drivers (hwmon, watchdog, parallel >> port...) to make use of it. And I am not the one to do this, I am busy >> enough as is with i2c and hwmon. >> > > Ok, I've mailed Jim Cromie, the author of the superio access patches > from end 2007 and told him we're still interested in this and asked him > if he is willing to drive this forward. I propose writing a subsystem driver. (Is that properly called "The SuperIO Bus Driver"?) If no one thinks it's a really bad idea I will put together some code and submit it for review, and maintain it. Some hwmon chips have odd / unique probe sequences. IMHO this is where the design needs to be inspected. One of those is the w83627ehf, which Jean and Hans are familiar enough with to check my work. Thoughts? David