From mboxrd@z Thu Jan 1 00:00:00 1970 X-GM-THRID: 6392494565268389888 X-Received: by 10.107.157.6 with SMTP id g6mr6007587ioe.79.1488804946232; Mon, 06 Mar 2017 04:55:46 -0800 (PST) X-BeenThere: outreachy-kernel@googlegroups.com Received: by 10.36.237.77 with SMTP id r74ls2399452ith.11.gmail; Mon, 06 Mar 2017 04:55:45 -0800 (PST) X-Received: by 10.99.23.95 with SMTP id 31mr6582222pgx.122.1488804945446; Mon, 06 Mar 2017 04:55:45 -0800 (PST) Return-Path: Received: from mail.linuxfoundation.org (mail.linuxfoundation.org. [140.211.169.12]) by gmr-mx.google.com with ESMTPS id s64si18254pfk.0.2017.03.06.04.55.45 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 06 Mar 2017 04:55:45 -0800 (PST) Received-SPF: pass (google.com: domain of gregkh@linuxfoundation.org designates 140.211.169.12 as permitted sender) client-ip=140.211.169.12; Authentication-Results: gmr-mx.google.com; spf=pass (google.com: domain of gregkh@linuxfoundation.org designates 140.211.169.12 as permitted sender) smtp.mailfrom=gregkh@linuxfoundation.org Received: from localhost (unknown [37.168.44.83]) by mail.linuxfoundation.org (Postfix) with ESMTPSA id 20925949; Mon, 6 Mar 2017 12:55:43 +0000 (UTC) Date: Mon, 6 Mar 2017 13:55:37 +0100 From: Greg KH To: Narcisa Ana Maria Vasile Cc: swarren@wwwdotorg.org, lee@kernel.org, eric@anholt.net, f.fainelli@gmail.com, rjui@broadcom.com, sbranden@broadcom.com, outreachy-kernel@googlegroups.com Subject: Re: [PATCH v3 1/2] staging: vc04_services: bcm2835-camera: Move min calculation outside the loop Message-ID: <20170306125537.GA26535@kroah.com> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.8.0 (2017-02-23) On Wed, Mar 01, 2017 at 05:37:13PM +0200, Narcisa Ana Maria Vasile wrote: > Instead of computing the minimum value in the for loop, store it in a variable. > > Signed-off-by: Narcisa Ana Maria Vasile > --- > drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c b/drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c > index 53e275e..7b9d33c 100644 > --- a/drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c > +++ b/drivers/staging/vc04_services/bcm2835-camera/bcm2835-camera.c > @@ -1526,6 +1526,7 @@ static int get_num_cameras(struct vchiq_mmal_instance *instance, > struct mmal_parameter_camera_info_t cam_info = {0}; > int param_size = sizeof(cam_info); > int i; > + int min_size; > > /* create a camera_info component */ > ret = vchiq_mmal_component_init(instance, "camera_info", > @@ -1541,11 +1542,10 @@ static int get_num_cameras(struct vchiq_mmal_instance *instance, > ¶m_size)) { > pr_info("Failed to get camera info\n"); > } > - for (i = 0; > - i < (cam_info.num_cameras > num_resolutions ? > - num_resolutions : > - cam_info.num_cameras); > - i++) { > + > + min_size = cam_info.num_cameras > num_resolutions ? > + num_resolutions : cam_info.num_cameras; Again, just use min(). thanks, greg k-h