Re: [U-Boot] [PATCH] efi_loader: Fix disk dp's for pre-DM/legacy devices

2017-10-09 Thread Jonathan Gray
On Mon, Oct 09, 2017 at 01:06:26PM -0400, Rob Clark wrote: > On Mon, Oct 9, 2017 at 12:41 PM, Jonathan Gray <j...@jsg.id.au> wrote: > > On Mon, Oct 09, 2017 at 12:06:24PM -0400, Rob Clark wrote: > >> On Mon, Oct 9, 2017 at 11:35 AM, Jonathan Gray <j...@jsg.id.au> wrot

Re: [U-Boot] [PATCH] efi_loader: Fix disk dp's for pre-DM/legacy devices

2017-10-09 Thread Jonathan Gray
On Mon, Oct 09, 2017 at 12:06:24PM -0400, Rob Clark wrote: > On Mon, Oct 9, 2017 at 11:35 AM, Jonathan Gray <j...@jsg.id.au> wrote: > > On Mon, Oct 09, 2017 at 10:17:18AM -0400, Rob Clark wrote: > >> On Mon, Oct 9, 2017 at 8:25 AM, Jonathan Gray <j...@jsg.id.au> wrot

Re: [U-Boot] [PATCH] efi_loader: Fix disk dp's for pre-DM/legacy devices

2017-10-09 Thread Jonathan Gray
On Mon, Oct 09, 2017 at 10:17:18AM -0400, Rob Clark wrote: > On Mon, Oct 9, 2017 at 8:25 AM, Jonathan Gray <j...@jsg.id.au> wrote: > > On Mon, Oct 09, 2017 at 06:52:18AM -0400, Rob Clark wrote: > >> On Sun, Oct 8, 2017 at 11:33 PM, Jonathan Gray <j...@jsg.id.au> wrot

[U-Boot] [PATCH] fs/fat: fix unaligned access regression

2017-10-09 Thread Jonathan Gray
: v7_outer_cache_inval_range - stop address is not aligned - 0x8f89e468 ... Switching the malloc() calls to malloc_cache_aligned() avoids the alignment errors and allows booting to continue. Signed-off-by: Jonathan Gray <j...@jsg.id.au> --- fs/fat/fat.c | 6 +++--- 1 file changed, 3 insertions

Re: [U-Boot] [PATCH] efi_loader: Fix disk dp's for pre-DM/legacy devices

2017-10-09 Thread Jonathan Gray
On Mon, Oct 09, 2017 at 06:52:18AM -0400, Rob Clark wrote: > On Sun, Oct 8, 2017 at 11:33 PM, Jonathan Gray <j...@jsg.id.au> wrote: > > On Sun, Oct 08, 2017 at 11:33:08AM -0400, Rob Clark wrote: > >> This fixes an issue with OpenBSD's bootloader, and I think should also

Re: [U-Boot] [PATCH] disk: part_dos: Use the original allocation scheme for the SPL case

2017-10-09 Thread Jonathan Gray
On Sun, Oct 08, 2017 at 11:12:28PM -0400, Tom Rini wrote: > On Sun, Oct 08, 2017 at 10:56:26PM -0300, Fabio Estevam wrote: > > On Sun, Oct 8, 2017 at 11:04 AM, Jonathan Gray <j...@jsg.id.au> wrote: > > > > > It turns out to be > > > > > >

Re: [U-Boot] [PATCH] efi_loader: Fix disk dp's for pre-DM/legacy devices

2017-10-08 Thread Jonathan Gray
arent device. > > Reported-by: Jonathan Gray <j...@jsg.id.au> > Signed-off-by: Rob Clark <robdcl...@gmail.com> Thanks for looking into this. While this lets armv7/bootarm.efi boot again on cubox-i and bbb it doesn't help rpi3. What is the easiest way to get U-Boot to display

Re: [U-Boot] [PATCH] disk: part_dos: Use the original allocation scheme for the SPL case

2017-10-08 Thread Jonathan Gray
On Sun, Oct 08, 2017 at 09:56:15AM -0300, Fabio Estevam wrote: > On Sat, Oct 7, 2017 at 10:06 PM, Jonathan Gray <j...@jsg.id.au> wrote: > > > I wonder if you have the same kind of fat filesystem. > > No, I have only used ext4 file system. > > > Can you try wr

Re: [U-Boot] [PATCH] disk: part_dos: Use the original allocation scheme for the SPL case

