Re: [U-Boot] [PATCH v2 5/7] armv8: layerscape: Eanble falcon boot

2017-09-17 Thread Albert ARIBAUD (U-Boot)
Spam detection software, running on the system "lists.denx.de", has identified this incoming email as possible spam. The original message has been attached to this so you can view it or label similar future email. If you have any questions, see @@CONTACT_ADDRESS@@ for details. Content preview:

Re: [U-Boot] [PATCH] arm: Update our 'ret' assembler macro slightly

2017-03-07 Thread Albert ARIBAUD
Hello Tom, LGTM. On Thu, 2 Mar 2017 09:59:30 -0500, Tom Rini <tr...@konsulko.com> wrote: > We only support cores that do Thumb-1 or later. So we add a comment to > explain this and remove the architecture test. > > Cc: Albert ARIBAUD <albert.u.b...@aribaud.net>

[U-Boot] [PATCH] armv5te: make 'ret lr' produce iinterworking 'bx lr'

2017-02-27 Thread Albert ARIBAUD
' for ARMv5TE. Signed-off-by: Albert ARIBAUD <albert.u.b...@aribaud.net> --- Note: this patch supersedes patch "openrd: disable private arch memset, memcpy and libgcc" dated Sun, 26 Feb 2017 16:29:32 +0100. arch/arm/include/asm/assembler.h | 2 +- 1 file changed, 1 insertion(+), 1 d

Re: [U-Boot] [PATCH] openrd: disable private arch memset, memcpy and libgcc

2017-02-27 Thread Albert ARIBAUD
Update: On Mon, 27 Feb 2017 07:52:10 +0100, Albert ARIBAUD <albert.u.b...@aribaud.net> wrote: > Hello Tom, > > On Sun, 26 Feb 2017 20:04:28 -0500, Tom Rini <tr...@konsulko.com> wrote: > > On Sun, Feb 26, 2017 at 04:29:32PM +0100, Albert ARIBAUD wrote: > > &

[U-Boot] [PATCH] openrd: disable private arch memset, memcpy and libgcc

2017-02-26 Thread Albert ARIBAUD
a fix to private libgcc and architecture-optimized memcpy and memset is mainlined, disable their use in all openrd configurations. Signed-off-by: Albert ARIBAUD <albert.u.b...@aribaud.net> --- configs/openrd_base_defconfig | 4 configs/openrd_client_defconfig | 4 c

Re: [U-Boot] [PATCH] armv7m: Add SysTick timer driver

2017-02-05 Thread Albert ARIBAUD
rtex > M3, M4, M7 devices and is always located at a fixed address. > > Signed-off-by: Phil Edworthy <phil.edwor...@renesas.com> > --- > Resend with Albert's new email address - get_maintainer.pl gave the old one. > --- Cordialement, Albert ARIBAUD 3ADEV _

[U-Boot] [PATCH] pcm052: fix DDR initialization sequence

2017-02-01 Thread Albert ARIBAUD (3ADEV)
The sequence erroneously launched the DDR controller initialization before the pad muxing was done, causing DRAM size computation to hang. Configuring the pads first then launching DDR controller initialization prevents the DRAM hanging. Signed-off-by: Albert ARIBAUD (3ADEV) <albert.a

Re: [U-Boot] [PATCH 1/2] arm: Remove unofficial mach-type number uses

2017-01-25 Thread Albert ARIBAUD
t; That said, your change is good, it may produce a discontent in particular > cases, but it won't be problematic to remove it by partial commit > reverting and adding an explanatory comment into the code. Agreed. > -- > With best wishes, &

Re: [U-Boot] [PATCH 0/6] Add support for the BK4R1 variant of PCM052

2016-10-10 Thread Albert ARIBAUD
Hi Stefano, Sorry for the delay. Le Fri, 7 Oct 2016 12:01:02 +0200, Stefano Babic <sba...@denx.de> a écrit : > Hi Albert, > > On 06/10/2016 15:43, Albert ARIBAUD wrote: > > > > > Hmm... What U-Boot commit do you apply above? > > It was on top of v

Re: [U-Boot] [PATCH 0/6] Add support for the BK4R1 variant of PCM052

2016-10-06 Thread Albert ARIBAUD
Hi Stefano, Le Thu, 6 Oct 2016 09:21:13 +0200, Stefano Babic <sba...@denx.de> a écrit : > Hi Albert, > > On 26/09/2016 09:08, Albert ARIBAUD (3ADEV) wrote: > > BK4R1 is basically PCM052 with the following differences > > or quirks: > > > > 1) it has

[U-Boot] [PATCH 0/6] Add support for the BK4R1 variant of PCM052

2016-09-26 Thread Albert ARIBAUD (3ADEV)
not apply, as target configs in this file never have descriptions. Albert ARIBAUD (3ADEV) (6): pcm052: fix MTD partitioning pcm052: remove target-specific dtb name from env pcm052: add 'm4go' command tools: mkimage: add support for Vybrid image format pcm052: allow specifying onboard DDR

[U-Boot] [PATCH 6/6] pcm052: add new BK4r1 target based on PCM052 SoM

2016-09-26 Thread Albert ARIBAUD (3ADEV)
Signed-off-by: Albert ARIBAUD (3ADEV) <albert.arib...@3adev.fr> --- arch/arm/Kconfig | 4 ++ arch/arm/dts/Makefile| 3 +- arch/arm/dts/bk4r1.dts | 48 + arch/arm/dts/vf.dtsi | 4 +- board/phytec/pcm052/Kconfig | 20 ++ board/

[U-Boot] [PATCH 4/6] tools: mkimage: add support for Vybrid image format

2016-09-26 Thread Albert ARIBAUD (3ADEV)
This format can be flashed directly at address 0 of the NAND FLASH, as it contains all necessary headers. Signed-off-by: Albert ARIBAUD (3ADEV) <albert.arib...@3adev.fr> --- Makefile | 6 ++ arch/arm/config.mk| 3 + arch/arm/cpu/armv7/vf610/Ma

