[U-Boot] [PATCH] fdt: Include arch specific gpio.h
Joe Hershberger
joe.hershberger at gmail.com
Thu Jul 12 18:37:31 CEST 2012
Hi Michal,
On Wed, Jul 11, 2012 at 6:21 AM, Michal Simek <monstr at monstr.eu> wrote:
> On 07/11/2012 12:59 PM, Simon Glass wrote:
>>
>> Hi,
>>
>>
>> On Wed, Jul 11, 2012 at 12:43 PM, Michal Simek <monstr at monstr.eu
>> <mailto:monstr at monstr.eu>> wrote:
>>
>> On 07/11/2012 11:59 AM, Simon Glass wrote:
>>
>> Hi Michal,
>>
>>
>> On Wed, Jul 11, 2012 at 10:19 AM, Michal Simek <monstr at monstr.eu
>> <mailto:monstr at monstr.eu> <mailto:monstr at monstr.eu
>> <mailto:monstr at monstr.eu>>> wrote:
>>
>> On 07/10/2012 11:13 PM, Simon Glass wrote:
>>
>> Hi Michal,
>>
>>
>> On Tue, Jul 10, 2012 at 11:56 AM, Michal Simek
>> <monstr at monstr.eu <mailto:monstr at monstr.eu> <mailto:monstr at monstr.eu
>> <mailto:monstr at monstr.eu>> <mailto:monstr at monstr.eu
>> <mailto:monstr at monstr.eu> <mailto:monstr at monstr.eu
>> <mailto:monstr at monstr.eu>>>> wrote:
>>
>> Also include arch specific gpio.h which can contain
>> gpio function implementation.
>>
>> Signed-off-by: Michal Simek <monstr at monstr.eu
>> <mailto:monstr at monstr.eu> <mailto:monstr at monstr.eu
>> <mailto:monstr at monstr.eu>> <mailto:monstr at monstr.eu
>> <mailto:monstr at monstr.eu> <mailto:monstr at monstr.eu
>> <mailto:monstr at monstr.eu>>>>
>> CC: Simon Glass <sjg at chromium.org
>> <mailto:sjg at chromium.org> <mailto:sjg at chromium.org
>> <mailto:sjg at chromium.org>> <mailto:sjg at chromium.org
>> <mailto:sjg at chromium.org> <mailto:sjg at chromium.org
>> <mailto:sjg at chromium.org>>>>
>>
>>
>>
>> ---
>> 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).
When you send an updated patch you need to add V2 or whatever to the
subject and you need to include a change log here.
I think you meant to reject this, not the other one here:
http://patchwork.ozlabs.org/patch/170457
> diff --git a/lib/fdtdec.c b/lib/fdtdec.c
> index cc09e06..50a5061 100644
> --- a/lib/fdtdec.c
> +++ b/lib/fdtdec.c
> @@ -25,7 +25,7 @@
>
> #include <fdtdec.h>
>
> /* we need the generic GPIO interface here */
> -#include <asm-generic/gpio.h>
> +#include <asm/gpio.h>
>
> DECLARE_GLOBAL_DATA_PTR;
Thanks,
-Joe
More information about the U-Boot
mailing list