Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Liu Ying <victor.liu@nxp.com>
To:     linux-arm-kernel@lists.infradead.org,
        dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org
Cc:     p.zabel@pengutronix.de, airlied@linux.ie, daniel@ffwll.ch,
        shawnguo@kernel.org, s.hauer@pengutronix.de, kernel@pengutronix.de,
        festevam@gmail.com, linux-imx@nxp.com, robh+dt@kernel.org,
        maarten.lankhorst@linux.intel.com, mripard@kernel.org,
        tzimmermann@suse.de, laurentiu.palcu@oss.nxp.com
Subject: [PATCH v3 4/6] drm/atomic: Avoid unused-but-set-variable warning on for_each_old_plane_in_state
Date:   Mon,  7 Dec 2020 11:20:58 +0800
Message-Id: <1607311260-13983-5-git-send-email-victor.liu@nxp.com>
In-Reply-To: <1607311260-13983-1-git-send-email-victor.liu@nxp.com>
References: <1607311260-13983-1-git-send-email-victor.liu@nxp.com>
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
Received: from localhost.localdomain (119.31.174.66) by SG2PR01CA0085.apcprd01.prod.exchangelabs.com (2603:1096:3:15::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256) id 15.20.3632.18 via Frontend Transport; Mon, 7 Dec 2020 03:29:21 +0000
X-MS-PublicTrafficType: Email
X-MS-Office365-Filtering-HT: Tenant
X-MS-Office365-Filtering-Correlation-Id: d73b3829-3fdc-4c22-a93a-08d89a604c79
X-MS-TrafficTypeDiagnostic: VE1PR04MB6349:
X-MS-Exchange-Transport-Forked: True
X-Microsoft-Antispam-PRVS: <VE1PR04MB634963CD3DD8CEB5B4ED7CE098CE0@VE1PR04MB6349.eurprd04.prod.outlook.com>
X-MS-Oob-TLC-OOBClassifiers: OLM:229;
X-MS-Exchange-SenderADCheck: 1
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info: pfWxtu5NWM4cTN4A1CYm8FUW/WDwuoNygPEjb6/wMYhd3T7eMFk20PIJyC7cPNltgQZF9A3TUuUG6BXmRZoDJfHGgKeS+VhRusuXdMSUPezrG4EuLkjeNldAAvYBmI+VFYp98ceECM5dbhnYdT4pm6gtd/NFR+wsAvzWLQXFfepqyQSnfn7zv84NlEjV/AUKVu9IJaCfsSLWN8sQcjfZcPMvA9lvXkUqxYdfbQbge+Zi8NxSSp/1b3ktbPdPCxo41WBBUCZoDM46Y+d5KYnyzv2YQp87h7i690M3VHgSLhTQ49TRojK+HVOSkIh1QsIWqRxTe0MpdVjI56crYCXZmEzNoBnBCxM6OSFekI0ut4q1N4urKhyGaK2lWaV6/CLM
X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:VI1PR04MB3983.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(4636009)(39860400002)(136003)(366004)(376002)(396003)(346002)(8936002)(6506007)(6512007)(2616005)(316002)(52116002)(26005)(478600001)(69590400008)(16526019)(186003)(6486002)(6666004)(4326008)(2906002)(7416002)(956004)(86362001)(36756003)(66946007)(66476007)(83380400001)(8676002)(66556008)(5660300002);DIR:OUT;SFP:1101;
X-MS-Exchange-AntiSpam-MessageData: =?us-ascii?Q?tMyO6jk4gAM6VaRuQUzR9cAeph5uyDbW+uXn4SaCXL5XgOVGItisSwW8iUgE?=
 =?us-ascii?Q?CUuwQIK1cFTT4sKVxxmeTbUgpbG9X45wrmMKI/h3wZDSw5lGA5qax6DHS+fF?=
 =?us-ascii?Q?1JGeB0TeAuF0+6hbTofHLda6BCMy69ZcyyhjPNp+n8ZH50lFMEK9vyODtEOB?=
 =?us-ascii?Q?49zUZwJfosxQ9yYvtFtR6eNFxPznQvpU5kSZjor9ao1ejZGZ8t/ZBZC1ClCF?=
 =?us-ascii?Q?zYrXheSPMZOlRGXAtCpfYyqeFTBB8M8Iwkq8v5FTxetuUMkOWNkeYn9c8PRY?=
 =?us-ascii?Q?mvPHVWE/mV0xDpO/m19qkknC4VHnR7gL/Hl1fiZ5sJbmFqBuhWQi0/31fIUL?=
 =?us-ascii?Q?1FmUCdIIiL/VkBSmbUTSIvj5Gjt31GOR4EBOmqJhNpiRZsv79J3QYcvUJNDQ?=
 =?us-ascii?Q?kE1Nxi1I5IZm/6CCj8h8xE8pDW2Y9/jzjXuBH9yYYVr2RRZgsRty8GtGHEeP?=
 =?us-ascii?Q?2g0Dcqzil/txyt8o06mBfbDwQ1G4Vm1s+4QhCZleM0kKxl/4keV7BG2xleA1?=
 =?us-ascii?Q?kP7LtLNbJOYUcB+Se4xM6FFzxuG4YV1zV7L3m6fvNuFaeiO8qp+CIymxYMkh?=
 =?us-ascii?Q?BKj9zcuhtYRfQ+YVqHnbjyKnOcMzdjBabK5jt0Y/0lT9o4HQRCuluvLqMxeH?=
 =?us-ascii?Q?2qV+MyLWuM3EQQ4jt+5gE7EOuBBWB4R37cbYuBxxGwHjL5gZujFDKUT3QiW0?=
 =?us-ascii?Q?Ijold28iODtdCiv0z3JbGMoumPj11c9LiSN4neZ+43tUk6VDgN3GBk73WA8q?=
 =?us-ascii?Q?fJ6LfovKTXLQzpHXqL3uwiQKQv1fMyM9TytJKPdL0Z6fl/2da+WDFFGrdYlQ?=
 =?us-ascii?Q?A1A8zYnZy0UFSKXwcbpiub+bisv1Wew89ERhljE1JWh4tDC3IU2ESvXLXpdR?=
 =?us-ascii?Q?L9MJ5lrlAfexpc+2TpL1b0EMGWNXjRvFhQE08LZmc38F/W+lBhxsNIynmPYD?=
 =?us-ascii?Q?RSk/ZloR4AXj9KIDbXXWWd+lhIOngN//aiJ46/1NSrcwlriOI785m92JFPbm?=
 =?us-ascii?Q?AFep?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: d73b3829-3fdc-4c22-a93a-08d89a604c79
