On 07/11/2012 11:02 PM, Joe Hershberger wrote:
Hi Michal,
On Wed, Jul 11, 2012 at 7:21 AM, Michal Simek<mon...@monstr.eu> wrote:
On 07/11/2012 02:00 PM, Simon Glass wrote:
Hi Michal,
On Wed, Jul 11, 2012 at 1:21 PM, Michal Simek<mon...@monstr.eu
<mailto:mon...@monstr.eu>> wrote:
On 07/11/2012 12:59 PM, Simon Glass wrote:
Please use text only email. your email is causing this ugly,
hard-to-follow formatting.
Will look at it. Thanks for pointing to it. I probably used gmail
for my reply. I believe that Thunderbird will be fine.
---
lib/fdtdec.c | 1 +
1 files changed, 1 insertions(+), 0
deletions(-)
diff --git a/lib/fdtdec.c b/lib/fdtdec.c
index cc09e06..b12eb77 100644
--- a/lib/fdtdec.c
+++ b/lib/fdtdec.c
@@ -25,6 +25,7 @@
#include<fdtdec.h>
/* we need the generic GPIO interface
here */
+#include<asm/gpio.h>
Can you not instead support the generic
functions? What are you missing from from there that fdtdec.c needs?
OK. The problem is with fdtdec_setup_gpio function
where you called gpio_request which
is gpio specific function which you haven't
included.
The most archs have this defined in
arch/<cpuname>/include/asm/______gpio.h.
This is there in my tree. Please see commit:
5f533aeb gpio: Modify common gpio.h to more closely match
Linux
This is generic gpio cleanup but I don't think this is the
same thing.
I think you should get some warnings for fdtdec compilation
around missing gpio_request
declaration or you include gpio.h in any other header file
which is in fdtdec.c/h.
Sorry I don't really understand that. For me fdtdec.c has #include
<asm-generic/gpio.h> and that is enough to get gpio_request(). Can you
please advise what warning you see and for what board?
Microblaze:
lib/libgeneric.o: In function `fdtdec_setup_gpio':
/mnt/projects/u-boot/lib/__fdtdec.c:477: undefined reference to
`gpio_request'
The same error will be for nios2, openrisc and blackfin because they
define
gpio_request as static inline function in
arch/<arch>/include/asm/gpio.h
Because I think there should be asm/gpio.h included which can include
asm-generic/gpio.h
(this is arm case).
OK I see, that makes sense.
It seems to me that those platforms simply haven't been updated to use
the generic gpio.h and should be changed.
That can be truth but on the other hand I think that including
asm-generic is not the best.
Also in connection to this issue that code in fdtdec should be used
when any CONFIG_GPIO config is enabled.
Thanks
Michal
--
Michal Simek, Ing. (M.Eng)
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel 2.6 Microblaze Linux - http://www.monstr.eu/fdt/
Microblaze U-BOOT custodian
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
