Closed thorsteneisbein closed 7 years ago
Pull request created to fix this. See #15
There is no spidev.c
. To which file do you refer?
Also, it does not appear that this change is related to the linuxspi
programmer type. I just checked the log and the only files that have been modified by me as part of this programmer type are:
avrdude/configure.ac
avrdude/doc/avrdude.texi
avrdude/.gitignore
avrdude/linuxspi.c
avrdude/linuxspi.h
avrdude/Makefile.am
avrdude/pgm_type.c
I don't recall any other files being modified as a result of pull requests, but those go back four years or so. In any case, none of these files make reference to the HAVE_LINUXGPIO
define. I did find a typo in pindefs.h
referencing HAVE_LINUX_GPIO
which has not been modified as part of the linuxspi
changes. This file is part of the mainline avrdude (managed with SVN on GNU savannah) and any changes to those files should be submitted there in order to keep this repository specific to the linuxspi
programmer type.
Note that pindefs.h
no longer exists in the mainline SVN repository, as they seem to have moved on from it. pindefs.c
still does exist.
Please see my comment on #15 as to why this change is important for your fork, even if it was not modified by you.
Edit: re: spidev.c - I actually skimmed over the bug report too fast, I missed this mention (filled in the blank once I saw HAVE_LINUXGPIO). I'm not positive what he is referring to there either
Your fix in #15 should fix the problem and I've merged it, though it would be nice if @thorsteneisbein could comment if it works for them due to the ambiguity of which file was being noted as the problem.
If using --enable-linuxgpio the ./configure script does: $as_echo "#define HAVE_LINUXGPIO 1" >>confdefs.h
In spidev.c you use HAVE_LINUX_GPIO. One of them is wrong.