From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933641AbdGTFtl (ORCPT ); Thu, 20 Jul 2017 01:49:41 -0400 Received: from mail3-relais-sop.national.inria.fr ([192.134.164.104]:2269 "EHLO mail3-relais-sop.national.inria.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751714AbdGTFtj (ORCPT ); Thu, 20 Jul 2017 01:49:39 -0400 X-IronPort-AV: E=Sophos;i="5.40,382,1496095200"; d="scan'208";a="232141645" Date: Thu, 20 Jul 2017 07:49:35 +0200 (CEST) From: Julia Lawall X-X-Sender: jll@hadrien To: laurentiu.tudor@nxp.com cc: devel@driverdev.osuosl.org, arnd@arndb.de, marc.zyngier@arm.com, roy.pledge@nxp.com, linux-kernel@vger.kernel.org, agraf@suse.de, catalin.horghidan@nxp.com, leoyang.li@nxp.com, bharat.bhushan@nxp.com, linux-arm-kernel@lists.infradead.org, Laurentiu Tudor , gregkh@linuxfoundation.org, stuyoder@gmail.com Subject: [PATCH] staging: fsl-mc: fix resource_size.cocci warnings Message-ID: User-Agent: Alpine 2.20 (DEB 67 2015-01-07) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Use resource_size function on resource object instead of explicit computation. Generated by: scripts/coccinelle/api/resource_size.cocci Fixes: a037b7ec2eb7 ("staging: fsl-mc: allow the driver compile multi-arch") CC: Laurentiu Tudor Signed-off-by: Julia Lawall Signed-off-by: Fengguang Wu --- url: https://github.com/0day-ci/linux/commits/laurentiu-tudor-nxp-com/staging-fsl-mc-make-the-driver-compile-on-other-architectures/20170718-021715 :::::: branch date: 2 days ago :::::: commit date: 2 days ago mc-io.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) --- a/drivers/staging/fsl-mc/bus/mc-io.c +++ b/drivers/staging/fsl-mc/bus/mc-io.c @@ -242,8 +242,7 @@ int __must_check fsl_mc_portal_allocate( goto error_cleanup_resource; mc_portal_phys_addr = dpmcp_dev->regions[0].start; - mc_portal_size = dpmcp_dev->regions[0].end - - dpmcp_dev->regions[0].start + 1; + mc_portal_size = resource_size(dpmcp_dev->regions); if (WARN_ON(mc_portal_size != mc_bus_dev->mc_io->portal_size)) goto error_cleanup_resource;