[U-Boot] [PATCH 5/6] pcm052: allow specifying onboard DDR size in configs

2016-09-26 Thread Albert ARIBAUD (3ADEV)
PCM052 SoMs may be equipped with various sizes of DDR. Keep default of 256MB; new PCM052-based targets will specify their actual DDR size. Linux command line is auto-adjusted to DDR size. Signed-off-by: Albert ARIBAUD (3ADEV) <albert.arib...@3adev.fr> --- board/phytec/pcm052/Kconf

[U-Boot] [PATCH 2/6] pcm052: remove target-specific dtb name from env

2016-09-26 Thread Albert ARIBAUD (3ADEV)
Signed-off-by: Albert ARIBAUD (3ADEV) <albert.arib...@3adev.fr> --- include/configs/pcm052.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/configs/pcm052.h b/include/configs/pcm052.h index 302c7dd..1858662 100644 --- a/include/configs/pcm052.h +++ b/include/c

[U-Boot] [PATCH 3/6] pcm052: add 'm4go' command

2016-09-26 Thread Albert ARIBAUD (3ADEV)
Add the 'm4go' command to pcm052-based targets. It loads scatter file images. Signed-off-by: Albert ARIBAUD (3ADEV) <albert.arib...@3adev.fr> --- board/phytec/pcm052/pcm052.c | 38 ++ 1 file changed, 38 insertions(+) diff --git a/board/phytec/pcm052/pc

[U-Boot] [PATCH 1/6] pcm052: fix MTD partitioning

2016-09-26 Thread Albert ARIBAUD (3ADEV)
arguments. Signed-off-by: Albert ARIBAUD (3ADEV) <albert.arib...@3adev.fr> --- include/configs/pcm052.h | 14 ++ 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/include/configs/pcm052.h b/include/configs/pcm052.h index 57a7630..302c7dd 100644 --- a/include/configs/pc

Re: [U-Boot] [PATCH] Revert "arm: Remove wireless_space board"

2016-04-02 Thread Albert ARIBAUD
On Sun, 31 Jan 2016 21:37:04 +0100, Albert ARIBAUD <albert.u.b...@aribaud.net> wrote: > Revert commit b352182a and complete generic board support. > > Signed-off-by: Albert ARIBAUD <albert.u.b...@aribaud.net> > --- > arch/arm/mach-kirkwood/Kconfig

Re: [U-Boot] [PATCH 5/5] lib: Enable private libgcc by default

2016-03-29 Thread Albert ARIBAUD
Hello Tom, On Sun, 27 Mar 2016 09:36:41 -0400, Tom Rini <tr...@konsulko.com> wrote: > On Fri, Mar 25, 2016 at 07:37:25AM +0100, Albert ARIBAUD wrote: > > Hello Tom, > > > > On Thu, 24 Mar 2016 20:49:42 -0400, Tom Rini <tr...@konsulko.com> wrote: > > >

Re: [U-Boot] [PATCH 5/5] lib: Enable private libgcc by default

2016-03-25 Thread Albert ARIBAUD
On Fri, 25 Mar 2016 07:37:25 +0100, Albert ARIBAUD <albert.u.b...@aribaud.net> wrote: > Hello Tom, > That way, > > 0) U-Boot gets the stable and controlled AEABI support you want; > > 1) GCC keeps its somewhat stable but uncontrolled internal "generated >cod

Re: [U-Boot] [PATCH 5/5] lib: Enable private libgcc by default

2016-03-25 Thread Albert ARIBAUD
Hello Sergey, On Thu, 24 Mar 2016 18:37:52 -0700 (PDT), Sergey Kubushyn wrote: > On Thu, 24 Mar 2016, Tom Rini wrote: > U-Boot is a standalone program not supposed to coexist with any external > applications i.e. it is totally self-sufficient, not living in some kind of > system

Re: [U-Boot] [PATCH 5/5] lib: Enable private libgcc by default

2016-03-25 Thread Albert ARIBAUD
Hello Tom, On Thu, 24 Mar 2016 20:49:42 -0400, Tom Rini <tr...@konsulko.com> wrote: > On Thu, Mar 24, 2016 at 08:50:03AM +0100, Albert ARIBAUD wrote: > > Hello Tom, > > > > On Wed, 23 Mar 2016 17:36:17 -0400, Tom Rini <tr...@konsulko.com> wrote: > > >

Re: [U-Boot] [PATCH 5/5] lib: Enable private libgcc by default

2016-03-24 Thread Albert ARIBAUD
Hello Tom, On Wed, 23 Mar 2016 17:36:17 -0400, Tom Rini <tr...@konsulko.com> wrote: > On Wed, Mar 23, 2016 at 06:08:45PM +0100, Albert ARIBAUD wrote: > > Hello Tom, > > > > On Wed, 23 Mar 2016 09:22:38 -0400, Tom Rini <tr...@konsulko.com> wrote: > > >

Re: [U-Boot] [PATCH 5/5] lib: Enable private libgcc by default

2016-03-23 Thread Albert ARIBAUD
Hello Tom, On Wed, 23 Mar 2016 09:22:38 -0400, Tom Rini <tr...@konsulko.com> wrote: > On Wed, Mar 23, 2016 at 01:53:35PM +0100, Albert ARIBAUD wrote: > > Hello Marek, > > > > On Sun, 20 Mar 2016 17:15:34 +0100, Marek Vasut <ma...@denx.de> wrote: > > &g

Re: [U-Boot] [PATCH 5/5] lib: Enable private libgcc by default

