Re: Android build failure

2011-08-01 Thread Paul Sokolovsky
Hello Angus,

I see that this configuration is still failing, and actually your other
builds have the same problem. I guess one productive way to deal with
that is you joining one of Infrastructure team hacking sessions
(https://wiki.linaro.org/Events/LinaroConnectQ3.11/Magdelene/Infrastructure)
and we sit together and investigate/fix this issue. 

Thanks,
Paul


On Thu, 28 Jul 2011 13:37:38 -0600
Angus Ainslie  wrote:

> I have a android manifest that builds fine on my local build server
> with -j4 for concurrent makes.
> 
> The same manifest fails to build on android-build.linaro.org The log
> is here
> 
> https://android-build.linaro.org/jenkins/job/angus-akkea_smdkv310/6/consoleText
> 
> Is there something I can add to the build variables  to disable the
> concurrent make ? This would be just to debug the manifest file and
> then re-enable it afterwards
> 
> Thanks
> Angus
> 
> 



-- 
Best Regards,
Paul

Linaro.org | Open source software for ARM SoCs
Follow Linaro: http://www.facebook.com/pages/Linaro
http://twitter.com/#!/linaroorg - http://www.linaro.org/linaro-blog

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Please register a Gerrit account, reminder #2

2011-08-01 Thread Paul Sokolovsky
Hello,

Some time this week we're switching to Gerrit for Android source tree
hosting. If you committed anything to git.linaro.org/android/* , please
make sure you registered an account in Gerrit, by following the
instructions below.

Thanks,
Paul


Begin forwarded message:

Date: Fri, 22 Jul 2011 00:45:13 +0300
From: Paul Sokolovsky 

Hello,

I'd like to invite all interested parties, Linaro Android team members
first of all, to register an account on the upcoming Linaro Gerrit
instance at http://android.git.linaro.org . You can find instructions
at https://wiki.linaro.org/Platform/Android/Gerrit .

[]

-- 
Best Regards,
Paul

Linaro.org | Open source software for ARM SoCs
Follow Linaro: http://www.facebook.com/pages/Linaro
http://twitter.com/#!/linaroorg - http://www.linaro.org/linaro-blog


-- 
Best Regards,
Paul

Linaro.org | Open source software for ARM SoCs
Follow Linaro: http://www.facebook.com/pages/Linaro
http://twitter.com/#!/linaroorg - http://www.linaro.org/linaro-blog

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: Android build failure

2011-08-01 Thread Paul Sokolovsky
Hello Bernhard,

On Thu, 28 Jul 2011 22:34:25 +0200
Bernhard Rosenkranzer  wrote:

> On 28 July 2011 21:37, Angus Ainslie  wrote:
> > I have a android manifest that builds fine on my local build server
> > with -j4 for concurrent makes.
> 
> Have you tried doing a re-download locally?
> The issue I've had that resulted in the same error (while it may not
> be elfcopy for you) was simply that a download failed - it didn't show
> up locally because I had it checked out, and apparently download
> errors don't show up in the build server's logs.

Seconding James' request, can you elaborate/submit bug for such case?
Do you mean exactly download or checkout here? Source checkout for sure
leads to error and build failure, and as for downloads, I had cases
when curl which was used in few places swallowed protocol
errors (like, when it got 404 it just downloaded error payload page
and exited with 0 status). In places I hit that I replaces it with
wget, maybe more complete code audit should be performed.

In either way, one of big android-build's is to avoid "false
positives", i.e. avoid building something not updated or not
properly downloaded. We run all build setup scripts with "set -e", so
any error with that execution immediately leads to build failure. So
again, if you faced any issue when some error was swallowed, please
report it.

> 
> I doubt the parallel build is the issue, given the other builds are
> pretty much doing the same thing and they tend to succeed.
> 
> ttyl
> bero
> 
> 
> 
> >
> > The same manifest fails to build on android-build.linaro.org The
> > log is here
> >
> > https://android-build.linaro.org/jenkins/job/angus-akkea_smdkv310/6/consoleText
> >
> > Is there something I can add to the build variables  to disable the
> > concurrent make ? This would be just to debug the manifest file and
> > then re-enable it afterwards
> >
> > Thanks
> > Angus
> >
> >
> > --
> > Angus Ainslie 
> > Team Lead, Samsung Landing Team
> >
> > ___
> > linaro-dev mailing list
> > linaro-dev@lists.linaro.org
> > http://lists.linaro.org/mailman/listinfo/linaro-dev
> >
> 
> ___
> linaro-dev mailing list
> linaro-dev@lists.linaro.org
> http://lists.linaro.org/mailman/listinfo/linaro-dev



-- 
Best Regards,
Paul

Linaro.org | Open source software for ARM SoCs
Follow Linaro: http://www.facebook.com/pages/Linaro
http://twitter.com/#!/linaroorg - http://www.linaro.org/linaro-blog

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: Android build failure

2011-08-01 Thread Paul Sokolovsky
Hello,

On Sat, 30 Jul 2011 13:18:36 +1200
Michael Hudson-Doyle  wrote:

> On Fri, 29 Jul 2011 13:10:48 -0600, Angus Ainslie
>  wrote:
> > On Thu, Jul 28, 2011 at 3:09 PM, James Westby
> >  wrote:
> > > On Thu, 28 Jul 2011 13:37:38 -0600, Angus Ainslie
> > >  wrote:
> > >> I have a android manifest that builds fine on my local build
> > >> server with -j4 for concurrent makes.
> > >>
> > >> The same manifest fails to build on android-build.linaro.org The
> > >> log is here
> > >>
> > >> https://android-build.linaro.org/jenkins/job/angus-akkea_smdkv310/6/consoleText
> > >>
> > >> Is there something I can add to the build variables  to disable
> > >> the concurrent make ? This would be just to debug the manifest
> > >> file and then re-enable it afterwards
> > >
> > > You can put
> > >
> > > MAKE_JOBS=1
> > >
> > 
> > I don't suppose there is a MAKE_TARGETS to go along with this is
> > there ?
> 
> There is indeed.

There's also doc at
https://wiki.linaro.org/Platform/Android/LinaroAndroidBuildService
describing this and other build config settings.


-- 
Best Regards,
Paul

Linaro.org | Open source software for ARM SoCs
Follow Linaro: http://www.facebook.com/pages/Linaro
http://twitter.com/#!/linaroorg - http://www.linaro.org/linaro-blog

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: [U-Boot] [PATCH v4 2/2] ORIGEN: Add MMC SPL support

2011-08-01 Thread Chander Kashyap
Dear Wolfgang Denk,

On 1 August 2011 15:39, Wolfgang Denk  wrote:

> Dear Chander Kashyap,
>
> In message  5pzhytn...@mail.gmail.com> you wrote:
> >
> [200+ lines of unrelated quote deleted.]
>
> > > > +void copy_uboot_to_ram(void)
> > > > +{
> > > > +   copy_sd_mmc_to_mem copy_bl2 = (copy_sd_mmc_to_mem)*(u32
> *)(0x02020030);
> > > > +   copy_bl2(BL2_START_OFFSET,\
> > > > +   BL2_SIZE_BLOC_COUNT, (u32 *)CONFIG_SYS_TEXT_BASE);
> > > > +}
> > >
> > > Curious to know, what the 0x02020030 hardcoding number means here?
> > > It is a pointer to a API or hardware register, or anything else, how
> > > this simple function perform the
> > > data transfer from mmc to ram?
> > >
> > >  0x02020030 is iROM address, which contains pointer to API (copy from
> mmc
> > from DDR) in iROM.
>
> Well, as you can see yourself here, an external reviewer has no
> chance to reand and understand this code. As mentioned before, this
> needs serious rework.
>
Yes I am reworking on it. I make it more readable. But still i need to use
the inbuilt function to copy u-boot from mmc to ddr.

>
> Best regards,
>
> Wolfgang Denk
>
> --
> DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
> Those who hate and fight must stop themselves -- otherwise it is  not
> stopped.
>-- Spock, "Day of the Dove", stardate unknown
>



-- 
with warm regards,
Chander Kashyap
___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: physical_package_id value

2011-08-01 Thread Daniel Lezcano
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

On 08/01/2011 09:42 AM, Vincent Guittot wrote:
> Hi,
> 
> You should see the mpidr values in the kernel log when the cpus are
> registered. Something like:
> CPU0: thread -1, cpu 0, socket xx, mpidr yy
> 
> Could you give me these values ?

Stupid me, the mpidr was not displayed and I thought I had to enable
early_printk but at this point I figured out sched_mc was not enabled in
the config :/

So never mind and sorry for the noise :)

Thanks
  -- Daniel

ps: I was puzzled by the existence of the different topology files, does
it make sense to show a file if the content is always invalid ?


> On 1 August 2011 01:11, Daniel Lezcano  wrote:
> On 08/01/2011 12:42 AM, Amit Kucheria wrote:
 On Sun, Jul 31, 2011 at 10:21 PM, Daniel Lezcano
  wrote:
> -BEGIN PGP SIGNED MESSAGE-
> Hash: SHA1
>
>
> Hi Vincent,
>
>
> I compiled the linux-linaro-3.0 kernel and checked the content of
> /sys/devices/system/cpu/cpu[0-1]/topology/physical_package_id and found
> it is equal to '-1'.
>
> Is it the expected value ?
>

 Was this on a OMAP3 device? Not sure whether MPIDR register is defined
 on Cortex-A8...
> 
> 
> Oh, sorry. I forgot to mention it is on a pandaboard.
> 
> 
>>

- -- 
  Linaro.org │ Open source software for ARM SoCs

Follow Linaro:   Facebook |
 Twitter |
 Blog

-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iQEcBAEBAgAGBQJONo8dAAoJEAKBbMCpUGYA2bIH/3B0tkX9eexxyNaa4pKzRfyX
r46EwauK3TEnpQyI6gegLVAkYqImrM+0GcZcmWKUcrjWgy/1X5XE3sZTW7xi5O2C
0HTH76D8ey9wKOOImZ18WZW/yWI7obCS/ymkdt7tbZMw8zlR1sbA1SzPDGZPTL1/
/pf8EyGrSPCvHz602yLrY7en19Dz+M4jpxyMghtY+Ea9iV3syhoQDNO/5RYFiLWx
CN+EhuX3+RBRlEIumR8CLT46GUxj6mWEv6B2m4WYO3CXz4wcCOb03iY0uzcsDY1r
R8o5y/j9D9nLO3+tLahzi/3NTi8SOALa3qMSn6fAYxGUr/ex7O93SghwFNh3grI=
=kAuE
-END PGP SIGNATURE-

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: [U-Boot] [PATCH v4 2/2] ORIGEN: Add MMC SPL support

2011-08-01 Thread Wolfgang Denk
Dear Chander Kashyap,

In message  
you wrote:
>
[200+ lines of unrelated quote deleted.]

> > > +void copy_uboot_to_ram(void)
> > > +{
> > > +   copy_sd_mmc_to_mem copy_bl2 = (copy_sd_mmc_to_mem)*(u32 
> > > *)(0x02020030);
> > > +   copy_bl2(BL2_START_OFFSET,\
> > > +   BL2_SIZE_BLOC_COUNT, (u32 *)CONFIG_SYS_TEXT_BASE);
> > > +}
> >
> > Curious to know, what the 0x02020030 hardcoding number means here?
> > It is a pointer to a API or hardware register, or anything else, how
> > this simple function perform the
> > data transfer from mmc to ram?
> >
> >  0x02020030 is iROM address, which contains pointer to API (copy from mmc
> from DDR) in iROM.

Well, as you can see yourself here, an external reviewer has no
chance to reand and understand this code. As mentioned before, this
needs serious rework.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
Those who hate and fight must stop themselves -- otherwise it is  not
stopped.
-- Spock, "Day of the Dove", stardate unknown

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: physical_package_id value

2011-08-01 Thread Vincent Guittot
Hi,

You should see the mpidr values in the kernel log when the cpus are
registered. Something like:
CPU0: thread -1, cpu 0, socket xx, mpidr yy

Could you give me these values ?

Vincent


On 1 August 2011 01:11, Daniel Lezcano  wrote:
> -BEGIN PGP SIGNED MESSAGE-
> Hash: SHA1
>
> On 08/01/2011 12:42 AM, Amit Kucheria wrote:
>> On Sun, Jul 31, 2011 at 10:21 PM, Daniel Lezcano
>>  wrote:
>>> -BEGIN PGP SIGNED MESSAGE-
>>> Hash: SHA1
>>>
>>>
>>> Hi Vincent,
>>>
>>>
>>> I compiled the linux-linaro-3.0 kernel and checked the content of
>>> /sys/devices/system/cpu/cpu[0-1]/topology/physical_package_id and found
>>> it is equal to '-1'.
>>>
>>> Is it the expected value ?
>>>
>>
>> Was this on a OMAP3 device? Not sure whether MPIDR register is defined
>> on Cortex-A8...
>
>
> Oh, sorry. I forgot to mention it is on a pandaboard.
>
>
> - --
>   Linaro.org ? Open source software for ARM SoCs
>
> Follow Linaro:   Facebook |
>  Twitter |
>  Blog
>
> -BEGIN PGP SIGNATURE-
> Version: GnuPG v1.4.11 (GNU/Linux)
> Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/
>
> iQEcBAEBAgAGBQJONeEVAAoJEAKBbMCpUGYAGZgH/3p44gyWysqp2iSnwDEIEBum
> 9eEiuUnuWx/89pPxcsdu3CZWkH9aK9D337ntOi5nKNlxFV2PHcZmKBdwvgd9e8pk
> Q0/5i+ISfg9+0+y1WZQ3PE9MneyWJA2Lo371JJk7VjMjavrPGQH3dPg9+bFALkTR
> Uc5P4McxtucAw3zWlJSsW3U+0vRXjIVSGgpctMTukAk1L2l8Va1FiAQvqk9FYzA9
> pVadRebzBdsXgW77sl8WogW3KCArdkJTnhFzVLr3Hsg+HUgZH82BSukvPFm97qtr
> HhWWd96L1P89fJ4gJ0crgdcWGdE2XVHlXoZ9vPhC6rPxljfuJTuCLdcig3S30BU=
> =oiR4
> -END PGP SIGNATURE-
>

___
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev


Re: [U-Boot] [PATCH v4 2/2] ORIGEN: Add MMC SPL support

2011-08-01 Thread Chander Kashyap
Hi Lei Wen,

On 31 July 2011 12:26, Lei Wen  wrote:

> Hi Chander,
>
> On Fri, Jul 29, 2011 at 12:41 PM, Chander Kashyap
>  wrote:
> > Adds mmc boot support.
> >
> > Signed-off-by: Chander Kashyap 
> > ---
> >  mmc_spl/board/samsung/origen/Makefile |  106
> 
> >  mmc_spl/board/samsung/origen/mmc_boot.c   |   57 +
> >  mmc_spl/board/samsung/origen/tools/mkv310_image.c |  140
> +
> >  mmc_spl/board/samsung/origen/u-boot.lds   |   88 +
> >  4 files changed, 391 insertions(+), 0 deletions(-)
> >  create mode 100644 mmc_spl/board/samsung/origen/Makefile
> >  create mode 100644 mmc_spl/board/samsung/origen/mmc_boot.c
> >  create mode 100644 mmc_spl/board/samsung/origen/tools/mkv310_image.c
> >  create mode 100644 mmc_spl/board/samsung/origen/u-boot.lds
> >
> > diff --git a/mmc_spl/board/samsung/origen/Makefile
> b/mmc_spl/board/samsung/origen/Makefile
> > new file mode 100644
> > index 000..1d61cc0
> > --- /dev/null
> > +++ b/mmc_spl/board/samsung/origen/Makefile
> > @@ -0,0 +1,106 @@
> > +#
> > +# (C) Copyright 2006-2007
> > +# Stefan Roese, DENX Software Engineering, s...@denx.de.
> > +#
> > +# (C) Copyright 2008
> > +# Guennadi Liakhovetki, DENX Software Engineering, 
> > +#
> > +# (C) Copyright 2011
> > +# Chander Kashyap, Samsung Electronics, 
> > +#
> > +# See file CREDITS for list of people who contributed to this
> > +# project.
> > +#
> > +# This program is free software; you can redistribute it and/or
> > +# modify it under the terms of the GNU General Public License as
> > +# published by the Free Software Foundation; either version 2 of
> > +# the License, or (at your option) any later version.
> > +#
> > +# This program is distributed in the hope that it will be useful,
> > +# but WITHOUT ANY WARRANTY; without even the implied warranty of
> > +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> > +# GNU General Public License for more details.
> > +#
> > +# You should have received a copy of the GNU General Public License
> > +# along with this program; if not, write to the Free Software
> > +# Foundation, Inc., 59 Temple Place, Suite 330, Boston,
> > +# MA 02111-1307 USA
> > +#
> > +
> > +CONFIG_MMC_SPL = y
> > +
> > +include $(TOPDIR)/config.mk
> > +
> > +LDSCRIPT= $(TOPDIR)/mmc_spl/board/$(BOARDDIR)/u-boot.lds
> > +LDFLAGS= -Bstatic -T $(mmcobj)u-boot.lds -Ttext
> $(CONFIG_SYS_TEXT_BASE) $(PLATFORM_LDFLAGS)
> > +AFLAGS += -DCONFIG_MMC_SPL
> > +AFLAGS += -DCONFIG_SPL_BUILD
> > +CFLAGS += -DCONFIG_MMC_SPL
> > +CFLAGS += -DCONFIG_SPL_BUILD
> > +
> > +SOBJS  = start.o mem_setup.o lowlevel_init.o
> > +COBJS  = mmc_boot.o
> > +
> > +SRCS   := $(SOBJS:.o=.S) $(COBJS:.o=.c)
> > +OBJS   := $(addprefix $(obj),$(SOBJS) $(COBJS))
> > +__OBJS := $(SOBJS) $(COBJS)
> > +LNDIR  := $(OBJTREE)/mmc_spl/board/$(BOARDDIR)
> > +
> > +mmcobj := $(OBJTREE)/mmc_spl/
> > +
> > +
> > +MKV310_MMC_SPL_EXEC = mkv310_mmc_spl_exec
> > +MMC_SPL_BIN = u-boot-mmc-spl.bin
> > +
> > +ALL = $(mmcobj)u-boot-spl $(mmcobj)u-boot-spl.bin
> $(mmcobj)$(MMC_SPL_BIN)
> > +
> > +all:$(obj).depend $(ALL)
> > +
> > +$(mmcobj)$(MMC_SPL_BIN):  $(mmcobj)u-boot-spl.bin
> tools/$(MKV310_MMC_SPL_EXEC)
> > +   ./tools/$(MKV310_MMC_SPL_EXEC) $(mmcobj)u-boot-spl.bin
> $(mmcobj)$(MMC_SPL_BIN)
> > +   rm -f tools/$(MKV310_MMC_SPL_EXEC)
> > +
> > +tools/$(MKV310_MMC_SPL_EXEC): tools/mkv310_image.c
> > +   $(HOSTCC) tools/mkv310_image.c -o tools/$(MKV310_MMC_SPL_EXEC)
> > +
> > +$(mmcobj)u-boot-spl.bin:   $(mmcobj)u-boot-spl
> > +   $(OBJCOPY) ${OBJCFLAGS} -O binary $< $@
> > +
> > +$(mmcobj)u-boot-spl:   $(OBJS) $(mmcobj)u-boot.lds
> > +   cd $(LNDIR) && $(LD) $(LDFLAGS) $(__OBJS) \
> > +   -Map $(mmcobj)u-boot-spl.map \
> > +   -o $(mmcobj)u-boot-spl
> > +
> > +$(mmcobj)u-boot.lds: $(LDSCRIPT)
> > +   $(CPP) $(CPPFLAGS) $(LDPPFLAGS) -ansi -D__ASSEMBLY__ -P - <$^ >$@
> > +
> > +# create symbolic links for common files
> > +
> > +# from cpu directory
> > +start.S:
> > +   @rm -f $@
> > +   @ln -s $(TOPDIR)/arch/arm/cpu/armv7/start.S $@
> > +
> > +# from board directory
> > +mem_setup.S:
> > +   @rm -f $@
> > +   @ln -s $(TOPDIR)/board/samsung/origen/mem_setup.S $@
> > +
> > +lowlevel_init.S:
> > +   @rm -f $@
> > +   @ln -s $(TOPDIR)/board/samsung/origen/lowlevel_init.S $@
> > +
> >
> +#
> > +
> > +$(obj)%.o: %.S
> > +   $(CC) $(AFLAGS) -c -o $@ $<
> > +
> > +$(obj)%.o: %.c
> > +   $(CC) $(CFLAGS) -c -o $@ $<
> > +
> > +# defines $(obj).depend target
> > +include $(SRCTREE)/rules.mk
> > +
> > +sinclude $(obj).depend
> > +
> >
> +#
> > diff --git a/mmc_spl/board/samsung/origen/mmc_boot.c
> b/mmc_spl/board/samsung/origen/mmc_boot.c
> > new file mode 100644
> > index 000..d9b44d8
> > --- /dev/null
> > +++ b/mmc_s