Message-ID: <CAMuHMdXgCR1HUwmwT21p=O2p+LuAqLfeM=Tn=ghvKQ+qgsNTTQ@mail.gmail.com>
Date:	Fri, 13 Jul 2012 22:02:47 +0200
From:	Geert Uytterhoeven <geert@...ux-m68k.org>
To:	Sonic Zhang <sonic.zhang@...log.com>, Bob Liu <lliubbo@...il.com>,
	Mike Frysinger <vapier@...too.org>
Cc:	uclinux-dist-devel@...ckfin.uclinux.org,
	linux-kernel@...r.kernel.org
Subject: Re: Build regressions/improvements in v3.5-rc4

On Tue, Jun 26, 2012 at 9:59 PM, Geert Uytterhoeven
<geert@...ux-m68k.org> wrote:
> + drivers/i2c/busses/i2c-bfin-twi.c: error: 'ANAK' undeclared (first use in this function):  => 154:19
>   + drivers/i2c/busses/i2c-bfin-twi.c: error: 'BUFRDERR' undeclared (first use in this function):  => 158:19
>   + drivers/i2c/busses/i2c-bfin-twi.c: error: 'BUFWRERR' undeclared (first use in this function):  => 160:19
>   + drivers/i2c/busses/i2c-bfin-twi.c: error: 'BUSBUSY' undeclared (first use in this function):  => 401:35, 301:35
>   + drivers/i2c/busses/i2c-bfin-twi.c: error: 'DNAK' undeclared (first use in this function):  => 156:19
>   + drivers/i2c/busses/i2c-bfin-twi.c: error: 'FAST' undeclared (first use in this function):  => 511:17, 360:18
>   + drivers/i2c/busses/i2c-bfin-twi.c: error: 'LOSTARB' undeclared (first use in this function):  => 152:19
>   + drivers/i2c/busses/i2c-bfin-twi.c: error: 'MCOMP' undeclared (first use in this function):  => 189:22, 489:31, 344:31
>   + drivers/i2c/busses/i2c-bfin-twi.c: error: 'MDIR' undeclared (first use in this function):  => 359:31, 567:32, 102:30
>   + drivers/i2c/busses/i2c-bfin-twi.c: error: 'MEN' undeclared (first use in this function):  => 358:51, 510:52, 196:33
>   + drivers/i2c/busses/i2c-bfin-twi.c: error: 'MERR' undeclared (first use in this function):  => 146:23, 489:24, 344:24
>   + drivers/i2c/busses/i2c-bfin-twi.c: error: 'RCVSERV' undeclared (first use in this function):  => 489:49, 116:23, 344:49
>   + drivers/i2c/busses/i2c-bfin-twi.c: error: 'RSTART' undeclared (first use in this function):  => 102:37
>   + drivers/i2c/busses/i2c-bfin-twi.c: error: 'SCLOVR' undeclared (first use in this function):  => 171:29
>   + drivers/i2c/busses/i2c-bfin-twi.c: error: 'SDAOVR' undeclared (first use in this function):  => 177:28
>   + drivers/i2c/busses/i2c-bfin-twi.c: error: 'SDASEN' undeclared (first use in this function):  => 168:33
>   + drivers/i2c/busses/i2c-bfin-twi.c: error: 'STOP' undeclared (first use in this function):  => 105:30
>   + drivers/i2c/busses/i2c-bfin-twi.c: error: 'TWI_ENA' undeclared (first use in this function):  => 398:30, 624:47, 298:30, 725:45
>   + drivers/i2c/busses/i2c-bfin-twi.c: error: 'XMTSERV' undeclared (first use in this function):  => 344:39, 489:39, 90:23