2017-10-07 Thread Jonathan Gray
On Sat, Oct 07, 2017 at 02:46:04PM -0300, Fabio Estevam wrote: > Hi Jonathan, > > On Thu, Oct 5, 2017 at 1:36 AM, Jonathan Gray <j...@jsg.id.au> wrote: > > > This does not seem to be a complete fix, cubox is still broken when > > U-Boot proper loads, unless the e

Re: [U-Boot] [PATCH] disk: part_dos: Use the original allocation scheme for the SPL case

2017-10-07 Thread Jonathan Gray
On Sat, Oct 07, 2017 at 08:23:31AM -0400, Rob Clark wrote: > On Fri, Oct 6, 2017 at 10:08 PM, Jonathan Gray <j...@jsg.id.au> wrote: > > On Fri, Oct 06, 2017 at 08:21:26AM -0400, Rob Clark wrote: > >> On Fri, Oct 6, 2017 at 12:35 AM, Jonathan Gray <j...@jsg.id.au> wrot

Re: [U-Boot] [PATCH] disk: part_dos: Use the original allocation scheme for the SPL case

2017-10-06 Thread Jonathan Gray
On Fri, Oct 06, 2017 at 08:21:26AM -0400, Rob Clark wrote: > On Fri, Oct 6, 2017 at 12:35 AM, Jonathan Gray <j...@jsg.id.au> wrote: > > On Thu, Oct 05, 2017 at 05:05:49AM -0400, Rob Clark wrote: > >> On Thu, Oct 5, 2017 at 12:36 AM, Jonathan Gray <j...@jsg.id.au> wrot

Re: [U-Boot] [PATCH] disk: part_dos: Use the original allocation scheme for the SPL case

2017-10-05 Thread Jonathan Gray
On Thu, Oct 05, 2017 at 05:05:49AM -0400, Rob Clark wrote: > On Thu, Oct 5, 2017 at 12:36 AM, Jonathan Gray <j...@jsg.id.au> wrote: > > On Wed, Oct 04, 2017 at 01:12:48PM -0400, Rob Clark wrote: > >> On Wed, Oct 4, 2017 at 12:29 PM, Fabio Estevam <fabio.este...@nxp.com

Re: [U-Boot] [PATCH] disk: part_dos: Use the original allocation scheme for the SPL case

2017-10-04 Thread Jonathan Gray
On Wed, Oct 04, 2017 at 01:12:48PM -0400, Rob Clark wrote: > On Wed, Oct 4, 2017 at 12:29 PM, Fabio Estevam wrote: > > Since commit ff98cb90514d ("part: extract MBR signature from partitions") > > SPL boot on i.MX6 starts to fail: > > > > U-Boot SPL 2017.09-00221-g0d6ab32

Re: [U-Boot] [PATCH] Revert "dm: arm: rpi: Drop CONFIG_OF_EMBED"

2017-09-06 Thread Jonathan Gray
U-Boot with CONFIG_OF_EMBED=y on rpi* since Mark Kettenis tracked it down to that commit. Tested-by: Jonathan Gray <j...@jsg.id.au> > --- > configs/rpi_2_defconfig | 1 + > configs/rpi_3_32b_defconfig | 1 + > configs/rpi_3_defconfig | 1 + > configs/rpi_defconfig

Re: [U-Boot] [ANN] U-Boot v2017.09-rc3 released

2017-08-29 Thread Jonathan Gray
On Mon, Aug 28, 2017 at 10:56:52PM -0400, Tom Rini wrote: > Hey all, > > It's release day and v2017.09-rc3 is out. I think there's still a few > outstanding important bugfixes that need to come in, and probably a few > more Kconfig migrations as well. The tag/changes not pushed to git yet? >

[U-Boot] [PATCH] bch: don't use __BSD_VISIBLE to test for fls

2017-08-27 Thread Jonathan Gray
and __FreeBSD__ to unbreak the build on OpenBSD. Signed-off-by: Jonathan Gray <j...@jsg.id.au> --- lib/bch.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/bch.c b/lib/bch.c index 7a2d9d3b86..d0d7e25c4e 100644 --- a/lib/bch.c +++ b/lib/bch.c @@ -117,7 +117,7 @@

Re: [U-Boot] U-boot 2017.07 not working on Raspberry Pi

2017-08-25 Thread Jonathan Gray
On Fri, Aug 25, 2017 at 10:46:55AM +0100, Paul Barker wrote: > On Fri, Aug 25, 2017 at 2:46 AM, Simon Glass wrote: > > Hi, > > > > On 20 August 2017 at 20:59, Simon Glass wrote: > >> Hi Paul, > >> > >> On 3 August 2017 at 11:42, Tom Rini

