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:   Minjie Du <duminjie@vivo.com>
To:     Harry Wentland <harry.wentland@amd.com>,
        Leo Li <sunpeng.li@amd.com>,
        Rodrigo Siqueira <Rodrigo.Siqueira@amd.com>,
        Alex Deucher <alexander.deucher@amd.com>,
        =?UTF-8?q?Christian=20K=C3=B6nig?= <christian.koenig@amd.com>,
        "Pan, Xinhui" <Xinhui.Pan@amd.com>,
        David Airlie <airlied@gmail.com>,
        Daniel Vetter <daniel@ffwll.ch>, Minjie Du <duminjie@vivo.com>,
        Stephen Rothwell <sfr@canb.auug.org.au>,
        amd-gfx@lists.freedesktop.org (open list:AMD DISPLAY CORE),
        dri-devel@lists.freedesktop.org (open list:DRM DRIVERS),
        linux-kernel@vger.kernel.org (open list)
Cc:     opensource.kernel@vivo.com
Subject: [PATCH v1] drivers: dcn315: remove duplicate assignments
Date:   Wed,  5 Jul 2023 16:18:44 +0800
Message-Id: <20230705081845.12621-1-duminjie@vivo.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?EjDvw9w5lUdRqh1O4eYXSPAdwsYaoh6QjgYbz2pgflOAlk4WDqGKtVGMuoYU?=
 =?us-ascii?Q?H3ymRlehjU8Wh++H7pkJ+GTyMDNLb8ZY31fMy/vkwQ9612PNW+dVufAnxB7D?=
 =?us-ascii?Q?YHOYExfgdEGhsB0yMRPJy7Q5C720idX4oyaK3ogH9zAIsoGccA94Evn5EGRz?=
 =?us-ascii?Q?vlo59/n1K4Fb60lFTQm5Xm0+SXFEKz75X+bg6xfy7ZNOSfspbSpzWwImqcSt?=
 =?us-ascii?Q?M7EC8K15u8R2ogvBA/tuQAkBBMrKTCdtnIFkepmWc0dcRIOskMNQ+QBnguWk?=
 =?us-ascii?Q?th3A7nAUqxzALuMzXQZZMTv9j15uaXs52VjF/JePtZvR8BpGt6aIqbASL7cr?=
 =?us-ascii?Q?b0ZDlLKi0wO4tG/8ECcfASIenIQb1oUleSb19UtUeQLZldmvDOGEac7BdYwP?=
 =?us-ascii?Q?wAYMd+A9Yd2tUz4cUdOO0ntm0bNIBqT0AcropAaXynO5k1V9RuRNXJl66uLJ?=
 =?us-ascii?Q?fxiOK+hJVxB2uVnc86u4mn/xFdtT4fEjFO4IwTmMyL1K7+CYMUNXcnsQVyde?=
 =?us-ascii?Q?DvtmZdaBcjD7W2hmlJRAi85cdAR1qkzn8vEEJJuSkEdOEQHd/3416Kydm9AT?=
 =?us-ascii?Q?7a0fr9a2UQ1Q138XrKbVHGhaQxchWrre5GGgBNYhAzuWMvpcRTsVFunBPCIG?=
 =?us-ascii?Q?KWFoC4ljfN5gmCmgootfjN2K1O8PN0MzxoMc3u1QyJGUJwQEUoprHU6j4Ga1?=
 =?us-ascii?Q?km/VrO5WaIwNtiLbrJmNL/Ut5DdEyM6FRIiKeBB+6qju3Us+7bk2j6zZSKVn?=
 =?us-ascii?Q?Z5Y91oEeXYwEeilIGMEJ+eFhtKq5p0HvkQu8TwjsAVRGucL8ykWRR7YRnqPj?=
 =?us-ascii?Q?9XhEfxTRZ63fE6sUFTi8UoAqK1xizEGrm8x7mJ1ctjcRsqw30Criqbr2jrW1?=
 =?us-ascii?Q?M8bnGzlwnqrs91Ri3lk8Grp+LI2Y5ZNBJJBYZBP6jGPRtnG18z4Nmz9KT3ZM?=
 =?us-ascii?Q?zTvpdT6RrXObcgT+YsbkAE8+qyRTFcmYIQV/5r5FE/VXyUZ7nH7x6ApRjASn?=
 =?us-ascii?Q?Rzc9E/LOy2JIb3XJOWfKxkTvutkXGElKAzXKRcOBMsCrjNuxlfm2oBh+jO1O?=
 =?us-ascii?Q?TV4Fg+SmcNiIWPYqZr9DDDMjznQ3vTfwunDV5VoQxS7X1Q5mN4K7jXMKVAck?=
 =?us-ascii?Q?JoEQJVXAnuPzvBYAeena0t01k2BDQpESQ3viNS2z/Iv5Fhe0bvtFkDy7lCyk?=
 =?us-ascii?Q?ml8G2UhN48QbWsrE6Mb4a1ZOalzxMg/hk30EwJQZcxlVgADDxHUpX/AN5jvP?=
 =?us-ascii?Q?oO92J+ARjbb7Zq3/3NPArdAD9vz81VfdGXI4exNFKo6pya0BOcaMPXQxgu1U?=
 =?us-ascii?Q?ap/zFlkHkxG92vHV9JbVbjV+XC0KHqAhW/RDWQz9YPCYQwxBWvD/EqmCcJpC?=
 =?us-ascii?Q?KkI7rYGAUfuHNhbOd7tw6eJji2NnyOf8Kf/HGgj7B/kyNrFiNJTuwZguElKN?=
 =?us-ascii?Q?EGbmKULRMhRPcF3qt1BwofekvM6FE3MAQ3UF3X5IVPvo5ktl4U0Ei1IBKD8O?=
 =?us-ascii?Q?XdJafxVHfd9EzSkNZ7AP14yq/ygsRMLUEv7cnevuQKQ9rmYkLZGv88cBeTce?=
 =?us-ascii?Q?iHBS+hha0S7AgKaSJArn8/5wuhJghr7dfeqBz/kQ?=
