Received: by 2002:a05:6a10:f347:0:0:0:0 with SMTP id d7csp1910332pxu; Tue, 24 Nov 2020 11:51:00 -0800 (PST) X-Google-Smtp-Source: ABdhPJxlyWYhRt7VRlO9oxAFl9FYNaIBgD6lZ6XbV9LkRHLPv3w19yR3BU9cpCo412hT5qhpiwUv X-Received: by 2002:a17:906:6546:: with SMTP id u6mr53673ejn.36.1606247459734; Tue, 24 Nov 2020 11:50:59 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1606247459; cv=none; d=google.com; s=arc-20160816; b=GFKRWiLMgN+JIwQeEJ6ktgojnB+cLZ/Hpi4mJ9OR/q+iTBXR3RcIZYMW+5gdyuCL4J sgxxfZzh9fIaVDIs8scoMpQT7w++GqS3IFs8XAxwaSWef6YYd3Va8orKtfIMtE3XYxrV 3dEuEAAk3FUeymJdeiUo2aEHfrmB/nzsk57xauB079Krwb+Snlxb/l01YzJZpzNiA4lS WQbuw2zPC98nA+g5Qw4wFegMIxqYSYCXfbOw6lyY1MKvOBqtO8GZGH4krKfWUor4dsTr uW3i9xe/v9Nj4Y7JmQiMO7glvlODqbif8d3IVSXunjuGfc842b1IQ0H3FqdFlxG9VzDo uldg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:subject:message-id:date:from:in-reply-to :references:mime-version:dkim-signature; bh=g7rAnBCNEOexDsLeiBc8AFxW7GC6geHmTufT6HDd6+M=; b=FAoi/kI6YErKXjNArmK96+4krJcqIJ90ohGqOkqFhSSwWFB5cMv/onVlEeFOiek+lB DS4xtEs69yj/zV1sa8AnrIKz3vpVBs9AK0S2IniUBOV2hUpnBMPAeaAssFeqL+WzO/Ga sPZYd+YqpU1+c6wSl1//kfUVuyinLX9mHUA62uatbTVmAk6tAvrBwAAi3m2dcE5wazTG zU3cHMKW6bNtIdedLONelcCmjBrEuJY5XvuXh4rSXxcmEAkB22JZNA3Yv3kZvkVbk9bH swNcMqo2ZWbDkG5kjuKMhEFqJYF//e8bUDngjx9SDrkUj1m2wkWul4Tfyz68l7/QeDoi tTbQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@cloud.ionos.com header.s=google header.b=EMjDhtqv; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=ionos.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id a11si9190266ejx.432.2020.11.24.11.50.36; Tue, 24 Nov 2020 11:50:59 -0800 (PST) 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; Authentication-Results: mx.google.com; dkim=pass header.i=@cloud.ionos.com header.s=google header.b=EMjDhtqv; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=ionos.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729583AbgKXGXX (ORCPT + 99 others); Tue, 24 Nov 2020 01:23:23 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:52300 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728601AbgKXGXX (ORCPT ); Tue, 24 Nov 2020 01:23:23 -0500 Received: from mail-ed1-x544.google.com (mail-ed1-x544.google.com [IPv6:2a00:1450:4864:20::544]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id CAE98C0613D6 for ; Mon, 23 Nov 2020 22:23:22 -0800 (PST) Received: by mail-ed1-x544.google.com with SMTP id v22so19622151edt.9 for ; Mon, 23 Nov 2020 22:23:22 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cloud.ionos.com; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=g7rAnBCNEOexDsLeiBc8AFxW7GC6geHmTufT6HDd6+M=; b=EMjDhtqvKPL+LhyRWhJ7GpK/6OhFaDspFcOFurIEKoAsnfdSiQH/4lbZOBLFEszfST t3NAqoNEGqJXwsuYU9Ruxskatx7rQP8LPXaz4BIL5tKybwTWrH6IuBAw+kGevXt0twOS 5ag1rMgG4v+X6GgUAbanNAQmI9M4AlShLlwH0w5iMaWgN7BuUbHdTLNU+xJ1GnkTuN9y HK3V2qg/n1YQ3juQRlx8hSy9BizRhSo/tvn410sDBBlDgtQ1rNyAmLs6wGZ+GihZvfr+ nqu6jdpBjtHlI6TJw22fr9BLwTJ8py4qL7AQzF/73OG9qZ/yZ/yysC25Zehk2sl09QW7 OIRA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=g7rAnBCNEOexDsLeiBc8AFxW7GC6geHmTufT6HDd6+M=; b=rWLrrh+SdzQAUtcbmPf9N6SGO+TO+tixGQFN9ei+qNhcsIEorZszbZt1pg8Huh4iVN 47besPGkvvsm4YVXfs2cVZEDO1L+NdO7xRfDhkQrO4ZF0ZhN36XKk6URmD0DDcuDvazk Z4Wnq+u/xa2E4k/WmcIpGwx6YI0iiDZZZlHsoyTS4sMFlyF+++ai1Z8AMLnl/Jmgb0XV jNwF5lXkKOcIpZcoyjPlDeZElmer6FM8+P4MHE9SkilF83J9GxxUh+IlBUY7EYXsDvV6 CKyoRDOiMaBm/BpBO6ZdpzrrHgyR3PGEz/5sYGrFypvXcxpcG3HeDxbDkeK+bbm4H54v xvsA== X-Gm-Message-State: AOAM5337mP1KEcT7aitI45YkDjQEt48v70lO9703RzK3aYaOmTReorMZ 4ae3UnuuccDkw2tEi8cHzAgWQXGiaEO8DyNWpxn8ag== X-Received: by 2002:aa7:d443:: with SMTP id q3mr2648810edr.262.1606199001305; Mon, 23 Nov 2020 22:23:21 -0800 (PST) MIME-Version: 1.0 References: <69dc34ff63adfa60b3f203ed2d58143b5692af57.1606192458.git.joe@perches.com> In-Reply-To: <69dc34ff63adfa60b3f203ed2d58143b5692af57.1606192458.git.joe@perches.com> From: Jinpu Wang Date: Tue, 24 Nov 2020 07:23:10 +0100 Message-ID: Subject: Re: [PATCH 1/2] scsi: pm8001: Convert pm8001_printk to pm8001_info To: Joe Perches Cc: "James E.J. Bottomley" , "Martin K. Petersen" , Linux SCSI Mailinglist , linux-kernel@vger.kernel.org Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Nov 24, 2020 at 5:36 AM Joe Perches wrote: > > Use the more common logging style. > > Signed-off-by: Joe Perches Acked-by: Jack Wang Thanks! > --- > drivers/scsi/pm8001/pm8001_init.c | 12 ++++++------ > drivers/scsi/pm8001/pm8001_sas.c | 4 ++-- > drivers/scsi/pm8001/pm8001_sas.h | 4 ++-- > 3 files changed, 10 insertions(+), 10 deletions(-) > > diff --git a/drivers/scsi/pm8001/pm8001_init.c b/drivers/scsi/pm8001/pm8001_init.c > index 13530d7fb8a6..38907f45c845 100644 > --- a/drivers/scsi/pm8001/pm8001_init.c > +++ b/drivers/scsi/pm8001/pm8001_init.c > @@ -1293,8 +1293,8 @@ static int pm8001_pci_suspend(struct pci_dev *pdev, pm_message_t state) > tasklet_kill(&pm8001_ha->tasklet[j]); > #endif > device_state = pci_choose_state(pdev, state); > - pm8001_printk(pm8001_ha, "pdev=0x%p, slot=%s, entering operating state [D%d]\n", > - pdev, pm8001_ha->name, device_state); > + pm8001_info(pm8001_ha, "pdev=0x%p, slot=%s, entering operating state [D%d]\n", > + pdev, pm8001_ha->name, device_state); > pci_save_state(pdev); > pci_disable_device(pdev); > pci_set_power_state(pdev, device_state); > @@ -1318,16 +1318,16 @@ static int pm8001_pci_resume(struct pci_dev *pdev) > pm8001_ha = sha->lldd_ha; > device_state = pdev->current_state; > > - pm8001_printk(pm8001_ha, "pdev=0x%p, slot=%s, resuming from previous operating state [D%d]\n", > - pdev, pm8001_ha->name, device_state); > + pm8001_info(pm8001_ha, "pdev=0x%p, slot=%s, resuming from previous operating state [D%d]\n", > + pdev, pm8001_ha->name, device_state); > > pci_set_power_state(pdev, PCI_D0); > pci_enable_wake(pdev, PCI_D0, 0); > pci_restore_state(pdev); > rc = pci_enable_device(pdev); > if (rc) { > - pm8001_printk(pm8001_ha, "slot=%s Enable device failed during resume\n", > - pm8001_ha->name); > + pm8001_info(pm8001_ha, "slot=%s Enable device failed during resume\n", > + pm8001_ha->name); > goto err_out_enable; > } > > diff --git a/drivers/scsi/pm8001/pm8001_sas.c b/drivers/scsi/pm8001/pm8001_sas.c > index 4562b0a5062a..d1e9dba2ef19 100644 > --- a/drivers/scsi/pm8001/pm8001_sas.c > +++ b/drivers/scsi/pm8001/pm8001_sas.c > @@ -1191,7 +1191,7 @@ int pm8001_abort_task(struct sas_task *task) > phy_id = pm8001_dev->attached_phy; > ret = pm8001_find_tag(task, &tag); > if (ret == 0) { > - pm8001_printk(pm8001_ha, "no tag for task:%p\n", task); > + pm8001_info(pm8001_ha, "no tag for task:%p\n", task); > return TMF_RESP_FUNC_FAILED; > } > spin_lock_irqsave(&task->task_state_lock, flags); > @@ -1313,7 +1313,7 @@ int pm8001_abort_task(struct sas_task *task) > task->slow_task = NULL; > spin_unlock_irqrestore(&task->task_state_lock, flags); > if (rc != TMF_RESP_FUNC_COMPLETE) > - pm8001_printk(pm8001_ha, "rc= %d\n", rc); > + pm8001_info(pm8001_ha, "rc= %d\n", rc); > return rc; > } > > diff --git a/drivers/scsi/pm8001/pm8001_sas.h b/drivers/scsi/pm8001/pm8001_sas.h > index 5266756a268b..f2c8cbad3853 100644 > --- a/drivers/scsi/pm8001/pm8001_sas.h > +++ b/drivers/scsi/pm8001/pm8001_sas.h > @@ -70,14 +70,14 @@ > #define PM8001_DEVIO_LOGGING 0x100 /* development io message logging */ > #define PM8001_IOERR_LOGGING 0x200 /* development io err message logging */ > > -#define pm8001_printk(HBA, fmt, ...) \ > +#define pm8001_info(HBA, fmt, ...) \ > pr_info("%s:: %s %d:" fmt, \ > (HBA)->name, __func__, __LINE__, ##__VA_ARGS__) > > #define pm8001_dbg(HBA, level, fmt, ...) \ > do { \ > if (unlikely((HBA)->logging_level & PM8001_##level##_LOGGING)) \ > - pm8001_printk(HBA, fmt, ##__VA_ARGS__); \ > + pm8001_info(HBA, fmt, ##__VA_ARGS__); \ > } while (0) > > #define PM8001_USE_TASKLET > -- > 2.26.0 >