Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754084AbdGNNLo (ORCPT ); Fri, 14 Jul 2017 09:11:44 -0400 Received: from userp1040.oracle.com ([156.151.31.81]:39914 "EHLO userp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753635AbdGNNLl (ORCPT ); Fri, 14 Jul 2017 09:11:41 -0400 Date: Fri, 14 Jul 2017 16:09:55 +0300 From: Dan Carpenter To: Arnd Bergmann Cc: devel@driverdev.osuosl.org, Hans Verkuil , Alan Cox , Greg Kroah-Hartman , Daeseok Youn , Linux Kernel Mailing List , dri-devel , adi-buildroot-devel@lists.sourceforge.net, Linux-Renesas , IDE-ML , Robert Jarzmik , Linux ARM , Niklas =?iso-8859-1?Q?S=F6derlund?= , Tejun Heo , Andrew Morton , Mauro Carvalho Chehab , Linus Torvalds , Guenter Roeck , Linux Media Mailing List Subject: Re: [PATCH 14/14] [media] fix warning on v4l2_subdev_call() result interpreted as bool Message-ID: <20170714130955.zqe26g6zpixr3xj2@mwanda> References: <20170714092540.1217397-1-arnd@arndb.de> <20170714093938.1469319-1-arnd@arndb.de> <20170714120512.ioe67nnloqivtbr7@mwanda> <20170714125525.kjemhcn4poon6r3i@mwanda> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170714125525.kjemhcn4poon6r3i@mwanda> User-Agent: NeoMutt/20170113 (1.7.2) X-Source-IP: userv0022.oracle.com [156.151.31.74] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 322 Lines: 9 On Fri, Jul 14, 2017 at 03:55:26PM +0300, Dan Carpenter wrote: > I don't agree with it as a static analysis dev... What I mean is if it's a macro that returns -ENODEV or a function that returns -ENODEV, they should both be treated the same. The other warnings this check prints are quite clever. regards, dan carpenter