Received: by 2002:ac0:950c:0:0:0:0:0 with SMTP id f12csp3521016imc; Wed, 13 Mar 2019 22:00:26 -0700 (PDT) X-Google-Smtp-Source: APXvYqxZJBTmtaxsKVlq7P6MUF2qqXCqLxG5Vx5oXWf1aolHBOGaha8fRf1T5GUaMpeVPmLhndw3 X-Received: by 2002:a62:aa0b:: with SMTP id e11mr634304pff.191.1552539626053; Wed, 13 Mar 2019 22:00:26 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1552539626; cv=none; d=google.com; s=arc-20160816; b=gzhSGFgyTzZ9wiuUTZrczCmaPDZ5mXfauZ6+DulBE8Pn6STOCgCyjA7NBbC6XFHjsq oQRXL+0C0zjvW8S4bJNjbQrSzoJJuJ0jPU0yJdi6wLpu6h/aBjjk/I5X0LMBql3hqITn WEyJLkuCkP+UAfH3myQAjb1v2CBEGB7DE2h7yGo1ntWYaTd4xHggjhjL4ZQLSM7OsGry NLV3YQ3Iv6SVQz5gMlO9WOsko6W0o8iD+s/mi0qSHHIOdCyl1mc4M4VkVBC2Mao/PnA/ S2fcrR1LYCoRanlbiAw3eGz4NnxMkmdwxsco19J8eOlVuJ44sGW4YOkidu/Sa4Bi4O9M Fv4g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:message-id:content-transfer-encoding :content-language:in-reply-to:mime-version:user-agent:date:from :references:cc:to:subject; bh=jpSVAT6YJY6Q7KCPNHj+0V6FQHb4N6Wj7Dmt2kYDMVE=; b=SlXYTQTJ4uVl6TTGQno8V47ydCfu/5YqUO2xeZ6Jv7XFSfeo3LNRtCMNTaBhe1qAQI DqiigZ9s63XN22ytEfqboNQWdFAfeCGZcLF2jH3El/q/5lb/DrP+QlZv93uOrQd73jpU BE9SknE599d3fCHlQPPbI8sAxo84az0x0T9fXc/yEDC0hQ0PFCqCJ2D0TParc6Zme23p EnYyABv5+U8GCU5nJGtAXyXvm6vcIDg8/RKmiCR7vqNUisPngRWBIwhlB4tgk5IR7v7Z qe2ao9EhSvB+fpnakwLNqmkBSBWRV2VKNdU/S4AOYnbo23jOKbQFp6/appcZiDCxLMEf y6Og== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=ibm.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id x64si12029335pfx.156.2019.03.13.21.59.35; Wed, 13 Mar 2019 22:00:26 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=ibm.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726527AbfCNE6q (ORCPT + 99 others); Thu, 14 Mar 2019 00:58:46 -0400 Received: from mx0b-001b2d01.pphosted.com ([148.163.158.5]:39504 "EHLO mx0a-001b2d01.pphosted.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1725897AbfCNE6q (ORCPT ); Thu, 14 Mar 2019 00:58:46 -0400 Received: from pps.filterd (m0098417.ppops.net [127.0.0.1]) by mx0a-001b2d01.pphosted.com (8.16.0.27/8.16.0.27) with SMTP id x2E4nLZ9117519 for ; Thu, 14 Mar 2019 00:58:45 -0400 Received: from e06smtp05.uk.ibm.com (e06smtp05.uk.ibm.com [195.75.94.101]) by mx0a-001b2d01.pphosted.com with ESMTP id 2r7fxdrx06-1 (version=TLSv1.2 cipher=AES256-GCM-SHA384 bits=256 verify=NOT) for ; Thu, 14 Mar 2019 00:58:44 -0400 Received: from localhost by e06smtp05.uk.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Thu, 14 Mar 2019 04:58:43 -0000 Received: from b06cxnps3075.portsmouth.uk.ibm.com (9.149.109.195) by e06smtp05.uk.ibm.com (192.168.101.135) with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted; (version=TLSv1/SSLv3 cipher=AES256-GCM-SHA384 bits=256/256) Thu, 14 Mar 2019 04:58:41 -0000 Received: from d06av22.portsmouth.uk.ibm.com (d06av22.portsmouth.uk.ibm.com [9.149.105.58]) by b06cxnps3075.portsmouth.uk.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id x2E4weJP46268556 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Thu, 14 Mar 2019 04:58:40 GMT Received: from d06av22.portsmouth.uk.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id 1A3714C044; Thu, 14 Mar 2019 04:58:40 +0000 (GMT) Received: from d06av22.portsmouth.uk.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id 79A5A4C050; Thu, 14 Mar 2019 04:58:39 +0000 (GMT) Received: from ozlabs.au.ibm.com (unknown [9.192.253.14]) by d06av22.portsmouth.uk.ibm.com (Postfix) with ESMTP; Thu, 14 Mar 2019 04:58:39 +0000 (GMT) Received: from [10.61.2.125] (haven.au.ibm.com [9.192.254.114]) (using TLSv1.2 with cipher AES128-SHA (128/128 bits)) (No client certificate requested) by ozlabs.au.ibm.com (Postfix) with ESMTPSA id 6DE37A00D1; Thu, 14 Mar 2019 15:58:38 +1100 (AEDT) Subject: Re: [PATCH 2/5] ocxl: Clean up printf formats To: "Alastair D'Silva" Cc: "Alastair D'Silva" , Frederic Barrat , Arnd Bergmann , Greg Kroah-Hartman , linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org References: <20190227045741.21412-1-alastair@au1.ibm.com> <20190313040702.14276-1-alastair@au1.ibm.com> <20190313040702.14276-3-alastair@au1.ibm.com> From: Andrew Donnellan Date: Thu, 14 Mar 2019 15:58:38 +1100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.5.1 MIME-Version: 1.0 In-Reply-To: <20190313040702.14276-3-alastair@au1.ibm.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-AU Content-Transfer-Encoding: 7bit X-TM-AS-GCONF: 00 x-cbid: 19031404-0020-0000-0000-0000032312E5 X-IBM-AV-DETECTION: SAVI=unused REMOTE=unused XFE=unused x-cbparentid: 19031404-0021-0000-0000-0000217514E6 Message-Id: <84392561-168e-11c6-f348-11b2e929d5c6@au1.ibm.com> X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:,, definitions=2019-03-14_01:,, signatures=0 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 priorityscore=1501 malwarescore=0 suspectscore=2 phishscore=0 bulkscore=0 spamscore=0 clxscore=1015 lowpriorityscore=0 mlxscore=0 impostorscore=0 mlxlogscore=999 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1810050000 definitions=main-1903140031 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 13/3/19 3:06 pm, Alastair D'Silva wrote: > From: Alastair D'Silva > > Use %# instead of using a literal '0x' > > Signed-off-by: Alastair D'Silva Acked-by: Andrew Donnellan > --- > drivers/misc/ocxl/config.c | 6 +++--- > drivers/misc/ocxl/context.c | 2 +- > drivers/misc/ocxl/trace.h | 10 +++++----- > 3 files changed, 9 insertions(+), 9 deletions(-) > > diff --git a/drivers/misc/ocxl/config.c b/drivers/misc/ocxl/config.c > index 8f2c5d8bd2ee..0ee7856b033d 100644 > --- a/drivers/misc/ocxl/config.c > +++ b/drivers/misc/ocxl/config.c > @@ -178,9 +178,9 @@ static int read_dvsec_vendor(struct pci_dev *dev) > pci_read_config_dword(dev, pos + OCXL_DVSEC_VENDOR_DLX_VERS, &dlx); > > dev_dbg(&dev->dev, "Vendor specific DVSEC:\n"); > - dev_dbg(&dev->dev, " CFG version = 0x%x\n", cfg); > - dev_dbg(&dev->dev, " TLX version = 0x%x\n", tlx); > - dev_dbg(&dev->dev, " DLX version = 0x%x\n", dlx); > + dev_dbg(&dev->dev, " CFG version = %#x\n", cfg); > + dev_dbg(&dev->dev, " TLX version = %#x\n", tlx); > + dev_dbg(&dev->dev, " DLX version = %#x\n", dlx); > return 0; > } > > diff --git a/drivers/misc/ocxl/context.c b/drivers/misc/ocxl/context.c > index c10a940e3b38..3498a0199bde 100644 > --- a/drivers/misc/ocxl/context.c > +++ b/drivers/misc/ocxl/context.c > @@ -134,7 +134,7 @@ static vm_fault_t ocxl_mmap_fault(struct vm_fault *vmf) > vm_fault_t ret; > > offset = vmf->pgoff << PAGE_SHIFT; > - pr_debug("%s: pasid %d address 0x%lx offset 0x%llx\n", __func__, > + pr_debug("%s: pasid %d address %#lx offset %#llx\n", __func__, > ctx->pasid, vmf->address, offset); > > if (offset < ctx->afu->irq_base_offset) > diff --git a/drivers/misc/ocxl/trace.h b/drivers/misc/ocxl/trace.h > index bcb7ff330c1e..68bf2f173a1a 100644 > --- a/drivers/misc/ocxl/trace.h > +++ b/drivers/misc/ocxl/trace.h > @@ -28,7 +28,7 @@ DECLARE_EVENT_CLASS(ocxl_context, > __entry->tidr = tidr; > ), > > - TP_printk("linux pid=%d spa=0x%p pasid=0x%x pidr=0x%x tidr=0x%x", > + TP_printk("linux pid=%d spa=%p pasid=%#x pidr=%#x tidr=%#x", > __entry->pid, > __entry->spa, > __entry->pasid, > @@ -61,7 +61,7 @@ TRACE_EVENT(ocxl_terminate_pasid, > __entry->rc = rc; > ), > > - TP_printk("pasid=0x%x rc=%d", > + TP_printk("pasid=%#x rc=%d", > __entry->pasid, > __entry->rc > ) > @@ -87,7 +87,7 @@ DECLARE_EVENT_CLASS(ocxl_fault_handler, > __entry->tfc = tfc; > ), > > - TP_printk("spa=%p pe=0x%llx dsisr=0x%llx dar=0x%llx tfc=0x%llx", > + TP_printk("spa=%p pe=%#llx dsisr=%#llx dar=%#llx tfc=%#llx", > __entry->spa, > __entry->pe, > __entry->dsisr, > @@ -127,7 +127,7 @@ TRACE_EVENT(ocxl_afu_irq_alloc, > __entry->irq_offset = irq_offset; > ), > > - TP_printk("pasid=0x%x irq_id=%d virq=%u hw_irq=%d irq_offset=0x%llx", > + TP_printk("pasid=%#x irq_id=%d virq=%u hw_irq=%d irq_offset=%#llx", > __entry->pasid, > __entry->irq_id, > __entry->virq, > @@ -150,7 +150,7 @@ TRACE_EVENT(ocxl_afu_irq_free, > __entry->irq_id = irq_id; > ), > > - TP_printk("pasid=0x%x irq_id=%d", > + TP_printk("pasid=%#x irq_id=%d", > __entry->pasid, > __entry->irq_id > ) > -- Andrew Donnellan OzLabs, ADL Canberra andrew.donnellan@au1.ibm.com IBM Australia Limited