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:
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>
' 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
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:
> >
&
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
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
_
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
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,
&
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
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
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
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/
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
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
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
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
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
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
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:
> > >
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
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
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:
> > >
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:
> > >
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
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
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
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
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>
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
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
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
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
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
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
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
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 +
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 -
>
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
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
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
>
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.
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.
>
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
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
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
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
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
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.
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
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.
>
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
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:
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
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
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
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
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
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
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
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
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
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
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>
--
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
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>
---
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
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
ptimized out.
> > > > > >
> > > > > > Signed-off-by: Marek Vasut <ma...@denx.de>
> > > > > > Cc: Dinh Nguyen <dingu...@opensource.altera.com>
> > > > > > Cc: Albert Aribaud <albert.u.b...@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
(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
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
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
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
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
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:
> >
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
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
.
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
, 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
.
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
, 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
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
... 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
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
.
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
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
>
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
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
> &
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
>
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
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
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
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
> >
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
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
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
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
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
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
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 - 100 of 4508 matches
Mail list logo