From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-qg0-f53.google.com ([209.85.192.53]:33789 "EHLO mail-qg0-f53.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753951AbaF3Dsx (ORCPT ); Sun, 29 Jun 2014 23:48:53 -0400 Received: by mail-qg0-f53.google.com with SMTP id i50so1475156qgf.26 for ; Sun, 29 Jun 2014 20:48:53 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: <53B02CA9.4020905@roeck-us.net> References: <53B02147.1000608@telliq.com> <53B02CA9.4020905@roeck-us.net> From: Naveen Krishna Ch Date: Mon, 30 Jun 2014 09:18:32 +0530 Message-ID: Subject: Re: hwmon: ntc_thermistor: Add another ntc(epcos) to the driver To: Guenter Roeck Cc: Jan Kardell , Johannes Pointner , Jean Delvare , Naveen Krishna Chatradhi , lm-sensors@lm-sensors.org, linux-iio@vger.kernel.org Content-Type: text/plain; charset=UTF-8 Sender: linux-iio-owner@vger.kernel.org List-Id: linux-iio@vger.kernel.org Hello Johannes Pointner skriver, On 29 June 2014 20:41, Guenter Roeck wrote: > On 06/29/2014 07:23 AM, Jan Kardell wrote: >> >> Johannes Pointner skriver: >>> >>> As I saw in the latest patches for the ntc_thermistor, in which as >>> part of the patch the module description changed to : >>> >>> MODULE_DESCRIPTION("NTC Thermistor Driver from Murata"); >>> >>> I wanted to know if it is intended to extend the driver for further >>> manufacturers like in my case for an epcos ntc in the future? The ntc_thermistor.c currently, supports ncpXXXX (all thermistors from Murata). Thus, My assumption. Please post your patches for supporting other thermistors. If there is enough compatibility. We can drop off the "from Murata" in MODULE_DESCRIPTION() and In the Kconfig file. I can submit a change for the same. >>> >>> Of course I would create the necessary patch. >>> >>> Thanks, >>> Hannes >>> -- >>> To unsubscribe from this list: send the line "unsubscribe linux-iio" in >>> the body of a message to majordomo@vger.kernel.org >>> More majordomo info at http://vger.kernel.org/majordomo-info.html >> >> >> Why not move the table to DT? There are lots of NTC types out there, and >> each type comes in an number of varietes. IMHO it seems impractical to >> change the code for each one, and the tables would grow to be very large. >> > > I would not mandate it, but I would he happy to accept a patch. > > Guenter > > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-iio" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Thanks & Regards, (: Nav :) From mboxrd@z Thu Jan 1 00:00:00 1970 From: Naveen Krishna Ch Date: Mon, 30 Jun 2014 03:51:44 +0000 Subject: Re: [lm-sensors] hwmon: ntc_thermistor: Add another ntc(epcos) to the driver Message-Id: List-Id: References: <53B02147.1000608@telliq.com> <53B02CA9.4020905@roeck-us.net> In-Reply-To: <53B02CA9.4020905@roeck-us.net> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Guenter Roeck Cc: Jan Kardell , Johannes Pointner , Jean Delvare , Naveen Krishna Chatradhi , lm-sensors@lm-sensors.org, linux-iio@vger.kernel.org Hello Johannes Pointner skriver, On 29 June 2014 20:41, Guenter Roeck wrote: > On 06/29/2014 07:23 AM, Jan Kardell wrote: >> >> Johannes Pointner skriver: >>> >>> As I saw in the latest patches for the ntc_thermistor, in which as >>> part of the patch the module description changed to : >>> >>> MODULE_DESCRIPTION("NTC Thermistor Driver from Murata"); >>> >>> I wanted to know if it is intended to extend the driver for further >>> manufacturers like in my case for an epcos ntc in the future? The ntc_thermistor.c currently, supports ncpXXXX (all thermistors from Murata). Thus, My assumption. Please post your patches for supporting other thermistors. If there is enough compatibility. We can drop off the "from Murata" in MODULE_DESCRIPTION() and In the Kconfig file. I can submit a change for the same. >>> >>> Of course I would create the necessary patch. >>> >>> Thanks, >>> Hannes >>> -- >>> To unsubscribe from this list: send the line "unsubscribe linux-iio" in >>> the body of a message to majordomo@vger.kernel.org >>> More majordomo info at http://vger.kernel.org/majordomo-info.html >> >> >> Why not move the table to DT? There are lots of NTC types out there, and >> each type comes in an number of varietes. IMHO it seems impractical to >> change the code for each one, and the tables would grow to be very large. >> > > I would not mandate it, but I would he happy to accept a patch. > > Guenter > > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-iio" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Thanks & Regards, (: Nav :) _______________________________________________ lm-sensors mailing list lm-sensors@lm-sensors.org http://lists.lm-sensors.org/mailman/listinfo/lm-sensors