From patchwork Wed May 18 12:08:36 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Guo Zhengkui X-Patchwork-Id: 12853580 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id B3F03C433EF for ; Wed, 18 May 2022 12:10:43 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-Id:Date:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=zKcNyrYnZXmhFZmI9N6sr8vhDNZjrhTXhni9NHYRgDA=; b=I1mdXntkZPgMeH 2mZNL1p51aMfvia7Cf9aHRLGK41MYSgIH8ujYjlJQ81Qj/2bjs4p9RQyAjS54/w3djmjqrtJHhT2A eLS5d0oIxBHdVzmuYc5Fj/54a3KKYDI32hUvDy4g6fdDm6/VuSZOwJr0rVxzsvWwOlnXt0aSrRJFA wduFcS7Cx23634NM2uVOvF3+rhsE79nAXQvW8xcSzWHgq51abWToRn2LSIDsVv6iFa5NVSLRhxaab z12cCg/55wLXaBHWNIiIGN1yxEmpcN5R7WVXwFh8Bo0XEDkLs53Dhq58sT+NGpVvwiP8o4m3b/FN2 IYt+vXe1vxETfiis6AoQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nrIUQ-001lSE-G5; Wed, 18 May 2022 12:09:18 +0000 Received: from mail-tyzapc01on20705.outbound.protection.outlook.com ([2a01:111:f403:704b::705] helo=APC01-TYZ-obe.outbound.protection.outlook.com) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nrIUN-001lQU-KG for linux-arm-kernel@lists.infradead.org; Wed, 18 May 2022 12:09:17 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=AH1aHo6KvlxGW3BrRbGXBCZ6Ul0cHO1g431cVNwUyMpI7h5yef81xp5hBLKILMREaJFvaPUhY3YLSy0qBxxaeBUSqtrFtJodXEwwuby8o7zZlaJVTu6bCJBWpirMyL8RHaoqcowLvSJUxyZqk+IssmuB/wGBUX+S3QuNSawCRgvKlHRmPkD5M/Nc9WEIw491W7E1+hxIPmkQnITiRhByONg8CT4a21fE5CEL37ZRQp533zJu5mlm/8G/DdLNNA0aTrEaA+HAHr906hAcb0R8bXGqaJeh2jK6IQYJCOJ7FWYVVew+KIFQG7rYj3CBb+f+6UcEDfnfoWTZZHySKYjNfg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=jTqE/2fnE7c6PaHnN/iy+mRqfso4q1FNVM0fKEq+dK4=; b=Av79Qu3lo+ylrfs9qzCUQBqojIuEKD2VT+NxkW9jA+pAY0NKbPt8U4dQqiDn9uXVFjgWuUzYBf8Yi/kFeDp+R8fMWK/49Hs6XoikkDpX8xeKGqpFsCesSfN5WwkmnV9xQzskU4mYrxmAiTdu2roNBI0GkDctZ84M3LROJJPhBvjpBsZqmfIMN4hywMflDMoOJphCi4KBu4+1wy87Dw1pKmtEKYnJJeL2AiKbtJpwKFknwF8+Eb0pl9xAjHOZmu+qQrY5hhmhWxfQtQDZFMc3/5U1i0L1IQ97oJHlG1okE1fdqd+p+cMdcA4DkUGMyJsU6xEg0HKbbli/6UIB4Ce30A== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=vivo.com; dmarc=pass action=none header.from=vivo.com; dkim=pass header.d=vivo.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=vivo0.onmicrosoft.com; s=selector2-vivo0-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=jTqE/2fnE7c6PaHnN/iy+mRqfso4q1FNVM0fKEq+dK4=; b=XnPAlyYZWU133oqd1+Z+67poYX3S7Mztm/GJ0W3OYmD+qadcmIvM6njWVIIl3uCmO7LEfX/PEdl7hPL045Ygc9Kzhl0jkLPe0jPW5JcJK+vSE+w00ib8eSPCRDU6z1+5QfC5ypZcOwDBhzV3/wT7Un0+TOWPrFj32fKJdTbZTEE= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=vivo.com; Received: from HK2PR06MB3492.apcprd06.prod.outlook.com (2603:1096:202:2f::10) by TYZPR06MB4352.apcprd06.prod.outlook.com (2603:1096:400:8a::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5273.14; Wed, 18 May 2022 12:09:08 +0000 Received: from HK2PR06MB3492.apcprd06.prod.outlook.com ([fe80::88e1:dc04:6851:ad08]) by HK2PR06MB3492.apcprd06.prod.outlook.com ([fe80::88e1:dc04:6851:ad08%7]) with mapi id 15.20.5250.018; Wed, 18 May 2022 12:09:08 +0000 From: Guo Zhengkui To: Andrzej Pietrasiewicz , Jacek Anaszewski , Sylwester Nawrocki , Mauro Carvalho Chehab , linux-arm-kernel@lists.infradead.org (moderated list:ARM/SAMSUNG S5P SERIES JPEG CODEC SUPPORT), linux-media@vger.kernel.org (open list:ARM/SAMSUNG S5P SERIES JPEG CODEC SUPPORT), linux-kernel@vger.kernel.org (open list) Cc: zhengkui_guo@outlook.com, Guo Zhengkui Subject: [PATCH v2] media: platform: samsung: s5p-jpeg: replace ternary operator with max() Date: Wed, 18 May 2022 20:08:36 +0800 Message-Id: <20220518120836.126368-1-guozhengkui@vivo.com> X-Mailer: git-send-email 2.20.1 In-Reply-To: <1761ab1a-68ce-4946-24d4-8f4f9575e735@xs4all.nl> References: <1761ab1a-68ce-4946-24d4-8f4f9575e735@xs4all.nl> X-ClientProxiedBy: TYAPR01CA0227.jpnprd01.prod.outlook.com (2603:1096:404:11e::23) To HK2PR06MB3492.apcprd06.prod.outlook.com (2603:1096:202:2f::10) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: d3337e54-5f6f-43f9-1c9a-08da38c7359d X-MS-TrafficTypeDiagnostic: TYZPR06MB4352:EE_ X-Microsoft-Antispam-PRVS: X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: JJ89OQnEgR2eEoP9p5d057lgI3yETDBO6MuQg7CZ4G3gBoxuSTfH7HqeJLXTKXiV3cHaC0Un16JEnTuA0N+gCUVvQkrRGF71frVwrYyB9KyVymZnjpkHFeabvNRHqvA3M3swZ73VfXL2hr3tvJbqQjR7DqwcA1/HzfE5jkq/Qc8Uz4uJgBJosVmcXCWkWjXR7meaGEMu8l4xq4EJzMIDpz8IGdDH4/q0OoDAVwfXO99CyrLEfAQj+CVTjoJu1QZwGPQsgd/I95uZ+FkiPr8SGBW+pLvH/taLX4OsUx8TKwQNrAU5+D1Hfd+VFZjlH8s/tUt/GxrNVSnraTefLXkGqxVlCqHhJl0pWkhLqC3gqG+yHa1kh1JU0gZk0oycZhm1+XNPIjQxYHJGiGybXDGSt4HInAwt2mslSv4cK+SaDjyaYRGEAPzQL+aq7Wu15+vNLs0suIxTvBCcDymS5FsJ3JHtq1gzClBiWuTcO1s/gAijuGFGbIcr7V/PYTpU/t/T9nCjplSGIX8YR16dnQ03LtPq9IfJYGcrM5f9hUuHnumT1rHIra0gR/jo7wxdOWiPLbdvxgamQ2ynM4tBGRwmnyWBLR7e8xl1KMKt3FrVgfAfdvAiSj2C/Np+ClBGCNcNk2HYKxHLSOq80GuBdNyHNJA7lnVkVN07HuJmBtAPTrDsnzokSS8p+DZ7KEm4uIkPNt+y9SsjDRKv/JPv6WHZ0g== X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:HK2PR06MB3492.apcprd06.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230001)(4636009)(366004)(316002)(2906002)(5660300002)(36756003)(110136005)(83380400001)(107886003)(52116002)(1076003)(86362001)(186003)(66946007)(2616005)(38350700002)(38100700002)(8936002)(8676002)(4326008)(6512007)(66556008)(66476007)(6506007)(6486002)(6666004)(26005)(508600001); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: s9OO6kcsHFl0EyFSW6SirXp/cAC/nEbIjlHcm9mSuJqkNhsxpLTs++DQCyB2dUm3FhrXWmaEJxhou7TjeHhzTUHNe0/GB8aXOg9gik9NrpcPCzY+FqqsFF2yg5yOjl7l0NOtnYN6szMrP+wnqJzT2E/87fvJ/q7xVTg20utSQwTEDPCNy/Wtagm+3ctpr9PKDoIkRwnGTuOKoIXrczU5LYLUocV5pt9O9/b/sHlvGmRnaqxj/ICtFFBPR7TZHRBI+c7GB2jUffLUxtl+nLGC/iDN19g+GkQ+11P0LucuZ656ZU7VJGFOz5pGcjph1QDIASlJkcDmQJzB3JUW2hh3zjoE70HnjdduQyncgNKUDzhihx3wQ0+G3hWSb7Os4ZXu5YXhFpTndFeyqwQEE3ya5m6OSABa9Qvh8I0MQQkkkxZ3pWM4pN7n52f2vCtTEXWXEV5Kd6gSdX7rdr/E6DF3omCkFgOz5eGpBpFb31zWWE7nJXK0kqH2Rolk0YYNDVjAX84C8dZJ3eKs9ABTk7QBZlh0Pl3xwTLR8NxTJQ2ilDXMWC9k2w7EmK3NeosLAeYzga2R6ZVCfHbb7bijBhKx821TNy+vrHFy+D/9MPRVVdO2xJfM4iVmnGnswAZI5nLdMIZntV/sNYMBVcutfV8KNytIoUeoJoE366lHd8qb6wjgktjB+dxw6R7HbYaVsOi2Oz0K5ftNuFBPuthYXT1jWXkYaAtbTYMPe+jMbCk2LRc4eY5uI2LGcsXYjUFkoPiSc14skrQgjpa0aUA3fcB3sT5eYNiWSfSzxPaqt6SnTv5JlEzEuDAHeL6ccSr1tXJFcABIgjk3tw5nX3r6BXcYVVHke9sRoiIe2Io5pXlsbc9SFlLGewo3coVFFex0JP6Jetx9DXyMvN8QE4lguDlvWtt2wDTwSVyd11aDhmwenGuqKUxUyUb4tLtaJVQtt3F2R9w4+db6MVf+YdGFjh/bxknuAd9nlbWlVLv5fCaZ4AB6rzsyl7/O0P6YYFVr48LgoV0A9piia0rR7OthE66BCyWoIS0IbyPC9z4FEQsjIerzpmn/GEhpE5t+U/FCYvDiVkxgOmeSoKwxHoyAs3sQzVSeaxwEWnTQsKfYoYqZrCgRnZYbo3pIyF0e26bD0d4TEW2uP5DvlKVNgGycW1JxM+4Fe/CQ8v2pT9mplKhTqn5vOA0L2oekIsY/qUUz9NQFbR8hqguBaWYlq5qxRvJEyonAWUwkeWD//e9hI6c2OPAM5dguxt4uc9lSvNtIDSzJf+M5wPZHX93tTuGH9BgdSu1rmmkzOKfCL9WORzep/aesCqS4GM0XP0guDhMKWKL2jdoU4vSSTJl/YmCYmnRsv0dCJeOwcrA9/2pRX4EPbuizNQFLQY9sdpsusgxY5uR5Bb2BaQcYR/mcH5Hv1SgAIWpCCbneGG4O2koJSRy7uuybGoDuTI2w4pdH6BARr0vAuw/wEjt5fO7oG/cxACBO1kYNPphwzgJDOa9AOpHhh2yqHhQ+He3ieNX4fyicYqZIigr9FyrKwIykRj106hQO5QOtlUMngRwAQnTUv/hAQIEJMak1p9m6AVAOykRF8/BOxXfgRH5LFnUquk8DDt2TUy1a2wpLMkbJtLpGgpfbwuelM9cQ5NZsov/YfwetOYRpGwKcFNbVfEealIssOnZy+Ogvl/Hsn75ElxJSlm7gkDuV/72vbKLAU3tfZTgFlZyPnC37oBESe6ISmn8HAJ58tQ== X-OriginatorOrg: vivo.com X-MS-Exchange-CrossTenant-Network-Message-Id: d3337e54-5f6f-43f9-1c9a-08da38c7359d X-MS-Exchange-CrossTenant-AuthSource: HK2PR06MB3492.apcprd06.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 May 2022 12:09:07.9388 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 923e42dc-48d5-4cbe-b582-1a797a6412ed X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: DNrLNFBoGa/LKCpc+BO8wA7gDQDUHXwRlUrcQ77Jjcv2BNtDNMmSm7yi6hGkbIRtINXHQe5pdvztHfAm9isX9w== X-MS-Exchange-Transport-CrossTenantHeadersStamped: TYZPR06MB4352 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220518_050915_708504_0812E824 X-CRM114-Status: UNSURE ( 9.46 ) X-CRM114-Notice: Please train this message. X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Fix the following coccicheck warning: drivers/media/platform/samsung/s5p-jpeg/jpeg-core.c:1712:24-25: WARNING opportunity for max() max() macro is defined in include/linux/minmax.h. It avoids multiple evaluations of the arguments when non-constant and performs strict type-checking. Signed-off-by: Guo Zhengkui Acked-by: Andrzej Pietrasiewicz --- v1 -> v2: Change the subject according to Hans Verkuil's suggestion. drivers/media/platform/samsung/s5p-jpeg/jpeg-core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/platform/samsung/s5p-jpeg/jpeg-core.c b/drivers/media/platform/samsung/s5p-jpeg/jpeg-core.c index 456287186ad8..55814041b8d8 100644 --- a/drivers/media/platform/samsung/s5p-jpeg/jpeg-core.c +++ b/drivers/media/platform/samsung/s5p-jpeg/jpeg-core.c @@ -1709,7 +1709,7 @@ static int exynos3250_jpeg_try_downscale(struct s5p_jpeg_ctx *ctx, w_ratio = ctx->out_q.w / r->width; h_ratio = ctx->out_q.h / r->height; - scale_factor = w_ratio > h_ratio ? w_ratio : h_ratio; + scale_factor = max(w_ratio, h_ratio); scale_factor = clamp_val(scale_factor, 1, 8); /* Align scale ratio to the nearest power of 2 */