[coreboot-gerrit] [XS] Change in coreboot[main]: util/intelmetool: Print the address in `map_physical` errors in hex

[Martin L Roth (Code Review)]

Attention is currently required from: Christian Walter, Máté Kukri, Nico Huber.

Martin L Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/81191?usp=email )

The change is no longer submittable: All-Comments-Resolved is unsatisfied now.

Change subject: util/intelmetool: Print the address in `map_physical` errors in hex
......................................................................

Patch Set 1:

(1 comment)

File util/intelmetool/mmap.c:

https://review.coreboot.org/c/coreboot/+/81191/comment/85fe2c37_81fa4c49 :
PS1, Line 22: 		       (intmax_t)phys_addr, len, err, strerror(err));
> That's for `j`, which is only a length modifier. `x` is the conversion specifier and unsigned.

I'm with Nico on this. An address value shouldn't be signed. I know that it's not *strictly* part of what you're changing here, but I think it's be best to update both at the same time if you wouldn't mind.