On Oct 09 2001, Colin Walters wrote: > Josh Huber <[EMAIL PROTECTED]> writes: > > > It seems to work fine on my powerbook, which is great. > > It works fine for me too on my TiBook.
It would work fine if I had more money to purchase one of these cool toys. :-) > > So, my real question is this...how should this be integrated with > > pmud-utils/pmud. (I think it should somehow be put in Debian) > > I think it should be integrated too, after applying the attached > patch. Doesn't anyone else compile with -Wall? Well, I think that it is wise to compile not only with -Wall but also with -ansi -pedantic -O2. This way, some further errors are catched. BTW, since n is an integer in that program, the format string of %ld is incorrect to deal with it and the correct fix is not to use a type cast, but to remove the l of the format string. The I think that the same should apply to the sz variable (but I'm not so sure about this one). Also, there is a problem with a format string, which has only %: and which should have %s instead. I'm sending a patch to the original program. Does it compile cleanly with other people's compilers? Here it doesn't but my build environment is broken anyway. :-( I haven't had the time to check this (I only glanced), but there are many variables called data, some defined with 32 and some defined with 16 bytes. Is that really correct? []s, Roger... P.S.: An unrelated question: I'm having some severe problems getting the pmac 9500 working with OF (I can't for the life of me make this thing send the output to the monitor). Is the situation any better with newer machines (i.e., is the default OF output the screen or does it still default to the serial port)? -- =-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-= Rogério Brito - [EMAIL PROTECTED] - http://www.ime.usp.br/~rbrito/ =-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
--- fnset.c.orig Tue Oct 9 23:50:41 2001 +++ fnset.c Tue Oct 9 23:44:37 2001 @@ -17,7 +17,9 @@ #include <sys/types.h> #include <sys/stat.h> #include <fcntl.h> +#include <unistd.h> #include <stdlib.h> +#include <string.h> #include <linux/adb.h> #include <linux/pmu.h> @@ -61,7 +63,7 @@ if (debug) { int i; - fprintf(stderr, "writing %ld bytes: {", sz); + fprintf(stderr, "writing %d bytes: {", sz); for (i = 0; i < sz; i++) { fprintf(stderr, " 0x%02x", data[i]); } @@ -74,7 +76,7 @@ fprintf(stderr, "%s: write(): %s\n", prog, strerror(errno)); } else { - fprintf(stderr, "%s: write(): expected %ld got %ld\n", + fprintf(stderr, "%s: write(): expected %d got %d\n", prog, sz, n); } return -1; @@ -96,7 +98,7 @@ if (debug) { int i; - fprintf(stderr, "read %ld bytes", n); + fprintf(stderr, "read %d bytes", n); if (n > 0) { fputs(": {", stderr); for (i = 0; i < n; i++) { @@ -129,7 +131,7 @@ return -1; } if (n != 2) { - fprintf(stderr, "%s: read(): expected 2 got %ld\n", + fprintf(stderr, "%s: read(): expected 2 got %d\n", prog, n); return -1; } @@ -255,10 +257,9 @@ } int -main(int argc, char *argv[], char *envp[]) +main(int argc, char *argv[]) { const char *adb = "/dev/adb"; - unsigned char data[32]; const char *optlet = "dvhsubvP:"; extern char *optarg; @@ -365,7 +366,7 @@ if (bitpos && !force) { fprintf(stderr, - "%: WARNING: use of bit offset other than 0 (-P option)" + "%s: WARNING: use of bit offset other than 0 (-P option)" "requires force (-f option) since it may damage hardware!\n", prog); }