Date: Thu, 24 Jan 2008 14:39:09 +0200
From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= <syrjala@sci.fi>
To: David Fries <david@fries.net>
Cc: linux-kernel@vger.kernel.org, Evgeniy Polyakov <johnpol@2ka.mipt.ru>
Subject: Re: [PATCH] W1: w1_therm.c standardize units to millidegrees C
Message-ID: <20080124123909.GY30987@sci.fi>
Mail-Followup-To: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= <syrjala@sci.fi>,
	David Fries <david@fries.net>, linux-kernel@vger.kernel.org,
	Evgeniy Polyakov <johnpol@2ka.mipt.ru>
References: <20080124020330.GA19155@spacedout.fries.net>
MIME-Version: 1.0
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <20080124020330.GA19155@spacedout.fries.net>
User-Agent: Mutt/1.5.16 (2007-06-09)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1013
Lines: 26

On Wed, Jan 23, 2008 at 08:03:30PM -0600, David Fries wrote:
> Here is the patch to standardize the temperature units to millidegrees
> C for the two sensor conversion routines.  Previously the routines
> were,
> 
> w1_DS18B20_convert_temp degrees C
> w1_DS18S20_convert_temp millidegrees C
> 
> Unfortunately this will break any program using the ds18b20 value as
> it will now be 1000 times bigger.  Fortunately there can't be that
> many users out there, or some of these bugs will have been fixed by
> now, such as the negative C error (see previous patch) that makes me
> think the ds18b20 is the better choice to change because of the
> current bugs.

Why isn't w1_therm presented as a hwmon device?

-- 
Ville Syrj?l?
syrjala@sci.fi
http://www.sci.fi/~syrjala/
--
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/