From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 4C828C433F5 for ; Wed, 13 Oct 2021 09:01:57 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 2FDF4610D1 for ; Wed, 13 Oct 2021 09:01:57 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234702AbhJMJD7 (ORCPT ); Wed, 13 Oct 2021 05:03:59 -0400 Received: from mail.kernel.org ([198.145.29.99]:35480 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229987AbhJMJD6 (ORCPT ); Wed, 13 Oct 2021 05:03:58 -0400 Received: by mail.kernel.org (Postfix) with ESMTPSA id 78EFA610A1; Wed, 13 Oct 2021 09:01:55 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1634115715; bh=jN/4FAaN7crTTlDuot+9D29of9NN4is61FaPC7nYdAc=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=L5Gn6sSPZHVB588QFAb+aGM67vjlUeequD5fC5h/Fs6v6wdKC/kTv7i3IGPeRtKr2 aGu86AWA424KGny9HsUqy4omNpd3CFLdwfl450O3W0Tz42aNFl65DHtLfFtGHD9kM5 rbtJCG0xpY3dhsyaLdlB0lVR4CbWXjBkRkmhlf54= Date: Wed, 13 Oct 2021 11:01:53 +0200 From: Greg Kroah-Hartman To: Hans de Goede Cc: Uwe =?iso-8859-1?Q?Kleine-K=F6nig?= , Arnd Bergmann , Eric Piel , Mark Gross , Mark Brown , kernel@pengutronix.de, linux-spi@vger.kernel.org, platform-driver-x86@vger.kernel.org Subject: Re: [PATCH v2 12/20] misc: lis3lv02d: Make lis3lv02d_remove_fs() return void Message-ID: References: <20211012153945.2651412-1-u.kleine-koenig@pengutronix.de> <20211012153945.2651412-13-u.kleine-koenig@pengutronix.de> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: platform-driver-x86@vger.kernel.org On Tue, Oct 12, 2021 at 05:54:46PM +0200, Hans de Goede wrote: > Hi, > > On 10/12/21 5:39 PM, Uwe Kleine-König wrote: > > Up to now lis3lv02d_remove_fs() returns zero unconditionally. Make it return > > void instead which makes it easier to see in the callers that there is > > no error to handle. > > > > Also the return value of i2c and spi remove callbacks is ignored anyway. > > > > Signed-off-by: Uwe Kleine-König > > Thanks, patch looks good to me: > > Reviewed-by: Hans de Goede > > Also please consider this as my ack for merging the > drivers/platform/x86/ part of this through whatever > tree is convenient. Thanks, I'll take it! greg k-h