From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jean Delvare Subject: Re: [PATCH] x86_pkg_temp_thermal: Do not expose as a hwmon device Date: Fri, 7 Mar 2014 09:07:05 +0100 Message-ID: <20140307090705.64b0a763@endymion.delvare> References: <20140302153335.0f1326b6@endymion.delvare> <53135650.3070909@roeck-us.net> <20140302180452.73fa2f8a@endymion.delvare> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: Received: from cantor2.suse.de ([195.135.220.15]:42394 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751082AbaCGIHI (ORCPT ); Fri, 7 Mar 2014 03:07:08 -0500 In-Reply-To: <20140302180452.73fa2f8a@endymion.delvare> Sender: linux-pm-owner@vger.kernel.org List-Id: linux-pm@vger.kernel.org To: Zhang Rui , Eduardo Valentin Cc: Guenter Roeck , linux-pm@vger.kernel.org On Sun, 2 Mar 2014 18:04:52 +0100, Jean Delvare wrote: > On Sun, 02 Mar 2014 08:03:28 -0800, Guenter Roeck wrote: > > On 03/02/2014 06:33 AM, Jean Delvare wrote: > > > The temperature value reported by x86_pkg_temp_thermal is already > > > reported by the coretemp driver. So, do not expose this thermal zone > > > as a hwmon device, because it would be redundant. > > > > > > Signed-off-by: Jean Delvare > > > Cc: Zhang Rui > > > Cc: Eduardo Valentin > > > > Acked-by: Guenter Roeck > > > > Does that make the "Fix thermal zone type" patch unnecessary ? > > I think both patches are needed, as type "pkg-temp-0" is wrong anyway, > regardless of the hwmon side of things. But it is less important with > this patch applied, indeed. > > > If so, this patch would be a candidate for stable. > > I'm still not 100% certain we want either patch in stable. After all, > they aren't fixing any critical bug, and they change user interfaces so > they could impact some users in unpredictable ways. I made up my mind, and now believe this patch should go to stable trees. The invalid name attribute is causing real trouble to lm-sensors users. The other patch OTOH doesn't need to go to stable trees if that one goes in. Rui, Eduardo, can either of you please review this patch, tag it for stable and apply it for 3.14? Thanks. -- Jean Delvare SUSE L3 Support