Re: [U-Boot] [PATCH] rpi: Enable USB keyboard support

2017-08-24 Thread Jonathan Gray
config but was with CONFIG_DM_KEYBOARD=y added. With CONFIG_DM_KEYBOARD=y having both HDMI and serial connected, input from serial and USB keyboard is output to both HDMI and serial as expected. Tested-by: Jonathan Gray <j...@jsg.id.au> > > Signed-off-by: Simon Glass <s...@chromium.org

Re: [U-Boot] [PATCH v3] Makefile: honor PYTHON configuration properly

2017-08-14 Thread Jonathan Gray
d from Python 3. > > Note on the libfdt imports: "@if ! PYTHONPATH=tools $(PYTHON) -c 'import > libfdt'; then..." is probably simpler than the currently sub-optimal > pipe. Reviewed-by: Jonathan Gray <j...@jsg.id.au> This is also helpful for OpenBSD where pyth

Re: [U-Boot] [PATCH] Makefile: honor PYTHON configuration properly

2017-08-13 Thread Jonathan Gray
On Sun, Aug 13, 2017 at 11:04:03PM +0200, Cl??ment B??sch wrote: > --- > Makefile | 2 +- > tools/Makefile | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) There is a similiar fragment in scripts/Makefile.spl with the checkdtoc target that should also be fixed. checkdtoc: tools

Re: [U-Boot] [RFC] efi_loader: Add test to boot OpenBSD's efi bootloader

2017-08-09 Thread Jonathan Gray
On Sun, Aug 06, 2017 at 03:06:17PM -0400, Rob Clark wrote: > On Sun, Aug 6, 2017 at 2:54 PM, Mark Kettenis wrote: > >> From: Rob Clark > >> Date: Sun, 6 Aug 2017 12:10:28 -0400 > >> > >> Signed-off-by: Rob Clark > >> --- > >>

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

2017-08-07 Thread Jonathan Gray
On Mon, Aug 07, 2017 at 12:16:54PM -0400, Rob Clark wrote: > On Mon, Aug 7, 2017 at 11:47 AM, Jonathan Gray <j...@jsg.id.au> wrote: > > On Sun, Aug 06, 2017 at 11:34:15AM -0400, Rob Clark wrote: > >> On Sun, Aug 6, 2017 at 10:45 AM, Rob Clark <robdcl...@gmail.com> wro

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

2017-08-07 Thread Jonathan Gray
On Sun, Aug 06, 2017 at 11:34:15AM -0400, Rob Clark wrote: > On Sun, Aug 6, 2017 at 10:45 AM, Rob Clark wrote: > > > > I've started trying to hack up test_efi_loader.py to add a test that > > loads OpenBSD's bootloader.. kinda muddling through it at this point, > > since not

Re: [U-Boot] [PATCH v0 13/20] efi_loader: use proper device-paths for partitions

2017-08-06 Thread Jonathan Gray
On Sun, Aug 06, 2017 at 11:34:15AM -0400, Rob Clark wrote: > On Sun, Aug 6, 2017 at 10:45 AM, Rob Clark wrote: > > > > I've started trying to hack up test_efi_loader.py to add a test that > > loads OpenBSD's bootloader.. kinda muddling through it at this point, > > since not

Re: [U-Boot] U-boot 2017.07 not working on Raspberry Pi

2017-08-03 Thread Jonathan Gray
On Wed, Aug 02, 2017 at 01:13:30PM +0100, Paul Barker wrote: > I'm currently testing U-boot 2017.07 on Raspberry Pi, both on a Model > B+ and a Raspberry Pi 3. On both devices I'm seeing two issues with > u-boot: > > * U-boot can't boot the Linux kernel, it just hangs at "Starting kernel ...". >

Re: [U-Boot] [PATCH] bch: Fix build on FreeBSD host

2017-07-19 Thread Jonathan Gray
On Tue, Jun 20, 2017 at 09:02:29AM +0200, Emmanuel Vadot wrote: > endian.h on FreeBSD system exist in sys/ subdirectory. > FreeBSD already have a fls function defined in strings.h which is included > in string.h if __BSD_VISIBLE is defined, as a check for this. This broke the build of u-boot on

Re: [U-Boot] [PATCH 2/2] mx6cuboxi: Add support for sata

