Thread (7 messages) 7 messages, 4 authors, 2012-11-27

OMAP build now completely broken in latest arm-soc

From: tony@atomide.com (Tony Lindgren)
Date: 2012-11-26 17:37:07
Also in: linux-omap

* Olof Johansson [off-list ref] [121125 22:02]:
Hi,

On Sun, Nov 25, 2012 at 6:07 AM, Russell King - ARM Linux
[off-list ref] wrote:
quoted
On Sun, Nov 25, 2012 at 05:56:58AM -0800, Olof Johansson wrote:
quoted
Hi,

On Sun, Nov 25, 2012 at 1:37 AM, Russell King - ARM Linux
[off-list ref] wrote:
quoted
Not much more to say...

arch/arm/plat-omap/i2c.c:35:1: error: expected identifier or '(' before '<<' token
Hmm. In for-next, line 35 is a blank line.

Could this be because of a merge conflict in your local version? This
is one of the files that has conflicts with mainline right now; I'll
look to see if we can resolve those in our tree later today.
Hmm, it looks like a merge conflict which didn't get fixed up... and
looking at it, it's beyond what I'd call trivial to fix.

So I'll shut down the build system until that can be resolved properly;
as OMAP is effectively unbuildable there's not much point it running the
builds.
Tony, Tomi,

I've taken a stab at resolving these conflicts. They seem to come from
the cleanups combined with fixes that went upstream, and it seems like
we want to stick to the arm-soc versions for most of the conflicts.

I've pushed a merge of 3.7-rc7 into next/cleanup (and for-next) of
arm-soc, can you please check and see if this is the correct
resolution? As far as I can tell it is, but a double-check would be
appreciated.

Conflicts were in:

        arch/arm/mach-omap2/omap_hwmod_44xx_data.c
        arch/arm/plat-omap/i2c.c
        drivers/video/omap2/dss/dss.c

The hwmod data was trivial (include files). i2c was a little hairier,
a revert in mainline -- I presumed we're still good with our arm-soc
contents so I stuck close to what we had there. The DSS change seems
to be done completely differently in mainline, i.e. the surrounding
code is different to what we have in arm-soc today, and it looks like
the bugfix (3630-specific stuff) is taken care of in the version we
had. So I stuck to that.
Looks like that i2c revert probably needs a fixup patch as we still
don't have the prerequisites merged. Anyways, the fixup should be done
in mach-omap2/i2c.c instead of plat-omap/i2c.c, so your merge fix
is OK.
 
But, please speak up ASAP if the above doesn't look correct, since we
want to fix it up before we merge much on top.
That's good, thanks.

Regards,

Tony
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help