linux-kernel - Re: linux-next: build failure after merge of the final tree (gpio tree related)

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] [day] [month] [year] [list]

Date:	Wed, 05 Mar 2014 17:42:06 +0100
From:	Fabian Vogt <fabian@...ter-vogt.de>
To:	Linus Walleij <linus.walleij@...aro.org>
Cc:	Stephen Rothwell <sfr@...b.auug.org.au>,
	"linux-next@...r.kernel.org" <linux-next@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: linux-next: build failure after merge of the final tree (gpio tree related)

Hi,

On Wed, Mar 5, 2014, 3:16 PM, Linus Walleij wrote:
> On Wed, Mar 5, 2014 at 2:35 PM, Stephen Rothwell <sfr@...b.auug.org.au> wrote:
> 
> > After merging the final tree, today's linux-next build (powerpc
> > allyesconfig) failed like this:
> >
> > drivers/gpio/gpio-zevio.c: In function 'zevio_gpio_port_get':
> > drivers/gpio/gpio-zevio.c:70:2: error: implicit declaration of function 'IOMEM' [-Werror=implicit-function-declaration]
> >   return readl(IOMEM(c->chip.regs + section_offset + port_offset));
> >   ^
> > drivers/gpio/gpio-zevio.c:70:2: warning: passing argument 1 of 'readl' makes pointer from integer without a cast [enabled by default]
> > In file included from include/linux/io.h:22:0,
> >                  from drivers/gpio/gpio-zevio.c:15:
> > arch/powerpc/include/asm/io-defs.h:6:16: note: expected 'const volatile void *' but argument is of type 'int'
> >  DEF_PCI_AC_RET(readl, u32, (const PCI_IO_ADDR addr), (addr), mem, addr)
> >                 ^
> 
> Fabian can you suggest a patch adding the proper depends added to
> the Kconfig. Check what things bring in the IOMEM etc that is needed
> to compile this.

It looks like powerpc doesn't have the IOMEM macro (which just casts to "void __iomem*").
Would "depends on ARM && OF_GPIO" be ok?

> Yours,
> Linus Walleij

Bye,
Fabian
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/