2016-03-23 Thread Albert ARIBAUD
Hello Marek, On Sun, 20 Mar 2016 17:15:34 +0100, Marek Vasut wrote: > This patch decouples U-Boot binary from the toolchain on systems where > private libgcc is available. Instead of pulling in functions provided > by the libgcc from the toolchain, U-Boot will use it's own set of

Re: [U-Boot] [PATCH 1/6] ARM: keystone2: Split monitor code / command code

2016-03-19 Thread Albert ARIBAUD
Hello Tom, On Wed, 16 Mar 2016 11:03:03 -0400, Tom Rini wrote: > When we switch to including all linker lists in U-Boot it is important > to not include commands as that may lead to link errors due to other > things we have already discarded. In this case, we split the code

Re: [U-Boot] [PATCH 00/18] sf: fix support of QSPI memories and controllers

2016-03-19 Thread Albert ARIBAUD
Hello Jagan, On Wed, 16 Mar 2016 19:44:26 +0530, Jagan Teki wrote: > On Wednesday 16 March 2016 07:00 PM, Cyrille Pitchen wrote: > > Le 15/03/2016 19:21, Jagan Teki a écrit : > >> On Tuesday 15 March 2016 11:42 PM, Cyrille Pitchen wrote: > >>> Hi all, > >>> > >>> This series

Re: [U-Boot] [PATCH 00/18] sf: fix support of QSPI memories and controllers

2016-03-18 Thread Albert ARIBAUD
Hello Jagan, On Wed, 16 Mar 2016 22:04:23 +0530, Jagan Teki <jt...@openedev.com> wrote: > Hi Albert, > > On Wednesday 16 March 2016 09:53 PM, Albert ARIBAUD wrote: > > Hello Jagan, > > > > On Wed, 16 Mar 2016 19:44:26 +0530, Jagan Teki <jt...@openedev.com>

Re: [U-Boot] [PATCH] sf: Correct data types in stm_is_locked_sr()

2016-03-11 Thread Albert ARIBAUD
Hello Jagan, On Sat, 12 Mar 2016 00:41:25 +0530, Jagan Teki <jagannadh.t...@gmail.com> wrote: > Hi Albert, > > On 12 March 2016 at 00:17, Albert ARIBAUD <albert.u.b...@aribaud.net> wrote: > > Hello Jagan, > > > > On Fri, 11 Mar 2016 12:09:37 +0530, Jagan Te

Re: [U-Boot] [PATCH] sf: Correct data types in stm_is_locked_sr()

2016-03-11 Thread Albert ARIBAUD
Hello Jagan, On Fri, 11 Mar 2016 12:09:37 +0530, Jagan Teki wrote: > On 11 March 2016 at 07:50, Marek Vasut wrote: > > The stm_is_locked_sr() function is picked from Linux kernel. For reason > > unknown, the 64bit data types used by the function and

Re: [U-Boot] [PATCH 3/5] kbuild: use $(call cmd, ) rather than $(call if_changed, ) where possible

2016-02-03 Thread Albert ARIBAUD
Hello Masahiro, On Wed, 3 Feb 2016 21:05:12 +0900, Masahiro Yamada wrote: > These build commands are constant (mostly, just concatenating images, > or just copying). No need to use $(call if_changed,...) for them. I am a total kbuild ignorant, so I'm probably

Re: [U-Boot] [PATCHv2] common/memsize.c: Simplify RAM size detection

2016-02-03 Thread Albert ARIBAUD
Hello Eddy, On Tue, 2 Feb 2016 22:15:28 +0200, Eddy Petrișor wrote: > The case of memory of size 0 is not that different from a memory of any other > size, so we remove the duplicate code and treat the small differences when it > is the case. > > Signed-off-by: Eddy

Re: [U-Boot] [PATCH 3/5] kbuild: use $(call cmd, ) rather than $(call if_changed, ) where possible