This seems to be caused by commit c55c89e939f2a0a83d5c61462be554d5d2408178
("blackfin: twi: move twi bit mask macro to twi head file").
However, just including <asm/bfin_twi.h> doesn't fix it, as commit
1e92bf6d80b5a0a137455c96bf6cdd9c1a5b531e ("blackfin: twi: Move TWI MMR access
macro to twi head file") created lots of duplicate definitions:

arch/blackfin/include/asm/bfin_twi.h:87:6: warning: "ANOMALY_05001001"
is not defined [-Wundef]
arch/blackfin/include/asm/bfin_twi.h:67:0: note: this is the location
of the previous definition
drivers/i2c/busses/i2c-bfin-twi.c:36:8: error: redefinition of 'struct
bfin_twi_iface'
arch/blackfin/include/asm/bfin_twi.h:46:8: note: originally defined here
arch/blackfin/include/asm/bfin_twi.h:73:90: note: previous definition
of 'read_CLKDIV' was here
arch/blackfin/include/asm/bfin_twi.h:73:259: note: previous definition
of 'write_CLKDIV' was here
arch/blackfin/include/asm/bfin_twi.h:74:90: note: previous definition
of 'read_CONTROL' was here
arch/blackfin/include/asm/bfin_twi.h:74:261: note: previous definition
of 'write_CONTROL' was here
arch/blackfin/include/asm/bfin_twi.h:75:90: note: previous definition
of 'read_SLAVE_CTL' was here
arch/blackfin/include/asm/bfin_twi.h:75:265: note: previous definition
of 'write_SLAVE_CTL' was here
arch/blackfin/include/asm/bfin_twi.h:76:90: note: previous definition
of 'read_SLAVE_STAT' was here
arch/blackfin/include/asm/bfin_twi.h:76:267: note: previous definition
of 'write_SLAVE_STAT' was here
arch/blackfin/include/asm/bfin_twi.h:77:90: note: previous definition
of 'read_SLAVE_ADDR' was here
arch/blackfin/include/asm/bfin_twi.h:77:267: note: previous definition
of 'write_SLAVE_ADDR' was here
arch/blackfin/include/asm/bfin_twi.h:78:90: note: previous definition
of 'read_MASTER_CTL' was here
arch/blackfin/include/asm/bfin_twi.h:78:267: note: previous definition
of 'write_MASTER_CTL' was here
arch/blackfin/include/asm/bfin_twi.h:79:90: note: previous definition
of 'read_MASTER_STAT' was here
arch/blackfin/include/asm/bfin_twi.h:79:269: note: previous definition
of 'write_MASTER_STAT' was here
arch/blackfin/include/asm/bfin_twi.h:80:90: note: previous definition
of 'read_MASTER_ADDR' was here
arch/blackfin/include/asm/bfin_twi.h:80:269: note: previous definition
of 'write_MASTER_ADDR' was here
arch/blackfin/include/asm/bfin_twi.h:81:90: note: previous definition
of 'read_INT_STAT' was here
arch/blackfin/include/asm/bfin_twi.h:81:263: note: previous definition
of 'write_INT_STAT' was here
arch/blackfin/include/asm/bfin_twi.h:82:90: note: previous definition
of 'read_INT_MASK' was here
arch/blackfin/include/asm/bfin_twi.h:82:263: note: previous definition
of 'write_INT_MASK' was here
arch/blackfin/include/asm/bfin_twi.h:83:90: note: previous definition
of 'read_FIFO_CTL' was here
arch/blackfin/include/asm/bfin_twi.h:83:263: note: previous definition
of 'write_FIFO_CTL' was here
arch/blackfin/include/asm/bfin_twi.h:84:90: note: previous definition
of 'read_FIFO_STAT' was here
arch/blackfin/include/asm/bfin_twi.h:84:265: note: previous definition
of 'write_FIFO_STAT' was here
arch/blackfin/include/asm/bfin_twi.h:85:90: note: previous definition
of 'read_XMT_DATA8' was here
arch/blackfin/include/asm/bfin_twi.h:85:265: note: previous definition
of 'write_XMT_DATA8' was here
arch/blackfin/include/asm/bfin_twi.h:86:90: note: previous definition
of 'read_XMT_DATA16' was here
arch/blackfin/include/asm/bfin_twi.h:86:267: note: previous definition
of 'write_XMT_DATA16' was here
arch/blackfin/include/asm/bfin_twi.h:88:90: note: previous definition
of 'read_RCV_DATA8' was here
arch/blackfin/include/asm/bfin_twi.h:88:265: note: previous definition
of 'write_RCV_DATA8' was here
arch/blackfin/include/asm/bfin_twi.h:89:90: note: previous definition
of 'read_RCV_DATA16' was here
arch/blackfin/include/asm/bfin_twi.h:89:267: note: previous definition
of 'write_RCV_DATA16' was here

So I gave up. A pity, considering BF537-STAMP_defconfig is the only
blackfin config
in http://kisskb.ellerman.id.au/kisskb/branch/3/.

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@...ux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
--
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/

Powered by blists - more mailing lists