From patchwork Mon Nov 21 17:39:50 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Kieran Bingham X-Patchwork-Id: 23969 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:adf:f944:0:0:0:0:0 with SMTP id q4csp1735024wrr; Mon, 21 Nov 2022 09:45:02 -0800 (PST) X-Google-Smtp-Source: AA0mqf5ROLiJtED5VdXp4tz9O9iFf/eQMEymsFf3tTid683WkOnb/yIV7hgmPiam7Ud2fg217A+3 X-Received: by 2002:aa7:8f08:0:b0:56d:4d37:89dc with SMTP id x8-20020aa78f08000000b0056d4d3789dcmr811814pfr.81.1669052702428; Mon, 21 Nov 2022 09:45:02 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1669052702; cv=none; d=google.com; s=arc-20160816; b=gZs8slwn4fA8+xIH54pepRyacDJWc2g8IdKO7pIRdzAJctet+sVYk+6sbGp4PAXBGl RkQiefAeB8eXb4DCwlOpNacZOKUm+2rYJQIfWZZ+/nGC4kRhaXtTrM9Ce+Wjc+FKK2C5 l/oLPTYwAjFLF5xdKFLUF2yaPRcsS7CdDAnoRI8FmrN1hiPVrWMEli+eH3eYTosFumVe wEB/s9hB7sGaviGZeGPDS2pNQtHzElrqLdzZPHw+4KkFut9CDvhfQmkXYG6o3VS31ymT RxtUAcU01e1HrPLokyaR4I98yNSYoQ1j3pgbU/YiD0lAuW84yVPvawR1CM7QmCY+IQ48 FG5g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:dkim-signature; bh=R0pRk0W95gnOA+jgrGzIJ4lNRZvTFe5Ts81/hTjuTM8=; b=W8A+5qsMNt6ZQmxAjQkLQnGSoKK/vzeepsTgS0tYSldpyBr0W5Qhmfr5KaH+ib2GBQ MSsmzTOCO1jxIemcAVNxHVP/Je98SMXvchAVTyzGbhwYArty4cwteJPv7ySh+eAwywIe y9KaYroejJxrNZuNKjyHlHJkq6ul9XOW+Z21BVbLmKWhKKxDlh6pzTNNRO5ME+cJkL5j Iclgph2Kkzs6snlsxs5V/SD8iJaNe67ySIoRX2AQGRNtwNeEau7ip2dsExBRALqrMTxx bNPkw+jSk3i6PnKhBsIu8bOzwjIg+XK+umXEjKu+MgQKsojHOjdk2EUfUvs6G68QCaBw b5Rg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass (test mode) header.i=@ideasonboard.com header.s=mail header.b=OcRBh240; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id h188-20020a6253c5000000b0056341219532si10726109pfb.89.2022.11.21.09.44.45; Mon, 21 Nov 2022 09:45:02 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass (test mode) header.i=@ideasonboard.com header.s=mail header.b=OcRBh240; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229836AbiKURkF (ORCPT + 99 others); Mon, 21 Nov 2022 12:40:05 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:60946 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229874AbiKURkB (ORCPT ); Mon, 21 Nov 2022 12:40:01 -0500 Received: from perceval.ideasonboard.com (perceval.ideasonboard.com [213.167.242.64]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id E03F9CFEB4; Mon, 21 Nov 2022 09:39:59 -0800 (PST) Received: from Monstersaurus.local (cpc89244-aztw30-2-0-cust3082.18-1.cable.virginm.net [86.31.172.11]) by perceval.ideasonboard.com (Postfix) with ESMTPSA id 1987674C; Mon, 21 Nov 2022 18:39:57 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=ideasonboard.com; s=mail; t=1669052397; bh=sExCtvh75xW72P8PYcJyOGr18zPBiZkEL2+R3XyI2kM=; h=From:To:Cc:Subject:Date:From; b=OcRBh240mIiGwQkSAyZwomaKrA4BJK19ATennWVFe9mAh+bLH+PjMry/5D8H9qolZ PLar/qVT8XQYfMOl2GrEOttkKnYo8NV6+hZ4hzYkGT5iTfw35YEbYG+/WDPBz/+8Qm bts+DXn2J4AnOQrJd1ZNARNeEkgUacsOU0hYH0wg= From: Kieran Bingham To: linux-media@vger.kernel.org Cc: Lukas Bulwahn , Kieran Bingham , kernel test robot , Jason Chen , Mauro Carvalho Chehab , Sakari Ailus , Shawn Tu , linux-kernel@vger.kernel.org (open list) Subject: [PATCH v2] media: i2c: ov08x40: Make remove callback return void Date: Mon, 21 Nov 2022 17:39:50 +0000 Message-Id: <20221121173950.390230-1-kieran.bingham+renesas@ideasonboard.com> X-Mailer: git-send-email 2.34.1 MIME-Version: 1.0 X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_PASS,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1750128429899942561?= X-GMAIL-MSGID: =?utf-8?q?1750128606667418928?= In commit ed5c2f5fd10d ("i2c: Make remove callback return void") drivers were updated to remove the return value of the remove callback, and return void. The OV08x40 was added after this commit but was not compile tested at the time due to a KConfig issue, and this warning was missed. Reported-by: kernel test robot Fixes: 38fc5136ac16 ("media: i2c: Add ov08x40 image sensor driver") Signed-off-by: Kieran Bingham --- How come it's always after sending that it is easier to notice the small mistakes like ordering of the commits to blame. v2: - Fix wording of commit message to show that the driver patch came after the change in remove call back API. drivers/media/i2c/ov08x40.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/media/i2c/ov08x40.c b/drivers/media/i2c/ov08x40.c index b4ade17a83f5..72ae7fba94eb 100644 --- a/drivers/media/i2c/ov08x40.c +++ b/drivers/media/i2c/ov08x40.c @@ -3281,7 +3281,7 @@ static int ov08x40_probe(struct i2c_client *client) return ret; } -static int ov08x40_remove(struct i2c_client *client) +static void ov08x40_remove(struct i2c_client *client) { struct v4l2_subdev *sd = i2c_get_clientdata(client); struct ov08x40 *ov08x = to_ov08x40(sd); @@ -3292,8 +3292,6 @@ static int ov08x40_remove(struct i2c_client *client) pm_runtime_disable(&client->dev); pm_runtime_set_suspended(&client->dev); - - return 0; } static const struct dev_pm_ops ov08x40_pm_ops = {