From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Wed, 3 Apr 2013 09:26:29 -0700 From: Greg KH To: Jonghwan Choi Cc: linux-kernel@vger.kernel.org, stable@vger.kernel.org, 'Zhang Rui' , 'Richard Guy Briggs' Subject: Re: [PATCH 3.8-stable] thermal: return an error on failure to register thermal Message-ID: <20130403162629.GA21969@kroah.com> References: <000001ce3003$a1b87aa0$e5296fe0$%choi@samsung.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <000001ce3003$a1b87aa0$e5296fe0$%choi@samsung.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: On Wed, Apr 03, 2013 at 09:38:55AM +0900, Jonghwan Choi wrote: > 3.8-stable review patch. If anyone has any objections, please let me know. Ok, I object, given that I told you to change the format of the message you send out. Please look at how others on the stable@ mailing list send "should this patch be applied" messages, and copy how that is done. > ------------------ > > From: "Richard Guy Briggs " > > commit da28d966f6aa942ae836d09729f76a1647932309 upstream. > > The return code from the registration of the thermal class is used to > unallocate resources, but this failure isn't passed back to the caller of > thermal_init. Return this failure back to the caller. > > This bug was introduced in changeset 4cb18728 which overwrote the return > code > when the variable was re-used to catch the return code of the registration > of > the genetlink thermal socket family. You linewrapped the patch message, careful, you might mess up the patch itself as well :( > Signed-off-by: Richard Guy Briggs > Signed-off-by: Zhang Rui You aren't signing off on it as well? Why not? thanks, greg k-h