2016-02-03 Thread Albert ARIBAUD
Hello Albert, On Wed, 3 Feb 2016 13:18:35 +0100, Albert ARIBAUD <albert.u.b...@aribaud.net> wrote: > Hello Masahiro, > > On Wed, 3 Feb 2016 21:05:12 +0900, Masahiro Yamada > <yamada.masah...@socionext.com> wrote: > > These build commands are constant (mos

Re: [U-Boot] [PATCH 2/3] common: env_flags: include common.h even for HOST_CC

2016-02-03 Thread Albert ARIBAUD
Hello Peter, On Wed, 3 Feb 2016 12:42:51 +, Peter Robinson wrote: > When compiling with gcc 6 we get the following error due to ARRAY_SIZE being > defined elsewhere. > > common/env_flags.c:155: undefined reference to `ARRAY_SIZE' > > Signed-off-by: Peter Robinson

[U-Boot] Patman use (was: [PATCHv2] common/memsize.c: Simplify RAM size detection)

2016-02-03 Thread Albert ARIBAUD
Hello Eddy, (re-titling) On Wed, 3 Feb 2016 18:32:45 +0200, Eddy Petrișor <eddy.petri...@gmail.com> wrote: > 2016-02-03 15:38 GMT+02:00 Albert ARIBAUD <albert.u.b...@aribaud.net>: > > Hello Eddy, > > > > On Tue, 2 Feb 2016 22:15:28 +0200, Eddy Petrișor &g

Re: [U-Boot] [PATCH 2/3] common: env_flags: include common.h even for HOST_CC

2016-02-03 Thread Albert ARIBAUD
Hello Peter, On Wed, 3 Feb 2016 16:11:38 +, Peter Robinson <pbrobin...@gmail.com> wrote: > Hi Albert, > > On Wed, Feb 3, 2016 at 1:41 PM, Albert ARIBAUD > <albert.u.b...@aribaud.net> wrote: > > Hello Peter, > > > > On Wed, 3 Feb 2016 12:42:51 +

Re: [U-Boot] 64-bit x86 U-Boot?

2016-02-02 Thread Albert ARIBAUD
Hello Bin and Simon, On Tue, 2 Feb 2016 15:25:48 +0800, Bin Meng wrote: > Hi Simon, > > On Tue, Feb 2, 2016 at 11:58 AM, Simon Glass wrote: > > +Bin (sorry, meant to copy you before) > >>> For non-FSP devices we don't init the RAM until much later - >

Re: [U-Boot] [PATCH] spl: define BOOT_DEVICE_USB

2016-02-02 Thread Albert ARIBAUD
Hello Masahiro, On Tue, 2 Feb 2016 15:45:13 +0900, Masahiro Yamada wrote: > This macro is referenced from common/spl/spl.c Nitpick: not a macro, but an enum value. Amicalement, -- Albert. ___ U-Boot mailing list

[U-Boot] [PATCH v2 0/1] Fix patman Series-notes handling for buildman

2016-02-02 Thread Albert ARIBAUD
atch. Changes in v2: - fix typo in commit message ("rathen" -> "rather") - actually limit array initialization to series.notes Albert ARIBAUD (1): patman: fix series-notes handling for buildman tools/patman/series.py | 5 - 1 file changed

Re: [U-Boot] [PATCH] patman: fix series-notes handling for buildman

2016-02-02 Thread Albert ARIBAUD
Hello Simon, > >> Hmm, actually I've had to drop this as it breaks 'Series-version'. > >> That currently does not expect a list. > > > > Hmm, I can't reproduce this here. How do you trigger the Series-version > > break? > > I created a commit with a Series-version: in it. Then, running patman >

[U-Boot] [PATCH v2] patman: fix series-notes handling for buildman

2016-02-02 Thread Albert ARIBAUD
A patman series with a 'Series-notes' section causes buildman to crash with: self.series.notes += self.section TypeError: cannot concatenate 'str' and 'list' objects Fix by initializing series.notes as a one-element array rather than a scalar. Signed-off-by: Albert ARIBAUD <albert.

Re: [U-Boot] [PATCH v3]net: Wrong Initialization in davinci-emac driver

2016-02-01 Thread Albert ARIBAUD
Hello Vishwas, On Mon, 1 Feb 2016 20:51:58 +0530, Vishwas Srivastava wrote: > Author: Vishwas Srivastava > Date: Mon Jan 25 21:28:17 2016 +0530 This is unneeded in the commit message, as the author and date are already provided in the mail. >

Re: [U-Boot] [PATCH] patman: fix series-notes handling for buildman

2016-01-31 Thread Albert ARIBAUD
Hello Simon, On Thu, 19 Nov 2015 20:29:11 -0700, Simon Glass <s...@chromium.org> wrote: > Hi Albert, > > On 13 November 2015 at 19:35, Simon Glass <s...@chromium.org> wrote: > > On 9 November 2015 at 14:36, Albert ARIBAUD <albert.u.b...@aribaud.net> > > wr

[U-Boot] Pull request: u-boot-arm/master

2016-01-31 Thread Albert ARIBAUD
to 735b1a2e5a61401868bb35702b6e5e18bce6eb97: arm: novena: Fix EEPROM i2c configuration (2016-01-31 16:32:56 +0100) Albert ARIBAUD (1): armv7: add cacheline sizes where missing Marek Vasut (4): arm: Replace test for CONFIG_ARMV7 with CONFIG_CPU_V7

[U-Boot] [PATCH] Revert "arm: Remove wireless_space board"

2016-01-31 Thread Albert ARIBAUD
Revert commit b352182a and complete generic board support. Signed-off-by: Albert ARIBAUD <albert.u.b...@aribaud.net> --- arch/arm/mach-kirkwood/Kconfig | 4 + board/LaCie/wireless_space/Kconfig | 12 ++ board/LaCie/wireless_space/MAINTAINERS | 6 + board

Re: [U-Boot] 64-bit x86 U-Boot?

2016-01-31 Thread Albert ARIBAUD
Hello Simon, On Sun, 31 Jan 2016 19:20:31 -0700, Simon Glass wrote: > Hi Bin, > > At present U-Boot supports booting a 64-bit kernel directly. It can > also be loaded as a 64-bit payload from EFI. But it cannot be built as > a 64-bit boot loader. > > I took a bit of a look

Re: [U-Boot] [PATCH v3 0/7] fdt: Replace u-boot-dtb.bin with u-boot.bin

2016-01-31 Thread Albert ARIBAUD
Hello Simon, Nitpicking: On Sun, 31 Jan 2016 18:10:48 -0700, Simon Glass wrote: > The original decision to use a separate u-boot-dtb.bin was aimed at allowing > any device tree file to be concatenated to the u-boot.bin image after the > build. However this no-longer seems so

Re: [U-Boot] [PATCH] OMAP3: omap3_logic: Remove YAFFS support.

2016-01-31 Thread Albert ARIBAUD
Hello Adam, On Sun, 31 Jan 2016 17:17:24 -0600, Adam Ford wrote: > UBIFS is the preferred OS, and YAFFS isn't officially included in > Linux. Removing this feature reduces the code size. Typo: "preferred FS", not "preferred OS". Amicalement, -- Albert.

Re: [U-Boot] Include patchwork patch ID in commit message?

2016-01-31 Thread Albert ARIBAUD
Hello Masahiro, On Fri, 29 Jan 2016 18:57:28 +0900, Masahiro Yamada wrote: > 2016-01-29 14:14 GMT+09:00 Heiko Schocher : > > Hello Joe, > > > > > > Am 28.01.2016 um 16:15 schrieb Joe Hershberger: > >> > >> Hi Heiko, > >> > >> On Thu, Jan 28, 2016 at

Re: [U-Boot] [PATCH V2] arm: config: enforce -fno-pic for gcc

2016-01-31 Thread Albert ARIBAUD
s not .got section and R_ARM_GOT_BREL in built-in.o. > And in the final u-boot elf image, all relocation entries are > R_ARM_RELATIVE. > > we can pass '-fno-pic' to xxx-gcc to disable pic. whether > the toolchain internally enables or disables pic, '-fno-pic' > can work well. >

Re: [U-Boot] [PATCH] arm: use common instructions applicable to armv7m & other arm archs

2016-01-31 Thread Albert ARIBAUD
Hello Vikas, On Sat, 30 Jan 2016 00:36:55 +0100, Vikas MANOCHA <vikas.mano...@st.com> wrote: > Hi Albert, > > > -Original Message- > > From: Albert ARIBAUD [mailto:albert.u.b...@aribaud.net] > > Sent: Friday, January 29, 2016 9:16 AM > > To: Vikas

Re: [U-Boot] [RFC V1] arm: config: enforce -fno-pic for gcc

2016-01-29 Thread Albert ARIBAUD
Hello Peng, On Sat, 5 Dec 2015 15:53:46 +0800, Peng Fan wrote: > Android's tool chain enable the -mandroid at default. > This option will enable the -fpic, which cause uboot compilation > failure: > " > LD u-boot > u-boot contains unexpected relocations:

Re: [U-Boot] [PATCH v2] ARM: Disable "DISCARD" for secure section if CONFIG_ARMV7_SECURE_BASE isn't defined

2016-01-29 Thread Albert ARIBAUD
Hello Tom, On Tue, 19 Jan 2016 10:57:11 -0500, Tom Rini wrote: > On Mon, Jan 18, 2016 at 11:02:40AM +0800, Dongsheng Wang wrote: > > > From: Wang Dongsheng > > > > "DISCARD" will remove ._secure.text relocate, but PSCI framework > > has already used

Re: [U-Boot] [PATCH 1/2] arm: Replace test for CONFIG_ARMV7 with CONFIG_CPU_V7

2016-01-29 Thread Albert ARIBAUD
eflect that change. > > Signed-off-by: Marek Vasut <ma...@denx.de> > Cc: Tom Rini <tr...@konsulko.com> > Cc: Albert Aribaud <albert.u.b...@aribaud.net> > Cc: Simon Glass <s...@chromium.org> > --- > arch/arm/include/asm/system.h | 4 ++-- > arch/arm/li

Re: [U-Boot] [PATCH] arm: use common instructions applicable to armv7m & other arm archs

2016-01-29 Thread Albert ARIBAUD
Hello Vikas, On Mon, 18 Jan 2016 18:52:57 -0800, Vikas Manocha wrote: > BIC instruction to clear the SP is not allowed in armv7m & is deprecated > in ARMv6T2 & above. This patch cleans the code by using instructions allowed > for armv7m as well as other Arm archs. I am not

Re: [U-Boot] [PATCH v2] armv7: add cacheline sizes where missing

2016-01-28 Thread Albert ARIBAUD
On Wed, 27 Jan 2016 08:46:11 +0100, Albert ARIBAUD <albert.u.b...@aribaud.net> wrote: > Some armv7 targets are missing a cache line size declaration. > In preparation for "arm: cache: Implement cache range check for v7" > patch, add these declarations with the appropriate

Re: [U-Boot] [PATCH v2] armv7: add cacheline sizes where missing

2016-01-28 Thread Albert ARIBAUD
On Wed, 27 Jan 2016 08:46:11 +0100, Albert ARIBAUD <albert.u.b...@aribaud.net> wrote: > Some armv7 targets are missing a cache line size declaration. > In preparation for "arm: cache: Implement cache range check for v7" > patch, add these declarations with the appropriate

Re: [U-Boot] patman dies cryptially when a (valid) e-mail address contains parentheses (or is UTF-8)

2016-01-27 Thread Albert ARIBAUD
Hello Simon, On Wed, 27 Jan 2016 15:53:45 -0700, Simon Glass <s...@chromium.org> wrote: > Hi Albert, > > On 27 January 2016 at 01:22, Albert ARIBAUD <albert.u.b...@aribaud.net> wrote: > > Hello Simon, > > > > I just noticed this while adding a Ser

[U-Boot] patman dies cryptially when a (valid) e-mail address contains parentheses (or is UTF-8)

2016-01-27 Thread Albert ARIBAUD
Hello Simon, I just noticed this while adding a Series-cc to my work address which has parentheses: running patman (without -n) will result in sh: 1: Syntax error: "(" unexpected Which initially left me wondering what was happening until I realized one of the Series-cc addresses had

Re: [U-Boot] [PATCH] ARM: zynq: Add function to get reboot status register value.

2016-01-27 Thread Albert ARIBAUD
Hello Moritz, On Wed, 27 Jan 2016 14:53:30 +0100, Moritz Fischer wrote: > Hi Michal, > > On Wed, Jan 27, 2016 at 2:15 PM, Michal Simek wrote: > > On 27.1.2016 12:22, Moritz Fischer wrote: > >> Signed-off-by: Moritz Fischer

Re: [U-Boot] [PATCH v2 2/2] net: phy: Add PHY driver for mv88e61xx switches

2016-01-27 Thread Albert ARIBAUD
Hello Kevin, On Wed, 27 Jan 2016 16:29:42 +, Kevin Smith wrote: > Hi Joe, > On 01/26/2016 06:11 PM, Joe Hershberger wrote: > >> + /* Replace the bus with the fake device */ > > Fake how? This is a confusing comment to me as written. > The genphy functions

Re: [U-Boot] [PATCH] armv7: add cacheline sizes where missing

2016-01-26 Thread Albert ARIBAUD
Hello Marek, On Wed, 27 Jan 2016 05:10:59 +0100, Marek Vasut <ma...@denx.de> wrote: > On Tuesday, January 26, 2016 at 05:40:49 PM, Albert ARIBAUD wrote: > > Some armv7 targets are missing a cache line size declaration. > > In preparation for "arm: cache: Implemen

[U-Boot] [PATCH v2] armv7: add cacheline sizes where missing

2016-01-26 Thread Albert ARIBAUD
Some armv7 targets are missing a cache line size declaration. In preparation for "arm: cache: Implement cache range check for v7" patch, add these declarations with the appropriate value for the target's SoC or CPU. Signed-off-by: Albert ARIBAUD <albert.u.b...@aribaud.net> --

Re: [U-Boot] [PATCH v2 0/2] net: phy: mv88e61xx: Revise as a PHY driver

2016-01-26 Thread Albert ARIBAUD
Hello Kevin, On Mon, 21 Dec 2015 21:45:32 +, Kevin Smith wrote: > The previous version of this driver implemented a shell command to manually > comfigure the switch. It did not integrate with the PHY infrastructure to > allow a MAC to use it as its PHY. This is

[U-Boot] [PATCH] armv7: add cacheline sizes where missing

2016-01-26 Thread Albert ARIBAUD
Some armv7 targets are missing a cache line size declaration. In preparation for "arm: cache: Implement cache range check for v7" patch, add these declarations with the appropriate value for the target's SoC or CPU. Signed-off-by: Albert ARIBAUD <albert.u.b...@aribaud.net> ---

Re: [U-Boot] [PATCH v2 0/2] net: phy: mv88e61xx: Revise as a PHY driver

2016-01-26 Thread Albert ARIBAUD
Hello Kevin, On Tue, 26 Jan 2016 16:56:21 +, Kevin Smith <kevin.sm...@elecsyscorp.com> wrote: > Hi Albert, > > On 01/26/2016 10:13 AM, Joe Hershberger wrote: > > Hi Albert, > > > > On Tue, Jan 26, 2016 at 10:09 AM, Albert ARIBAUD > > <albert.u

Re: [U-Boot] [PATCH v2 5/5] tools: Add tool to add crc8 to a mac address

2016-01-26 Thread Albert ARIBAUD
Hello Joe, On Mon, 25 Jan 2016 18:45:52 -0600, Joe Hershberger wrote: > On Mon, Dec 14, 2015 at 6:41 AM, Olliver Schinagl > wrote: > > +++ b/tools/gen_mac_addr.c > > This is not "generating a mac address", right? Its point is to create > a

Re: [U-Boot] [PATCH] arm: cache: Implement cache range check for v7

2016-01-25 Thread Albert ARIBAUD
ptimized out. > > > > > > > > > > > > Signed-off-by: Marek Vasut <ma...@denx.de> > > > > > > Cc: Dinh Nguyen <dingu...@opensource.altera.com> > > > > > > Cc: Albert Aribaud <albert.u.b...@aribaud.

Re: [U-Boot] axm SPL image too big

2016-01-12 Thread Albert ARIBAUD
Hello Heiko, On Tue, 12 Jan 2016 11:48:17 +0100, Heiko Schocher <h...@denx.de> wrote: > Hello Albert, > > Am 12.01.2016 um 11:26 schrieb Albert ARIBAUD: > > (cc:ing Heiko as the maintainer for axm/taurus) > > > > Hello, > > > > AXM is currently

[U-Boot] axm SPL image too big

2016-01-12 Thread Albert ARIBAUD
(cc:ing Heiko as the maintainer for axm/taurus) Hello, AXM is currently the only board failing 'buildman arm aarch64': +arm-unknown-linux-gnueabi-ld.bfd: SPL image too big +make[2]: *** [spl/u-boot-spl] Error 1 +make[1]: *** [spl/u-boot-spl] Error 2 +make: *** [sub-make] Error 2 Apparently SPL

Re: [U-Boot] [PATCH 1/2] arm: Replace test for CONFIG_ARMV7 with CONFIG_CPU_V7

2015-12-30 Thread Albert ARIBAUD
Hello Marek, On Tue, 29 Dec 2015 19:44:01 +0100, Marek Vasut wrote: > The arch/arm/lib/cache-cp15.c checks for CONFIG_ARMV7 and if this macro is > set, it configures TTBR0 register. This register must be configured for the > cache on ARMv7 to operate correctly. > > The problem is

Re: [U-Boot] ARMv7 MMU shareability issue

2015-12-14 Thread Albert ARIBAUD
Hello Pavel, On Mon, 14 Dec 2015 08:48:16 +0100, Pavel Machek wrote: > Hi! > > > This patch has several effects: > > > > - it selects proper ARMv7 translation table level 1 bit definitions; > > - it provides proper ARMv7 definitions for WT/WB/WA; > > - it selects proper ARMv7

Re: [U-Boot] ARMv7 MMU shareability issue

2015-12-13 Thread Albert ARIBAUD
3 PM, Stefan Roese wrote: > > > > Hi All! > > > > > > > > On 09.12.2015 09:09, Albert ARIBAUD wrote: > > > > > On Tue, 8 Dec 2015 14:43:42 +0100, Marek Vasut <ma...@denx.de> wrote: > > > > >> The arch/arm/lib/cache-cp15.c checks for

[U-Boot] ARMv7 MMU shareability issue (was: [PATCH] arm: Replace test for CONFIG_ARMV7 with CONFIG_CPU_V7)

2015-12-09 Thread Albert ARIBAUD
Hello Marek, On Tue, 8 Dec 2015 14:43:42 +0100, Marek Vasut wrote: > The arch/arm/lib/cache-cp15.c checks for CONFIG_ARMV7 and if this macro is > set, it configures TTBR0 register. This register must be configured for the > cache on ARMv7 to operate correctly. > > The problem is

Re: [U-Boot] [PATCH] arm, ubifs: fix gcc5.x compiler warning

2015-11-30 Thread Albert ARIBAUD
Hello Tom, On Mon, 30 Nov 2015 11:28:53 -0500, Tom Rini wrote: > On Mon, Nov 30, 2015 at 11:03:53AM +0100, Heiko Schocher wrote: > > Hello Jeroen, > > > > Am 30.11.2015 um 10:20 schrieb Jeroen Hofstee: > > >Hello Heiko, > > > > > >On 30-11-15 08:47, Heiko Schocher wrote: > >

Re: [U-Boot] [RFC] Removal of superfluous gd assignments

2015-11-26 Thread Albert ARIBAUD
Hello Stefan, On Thu, 26 Nov 2015 07:23:51 +0100, Stefan Roese <stefan.ro...@gmail.com> wrote: > Hi Albert, > > first, thank you very much for taking the time to go over this > in such depth. > > On 25.11.2015 16:20, Albert ARIBAUD wrote: > > Hello al

[U-Boot] [RFC] Removal of superfluous gd assignments

2015-11-25 Thread Albert ARIBAUD
Hello all, This is a follow-up to discussions on the IRC chan re: the fact that gd (the global data pointer) is being assigned in various places, which is not too much of a good thing. In all architectures, common/init/board_init.c is built in, which provides board_init_f_{mem,init_reserve}, in

[U-Boot] [PATCH v7 1/2] Fix board init code to respect the C runtime environment

2015-11-25 Thread Albert ARIBAUD
. Signed-off-by: Albert ARIBAUD <albert.u.b...@aribaud.net> --- Copying custodians for all architectures which this patch affects. This patch has been build-tested for all affected architectures except NIOS2, and run-tested on ARM OpenRD Client. For NIOS2, this patch contains all manual v1

[U-Boot] [PATCH v7 2/2] arm: move gd handling outside of C code

2015-11-25 Thread Albert ARIBAUD
, causing U-Boot to use a bad address to access GD. Fix this by never calling arch_setup_gd() for ARM, and instead setting r9 in arch/arm/lib/crt0.S, to the value returned by board_init_f_alloc_reserve(). Signed-off-by: Albert ARIBAUD <albert.u.b...@aribaud.net> --- Changes in v7: - also fix

[U-Boot] [PATCH v6 1/2] Fix board init code to respect the C runtime environment

2015-11-21 Thread Albert ARIBAUD
. Signed-off-by: Albert ARIBAUD <albert.u.b...@aribaud.net> --- Copying custodians for all architectures which this patch affects. This patch has been build-tested for all affected architectures except NIOS2, and run-tested on ARM OpenRD Client. For NIOS2, this patch contains all manual v1

[U-Boot] [PATCH v6 2/2] arm: move gd handling outside of C code

2015-11-21 Thread Albert ARIBAUD
, causing U-Boot to use a bad address to access GD. Fix this by never calling arch_setup_gd() for ARM, and instead setting r9 in arch/arm/lib/crt0.S, to the value returned by board_init_f_alloc_reserve(). Signed-off-by: Albert ARIBAUD <albert.u.b...@aribaud.net> --- Changes in v6: - moved ARM

Re: [U-Boot] [PATCH v5] Fix board init code to respect the C runtime environment

2015-11-20 Thread Albert ARIBAUD
Hello Simon, On Wed, 18 Nov 2015 18:05:07 -0700, Simon Glass wrote: > Hi Albert, > > >> early malloc() area > > > > "Arena", and "malloc arena", are established designations for the > > memory space used by malloc implementations; and I prefer to use this > > more specific

Re: [U-Boot] [PATCH v5] Fix board init code to respect the C runtime environment

2015-11-17 Thread Albert ARIBAUD
... and of course, I forgot to add a note at '(*)' re the malloc arena size alignment... On Tue, 17 Nov 2015 13:59:32 +0100, Albert ARIBAUD <albert.u.b...@aribaud.net> wrote: > Hello Simon, > Indeed, no attempt is made to check that it is aligned (and no attempt > is made in th

Re: [U-Boot] [PATCH v5] Fix board init code to respect the C runtime environment

2015-11-17 Thread Albert ARIBAUD
Hello Simon, On Mon, 16 Nov 2015 21:11:38 -0700, Simon Glass wrote: > > +++ b/arch/arm/lib/crt0.S > > @@ -82,9 +82,11 @@ ENTRY(_main) > > #else > > bic sp, sp, #7 /* 8-byte alignment for ABI compliance */ > > #endif > > + bl

[U-Boot] [PATCH v5] Fix board init code to respect the C runtime environment

2015-11-16 Thread Albert ARIBAUD
. Signed-off-by: Albert ARIBAUD <albert.u.b...@aribaud.net> --- Cc:ing custodians for all architectures which this patch affects. This patch has been build-tested for all there architectures, and run-tested on ARM OpenRD Client. For NIOS2, this patch contains all manual v1 and v2 fixes by

Re: [U-Boot] U-Boot as a next bootloader in a chain - reading the kernel boot args provided by the previous stage bootlader?

2015-11-16 Thread Albert ARIBAUD
Hello Wojciech, On Mon, 16 Nov 2015 13:09:43 +0100, Wojciech Zabolotny <wza...@gmail.com> wrote: > 2015-11-16 11:52 GMT+01:00 Albert ARIBAUD <albert.u.b...@aribaud.net>: > > Hello Wojciech, > > > > On Mon, 16 Nov 2015 10:42:50 +0100, Wojciech Zabolotny >

Re: [U-Boot] [PATCH v4] Fix board init code to use a valid C runtime environment

2015-11-16 Thread Albert ARIBAUD
Hello Alexey, On Mon, 16 Nov 2015 13:12:15 +, Alexey Brodkin <alexey.brod...@synopsys.com> wrote: > Hi Albert, > > On Sun, 2015-11-15 at 19:25 +0100, Albert ARIBAUD wrote: > > board_init_f_mem() alters the C runtime environment's > > stack it ls actually already

Re: [U-Boot] [PATCH v4] Fix board init code to use a valid C runtime environment

2015-11-16 Thread Albert ARIBAUD
Hello Alexey, On Mon, 16 Nov 2015 13:43:19 +, Alexey Brodkin <alexey.brod...@synopsys.com> wrote: > Hi Albert, > > On Mon, 2015-11-16 at 14:34 +0100, Albert ARIBAUD wrote: > > Hello Alexey, > > > > On Mon, 16 Nov 2015 13:12:15 +, Alexey Brodkin > &

Re: [U-Boot] U-Boot as a next bootloader in a chain - reading the kernel boot args provided by the previous stage bootlader?

2015-11-16 Thread Albert ARIBAUD
Hello Wojciech, On Mon, 16 Nov 2015 10:42:50 +0100, Wojciech Zabolotny wrote: > Hi, > > I need to use U-Boot (or barebox) as a bootloader in a Raspberry Pi > based system to extend booting flexibility. > The problem however is that certain kernel boot arguments are prepared >

Re: [U-Boot] [PATCH v3] Fix board init code to use a valid C runtime environment

2015-11-15 Thread Albert ARIBAUD
Hello Simon, On Sat, 14 Nov 2015 08:41:01 -0700, Simon Glass wrote: > Reading between the lines, there isn't actually a problem with this in > practice, but it is doing things that C should not really be doing. Is > that right? You did not even need to read between the lines

[U-Boot] [PATCH v4] Fix board init code to use a valid C runtime environment

2015-11-15 Thread Albert ARIBAUD
board_init_f_mem() alters the C runtime environment's stack it ls actually already using. This is not a valid C runtime environment. Split board_init_f_mem into C functions which do not alter their own stack and therefore run in a valid C runtime environment. Signed-off-by: Albert ARIBAUD

Re: [U-Boot] [PATCH v4] Fix board init code to use a valid C runtime environment

2015-11-15 Thread Albert ARIBAUD
Hello Albert, On Sun, 15 Nov 2015 19:25:25 +0100, Albert ARIBAUD <albert.u.b...@aribaud.net> wrote: > board_init_f_mem() alters the C runtime environment's > stack it ls actually already using. This is not a valid > C runtime environment. > > Split board_init_f_mem into

Re: [U-Boot] [PATCH v3] Fix board init code to use a valid C runtime environment

2015-11-14 Thread Albert ARIBAUD
Hello Simon, On Fri, 13 Nov 2015 19:29:21 -0700, Simon Glass <s...@chromium.org> wrote: > Hi Albert, > > On 13 November 2015 at 07:43, Albert ARIBAUD <albert.u.b...@aribaud.net> > wrote: > > board_init_f_mem() alters the C runtime environment's > >

Re: [U-Boot] [PATCH v3] Fix board init code to use a valid C runtime environment

2015-11-14 Thread Albert ARIBAUD
Hello Thomas, On Sat, 14 Nov 2015 14:23:31 +0800, Thomas Chou <tho...@wytron.com.tw> wrote: > Hi Albert, > > On 2015年11月13日 22:43, Albert ARIBAUD wrote: > > +/* > > + * Initialize reserved space (which has been safely allocated on the C > > + * stack from the C

Re: [U-Boot] [PATCH] ARM: crt0: Pass malloc base address

2015-11-14 Thread Albert ARIBAUD
Hello Simon, On Fri, 13 Nov 2015 19:12:13 -0700, Simon Glass wrote: > > IMO, keeping it simple does not play well with making DM and malloc > > available pre-SDRAM init :) --- but don't miscontrue my meaning: I'm > > fine with DM and malloc pre-SDRAM on platforms which have

[U-Boot] [PATCH v3] Fix board init code to use a valid C runtime environment

2015-11-13 Thread Albert ARIBAUD
board_init_f_mem() alters the C runtime environment's stack it ls actually already using. This is not a valid C runtime environment. Split board_init_f_mem into C functions which do not alter their own stack and therefore run in a valid C runtime environment. Signed-off-by: Albert ARIBAUD

Re: [U-Boot] [PATCH v2] Fix board init code to use a valid C runtime environment

2015-11-13 Thread Albert ARIBAUD
Hello Bin, On Fri, 13 Nov 2015 17:08:22 +0800, Bin Meng wrote: > Hi Albert, > > > + subl%esp, %eax > > I guess you are confused by the AT assembly syntax. This should be: > subl %eax, %esp. > > + subl%esp, %eax > > Ditto. Thanks -- last time I touched

Re: [U-Boot] [PATCH v2] Fix board init code to use a valid C runtime environment

2015-11-12 Thread Albert ARIBAUD
Hello Thomas, On Thu, 12 Nov 2015 16:28:38 +0800, Thomas Chou <tho...@wytron.com.tw> wrote: > Hi Albert, > > On 2015年11月12日 15:17, Albert ARIBAUD wrote: > >> > >> diff --git a/commo

Re: [U-Boot] QSPI XIP boot on am437x

2015-11-12 Thread Albert ARIBAUD
Hello Vignesh, On Fri, 13 Nov 2015 11:22:19 +0530, Vignesh R <vigne...@ti.com> wrote: > Hi, > > On 11/11/2015 02:33 PM, Albert ARIBAUD wrote: > > [...] > > > Alternatively, you could test the patch at > > > > http://patchwork.ozlabs.org/patch/5

Re: [U-Boot] [PATCH v2] Fix board init code to use a valid C runtime environment

2015-11-12 Thread Albert ARIBAUD
Hello Thomas, On Thu, 12 Nov 2015 13:59:28 +0800, Thomas Chou wrote: > Hi Albert, > - /* Update stack- and frame-pointers */ > - mov sp, r2 > - mov fp, sp Just a sec here on second thought. I understand the 'mov sp, r2' must be removed as it is

  1   2   3   4   5   6   7   8   9   10   >