linux-kernel - Re: [PATCH 1/2] ARM: ixp4xx: fix compilation, add gpiolib support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Thu, 09 Feb 2012 14:37:34 +0100
From:	Marc Kleine-Budde <mkl@...gutronix.de>
To:	"Arnaud Patard (Rtp)" <arnaud.patard@...-net.org>
CC:	Mikael Pettersson <mikpe@...uu.se>,
	Marc Kleine-Budde <mkl@...ckshift.org>,
	Imre Kaloz <kaloz@...nwrt.org>, linux-kernel@...r.kernel.org,
	linux-arm-kernel@...ts.infradead.org,
	Krzysztof Halasa <khc@...waw.pl>
Subject: Re: [PATCH 1/2] ARM: ixp4xx: fix compilation, add gpiolib support

On 02/09/2012 11:57 AM, Arnaud Patard (Rtp) wrote:
> Mikael Pettersson <mikpe@...uu.se> writes:
> 
>> Marc Kleine-Budde writes:
>>  > From: Imre Kaloz <kaloz@...nwrt.org>
>>  > 
>>  > The problem was introduced with commit:
>>  > "eb9ae7f gpio: fix build error in include/asm-generic/gpio.h"
>>  > 
>>  > This patch adds gpiolib support for the IXP4xx platform, which fixes the
>>  > compilation of several ixp4xx platforms, e.g.:
>>  > 
>>  > In file included from arch/arm/mach-ixp4xx/include/mach/gpio.h:72,
>>  >                  from /home/frogger/projects/server/linux/arch/arm/include/asm/gpio.h:9,
>>  >                  from include/linux/gpio.h:30,
>>  >                  from arch/arm/mach-ixp4xx/nslu2-setup.c:19:
>>  > include/asm-generic/gpio.h: In function 'gpio_get_value_cansleep':
>>  > include/asm-generic/gpio.h:218: error: implicit declaration of function '__gpio_get_value'
>>  > include/asm-generic/gpio.h: In function 'gpio_set_value_cansleep':
>>  > include/asm-generic/gpio.h:224: error: implicit declaration of function '__gpio_set_value'
>>
>> This fix is needed for 3.2 too, right?  I got the same errors when
>> updating my (out-of-tree) ixp4xx/ds101 platform support to the 3.2 kernel
>> a few days ago.
> 
> fwiw, it's even needed for 3.1. We've added it in the debian kernel when it
> was sent to the linux-arm-kernel ml and it was for 3.1.

Yep v3.1.10 fails in drivers/input/touchscreen/ads7846.c with:

drivers/input/touchscreen/ads7846.c: In function 'ads7846_setup_pendown':
drivers/input/touchscreen/ads7846.c:970: error: implicit declaration of
function 'gpio_request_one'

Marc

-- 
Pengutronix e.K.                  | Marc Kleine-Budde           |
Industrial Linux Solutions        | Phone: +49-231-2826-924     |
Vertretung West/Dortmund          | Fax:   +49-5121-206917-5555 |
Amtsgericht Hildesheim, HRA 2686  | http://www.pengutronix.de   |


Download attachment "signature.asc" of type "application/pgp-signature" (263 bytes)