X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB3983.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 Dec 2020 03:29:26.7152
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: yIJ7ZnIHPE3gvY6HGKEdc8y3E6QTxTfBLy0VEQOdBUN4ggQ2zIqfVXgLsF0VRTgQucBfE/xnMR+d+RGlzGFrew==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VE1PR04MB6349
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Artifically use 'plane' and 'old_plane_state' to avoid 'not used' warning.
The precedent has already been set by other macros in the same file.

Signed-off-by: Liu Ying <victor.liu@nxp.com>
---
v2->v3:
* Add a missing blank line.

v1->v2:
* No change.

 include/drm/drm_atomic.h | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/include/drm/drm_atomic.h b/include/drm/drm_atomic.h
index 54e051a..2e087d7 100644
--- a/include/drm/drm_atomic.h
+++ b/include/drm/drm_atomic.h
@@ -888,7 +888,10 @@ void drm_state_dump(struct drm_device *dev, struct drm_printer *p);
 	     (__i)++)							\
 		for_each_if ((__state)->planes[__i].ptr &&		\
 			     ((plane) = (__state)->planes[__i].ptr,	\
-			      (old_plane_state) = (__state)->planes[__i].old_state, 1))
+			      (void)(plane) /* Only to avoid unused-but-set-variable warning */, \
+			      (old_plane_state) = (__state)->planes[__i].old_state, \
+			      (void)(old_plane_state) /* Only to avoid unused-but-set-variable warning */, 1))
+
 /**
  * for_each_new_plane_in_state - iterate over all planes in an atomic update
  * @__state: &struct drm_atomic_state pointer
-- 
2.7.4