Re: [PATCH] 2.5.17 /dev/ports

[Riley Williams]

Hi Martin.

> #include <linux/io.h>
> #include <stdio.h>
> #include <stdlib.h>
> 
> int main(char *argv[], int argc)
> {
>	int port = aoit(argv[0]);
>	int byte = aoit(argv[1]);
>
>	if (port > 0)
>		return inb(port);		
>	else {
>		outb(port, byte);
>		return 0;
>	}
> }

Interesting code. Did you really mean to have the program name as the
port number, or should that be argv[1] and argv[2] instead? Also, what
is it supposed to do? A quick analysis makes no sense of it.

Would the following be what you had intended?

	#include <linux/io.h>
	#include <stdio.h>
	#include <stdlib.h>

	int main(int argc, char **argv) {
	    int byte, port, result = 0;

	    switch (argc) {
		case 2:
		    port = atoi(argv[1]);
		    result = inb(port);
		    break;

		case 3:
		    port = atoi(argv[1]);
		    byte = atoi(argv[2]);
		    outb(port, byte);
		    break;

		default:
		    fprintf(stderr, "Usage: %s port [byte]\n", argv[0]);
		    break;
	    }
	    return result;
	}

Best wishes from Riley.