2017-05-15 Thread Jonathan Gray
On Mon, May 15, 2017 at 06:24:46PM +0100, Peter Robinson wrote: > The Cubox-i and Hummingboard series of devices have an option of > SATA on board, and depending on how the fuses are blown even the > option to boot SPL from SATA. So enable support for it so it can > be used to boot the OS from if

[U-Boot] [PATCH 2/2] efi_loader: check CreateEvent() parameters

2017-03-12 Thread Jonathan Gray
Add some of the invalid parameter checks described in the UEFI specification for CreateEvent(). This does not include checking the validity of the type and tpl parameters. Signed-off-by: Jonathan Gray <j...@jsg.id.au> --- lib/efi_loader/efi_boottime.c | 10 ++ 1 file chang

[U-Boot] [PATCH 1/2] efi_loader: run CreateEvent() notify function based on flags

2017-03-12 Thread Jonathan Gray
. Signed-off-by: Jonathan Gray <j...@jsg.id.au> --- include/efi_api.h | 3 +++ lib/efi_loader/efi_boottime.c | 4 +++- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/include/efi_api.h b/include/efi_api.h index 5c3836a51b..f071b36b53 100644 --- a/include/efi_api.h

Re: [U-Boot] [linux-sunxi] Re: [PATCH 14/17] sunxi: Pine64: defconfig: enable SPL FIT support

2017-03-04 Thread Jonathan Gray
On Fri, Mar 03, 2017 at 09:55:25AM +, Andre Przywara wrote: > Hi, > > On 03/03/17 09:22, Maxime Ripard wrote: > > On Thu, Mar 02, 2017 at 12:03:20AM +0800, Icenowy Zheng wrote: > >> > >> 2017???3???1??? 23:51??? Maxime Ripard > >> ? > >>> > >>> Hi

Re: [U-Boot] [PATCH] binman: Explicitly request python2 instead of python from env

2017-02-22 Thread Jonathan Gray
On Mon, Feb 20, 2017 at 07:41:34PM +0100, Paul Kocialkowski wrote: > We now live in a world where python cannot be assumed to be python2. > As a matter of fact, it is no longer the default for python on many > GNU/Linux distributions. > > Running binman with python3 fails, so explicitly request

Re: [U-Boot] [PATCH 1/2] rsa: Fix build with OpenSSL 1.1.x

2017-02-15 Thread Jonathan Gray
These version tests should be #if (OPENSSL_VERSION_NUMBER < 0x1010L) || defined(LIBRESSL_VERSION_NUMBER) or better yet have tests based on functionality rather than version. opensslv.h on OpenBSD-current/LibreSSL portable master has /* $OpenBSD: opensslv.h,v 1.39 2017/02/14 03:50:25 bcook

Re: [U-Boot] am335x: musb: mass storage device issue

2017-02-13 Thread Jonathan Gray
On Mon, Feb 13, 2017 at 04:02:31PM +0100, Yegor Yefremov wrote: > On Mon, Feb 13, 2017 at 3:17 PM, Belisko Marek > wrote: > > Hi Yegor, > > > > On Mon, Feb 13, 2017 at 12:57 PM, Yegor Yefremov > > wrote: > >> > >> My am335x based board

Re: [U-Boot] [PATCH] relocate-rela: use compiler.h endian macros

2016-12-23 Thread Jonathan Gray
Any objections to this? On Sun, Dec 11, 2016 at 02:51:13PM +1100, Jonathan Gray wrote: > Use the endian macros from u-boot's compiler.h instead of duplicating > the definitions. > > This also avoids a build error on OpenBSD by removing swap64 which > collides with a system definit

[U-Boot] [PATCH] relocate-rela: use compiler.h endian macros

2016-12-10 Thread Jonathan Gray
Use the endian macros from u-boot's compiler.h instead of duplicating the definitions. This also avoids a build error on OpenBSD by removing swap64 which collides with a system definition in endian.h pulled in by inttypes.h. Signed-off-by: Jonathan Gray <j...@jsg.id.au> --- tools/re

[U-Boot] [PATCH] compiler.h: use u-boot endian macros on OpenBSD

2016-11-25 Thread Jonathan Gray
and define glibc __BYTE_ORDER __BIG_ENDIAN __LITTLE_ENDIAN names, as at least some parts of the non-cross build assumes those names are present (ie crc32.c). Signed-off-by: Jonathan Gray <j...@jsg.id.au> --- include/compiler.h | 18 -- 1 file changed, 4 insertions(+), 14 del

[U-Boot] kwbimage use after free

