Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759319Ab2FAUD2 (ORCPT ); Fri, 1 Jun 2012 16:03:28 -0400 Received: from zoneX.GCU-Squad.org ([194.213.125.0]:27715 "EHLO services.gcu-squad.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759236Ab2FAUD1 (ORCPT ); Fri, 1 Jun 2012 16:03:27 -0400 Date: Fri, 1 Jun 2012 22:03:15 +0200 From: Jean Delvare To: Geert Uytterhoeven Cc: Meelis Roos , Dean Nelson , Linux Kernel list Subject: Re: f71805f arch dependencies? Message-ID: <20120601220315.246ba5a5@endymion.delvare> In-Reply-To: References: X-Mailer: Claws Mail 3.7.10 (GTK+ 2.24.7; x86_64-suse-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1912 Lines: 45 Hi there, On Fri, 1 Jun 2012 15:23:04 +0200, Geert Uytterhoeven wrote: > On Fri, Jun 1, 2012 at 3:12 PM, Meelis Roos wrote: > >> > Loading f71805f on sparc64 causes hard reboot beacuse on invalid > >> > hardware access. Should this driver be restricted to PC, or some wider > >> > subset of platforms where this LPC SUper-IO chip is present? > >> > >> Which tree is f71805f in? Not in Linus'. > > > > Running upstream 3.4.0-09547-gfb21aff and it is there as > > SENSORS_F71805F. > > > > In fact, there are more LPC hwmon drivers that people can use to "shoot > > themselves in the leg" - SMSC LPC*, SMSC SCH*, SMSC DME1737 (partly I2C, > > partly ISA IO), and Fintek F71882FG and compatibles. Maybe more. > > Sorry, I thought it was a git SHA1 commit ID. > Didn't know hardware manufacturers christen their chips to the abbreviated > commit ID in their VHDL repository ;-) > > Indeed, it seems commit 746cdfbf01c0a30d59f6e1b6942d432658d7c7cd > ("hwmon: Avoid building drivers for powerpc that read/write ISA addresses") > was a bit too limited... I am fine extending that commit to other architectures. Now the question is, do we just add SPARC64 to the exception list, or do we restrict these drivers to X86? I know they are mostly used on X86, but I have no proof they aren't used on other architectures. That's why Dean and myself decided to play it safe at first and only exclude PPC. Apparently other drivers (e.g. watchdog drivers) for similar chips are limited to X86 so we might as well do the same here. And see if anybody complains... Meelis, do you feel like sending a patch, or do you prefer if I take care of it? -- Jean Delvare -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/