From patchwork Wed Mar 29 04:50:25 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Thomas_Wei=C3=9Fschuh?= X-Patchwork-Id: 76359 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:b0ea:0:b0:3b6:4342:cba0 with SMTP id b10csp169300vqo; Tue, 28 Mar 2023 22:09:07 -0700 (PDT) X-Google-Smtp-Source: AKy350arBjA3udSLz7iNCaLKLyX+PSLMj7EBP6MB30PHImc8VHkV3YUQ3F8puHY+ehXXB5uaP7nV X-Received: by 2002:a17:906:d97:b0:8b1:2c37:ae97 with SMTP id m23-20020a1709060d9700b008b12c37ae97mr20398371eji.43.1680066547234; Tue, 28 Mar 2023 22:09:07 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1680066547; cv=none; d=google.com; s=arc-20160816; b=yYkqR9r2RTkJsB48QVvq66hrKvzV+mcQemKfSaMYPZl19wlirTz38+txiNmu0g05ln uG5VHh9IFxXYpFld5kn80uv6POVM0pt8VU/oPTNAni4Qscb7Jz0cYq21SC6B9/sLwxqQ uWYvPQzrWyaXLTEFQIrLPj5bMHGKikjI7ohyzhUyy3WnSaOob5alAN7Dzic/RPmUUuWe cg14/Ms2xjNiJ2FdxcZ2ZfUWGn+47iGXsQrcoYJn2AuKDwLMYcEa8KcjsJGo1nZnH6iy caOOLAzqpvA7zDvgUB0tqtPXg4D5IcdRFryXMuYHMNnab/n20qqlPtaFUqCYPk2FJrF0 A8zw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:message-id:content-transfer-encoding :mime-version:subject:date:dkim-signature:from; bh=ms6G0spWr1+ZRDcoWrNluc9nehlMV4SpsUtTr69DS1c=; b=yQ+3bX3NpJU8GQHzEG+XgVT23I52yu9COeahuRWlEQkMhtYKWVqwnMPU7Ic+zt1UR2 xc6xPV+jQriJpf4rBRVZmrtjuYRh+wQc3ZBqvcnRRTjgw3Z+o62BW3bKJQpoVcZ5BD8J 9HrDbahHyKZeJuhysrfjtjJpRbLiZTGJabRuuwPcmdlBfN4TvB4gH23UMBLq8bLduQTh qc1CzqsxTuTye5LbPGdakt7T7nfXIkK72jS1EVgxg8tRVwXxixWqO1S8j7b0ofRayxgh uC1Yk0vq0ZJuvuNZ7D7Zavr/gxfUTgrhm1a/roqjYFOUwkiTQ2VAuZPeVLDFza6mzWUw d6Bg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@weissschuh.net header.s=mail header.b=bsYwTtfS; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id tx26-20020a1709078e9a00b0093348efaf0asi13390675ejc.198.2023.03.28.22.08.43; Tue, 28 Mar 2023 22:09:07 -0700 (PDT) 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; Authentication-Results: mx.google.com; dkim=pass header.i=@weissschuh.net header.s=mail header.b=bsYwTtfS; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229753AbjC2Eud (ORCPT + 99 others); Wed, 29 Mar 2023 00:50:33 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:35328 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229456AbjC2Euc (ORCPT ); Wed, 29 Mar 2023 00:50:32 -0400 Received: from todd.t-8ch.de (todd.t-8ch.de [IPv6:2a01:4f8:c010:41de::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 209662D46; Tue, 28 Mar 2023 21:50:30 -0700 (PDT) From: =?utf-8?q?Thomas_Wei=C3=9Fschuh?= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=weissschuh.net; s=mail; t=1680065428; bh=3uDKL3dxAD8wUGaHcCtXGJ0ik0XhHCgJ0XsIoOMR6mg=; h=From:Date:Subject:To:Cc:From; b=bsYwTtfS+4xkWfPFqC/VUCJHKjRkhFCGJa9s8462OKpDQOGfGablzqi7G1aSPs2qq xH4JGx04WWJFBXkIv86/0xZEVopT3qol6ylsVdkPNVTf7ghf0yiQB2R8NAP+TTAwyY 4qJKH1QB5MUYwxaUEFn9+s0cJuhmbeBoBg47j4PQ= Date: Wed, 29 Mar 2023 04:50:25 +0000 Subject: [PATCH] platform/x86: think-lmi: Remove unnecessary casts for attributes MIME-Version: 1.0 Message-Id: <20230329-think-lmi-attrs-v1-1-5794f2367cc2@weissschuh.net> X-B4-Tracking: v=1; b=H4sIAJDDI2QC/x2N0QrCMAwAf2Xk2cDsJrP+iviQbtEGa5SmijD27 wYf7+C4FYyrsMGpW6HyR0ye6rDfdTBn0hujLM4Q+jD0Q4jYsugdy0OQWquGMY6U0nSYOB7Bq0T GmCrpnL3TdykuX5Wv8v1vzpdt+wFpjO6TdgAAAA== To: Mark Pearson , Hans de Goede , Mark Gross Cc: platform-driver-x86@vger.kernel.org, linux-kernel@vger.kernel.org, =?utf-8?q?Thomas_Wei=C3=9Fschuh?= X-Mailer: b4 0.12.2 X-Developer-Signature: v=1; a=ed25519-sha256; t=1680065425; l=1970; i=linux@weissschuh.net; s=20221212; h=from:subject:message-id; bh=3uDKL3dxAD8wUGaHcCtXGJ0ik0XhHCgJ0XsIoOMR6mg=; b=VLmT86d5sNkUhvSPhj+1JdaQQqAB7clQbigXTWogDddXcTmRsmJVLQ8BL/b79XBx3jqBPxDaP SPk90swaAUFCYYddQGWptkFLOYldeyfQx+s3teWsmFooWrN8KSPdJ0M X-Developer-Key: i=linux@weissschuh.net; a=ed25519; pk=KcycQgFPX2wGR5azS7RhpBqedglOZVgRPfdFSPB1LNw= X-Spam-Status: No, score=-0.2 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_NONE,SPF_PASS autolearn=unavailable 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: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1761677460252183650?= X-GMAIL-MSGID: =?utf-8?q?1761677460252183650?= These casts are unnecessary and could break if structure layouts are randomized or implementation details change. Use the proper syntax that works without casts. Also remove some unnecessary braces that checkpatch complains about. Signed-off-by: Thomas Weißschuh --- drivers/platform/x86/think-lmi.c | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) --- base-commit: fcd476ea6a888ef6e6627f4c21a2ea8cca3e9312 change-id: 20230329-think-lmi-attrs-994abb757e98 Best regards, diff --git a/drivers/platform/x86/think-lmi.c b/drivers/platform/x86/think-lmi.c index c816646eb661..02bfd6e3fed2 100644 --- a/drivers/platform/x86/think-lmi.c +++ b/drivers/platform/x86/think-lmi.c @@ -862,19 +862,18 @@ static umode_t auth_attr_is_visible(struct kobject *kobj, struct tlmi_pwd_setting *setting = to_tlmi_pwd_setting(kobj); /* We only want to display level and index settings on HDD/NVMe */ - if ((attr == (struct attribute *)&auth_index) || - (attr == (struct attribute *)&auth_level)) { + if (attr == &auth_index.attr || attr == &auth_level.attr) { if ((setting == tlmi_priv.pwd_hdd) || (setting == tlmi_priv.pwd_nvme)) return attr->mode; return 0; } /* We only display certificates on Admin account, if supported */ - if ((attr == (struct attribute *)&auth_certificate) || - (attr == (struct attribute *)&auth_signature) || - (attr == (struct attribute *)&auth_save_signature) || - (attr == (struct attribute *)&auth_cert_thumb) || - (attr == (struct attribute *)&auth_cert_to_password)) { + if (attr == &auth_certificate.attr || + attr == &auth_signature.attr || + attr == &auth_save_signature.attr || + attr == &auth_cert_thumb.attr || + attr == &auth_cert_to_password.attr) { if ((setting == tlmi_priv.pwd_admin) && tlmi_priv.certificate_support) return attr->mode; return 0;