Skip to content

Commit 947df02

Browse files
committed
libnvdimm, pmem: clean up resource print / request
The leading '0x' in front of %pa is redundant, also we can just use %pR to simplify the print statement. The request parameters can be directly taken from the resource as well. Reviewed-by: Johannes Thumshirn <[email protected]> Signed-off-by: Dan Williams <[email protected]>
1 parent 030b99e commit 947df02

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

drivers/nvdimm/pmem.c

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -224,10 +224,9 @@ static struct pmem_device *pmem_alloc(struct device *dev,
224224
if (!arch_has_wmb_pmem())
225225
dev_warn(dev, "unable to guarantee persistence of writes\n");
226226

227-
if (!devm_request_mem_region(dev, pmem->phys_addr, pmem->size,
228-
dev_name(dev))) {
229-
dev_warn(dev, "could not reserve region [0x%pa:0x%zx]\n",
230-
&pmem->phys_addr, pmem->size);
227+
if (!devm_request_mem_region(dev, res->start, resource_size(res),
228+
dev_name(dev))) {
229+
dev_warn(dev, "could not reserve region %pR\n", res);
231230
return ERR_PTR(-EBUSY);
232231
}
233232

0 commit comments

Comments
 (0)