[coreboot-gerrit] [S] Change in coreboot[master]: [NEEDS TEST] util/superiotool: Replace iopl with ioperm

[Nicholas Chin (Code Review)]

Attention is currently required from: Nicholas Chin.

Nicholas Chin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/69981 )

Change subject: [NEEDS TEST] util/superiotool: Replace iopl with ioperm
......................................................................

Patch Set 2:

(1 comment)

File util/superiotool/superiotool.c:

https://review.coreboot.org/c/coreboot/+/69981/comment/63580206_374eb8e7 
PS2, Line 330: 		printf("Superiotool must be run as root.\n");
Should we actually check what the return value is to see if it is a permission error? Right now this message assumes that any error means it wasn't run as root, which wasn't the case in bug #440. Although it was run as root, iopl returned a different error as it wasn't implemented but still confusingly printed this message.