From patchwork Thu Sep 14 11:07:27 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Arnd Bergmann X-Patchwork-Id: 112533 Delivered-To: patch@linaro.org Received: by 10.140.106.117 with SMTP id d108csp610440qgf; Thu, 14 Sep 2017 04:07:51 -0700 (PDT) X-Google-Smtp-Source: ADKCNb7O8Ilm9fK7Oo1bscykjIsI+2vEuo5995av0LoFvY05DUt5qExajthRTT//mPNE7OcBvZ4U X-Received: by 10.84.143.100 with SMTP id 91mr23764686ply.136.1505387271700; Thu, 14 Sep 2017 04:07:51 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1505387271; cv=none; d=google.com; s=arc-20160816; b=YZQW+HV6Hlyqj1Xb7sUka5ckqISIADtq6cuUF+ZMcRYD/CgalMto4cpsXA2XStv4XH 6HIQEfrWBm9KH3I87LMdhwYmWQ8ctbZiYGXwBC/JNDFZ2RQZIEmQwau92gjMKYXXPwJ6 1/a7nGWzw9LjgP38dgiQALAch6lMgnBt1q7KNO2pS8AW6le7s/1+C7xvbexVxKRqtc4B O9rcC1jb0rxHuLZB1mTkXhALWEeqAADwYsY7b2m5/ffHOtV4fYnIEZdE7cSUtW1qNz6s 42BE+rJnY3G5sKxqi/xKmWMZoJTOiZoWaOQ+TZELvx75WWaCGzTc0Xq+ILsGb3DZUMJS pNdA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:arc-authentication-results; bh=AedMKJwDKqTrmsuETcaffJBeujzUdO6UIBaab93QLyc=; b=h2lNSuFyHU11DWnmwH9igGaAMlLHsY+WUb+z5rZt1DQ1pdW0CFaOn7aHdgvF3WH/Sk oVdPuv7RvrL1asBWJ+JTtCkiRfvGHoKym3/J68MpvQIxtRMqlBdJh8oc58tWtRL754GJ VTyyleppTw4OkArq1/lpPGS10a84PIjAOZqodNp8tydpbTw8/o1PsRZuwtck6vRMMAiA jhE57oIGzA3SRlcF/IT11v3dMLVIvopFKSEYF7DET1tdmlM8qoBjOMa3d7g18C5d402Q a+FssGwa8VvqxqxtiSxDYEa6MdUyklKyjXEv40S9T2uTOP2WjH5XvOPuB4fhlnNNjRm7 dcZg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id 64si6862541plf.247.2017.09.14.04.07.50; Thu, 14 Sep 2017 04:07:51 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751925AbdINLHs (ORCPT + 26 others); Thu, 14 Sep 2017 07:07:48 -0400 Received: from mout.kundenserver.de ([212.227.126.130]:54249 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751413AbdINLHq (ORCPT ); Thu, 14 Sep 2017 07:07:46 -0400 Received: from wuerfel.lan ([95.208.190.237]) by mrelayeu.kundenserver.de (mreue002 [212.227.15.129]) with ESMTPA (Nemesis) id 0LZqH7-1d9kEs112r-00lntY; Thu, 14 Sep 2017 13:07:35 +0200 From: Arnd Bergmann To: Ramesh Shanmugasundaram , Mauro Carvalho Chehab Cc: Arnd Bergmann , Hans Verkuil , linux-media@vger.kernel.org, linux-renesas-soc@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH] [media] rcar_drif: fix potential uninitialized variable use Date: Thu, 14 Sep 2017 13:07:27 +0200 Message-Id: <20170914110733.3592437-1-arnd@arndb.de> X-Mailer: git-send-email 2.9.0 MIME-Version: 1.0 X-Provags-ID: V03:K0:QLXhlsDhx9RvT6PvTfgzwZNTSn81PQw7QRj5sRJ5e0RNOTO7dd1 cMhKQE0G3Sji8dR7Mz/uhKdGW/GbePuwqwELGt0beTCK6uBmnSNbjTldXXpITJEuJnWvE05 vBxaQc56EyzIYIlaHmzl+4CiPVAUHTnnVbGDya/sVFX0IPOyPAnciq3qD0BTrChxCBYpYsu k5pqXHNuyafjfeZ35ZRHA== X-UI-Out-Filterresults: notjunk:1; V01:K0:xddQl4lvQ44=:UF9sLbm2AglC16p5cGSk7E wmqlExnG6zh+CemLOAZ5adtuABHXo4OKl2OWpTT+maqwWm56gr4KCBiunycazpaYiRf4C/ECd KbWMcSDe+oOz8olTjdKzWNT1h96a+OgqB/PmauPKFM4BaiB8YBG9DFpPtYc4J0oOoQH8kQTEn 0eTgf15e26LZwVCJCtmQtjcUG8LKLKcXUpXolDZ8YLAdaTzpcpEmXbuQweNbAiL9xhfZmz7Js hyF7huuFkXiKXNnV53oZC0dtS06xwr9n+0UCXEQFKvDdyY6iZXrjCWGBgPPv37VDWa8b/evx2 PPtgwj3DK74Z4fpijoTdunWV10N/VQ+V89DDKOD7YG7HZpRsfuIA/16wqc6+1vInsO+J27T0f gwH0BmQu3NfZ4iepG65/8+lrJl9iTljWiPy7s1XGuu/3cAOVOvsYSg0gcB2oZrjQ3/CdidcI/ +UlHSl5Zl990cWCKRUGmKPPt5KpKk/GRb2XmPhvDAmSpV/FZ1SySWvNd8NBCQ1UH/Oly3Msop iRGCQIzPC1p2AeWIdMNQH2YmKt5G/E8/YcOmvMiKiWzTolzfD6507wwjgMAUK8MhZzuRm7cS7 jE9EFvJatcOpUS4p1roQ9oARe1clAN2dtYWB6TjZME22LU63fVeUJK72m38WLDPckjgi5PB6Z oqqbFPN67WzI2YaKr0rVw1/LtotDfXp8Nbi7PB3h5yarR3ijYO3cpaR3PJwK8XYctj8fIM9SA i44kxUZIETGHNWS7m0mUxGyeYp1XyVpX3Mtb5c8vyywi7wUZNxs+nRqNOKs= Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Older compilers like gcc-4.6 may run into a case that returns an uninitialized variable from rcar_drif_enable_rx() if that function was ever called with an empty cur_ch_mask: drivers/media/platform/rcar_drif.c:658:2: error: ‘ret’ may be used uninitialized in this function [-Werror=uninitialized] Newer compilers don't have that problem as they optimize the 'ret' variable away and just return zero in that case. This changes the function to return -EINVAL for this particular failure, to make it consistent across all compiler versions. In case gcc gets changed to report a warning for it in the future, it's also a good idea to shut it up now. Link: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82203 Signed-off-by: Arnd Bergmann --- drivers/media/platform/rcar_drif.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- 2.9.0 diff --git a/drivers/media/platform/rcar_drif.c b/drivers/media/platform/rcar_drif.c index 522364ff0d5d..2c6afd38b78a 100644 --- a/drivers/media/platform/rcar_drif.c +++ b/drivers/media/platform/rcar_drif.c @@ -630,7 +630,7 @@ static int rcar_drif_enable_rx(struct rcar_drif_sdr *sdr) { unsigned int i; u32 ctr; - int ret; + int ret = -EINVAL; /* * When both internal channels are enabled, they can be synchronized