From patchwork Tue Nov 15 11:28:51 2022 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Xuezhi Zhang X-Patchwork-Id: 20324 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a5d:6687:0:0:0:0:0 with SMTP id l7csp2665263wru; Tue, 15 Nov 2022 03:36:18 -0800 (PST) X-Google-Smtp-Source: AA0mqf6MTrWsC1MeNT0Zym+SD2H+gjlvjVuG42pJLpuAW7RqqEhrpIGnBUGfYv5szfwsCv87BMBN X-Received: by 2002:a17:903:2151:b0:172:d410:2a91 with SMTP id s17-20020a170903215100b00172d4102a91mr3864614ple.30.1668512177806; Tue, 15 Nov 2022 03:36:17 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1668512177; cv=none; d=google.com; s=arc-20160816; b=kpQkbFJArvbJ9hNDH6RCX+EY7KFwhJTG7/Y5C9GN35gP/NeREGRr0MShDl0bnyEqsz xTFebajOWXvXd7raJpUXOyWL5XIdTElnkzAepMvW6NIGmeT3mepyTpMbBhFP1/71C8FB L/s+rvcOfINX/GMM+kre/dEQDM4DCbpBqn2K6l4fCKpo0IYedrMWWQU7JIXVLWMAAiUU c/vbXHeMxDiu3Rx7L771GtFZN4/hBiRbld0RLtwpS/jvnOm+jeI8u7dD0TlC91XZCqFN q8VJspiBXtXf1j1/NBXmjJhMyEI4py3q3B6GdTxvZkYsJQKaejTMt/6ubY4sbd0cLBQ2 sUcQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:dkim-signature; bh=3o0IP0jRO/k6EuewxkJJcEod0HUZCfXHgiOkxOYiCag=; b=b9+/b5/WhdyS+yfSCstUMXe0pWWzTNHi092Ydktv3OX9gqJtiI/DuDay4RxWmN6SKv b0TLzqDvN6dJGgvLYR7kunPP955L+djWY+CRAJTxtbKvVEMI8S2fIG4BIyV1tkOleZJK ky6z8jHBvTtmIBVZluE0q7zUOxWmQlXqhkpVLaOSDyTwxD8RRQi+LY3elWAHGQURbFh3 Vx0V2BKdtpjxgut9VM1tM8H5/bHvEcODC6yDYJ1j5mPD1G9MDu+NHJVMFd7t9mWiXXaJ HN0abF7lPx9AcHGguq4rk6VrET3joe7k2pR0zkv2XCmMP/oLP/eRlvNSlIfbesHOOMku obrQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20210112 header.b=EwXpXdYm; 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; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id u26-20020a63471a000000b0046f5a213b6bsi11702265pga.750.2022.11.15.03.36.04; Tue, 15 Nov 2022 03:36:17 -0800 (PST) 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=@gmail.com header.s=20210112 header.b=EwXpXdYm; 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; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232318AbiKOL3J (ORCPT + 99 others); Tue, 15 Nov 2022 06:29:09 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:35268 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229871AbiKOL26 (ORCPT ); Tue, 15 Nov 2022 06:28:58 -0500 Received: from mail-pj1-x1034.google.com (mail-pj1-x1034.google.com [IPv6:2607:f8b0:4864:20::1034]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 7B6731010; Tue, 15 Nov 2022 03:28:57 -0800 (PST) Received: by mail-pj1-x1034.google.com with SMTP id h14so13050973pjv.4; Tue, 15 Nov 2022 03:28:57 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=3o0IP0jRO/k6EuewxkJJcEod0HUZCfXHgiOkxOYiCag=; b=EwXpXdYmCH55f66LcK5rJS3Qrmzr6sZjvFNr4efYWyP4Ykh/W1BOLZ3/7UM2LwRsZx 2lPQZRJWTmWsRULXVOfpEnxyUWLlHsmXwDrrx8uLxnQW8wdWZtTT6EsOqWOvxtC8Kn99 Bk+/Yccd+SaAKHP/QQVowATqCPuUK43n4EZOpMzdgLqfrcfj3t3eoHymfd14TXHKOmB6 X8RYMQUxJUpUdjNBDgjYzlWMMhUKIBfnG3jsAts74/Vx1wgp8/BZ/fYEnMPSUOoJfSNB FJtJyPTLA5dawWuJBtqo6WOrjeHBYFo3bWC+LFGtgM1UMtOENCDPSeIT3le5sZ+fp+hv 5geA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=3o0IP0jRO/k6EuewxkJJcEod0HUZCfXHgiOkxOYiCag=; b=n2onegP1FbiIEE+Uy9t3uc6YJa1QsDrMsS8bp4d35fQnXV2GTlbUSatp9A5HYeHjts InxFcpJEN5sU828FvGTG2izgMWf46XQb6yuZBtSZG8roUZsAC6f5z/EJI2JRcdbKTPfJ QK4bN8h0GenAc0sdnI8zdWkTbESYV1BpyXRAB46NXQYM4jWWGe4EPxQaqGChU5BHVkl4 IJ/3rV0x0fWg9hTdiDlt1QuAKHF1tI5TUxqVksJa0WpOc+d7U8BpLxEVbOoelKNg+Ykg doAp0QItmnBdeE/DERdGqC6Al+UvlqcfGVtTnD2IDXO2ezd1B0cjVqUVfutPqk+Fzrqu dW+Q== X-Gm-Message-State: ANoB5pnRcG+HcmcMmq+EN1tYMrmkf4OpINMy0cZEdbCxAZyfgQo419/D d9s5AvwuYFOp14NgcwcRiF91NdMw1QI= X-Received: by 2002:a17:903:2d0:b0:188:7dca:6f41 with SMTP id s16-20020a17090302d000b001887dca6f41mr3580296plk.72.1668511736984; Tue, 15 Nov 2022 03:28:56 -0800 (PST) Received: from carlis-virtual-machine.localdomain ([156.236.96.164]) by smtp.gmail.com with ESMTPSA id e13-20020a17090ab38d00b0021806f631ccsm8127582pjr.30.2022.11.15.03.28.55 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 15 Nov 2022 03:28:56 -0800 (PST) From: Xuezhi Zhang To: zhangxuezhi1@coolpad.com, jinpu.wang@cloud.ionos.com, jejb@linux.ibm.com Cc: linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH] scsi: pm8001: convert sysfs snprintf to sysfs_emit Date: Tue, 15 Nov 2022 19:28:51 +0800 Message-Id: <20221115112851.1271409-1-zhangxuezhi3@gmail.com> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 X-Spam-Status: No, score=-1.8 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_ENVFROM_END_DIGIT, FREEMAIL_FROM,RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,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: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1749561825537302805?= X-GMAIL-MSGID: =?utf-8?q?1749561825537302805?= From: Xuezhi Zhang Follow the advice of the Documentation/filesystems/sysfs.rst and show() should only use sysfs_emit() or sysfs_emit_at() when formatting the value to be returned to user space. Signed-off-by: Xuezhi Zhang --- drivers/scsi/pm8001/pm8001_ctl.c | 38 +++++++++++++++----------------- 1 file changed, 18 insertions(+), 20 deletions(-) diff --git a/drivers/scsi/pm8001/pm8001_ctl.c b/drivers/scsi/pm8001/pm8001_ctl.c index 73f036bed128..3a08ece321a8 100644 --- a/drivers/scsi/pm8001/pm8001_ctl.c +++ b/drivers/scsi/pm8001/pm8001_ctl.c @@ -61,10 +61,10 @@ static ssize_t pm8001_ctl_mpi_interface_rev_show(struct device *cdev, struct pm8001_hba_info *pm8001_ha = sha->lldd_ha; if (pm8001_ha->chip_id == chip_8001) { - return snprintf(buf, PAGE_SIZE, "%d\n", + return sysfs_emit(buf, "%d\n", pm8001_ha->main_cfg_tbl.pm8001_tbl.interface_rev); } else { - return snprintf(buf, PAGE_SIZE, "%d\n", + return sysfs_emit(buf, "%d\n", pm8001_ha->main_cfg_tbl.pm80xx_tbl.interface_rev); } } @@ -86,8 +86,7 @@ static ssize_t controller_fatal_error_show(struct device *cdev, struct sas_ha_struct *sha = SHOST_TO_SAS_HA(shost); struct pm8001_hba_info *pm8001_ha = sha->lldd_ha; - return snprintf(buf, PAGE_SIZE, "%d\n", - pm8001_ha->controller_fatal_error); + return sysfs_emit(buf, "%d\n", pm8001_ha->controller_fatal_error); } static DEVICE_ATTR_RO(controller_fatal_error); @@ -107,13 +106,13 @@ static ssize_t pm8001_ctl_fw_version_show(struct device *cdev, struct pm8001_hba_info *pm8001_ha = sha->lldd_ha; if (pm8001_ha->chip_id == chip_8001) { - return snprintf(buf, PAGE_SIZE, "%02x.%02x.%02x.%02x\n", + return sysfs_emit(buf, "%02x.%02x.%02x.%02x\n", (u8)(pm8001_ha->main_cfg_tbl.pm8001_tbl.firmware_rev >> 24), (u8)(pm8001_ha->main_cfg_tbl.pm8001_tbl.firmware_rev >> 16), (u8)(pm8001_ha->main_cfg_tbl.pm8001_tbl.firmware_rev >> 8), (u8)(pm8001_ha->main_cfg_tbl.pm8001_tbl.firmware_rev)); } else { - return snprintf(buf, PAGE_SIZE, "%02x.%02x.%02x.%02x\n", + return sysfs_emit(buf, "%02x.%02x.%02x.%02x\n", (u8)(pm8001_ha->main_cfg_tbl.pm80xx_tbl.firmware_rev >> 24), (u8)(pm8001_ha->main_cfg_tbl.pm80xx_tbl.firmware_rev >> 16), (u8)(pm8001_ha->main_cfg_tbl.pm80xx_tbl.firmware_rev >> 8), @@ -138,7 +137,7 @@ static ssize_t pm8001_ctl_ila_version_show(struct device *cdev, struct pm8001_hba_info *pm8001_ha = sha->lldd_ha; if (pm8001_ha->chip_id != chip_8001) { - return snprintf(buf, PAGE_SIZE, "%02x.%02x.%02x.%02x\n", + return sysfs_emit(buf, "%02x.%02x.%02x.%02x\n", (u8)(pm8001_ha->main_cfg_tbl.pm80xx_tbl.ila_version >> 24), (u8)(pm8001_ha->main_cfg_tbl.pm80xx_tbl.ila_version >> 16), (u8)(pm8001_ha->main_cfg_tbl.pm80xx_tbl.ila_version >> 8), @@ -164,7 +163,7 @@ static ssize_t pm8001_ctl_inactive_fw_version_show(struct device *cdev, struct pm8001_hba_info *pm8001_ha = sha->lldd_ha; if (pm8001_ha->chip_id != chip_8001) { - return snprintf(buf, PAGE_SIZE, "%02x.%02x.%02x.%02x\n", + return sysfs_emit(buf, "%02x.%02x.%02x.%02x\n", (u8)(pm8001_ha->main_cfg_tbl.pm80xx_tbl.inc_fw_version >> 24), (u8)(pm8001_ha->main_cfg_tbl.pm80xx_tbl.inc_fw_version >> 16), (u8)(pm8001_ha->main_cfg_tbl.pm80xx_tbl.inc_fw_version >> 8), @@ -191,10 +190,10 @@ static ssize_t pm8001_ctl_max_out_io_show(struct device *cdev, struct pm8001_hba_info *pm8001_ha = sha->lldd_ha; if (pm8001_ha->chip_id == chip_8001) { - return snprintf(buf, PAGE_SIZE, "%d\n", + return sysfs_emit(buf, "%d\n", pm8001_ha->main_cfg_tbl.pm8001_tbl.max_out_io); } else { - return snprintf(buf, PAGE_SIZE, "%d\n", + return sysfs_emit(buf, "%d\n", pm8001_ha->main_cfg_tbl.pm80xx_tbl.max_out_io); } } @@ -215,11 +214,11 @@ static ssize_t pm8001_ctl_max_devices_show(struct device *cdev, struct pm8001_hba_info *pm8001_ha = sha->lldd_ha; if (pm8001_ha->chip_id == chip_8001) { - return snprintf(buf, PAGE_SIZE, "%04d\n", + return sysfs_emit(buf, "%04d\n", (u16)(pm8001_ha->main_cfg_tbl.pm8001_tbl.max_sgl >> 16) ); } else { - return snprintf(buf, PAGE_SIZE, "%04d\n", + return sysfs_emit(buf, "%04d\n", (u16)(pm8001_ha->main_cfg_tbl.pm80xx_tbl.max_sgl >> 16) ); } @@ -242,11 +241,11 @@ static ssize_t pm8001_ctl_max_sg_list_show(struct device *cdev, struct pm8001_hba_info *pm8001_ha = sha->lldd_ha; if (pm8001_ha->chip_id == chip_8001) { - return snprintf(buf, PAGE_SIZE, "%04d\n", + return sysfs_emit(buf, "%04d\n", pm8001_ha->main_cfg_tbl.pm8001_tbl.max_sgl & 0x0000FFFF ); } else { - return snprintf(buf, PAGE_SIZE, "%04d\n", + return sysfs_emit(buf, "%04d\n", pm8001_ha->main_cfg_tbl.pm80xx_tbl.max_sgl & 0x0000FFFF ); } @@ -315,7 +314,7 @@ static ssize_t pm8001_ctl_host_sas_address_show(struct device *cdev, struct Scsi_Host *shost = class_to_shost(cdev); struct sas_ha_struct *sha = SHOST_TO_SAS_HA(shost); struct pm8001_hba_info *pm8001_ha = sha->lldd_ha; - return snprintf(buf, PAGE_SIZE, "0x%016llx\n", + return sysfs_emit(buf, "0x%016llx\n", be64_to_cpu(*(__be64 *)pm8001_ha->sas_addr)); } static DEVICE_ATTR(host_sas_address, S_IRUGO, @@ -336,7 +335,7 @@ static ssize_t pm8001_ctl_logging_level_show(struct device *cdev, struct sas_ha_struct *sha = SHOST_TO_SAS_HA(shost); struct pm8001_hba_info *pm8001_ha = sha->lldd_ha; - return snprintf(buf, PAGE_SIZE, "%08xh\n", pm8001_ha->logging_level); + return sysfs_emit(buf, "%08xh\n", pm8001_ha->logging_level); } static ssize_t pm8001_ctl_logging_level_store(struct device *cdev, @@ -517,7 +516,7 @@ static ssize_t event_log_size_show(struct device *cdev, struct sas_ha_struct *sha = SHOST_TO_SAS_HA(shost); struct pm8001_hba_info *pm8001_ha = sha->lldd_ha; - return snprintf(buf, PAGE_SIZE, "%d\n", + return sysfs_emit(buf, "%d\n", pm8001_ha->main_cfg_tbl.pm80xx_tbl.event_log_size); } static DEVICE_ATTR_RO(event_log_size); @@ -604,8 +603,7 @@ static ssize_t non_fatal_count_show(struct device *cdev, struct sas_ha_struct *sha = SHOST_TO_SAS_HA(shost); struct pm8001_hba_info *pm8001_ha = sha->lldd_ha; - return snprintf(buf, PAGE_SIZE, "%08x", - pm8001_ha->non_fatal_count); + return sysfs_emit(buf, "%08x\n", pm8001_ha->non_fatal_count); } static ssize_t non_fatal_count_store(struct device *cdev, @@ -884,7 +882,7 @@ static ssize_t pm8001_show_update_fw(struct device *cdev, if (pm8001_ha->fw_status != FLASH_IN_PROGRESS) pm8001_ha->fw_status = FLASH_OK; - return snprintf(buf, PAGE_SIZE, "status=%x %s\n", + return sysfs_emit(buf, "status=%x %s\n", flash_error_table[i].err_code, flash_error_table[i].reason); }