[coreboot] [PATCH]ich_gpio_dump util ACK?
ron minnich
rminnich at gmail.com
Sun Mar 9 06:48:37 CET 2008
A few things caught my eye:
+ if (iopl(3)) { printf("You need to be root.\n"); exit(1); }
Just use perror. This is going to be confusing if there is an error
and you're running as root :-)
There are a few cases where there are pci cleanups and frees and such
and then an immediate exit. I applaud the sentiment but, really,
you're just as safe exiting without the cleanup. Just a thought ...
thanks, this is neat stuff.
ron
More information about the coreboot
mailing list