Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Angus Chen <angus.chen@jaguarmicro.com>
To:     eric.auger@redhat.com, alex.williamson@redhat.com,
        cohuck@redhat.com
Cc:     kvm@vger.kernel.org, linux-kernel@vger.kernel.org,
        Angus Chen <angus.chen@jaguarmicro.com>
Subject: [PATCH] vfio: platform: No need to check res again
Date:   Sat,  7 Jan 2023 11:47:20 +0800
Message-Id: <20230107034721.2127-1-angus.chen@jaguarmicro.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?C4FAmbiW5QeNrU5j1PBozXH8aPZjQ2033oxZ7IYNLJax6XdNF/QizkR+oD3d?=
 =?us-ascii?Q?jixScKHqAgItbY9TZklBMHiC6sD+W4AlcpLzr5TMVzJmOuIHtVDeGoQ23LAf?=
 =?us-ascii?Q?jKFmWecWFUkbUDH2gbrqOzwfFwDJyW/JT47O+ilvCqu+GuXPWmTreli87m9j?=
 =?us-ascii?Q?/sqK0hxRdlg/B158prHLp1IWkjkEPjTq1L8q+yr4bl8fmIO7hEPBTr+6Iilq?=
 =?us-ascii?Q?UHsBhTCfl0NH08Nllyd0Ro9bGf7UWV6DPLrD156Jkgznu6bYgc/tLkigUAay?=
 =?us-ascii?Q?xInDkc35470UrdgukfjwOLSPw3PY0vMe5PmiY2mjNoKxaa4fswO652HjnT92?=
 =?us-ascii?Q?MRwD2N/njYRDpSEHn1+xerF6YXYGZwjkM1r0PwcXzgtnUxR6g4AwFzr9MGJQ?=
 =?us-ascii?Q?PEl7yJDXsSGA+kFdbqQMpaoKkGN/f/2BakntA1AFTKkGT4OBnzSDsbvA1d4O?=
 =?us-ascii?Q?u2FEhx/xBcUkkNJ834t7+hkspPW4PpMbqspqPiAVLGiH44PYBkSsYUpOEbOu?=
 =?us-ascii?Q?KtphzOvrOnzrOcxImTv7/yTVA64wOAYi3RyQ5vqAmJQxiEJ0BDr+E4t95OQx?=
 =?us-ascii?Q?QHn6H0rTnOlF8MxWLBd/fTT7sHYdOaW5VQBCjxzMlK28kXf4jwOFiX2nEufO?=
 =?us-ascii?Q?6fnnWJy4TSfgN4N1mYEnmBaHD9vI0azE6/uodYC7nHoUn6QCjpUb3l/YjWrr?=
 =?us-ascii?Q?NmMMx5EUwsYHOVWctNy/GMhu0SGaL7mq1+msZKWYGwyYHtoQWtz8CvCwIDr9?=
 =?us-ascii?Q?Hf8tZTlgJeC2CmlcxOUVM7YteMIen3sjVqc1IZoFJAv1iL9Td0qNC3utHU44?=
 =?us-ascii?Q?csrxMA5NYKm1K7gtZQ2LvJl2aqWwtPARWy5yrPc3YmmNEj+WMoXdqkkW9h7w?=
 =?us-ascii?Q?n2oQfkTnL1EA6PpKaHOA71Hf/7bbC7k1ygCJ6gZ7ksJm2w3c2eWq7kkThAlO?=
 =?us-ascii?Q?ewf2R9WwLlxfTA+EvcjChh+qeq90y1CaO+iBIan/VeGu69ZkwL+dr9IRmD8x?=
 =?us-ascii?Q?uLld+nf51P9uBmRPBahMIfAGTK7ijQx9ZtL2ITq/H5+A+L14EYqYg4qKtJAR?=
 =?us-ascii?Q?BxtC216rE+breszFAqAYhWd2FQ2O8qXYjC8t48a9oEHKBLNjNybZuXAqpokI?=
 =?us-ascii?Q?VVwgif5v3++c0mNirD80ZZWJnczu9OJlm3jOjQDCrBnjiYYVYiiqWrCSChZb?=
 =?us-ascii?Q?+2iBxveVhFmIGcoqbLidauL3KWlfKxuRJeF66G0GUksLY9pww5ptXmr41mc3?=
 =?us-ascii?Q?n64oJIvsA16Ffn5DryUQKGjkRBQN6CS7W7eWshaMaYCxzVuUs06ZO8+PFhW2?=
 =?us-ascii?Q?rturOFamrFanPuQnYlgsAm9wSI0LZLwTwp4fz9dPZxo5M9JiUD4Jfl6YjHdf?=
 =?us-ascii?Q?9PPVXj/ZPkTkxl7PiD6vY1V7ZjBAQxeZYDOx8lo7vgSCPkm2MtRE9zJbbJ5f?=
 =?us-ascii?Q?FkrFaF9m/rKQ4w5bo6w9Ez3wyOMEuZNLyD7KhnuXUydDcDzLE1sgNL3xW+pH?=
 =?us-ascii?Q?5jKCy3zUFBjGt+sylE5vrYINW9XNX3KbH7OgBZGIBlRzrCuYXCVTUn3gIPP+?=
 =?us-ascii?Q?5SnajFFz7j36aB7Xhdq2NMRvXyyBAPJ4KGFoDWM+KbGN5ACMN40ylp+vu5z9?=
 =?us-ascii?Q?EA=3D=3D?=
X-OriginatorOrg: jaguarmicro.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 77b7bf46-096a-4ad2-9188-08daf061edb0
X-MS-Exchange-CrossTenant-AuthSource: TY2PR06MB3424.apcprd06.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 Jan 2023 03:47:41.9737
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 1e45a5c2-d3e1-46b3-a0e6-c5ebf6d8ba7b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: TzfYFPaIFcdJG1IZlopegqLYaRRvPPcvkAA63YXGLhDI0R6t1udytM7yjigTnPMJSSjQ6WPWiWGUHIILo+ILgT/1GUjw3n4BcHrefGWLoyA=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: KL1PR0601MB4615
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_PASS,SPF_PASS
        autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

In function vfio_platform_regions_init(),we did check res implied
by using while loop,
so no need to check whether res be null or not again.

No functional change intended.

Signed-off-by: Angus Chen <angus.chen@jaguarmicro.com>
---
 drivers/vfio/platform/vfio_platform_common.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/drivers/vfio/platform/vfio_platform_common.c b/drivers/vfio/platform/vfio_platform_common.c
index 1a0a238ffa35..a9ad3f4d2613 100644
--- a/drivers/vfio/platform/vfio_platform_common.c
+++ b/drivers/vfio/platform/vfio_platform_common.c
@@ -150,9 +150,6 @@ static int vfio_platform_regions_init(struct vfio_platform_device *vdev)
 		struct resource *res =
 			vdev->get_resource(vdev, i);
 
-		if (!res)
-			goto err;
-
 		vdev->regions[i].addr = res->start;
 		vdev->regions[i].size = resource_size(res);
 		vdev->regions[i].flags = 0;
-- 
2.25.1