Received: by 2002:a05:6a10:af89:0:0:0:0 with SMTP id iu9csp4237193pxb; Tue, 25 Jan 2022 06:25:46 -0800 (PST) X-Google-Smtp-Source: ABdhPJy+Y9SaXwnsY/LAenpyWDSyBiFP5hho3SgVbr3/1CzoySb8xgIfuHEGDlKE1HwKNTbmVTtA X-Received: by 2002:a17:90b:4c8e:: with SMTP id my14mr3786717pjb.18.1643120745970; Tue, 25 Jan 2022 06:25:45 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1643120745; cv=none; d=google.com; s=arc-20160816; b=rUx1H47/PhUrfUXzyKOcSIYMzopJkQ4rKg4mG/wIAVoAX75yFlyAfh59CRk6b6+1Kn hnlX1sK4EobbSvjYI5i1sm0hW+nBRko8+SfOUyN3VUvpob60YoO4gNjjiz3baD7897CM ImlVK+aY4t+ojW7nfYieZi3M8v9/0GyVu7XYYmhesOp6kZP0mVBjXqp9fiikwOB2BsnU GMvBgWkj2fvuEWMzGyN+r80uLN3LH5jCbwJ7Ia9M1UsfNpSE6DXH1TAK3Lfx8jRbNjnW FYSXcXJlegRUI7r+Q4nAJtO/BEw5f1OJ7BdF3lWBkb97/QLPw/9K409jPUDpOz7+Gy0W rskA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:cc:to:subject :message-id:date:from:in-reply-to:references:mime-version :dkim-signature; bh=1mm7+Qmz3LImo4aFwbhXIFkzVEgm/HAOzd6fbdAVMrg=; b=xntmFCtbdX5Fi59Tv3+bucFh23Oci+XipJUp70jPUsuQ+ARylG5dtF++dmxBF+txy1 Oj42ygqWuGNjcXwK8Pb9QFDEdhpkT1CsYqnPzADUOEOgxEO1xfADWtH50OBia6DBmSDX uLPhk8mZ9z5oaRKQwIELMGWGyjs8LbEnRL7vX5D8xkvwG+56EJCvZ9DQ4Fbr/2wEV0G+ +B+AbhBh9F0VRdJ3jcB7w583Ge1BuWqv5IJBB1KpZJp5O8zmH7tyrQagVUvV1BEBYz74 AF638bPmNdgDmHQY1W97CM99g8KRzb552xLMbyadJwQX8b/mwO2J1W7mhUsQ3Px8m95n VEPg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=kCmCEgQ+; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id i21si13663755pgk.309.2022.01.25.06.25.31; Tue, 25 Jan 2022 06:25:45 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=kCmCEgQ+; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1574434AbiAYJeO (ORCPT + 99 others); Tue, 25 Jan 2022 04:34:14 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:53354 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1573979AbiAYJa4 (ORCPT ); Tue, 25 Jan 2022 04:30:56 -0500 Received: from mail-ua1-x931.google.com (mail-ua1-x931.google.com [IPv6:2607:f8b0:4864:20::931]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 3BCE0C0613F0 for ; Tue, 25 Jan 2022 01:30:55 -0800 (PST) Received: by mail-ua1-x931.google.com with SMTP id p7so29029593uao.6 for ; Tue, 25 Jan 2022 01:30:55 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc:content-transfer-encoding; bh=1mm7+Qmz3LImo4aFwbhXIFkzVEgm/HAOzd6fbdAVMrg=; b=kCmCEgQ+xOVJqgvY6mIiDwGpng9RE64Ri/x5bVEJVgfKmrbder8yB0SJa9j3j/iO6u VSlwxJi1GtMfRaX9rz0Fi8V70QZmpTm/yg+yqKEhFTP/kvAigM6NlkG016BHxBuEGBud aKKG3vXyGKhkWECz7g02ij5pN8RBjRbfSAGJhF2s+s/E8Wzq61/RmUZzmkHINs6y38kj 8W/C1tUsfkR71I5xYVxyEgRhxjIz/FmTGO/CwLqL3papla/+aWUpb3kyrOT/GpbDGqeC c1kCPdzAMoEjl9Ka3L/ozlubTR502/GpTZoRLS5kuqKqPkl9NQ2lJSPzTzJVIDppuhpp EfLg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc:content-transfer-encoding; bh=1mm7+Qmz3LImo4aFwbhXIFkzVEgm/HAOzd6fbdAVMrg=; b=L+ugmTcXJC9324BzvC8J7zFItOMGd5RZNBLTybWUxQeA0oFkfX2wvXD/GDHIKXlrlr QiwdExuPP0jITw6oCqmetVtiDqeRLic4koCeTpeiDveD/UxsDEA4Zk6VyJCFyGsS0R92 oaRM9Pz5oZ422E3KMPlPY0qtoLjBtf+YlgFhAtgcfyQZiXH5h0pq9Kk9HU6uBoJSDFoV 8dRE1hmbmLK+vsg64YIHqDFzYOAP81pDQ8j5jTIBJOU0Pv+RQTPVpeneeNjz9WIiZdTV GRQU8GNxMAGKDF75xwAeinjMQKnPPUZZSwjByXS3bPioCBUUVYqI7yvtpRdZ8jqBKyoT TxvA== X-Gm-Message-State: AOAM532xLoKxIsugKM/eiwEUVOXWJFKC7pIzQKCnw4btKG7JKxYJ+NYR /UXqPaNfaccLgAF4opo/S+fARzqUaruUQeGLPTEx9A== X-Received: by 2002:a67:8c2:: with SMTP id 185mr712613vsi.19.1643103053912; Tue, 25 Jan 2022 01:30:53 -0800 (PST) MIME-Version: 1.0 References: <20220123175201.34839-1-u.kleine-koenig@pengutronix.de> In-Reply-To: <20220123175201.34839-1-u.kleine-koenig@pengutronix.de> From: Lee Jones Date: Tue, 25 Jan 2022 09:30:43 +0000 Message-ID: Subject: Re: [PATCH 0/5] spi: make remove callback a void function To: =?UTF-8?Q?Uwe_Kleine=2DK=C3=B6nig?= Cc: Mark Brown , =?UTF-8?B?TWFyZWsgQmVow7pu?= , Peter Huewe , Jarkko Sakkinen , Jason Gunthorpe , Michael Turquette , Stephen Boyd , Linus Walleij , Bartosz Golaszewski , Thierry Reding , Sam Ravnborg , David Airlie , Daniel Vetter , Markuss Broks , Emma Anholt , David Lechner , Kamlesh Gurudasani , =?UTF-8?Q?Noralf_Tr=C3=B8nnes?= , Jean Delvare , Guenter Roeck , Dan Robertson , Jonathan Cameron , Lars-Peter Clausen , Michael Hennerich , Marcus Folkesson , Kent Gustavsson , Rui Miguel Silva , Dmitry Torokhov , Pavel Machek , Yasunari Takiguchi , Mauro Carvalho Chehab , Charles-Antoine Couret , Antti Palosaari , Support Opensource , Charles Keepax , Richard Fitzgerald , Maxime Coquelin , Alexandre Torgue , Arnd Bergmann , Greg Kroah-Hartman , Eric Piel , Ulf Hansson , Miquel Raynal , Richard Weinberger , Vignesh Raghavendra , Wolfgang Grandegger , Marc Kleine-Budde , "David S. Miller" , Jakub Kicinski , Manivannan Sadhasivam , Thomas Kopp , Florian Fainelli , Andrew Lunn , Vivien Didelot , Vladimir Oltean , Woojung Huh , UNGLinuxDriver@microchip.com, =?UTF-8?Q?=C5=81ukasz_Stelmach?= , Alexander Aring , Stefan Schmidt , Harry Morris , Varka Bhadram , Xue Liu , Alan Ott , Heiner Kallweit , Russell King , Christian Lamparter , Kalle Valo , Ajay Singh , Claudiu Beznea , Solomon Peachy , Krzysztof Kozlowski , Mark Greer , Benson Leung , Hans de Goede , Mark Gross , Alessandro Zummo , Alexandre Belloni , =?UTF-8?B?SsOpcsO0bWUgUG91aWxsZXI=?= , Jiri Slaby , Felipe Balbi , Daniel Thompson , Jingoo Han , Helge Deller , James Schulman , David Rhodes , Lucas Tanure , Jaroslav Kysela , Takashi Iwai , =?UTF-8?B?TnVubyBTw6E=?= , Liam Girdwood , Daniel Mack , Thomas Zimmermann , Maxime Ripard , Andy Shevchenko , Stephan Gerhold , Alexandru Ardelean , Mike Looijmans , Gwendal Grignou , Cai Huoqing , Minghao Chi , Antoniu Miclaus , Julia Lawall , =?UTF-8?Q?Ronald_Tschal=C3=A4r?= , Marco Felsch , =?UTF-8?Q?Jonathan_Neusch=C3=A4fer?= , Emmanuel Gil Peyrot , Jon Hunter , Andrew Morton , Heiko Schocher , Fabio Estevam , Colin Ian King , Dan Carpenter , Tudor Ambarus , Matt Kline , Torin Cooper-Bennun , Vincent Mailhol , =?UTF-8?Q?Stefan_M=C3=A4tje?= , Frieder Schrempf , Wei Yongjun , Randy Dunlap , Nanyong Sun , Yang Shen , dingsenjie , Aditya Srivastava , Stefan Wahren , Peter Zijlstra , Michael Walle , Yang Li , Geert Uytterhoeven , wengjianfeng , Sidong Yang , Paulo Miguel Almeida , Zhang Qilong , Will Deacon , Mark Tomlinson , Davidlohr Bueso , Claudius Heine , Jiri Prchal , linux-kernel@vger.kernel.org, linux-integrity@vger.kernel.org, linux-clk@vger.kernel.org, linux-gpio@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-hwmon@vger.kernel.org, linux-iio@vger.kernel.org, linux-input@vger.kernel.org, linux-leds@vger.kernel.org, linux-media@vger.kernel.org, patches@opensource.cirrus.com, alsa-devel@alsa-project.org, linux-stm32@st-md-mailman.stormreply.com, linux-arm-kernel@lists.infradead.org, linux-mmc@vger.kernel.org, linux-mtd@lists.infradead.org, linux-can@vger.kernel.org, netdev@vger.kernel.org, linux-wpan@vger.kernel.org, linux-wireless@vger.kernel.org, libertas-dev@lists.infradead.org, platform-driver-x86@vger.kernel.org, linux-rtc@vger.kernel.org, linux-spi@vger.kernel.org, linux-fbdev@vger.kernel.org, linux-staging@lists.linux.dev, linux-serial@vger.kernel.org, linux-usb@vger.kernel.org, linux-omap@vger.kernel.org, kernel@pengutronix.de, Noralf Tronnes , Thomas Petazzoni , Guenter Roeck , Enric Balletbo i Serra Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org My usual mailer won't let me reply to this many people, so I'm using Gmail. No idea what chaos this will cause, but here goes ... > The value returned by an spi driver's remove function is mostly ignored. > (Only an error message is printed if the value is non-zero that the > error is ignored.) > > So change the prototype of the remove function to return no value. This > way driver authors are not tempted to assume that passing an error to > the upper layer is a good idea. All drivers are adapted accordingly. > There is no intended change of behaviour, all callbacks were prepared to > return 0 before. > > Signed-off-by: Uwe Kleine-K=C3=B6nig > --- [...] > drivers/mfd/arizona-spi.c | 4 +--- > drivers/mfd/da9052-spi.c | 3 +-- > drivers/mfd/ezx-pcap.c | 4 +--- > drivers/mfd/madera-spi.c | 4 +--- > drivers/mfd/mc13xxx-spi.c | 3 +-- > drivers/mfd/rsmu_spi.c | 4 +--- > drivers/mfd/stmpe-spi.c | 4 +--- > drivers/mfd/tps65912-spi.c | 4 +--- > drivers/video/backlight/ams369fg06.c | 3 +-- > drivers/video/backlight/corgi_lcd.c | 3 +-- > drivers/video/backlight/ili922x.c | 3 +-- > drivers/video/backlight/l4f00242t03.c | 3 +-- > drivers/video/backlight/lms501kf03.c | 3 +-- > drivers/video/backlight/ltv350qv.c | 3 +-- > drivers/video/backlight/tdo24m.c | 3 +-- > drivers/video/backlight/tosa_lcd.c | 4 +--- > drivers/video/backlight/vgg2432a4.c | 4 +--- If it's okay with Mark, it's okay with me. Acked-by: Lee Jones --=20 Lee Jones [=E6=9D=8E=E7=90=BC=E6=96=AF] Linaro Services Principle Technical Lead Linaro.org =E2=94=82 Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog