From patchwork Sun Sep 17 13:35:20 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: SF Markus Elfring X-Patchwork-Id: 9954819 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 8FD9A6039A for ; Sun, 17 Sep 2017 13:35:55 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 7A72A28881 for ; Sun, 17 Sep 2017 13:35:55 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 6F58328974; Sun, 17 Sep 2017 13:35:55 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.9 required=2.0 tests=BAYES_00,RCVD_IN_DNSWL_HI autolearn=unavailable version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 16CF6289E2 for ; Sun, 17 Sep 2017 13:35:48 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751466AbdIQNfd (ORCPT ); Sun, 17 Sep 2017 09:35:33 -0400 Received: from mout.web.de ([217.72.192.78]:64348 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751302AbdIQNfc (ORCPT ); Sun, 17 Sep 2017 09:35:32 -0400 Received: from [192.168.1.2] ([77.181.201.34]) by smtp.web.de (mrweb101 [213.165.67.124]) with ESMTPSA (Nemesis) id 0Mf0cv-1e4kx128Px-00OZsE; Sun, 17 Sep 2017 15:35:22 +0200 Subject: [PATCH 4/4] [media] cpia2: Delete an unnecessary return statement in process_frame() From: SF Markus Elfring To: linux-media@vger.kernel.org, Arvind Yadav , Hans Verkuil , Laurent Pinchart , Mauro Carvalho Chehab , Sakari Ailus , Shyam Saini Cc: LKML , kernel-janitors@vger.kernel.org References: Message-ID: Date: Sun, 17 Sep 2017 15:35:20 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 MIME-Version: 1.0 In-Reply-To: Content-Language: en-GB X-Provags-ID: V03:K0:KknCxNULwBDXlvdIlhGjgPdEVzVbCRe37E3YFL43WphhC3KSkkn P1gnVtfuIFMhLZmlnLuaU/mdvYNfpXjmG0vjyJj+eRNEc9a7iL0i73iLVKWg+2qIrZjTFV6 R+U5p4w0R9oQNo3hxJAEYkwsAG5hcuEtWK5HHdRBroENqWNXv9g0CYkl3OjoGWJiRV+hTEO xLqp3dpNN7QtQEnZGnEoQ== X-UI-Out-Filterresults: notjunk:1; V01:K0:TyKotDwENuI=:J+jHywGYqUtPCkPKyD7hOA /Usu2SVyopB2WHhFTXw73/VWg0uYsrWTx/2UBOzTSGTIP8cLDplxbwpHuLuF0aNZ6+pInIX1D a8EO5A4rm2aFJ3ZpqGJ9CPTlAlI5oqSlPQhrGgG6mbPYWfMEviwyeoVMXrQG128HBy+FxM3yi D4Bfgqx52kdNPjudDTA+SnCOaJAlfqGNcKur86B3w5mOVNhTv1RJb8lO/YKXMzLMxhD+wq99G GOFlzL1UnMnI0MmToM3fyjifTSOPF9mo4BycQRQxrsV5j4Q+FC3zV5y0s0H2GlMOmfbCbH6J/ zBCxIm3N6Upto2y4WRi1+VRkXuVVJ40Jo8MaG2r8qWsdwTNEbomnS87AJ2p4rEXVR4N66j2On GjESVbUqDXNVy3I3x5VUULCwN7sOxibmNba0hSBJp5ADxP15Exm7ran5K8ZcawV1f35NJe9WE GiV794MgvcUJ5ZPbifpUJnb7G0Z27z8Wj4pnUkd9X+3fvBLAkE1iUKtfYlPsXBlv2rqs/yQPJ DwP/FBZSoRedGqbmRS4HIsoHKu0ztURUHe3/aAZgbXze6Hd4Hnl2r4YYc/48btoVM0pFYF86W +zvE99EjotX9t9mxiiU7oB2xbW+uq1DlkhXkOAeLgvKg8yhuKBEVFOhDpQN2KMbz4Cyp2dK59 j+5BxxRFIP3hRXFt6hpCmQ/v6GNHUwdrPs6gizgaXnF6x/UbKaRw+/yVezOAiV5Z7G+j9AFt/ WZPHJfzRCIxKzGz2h8i+bZatHHAD2+t8cXsQzQwrpfbeJEKv0K4dANh4RfhFEmawRDbZCGoiU C1AsdCDqv/b4sjgpFoglJgD54Mu1/T6s+OXsbmzELJTJXWNvfg= Sender: linux-media-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-media@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP From: Markus Elfring Date: Sun, 17 Sep 2017 13:32:45 +0200 The script "checkpatch.pl" pointed information out like the following. WARNING: void function return statements are not generally useful Thus remove such a statement in the affected function. Signed-off-by: Markus Elfring --- drivers/media/usb/cpia2/cpia2_usb.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/media/usb/cpia2/cpia2_usb.c b/drivers/media/usb/cpia2/cpia2_usb.c index d1750fa48aa8..f224992590b8 100644 --- a/drivers/media/usb/cpia2/cpia2_usb.c +++ b/drivers/media/usb/cpia2/cpia2_usb.c @@ -154,6 +154,5 @@ static void process_frame(struct camera_data *cam) } cam->workbuff->status = FRAME_ERROR; - return; }