X-OriginatorOrg: vivo.com
X-MS-Exchange-CrossTenant-Network-Message-Id: ced1a88d-d226-4506-29d9-08db7d307944
X-MS-Exchange-CrossTenant-AuthSource: SEZPR06MB5269.apcprd06.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 05 Jul 2023 08:18:55.2032
 (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: I6mBl2J7XXFM2yNyGqWJ9hnHuvNbhXb4PwWtu3rcH2nnWKxpQgzfHyiry7Bl7MZTjClyxoXaEgUqofpUyhrNsg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: TY0PR06MB5306
X-Spam-Status: No, score=-0.8 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,RCVD_IN_VALIDITY_RPBL,SPF_HELO_PASS,SPF_PASS,
        T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED autolearn=no 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

make clk_mgr_base->clks.zstate_support avoid double assignment.
It looks like the value of new_clocks is unchanged, so there 
may be no need for repeated assignments.


Signed-off-by: Minjie Du <duminjie@vivo.com>
---
 drivers/gpu/drm/amd/display/dc/clk_mgr/dcn315/dcn315_clk_mgr.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/display/dc/clk_mgr/dcn315/dcn315_clk_mgr.c b/drivers/gpu/drm/amd/display/dc/clk_mgr/dcn315/dcn315_clk_mgr.c
index b2c4f97af..45b811610 100644
--- a/drivers/gpu/drm/amd/display/dc/clk_mgr/dcn315/dcn315_clk_mgr.c
+++ b/drivers/gpu/drm/amd/display/dc/clk_mgr/dcn315/dcn315_clk_mgr.c
@@ -143,7 +143,6 @@ static void dcn315_update_clocks(struct clk_mgr *clk_mgr_base,
 	 * if it is safe to lower, but we are already in the lower state, we don't have to do anything
 	 * also if safe to lower is false, we just go in the higher state
 	 */
-	clk_mgr_base->clks.zstate_support = new_clocks->zstate_support;
 	if (safe_to_lower) {
 		/* check that we're not already in lower */
 		if (clk_mgr_base->clks.pwr_state != DCN_PWR_STATE_LOW_POWER) {
-- 
2.39.0