From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from wa-out-1112.google.com (wa-out-1112.google.com [209.85.146.182]) by ozlabs.org (Postfix) with ESMTP id 0F8CADDE17 for ; Mon, 14 Jul 2008 07:11:25 +1000 (EST) Received: by wa-out-1112.google.com with SMTP id n7so2518290wag.13 for ; Sun, 13 Jul 2008 14:11:24 -0700 (PDT) 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 In-Reply-To: <4879A144.8060203@hhs.nl> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 References: <48768018.2070704@hhs.nl> <20080711085246.1ead773b@hyperion.delvare> <48770B5E.7000308@hhs.nl> <20080711093650.4b98e3b7@hyperion.delvare> <4879A144.8060203@hhs.nl> Cc: Samuel Ortiz , linux-kernel@vger.kernel.org, Milton Miller , lm-sensors@lm-sensors.org, linuxppc-dev@ozlabs.org, Jean Delvare List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , 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