[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <00e3c386-27ed-418a-8af6-e1e7eddc8284@intel.com>
Date: Fri, 9 May 2025 14:08:35 -0700
From: Dave Jiang <dave.jiang@...el.com>
To: Ira Weiny <ira.weiny@...el.com>, Dan Carpenter
<dan.carpenter@...aro.org>, Davidlohr Bueso <dave@...olabs.net>
Cc: Jonathan Cameron <jonathan.cameron@...wei.com>,
Alison Schofield <alison.schofield@...el.com>,
Vishal Verma <vishal.l.verma@...el.com>,
Dan Williams <dan.j.williams@...el.com>, Robert Richter <rrichter@....com>,
linux-cxl@...r.kernel.org, linux-kernel@...r.kernel.org,
kernel-janitors@...r.kernel.org
Subject: Re: [PATCH] cxl/hdm: Clean up a debug printk
On 5/9/25 2:01 PM, Ira Weiny wrote:
> Dan Carpenter wrote:
>> Smatch complains that %pa is for phys_addr_t types and "size" is a u64.
>>
>> drivers/cxl/core/hdm.c:521 cxl_dpa_alloc() error: '%pa' expects
>> argument of type 'phys_addr_t*', argument 4 has type 'ullong*
>>
>> Looking at this, to me it seems more useful to print the sizes as
>> decimal instead of hex. Let's do that.
>>
>> Signed-off-by: Dan Carpenter <dan.carpenter@...aro.org>
>
> Reviewed-by: Ira Weiny <ira.weiny@...el.com>
Applied to cxl/next
>
> [snip]
Powered by blists - more mailing lists