2016-10-21 Thread Jonathan Gray
I didn't see a dedicated list to send bug reports so sending it here: There is a use after free in kwbimage, found by building u-boot with the use after free detection enabled with OpenBSD's malloc. When building the clearfog target: MKIMAGE u-boot-spl.kwb Segmentation fault (core dumped)

Re: [U-Boot] [PATCH] Revert "image-fit: switch ENOLINK to ENOENT"

2016-09-18 Thread Jonathan Gray
On Sun, Sep 18, 2016 at 03:14:30PM +0200, Marek Vasut wrote: > On 09/17/2016 05:42 AM, Jonathan Gray wrote: > > On Fri, Sep 16, 2016 at 10:21:43PM +0100, Paul Burton wrote: > >> Commit bac17b78dace ("image-fit: switch ENOLINK to ENOENT") changed > >> fit_get_no

Re: [U-Boot] [PATCH] Revert "image-fit: switch ENOLINK to ENOENT"

2016-09-16 Thread Jonathan Gray
INK to > ENOENT"). > > Signed-off-by: Paul Burton <paul.bur...@imgtec.com> > Cc: Jonathan Gray <j...@jsg.id.au> > > --- > > common/image-fit.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/common/image-fit.c b/common/

Re: [U-Boot] [PATCH] image-fit: switch ENOLINK to ENOENT

2016-09-15 Thread Jonathan Gray
return 0; > else if (rd_noffset < 0) > return 1; > > So, yes, 2016.09 breaks Armada, apparently. The callers of the proposed/rejected kwboot.c errno diff were all checked but the test of the result in the caller of this one seems to have bee

Re: [U-Boot] image-fit: switch ENOLINK to ENOENT

2016-09-08 Thread Jonathan Gray
On Thu, Sep 08, 2016 at 11:17:12AM -0400, Tom Rini wrote: > On Thu, Sep 08, 2016 at 11:53:48PM +1000, Jonathan Gray wrote: > > On Thu, Sep 08, 2016 at 09:15:45AM -0400, Tom Rini wrote: > > > On Thu, Sep 08, 2016 at 11:06:34PM +1000, Jonathan Gray wrote: > > > > On T

Re: [U-Boot] image-fit: switch ENOLINK to ENOENT

2016-09-08 Thread Jonathan Gray
On Thu, Sep 08, 2016 at 09:15:45AM -0400, Tom Rini wrote: > On Thu, Sep 08, 2016 at 11:06:34PM +1000, Jonathan Gray wrote: > > On Thu, Sep 08, 2016 at 08:48:53AM -0400, Tom Rini wrote: > > > On Thu, Sep 08, 2016 at 10:01:52PM +1000, Jonathan Gray wrote: > > > > On W

Re: [U-Boot] image-fit: switch ENOLINK to ENOENT

2016-09-08 Thread Jonathan Gray
On Thu, Sep 08, 2016 at 08:48:53AM -0400, Tom Rini wrote: > On Thu, Sep 08, 2016 at 10:01:52PM +1000, Jonathan Gray wrote: > > On Wed, Sep 07, 2016 at 02:00:19PM -0400, Tom Rini wrote: > > > On Sat, Sep 03, 2016 at 08:30:14AM +1000, Jonathan Gray wrote: > > > >

Re: [U-Boot] image-fit: switch ENOLINK to ENOENT

2016-09-08 Thread Jonathan Gray
On Wed, Sep 07, 2016 at 02:00:19PM -0400, Tom Rini wrote: > On Sat, Sep 03, 2016 at 08:30:14AM +1000, Jonathan Gray wrote: > > > ENOLINK is not required by POSIX and does not exist on OpenBSD > > and likely other systems. > > > > Signed-off-by: Jonathan Gray <

[U-Boot] [PATCH] image-fit: switch ENOLINK to ENOENT

2016-09-03 Thread Jonathan Gray
ENOLINK is not required by POSIX and does not exist on OpenBSD and likely other systems. Signed-off-by: Jonathan Gray <j...@jsg.id.au> --- common/image-fit.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/common/image-fit.c b/common/image-fit.c index d8d4e95..79c0375

[U-Boot] [PATCH] compiler.h: use system endian macros on OpenBSD

2016-09-03 Thread Jonathan Gray
The u-boot endian macros map directly to system endian macros on OpenBSD. Signed-off-by: Jonathan Gray <j...@jsg.id.au> --- include/compiler.h | 19 +-- 1 file changed, 17 insertions(+), 2 deletions(-) diff --git a/include/compiler.h b/include/compiler.h index 47c296e..6

<    1   2