From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S1753191AbbC3IBu (ORCPT <rfc822;w@1wt.eu>);
	Mon, 30 Mar 2015 04:01:50 -0400
Received: from lb3-smtp-cloud6.xs4all.net ([194.109.24.31]:59141 "EHLO
	lb3-smtp-cloud6.xs4all.net" rhost-flags-OK-OK-OK-OK)
	by vger.kernel.org with ESMTP id S1753160AbbC3IBs (ORCPT
	<rfc822;linux-kernel@vger.kernel.org>);
	Mon, 30 Mar 2015 04:01:48 -0400
Message-ID: <1427702501.2408.21.camel@x220>
Subject: Re: [PATCH 2/2] hwmon: Allow to compile dell-smm-hwmon driver
 without /proc/i8k
From: Paul Bolle <pebolle@tiscali.nl>
To: Guenter Roeck <linux@roeck-us.net>
Cc: Pali =?ISO-8859-1?Q?Roh=E1r?= <pali.rohar@gmail.com>,
        Steven Honeyman <stevenhoneyman@gmail.com>,
        Arnd Bergmann <arnd@arndb.de>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Jean Delvare <jdelvare@suse.de>,
        Valdis Kletnieks <Valdis.Kletnieks@vt.edu>,
        Jochen Eisinger <jochen@penguin-breeder.org>,
        Gabriele Mazzotta <gabriele.mzt@gmail.com>,
        linux-kernel@vger.kernel.org, lm-sensors@lm-sensors.org
Date: Mon, 30 Mar 2015 10:01:41 +0200
In-Reply-To: <55174D7B.8070401@roeck-us.net>
References: <1427538255-10860-1-git-send-email-pali.rohar@gmail.com>
		 <1427540660.2334.14.camel@x220>
		 <CABz95_BkAvPoBAm_tRLH6OhCk_pDfJQ8UuwfhuEYuPNt3P-PGw@mail.gmail.com>
		 <201503282300.48749@pali> <55172939.4070100@roeck-us.net>
	 <1427582640.3377.0.camel@tiscali.nl> <55174D7B.8070401@roeck-us.net>
Content-Type: text/plain; charset="UTF-8"
X-Mailer: Evolution 3.10.4 (3.10.4-4.fc20) 
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Sat, 2015-03-28 at 17:55 -0700, Guenter Roeck wrote:
> On 03/28/2015 03:44 PM, Paul Bolle wrote:
> > I haven't actually looked into this part of the patch, since as far as
> > I'm concerned this patch is broken.
> 
> You mean beyond the "ON" and the above problem ? Can you elaborate ?

No, I was talking about the ON typo. Because a patch that mainly juggles
Kconfig symbols but adds a Kconfig syntax error while doing that is a
broken patch.


Paul Bolle