[U-Boot] [PATCH][v2] driver: net: fsl-mc: Memset MC reserve ram memory before usage

2018-08-26 Thread Prabhakar Kushwaha
It is required to clean Management Complex reserve memory before any usage. So memset MC reserve memory. Signed-off-by: Prabhakar Kushwaha --- Changes for v2: - updated patch subject - Consider case of dpl apply and mc_boot failure drivers/net/fsl-mc/mc.c | 7 +++ 1 file ch

Re: [U-Boot] [PATCH 11/25] clk: Allow clock defaults to be set also during re-reloc state

2018-08-26 Thread Lokesh Vutla
Hi Philipp, On Friday 24 August 2018 08:12 PM, Dr. Philipp Tomsich wrote: > +Kever > >> On 24 Aug 2018, at 16:12, Tom Rini wrote: >> >> On Tue, Aug 21, 2018 at 08:01:49PM +0530, Lokesh Vutla wrote: >> >>> From: Andreas Dannenberg >>> >>> The earlier commit f4fcba5c5ba ("clk: implement clk_set_d

[U-Boot] mtd: spi: Full DM conversion timeline

2018-08-26 Thread Ashish Kumar
Hello Jagan, > -Original Message- > From: Jagan Teki [mailto:jagannadh.t...@gmail.com] > Sent: Monday, May 14, 2018 3:18 PM > To: Ashish Kumar > Cc: Jagan Teki ; York Sun > ; Prabhakar Kushwaha > ; Yogesh Narayan Gaur > ; Poonam Aggrwal > ; u-boot@lists.denx.de > Subject: Re: mtd: spi-nor

Re: [U-Boot] [PATCH 11/25] clk: Allow clock defaults to be set also during re-reloc state

2018-08-26 Thread Kever Yang
Hi Philipp, Andreas, On 08/25/2018 12:00 AM, Dr. Philipp Tomsich wrote: >> On 24 Aug 2018, at 17:54, Andreas Dannenberg wrote: >> >> Philipp, >> >> On Fri, Aug 24, 2018 at 04:42:15PM +0200, Dr. Philipp Tomsich wrote: >>> +Kever >>> On 24 Aug 2018, at 16:12, Tom Rini wrote: On Tue

[U-Boot] [PATCH v2 02/13] mmc: Fix signed shift overflow

2018-08-26 Thread Eugeniu Rosca
Below is reproduced both with sandbox and R-Car Gen3 arm64 U-Boot: === UBSAN: Undefined behaviour in drivers/mmc/mmc.c:1147:21 left shift of 1 by 31 places cannot be represented in type 'int' ==

[U-Boot] [PATCH v2 10/13] mmc: Fix read-past-end-of-array

2018-08-26 Thread Eugeniu Rosca
Running "mmc dev 0" on R-Car H3 Salvator-X (CONFIG_UBSAN=y) occasionally results in: => mmc dev 0 = UBSAN: Undefined behaviour in drivers/mmc/mmc.c:2233:14 index 7 is out of range for type 'int [4]' ===

[U-Boot] [PATCH v2 01/13] UBSAN: run-time undefined behavior sanity checker

2018-08-26 Thread Eugeniu Rosca
Import Undefined Behavior SANitizer from Linux Kernel v4.18, as implemented by Andrey Ryabinin . Roughly, the UBSAN development history in Linux kernel looks like: v4.18 3ca17b1f3628 ("lib/ubsan: remove null-pointer checks") v4.17-rc1 317506009216 ("lib/test_ubsan.c: make test_ubsan_misaligne

[U-Boot] [PATCH v2 00/13] Import Undefined Behavior Sanitizer

2018-08-26 Thread Eugeniu Rosca
While certain classes of bugs (e.g. locking related) are totally irrelevant for U-Boot, undefined behavior is something U-Boot may experience all over the place and this certainly can lead to hidden and difficult to debug issues. As of v4.18, Linux kernel contains roughly 119 UBSAN fixes [1], so i

Re: [U-Boot] [PATCH v2 06/13] net: ravb: Fix signed shift overflow

2018-08-26 Thread Marek Vasut
On 08/27/2018 01:13 AM, Eugeniu Rosca wrote: > Running "tftp" on R-Car H3 Salvator-X with CONFIG_UBSAN=y results in: > > => tftp > = > UBSAN: Undefined behaviour in drivers/net/ravb.c:237:28 > left shift of 10 by 28 places cann

[U-Boot] [PATCH v2 12/13] input: Fix zero-sized array

2018-08-26 Thread Eugeniu Rosca
Fix below UBSAN reports, thrown on sandbox: UBSAN: Undefined behaviour in drivers/input/input.c:512:7 variable length array bound value 0 <= 0

[U-Boot] [PATCH v2 08/13] disk: part_dos: Fix signed shift overflow

2018-08-26 Thread Eugeniu Rosca
Fix the following UBSAN report: UBSAN: Undefined behaviour in disk/part_dos.c:30:22 left shift of 209 by 24 places cannot be represented in type 'int' Steps

[U-Boot] [PATCH v2 13/13] configs: sandbox*: Enable UBSAN

2018-08-26 Thread Eugeniu Rosca
On Sun, Aug 19, 2018 at 09:51:32PM -0400, Tom Rini wrote: > [..] we should be able to say more broadly that just about everyone > can enable this, but only out of the box sandbox should. Hence, turn UBSAN on for every available sandbox flavor. Make sure the inserted line complies with `make savede

[U-Boot] [PATCH v2 07/13] x86: Fix signed shift overflow in MSR_IA32_APICBASE_BASE

2018-08-26 Thread Eugeniu Rosca
Fix the following UBSAN report: == UBSAN: Undefined behaviour in arch/x86/cpu/lapic.c:73:14 left shift of 1048575 by 12 places cannot be represented in type 'int'

[U-Boot] [PATCH v2 05/13] net: phy: Fix signed shift overflow

2018-08-26 Thread Eugeniu Rosca
Booting R-Car Gen3 arm64 U-Boot with CONFIG_UBSAN=y results in: = UBSAN: Undefined behaviour in drivers/net/phy/phy.c:728:19 left shift of 1 by 31 places cannot be represented in type 'int'

[U-Boot] [PATCH v2 11/13] hashtable: Fix zero-sized array

2018-08-26 Thread Eugeniu Rosca
Enabling CONFIG_UBSAN=y, below runtime warning occurs both in sandbox and R-Car H3 Salvator-X U-Boot: = UBSAN: Undefined behaviour in lib/hashtable.c:784:8 variable length array bound value 0 <= 0 =

[U-Boot] [PATCH v2 09/13] common.h: Fix signed shift overflow in cpumask_next()

2018-08-26 Thread Eugeniu Rosca
Fix the following UBSAN report: = UBSAN: Undefined behaviour in include/common.h:322:19 left shift of 1 by 31 places cannot be represented in type 'int' = Steps to re

[U-Boot] [PATCH v2 06/13] net: ravb: Fix signed shift overflow

2018-08-26 Thread Eugeniu Rosca
Running "tftp" on R-Car H3 Salvator-X with CONFIG_UBSAN=y results in: => tftp = UBSAN: Undefined behaviour in drivers/net/ravb.c:237:28 left shift of 10 by 28 places cannot be represented in type 'int'

[U-Boot] [PATCH v2 04/13] pinctrl: renesas: Fix signed shift overflow

2018-08-26 Thread Eugeniu Rosca
Booting R-Car H3-Salvator-X (CONFIG_UBSAN=y) consistently results in: = UBSAN: Undefined behaviour in drivers/pinctrl/renesas/pfc.c:402:40 left shift of 1 by 31 places cannot be represented in type 'int' ==

[U-Boot] [PATCH v2 03/13] armv8: mmu: Fix signed shift overflow

2018-08-26 Thread Eugeniu Rosca
Fix the following UBSAN warnings: --8<- CPU: Renesas Electronics R8A7795 rev 2.0 Model: Renesas Salvator-X board based on r8a7795 ES2.0+ UBSAN: Undefined behaviour in arch/arm/cpu/armv8/cache_v8.c:72:9 left shift of 1

Re: [U-Boot] [PATCH 11/15] efi_loader: capitalization table

2018-08-26 Thread Alexander Graf
On 26.08.18 21:00, Heinrich Schuchardt wrote: > On 08/26/2018 08:22 PM, Alexander Graf wrote: >> >> >> On 11.08.18 17:28, Heinrich Schuchardt wrote: >>> This patch provides a define to initialize a table that maps lower to >>> capital letters for Unicode code point 0x - 0x. >>> >>> Signed

Re: [U-Boot] [PATCH 07/15] efi_loader: remove limit on variable length

2018-08-26 Thread Alexander Graf
On 26.08.18 20:40, Heinrich Schuchardt wrote: > On 08/26/2018 08:13 PM, Alexander Graf wrote: >> >> >> On 11.08.18 17:28, Heinrich Schuchardt wrote: >>> The EFI spec does not provide a length limit for variables. >>> >>> Signed-off-by: Heinrich Schuchardt >>> --- >>> lib/efi_loader/efi_variable

Re: [U-Boot] [PATCH 05/15] lib: vsprintf: correct printing of Unicode strings

2018-08-26 Thread Alexander Graf
On 26.08.18 20:34, Heinrich Schuchardt wrote: > On 08/26/2018 08:05 PM, Alexander Graf wrote: >> >> >> On 11.08.18 17:28, Heinrich Schuchardt wrote: >>> The width and precision of the printf() function refer to the number of >>> characters not to the number of bytes printed. >>> >>> Signed-off-by

Re: [U-Boot] [PATCH 02/15] efi_loader: rename utf16_strlen, utf16_strnlen

2018-08-26 Thread Alexander Graf
On 26.08.18 21:36, Heinrich Schuchardt wrote: > On 08/26/2018 08:33 PM, Alexander Graf wrote: >> >> >> On 26.08.18 20:21, Heinrich Schuchardt wrote: >>> On 08/26/2018 07:52 PM, Alexander Graf wrote: On 11.08.18 17:28, Heinrich Schuchardt wrote: > The function names utf16_strlen

Re: [U-Boot] cmd: ubi: change 'default y' for SUNXI to 'imply' in Kconfig

2018-08-26 Thread Tom Rini
On Mon, Jul 09, 2018 at 03:12:37PM +0900, Masahiro Yamada wrote: > It is not preferred to put SUNXI-specific code in the common place. > > Change it to 'imply' property of ARCH_SUNXI. > > Signed-off-by: Masahiro Yamada Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP

Re: [U-Boot] configs: am57xx: change default board name to beagle_x15

2018-08-26 Thread Tom Rini
On Tue, Aug 21, 2018 at 11:47:49PM -0500, Praneeth Bajjuri wrote: > beagleboard x15 is the first supported platform variant of am57xx > in AOSP (android open source project) now. > > changing board name to stay in consistent with aosp target name. > > Signed-off-by: Praneeth Bajjuri Applied to

Re: [U-Boot] [PATCH] GPIO: omap_gpio: Fix gpio name names with device tree

2018-08-26 Thread Derald D. Woods
On Fri, Aug 17, 2018 at 02:37:58PM -0500, Adam Ford wrote: > The GPIO bank numbers do not appear in the device tree, so this > patch makes the gpio name based on the address > (ie gpio@49054000_31 vs gpio4_31) > > adam > > Signed-off-by: Adam Ford Works for omap3-evm and omap3-beagle-xm. OMAP3

Re: [U-Boot] [PATCH 02/15] efi_loader: rename utf16_strlen, utf16_strnlen

2018-08-26 Thread Heinrich Schuchardt
On 08/26/2018 08:33 PM, Alexander Graf wrote: > > > On 26.08.18 20:21, Heinrich Schuchardt wrote: >> On 08/26/2018 07:52 PM, Alexander Graf wrote: >>> >>> >>> On 11.08.18 17:28, Heinrich Schuchardt wrote: The function names utf16_strlen() and utf16_strnlen() are misnomers. The functions

Re: [U-Boot] [PATCH 11/15] efi_loader: capitalization table

2018-08-26 Thread Heinrich Schuchardt
On 08/26/2018 08:22 PM, Alexander Graf wrote: > > > On 11.08.18 17:28, Heinrich Schuchardt wrote: >> This patch provides a define to initialize a table that maps lower to >> capital letters for Unicode code point 0x - 0x. >> >> Signed-off-by: Heinrich Schuchardt >> --- >> MAINTAINERS

Re: [U-Boot] [PATCH 07/15] efi_loader: remove limit on variable length

2018-08-26 Thread Heinrich Schuchardt
On 08/26/2018 08:13 PM, Alexander Graf wrote: > > > On 11.08.18 17:28, Heinrich Schuchardt wrote: >> The EFI spec does not provide a length limit for variables. >> >> Signed-off-by: Heinrich Schuchardt >> --- >> lib/efi_loader/efi_variable.c | 52 --- >> 1 file c

Re: [U-Boot] [PATCH 06/15] test: test printing Unicode

2018-08-26 Thread Heinrich Schuchardt
On 08/26/2018 08:06 PM, Alexander Graf wrote: > > > On 11.08.18 17:28, Heinrich Schuchardt wrote: >> Test printing of Unicode strings >> >> Signed-off-by: Heinrich Schuchardt >> --- >> test/unicode_ut.c | 37 + >> 1 file changed, 37 insertions(+) >> >> diff -

Re: [U-Boot] [PATCH 02/15] efi_loader: rename utf16_strlen, utf16_strnlen

2018-08-26 Thread Alexander Graf
On 26.08.18 20:21, Heinrich Schuchardt wrote: > On 08/26/2018 07:52 PM, Alexander Graf wrote: >> >> >> On 11.08.18 17:28, Heinrich Schuchardt wrote: >>> The function names utf16_strlen() and utf16_strnlen() are misnomers. >>> The functions do not count utf-16 characters but non-zero words. >>> So

Re: [U-Boot] [PATCH 05/15] lib: vsprintf: correct printing of Unicode strings

2018-08-26 Thread Heinrich Schuchardt
On 08/26/2018 08:05 PM, Alexander Graf wrote: > > > On 11.08.18 17:28, Heinrich Schuchardt wrote: >> The width and precision of the printf() function refer to the number of >> characters not to the number of bytes printed. >> >> Signed-off-by: Heinrich Schuchardt >> --- >> lib/vsprintf.c | 22 +

Re: [U-Boot] [PATCH 14/15] efi_loader: EFI_UNICODE_COLLATION_PROTOCOL

2018-08-26 Thread Alexander Graf
On 11.08.18 17:28, Heinrich Schuchardt wrote: > The patch implements the EFI_UNICODE_COLLATION_PROTOCOL. > > Signed-off-by: Heinrich Schuchardt Can you please add documentation to each function? Alex ___ U-Boot mailing list U-Boot@lists.denx.de ht

Re: [U-Boot] [PATCH v9 16/18] efi: Add more debugging for memory allocations

2018-08-26 Thread Simon Glass
Hi Heinich, On 23 August 2018 at 14:49, Heinrich Schuchardt wrote: > > On 08/08/2018 11:54 AM, Simon Glass wrote: > > Add some more verbose debugging when doing memory allocations. This might > > help to find bugs later. > > > > Signed-off-by: Simon Glass > > --- > > > > Changes in v9: None > >

Re: [U-Boot] [PATCH 04/15] test: unit tests for Unicode functions

2018-08-26 Thread Heinrich Schuchardt
On 08/26/2018 08:02 PM, Alexander Graf wrote: > > > On 11.08.18 17:28, Heinrich Schuchardt wrote: >> Provide unit tests for Unicode functions. >> >> Signed-off-by: Heinrich Schuchardt >> --- >> MAINTAINERS | 1 + >> include/test/suites.h | 3 +- >> test/Kconfig | 8 + >>

Re: [U-Boot] [PATCH 11/15] efi_loader: capitalization table

2018-08-26 Thread Alexander Graf
On 11.08.18 17:28, Heinrich Schuchardt wrote: > This patch provides a define to initialize a table that maps lower to > capital letters for Unicode code point 0x - 0x. > > Signed-off-by: Heinrich Schuchardt > --- > MAINTAINERS |1 + > include/capitalization.h | 1909 ++

Re: [U-Boot] [PATCH 02/15] efi_loader: rename utf16_strlen, utf16_strnlen

2018-08-26 Thread Heinrich Schuchardt
On 08/26/2018 07:52 PM, Alexander Graf wrote: > > > On 11.08.18 17:28, Heinrich Schuchardt wrote: >> The function names utf16_strlen() and utf16_strnlen() are misnomers. >> The functions do not count utf-16 characters but non-zero words. >> So let's rename them to u16_strlen and u16_strnlen(). >>

Re: [U-Boot] [PATCH 09/15] efi_loader: buffer size for load options

2018-08-26 Thread Alexander Graf
On 11.08.18 17:28, Heinrich Schuchardt wrote: > The number of bytes in an utf-8 string is an upper limit for the number of > words in the equivalent utf-16 string. In so far the inumbant coding works > correctly. For non-ASCII characters the utf-16 string is shorter. With the > patch only the nec

Re: [U-Boot] [PATCH 08/15] efi_loader: don't use unlimited stack as buffer

2018-08-26 Thread Alexander Graf
On 11.08.18 17:28, Heinrich Schuchardt wrote: > The length of a string printed to the console by the > EFI_SIMPLE_TEXT_OUTPUT_PROTOCOL is not limited by the UEFI spec. > Hence should not allocate a buffer for it on the stack. > > Signed-off-by: Heinrich Schuchardt Reviewed-by: Alexander Graf

Re: [U-Boot] [PATCH 07/15] efi_loader: remove limit on variable length

2018-08-26 Thread Alexander Graf
On 11.08.18 17:28, Heinrich Schuchardt wrote: > The EFI spec does not provide a length limit for variables. > > Signed-off-by: Heinrich Schuchardt > --- > lib/efi_loader/efi_variable.c | 52 --- > 1 file changed, 30 insertions(+), 22 deletions(-) > > diff --git

Re: [U-Boot] [PATCH 01/15] lib: build charset.o only if needed

2018-08-26 Thread Heinrich Schuchardt
On 08/26/2018 07:45 PM, Alexander Graf wrote: > > > On 11.08.18 17:28, Heinrich Schuchardt wrote: >> charset.o is only needed for the EFI subsystem >> >> Signed-off-by: Heinrich Schuchardt >> --- >> lib/Makefile | 5 - >> lib/vsprintf.c | 12 >> 2 files changed, 12 insertion

Re: [U-Boot] [PATCH 06/15] test: test printing Unicode

2018-08-26 Thread Alexander Graf
On 11.08.18 17:28, Heinrich Schuchardt wrote: > Test printing of Unicode strings > > Signed-off-by: Heinrich Schuchardt > --- > test/unicode_ut.c | 37 + > 1 file changed, 37 insertions(+) > > diff --git a/test/unicode_ut.c b/test/unicode_ut.c > index 29316

Re: [U-Boot] [PATCH 05/15] lib: vsprintf: correct printing of Unicode strings

2018-08-26 Thread Alexander Graf
On 11.08.18 17:28, Heinrich Schuchardt wrote: > The width and precision of the printf() function refer to the number of > characters not to the number of bytes printed. > > Signed-off-by: Heinrich Schuchardt > --- > lib/vsprintf.c | 22 +- > 1 file changed, 13 insertions(+)

Re: [U-Boot] [PATCH 04/15] test: unit tests for Unicode functions

2018-08-26 Thread Alexander Graf
On 11.08.18 17:28, Heinrich Schuchardt wrote: > Provide unit tests for Unicode functions. > > Signed-off-by: Heinrich Schuchardt > --- > MAINTAINERS | 1 + > include/test/suites.h | 3 +- > test/Kconfig | 8 + > test/Makefile | 1 + > test/cmd_ut.c |

Re: [U-Boot] [PATCH 03/15] lib: charset: utility functions for Unicode

2018-08-26 Thread Alexander Graf
On 11.08.18 17:28, Heinrich Schuchardt wrote: > utf8_get() - get next UTF-8 code point from buffer > utf8_put() - write UTF-8 code point to buffer > utf8_utf16_strnlen() - length of a utf-8 string after conversion to utf-16 > utf8_utf16_strncpy() - copy a utf-8 string to utf-16 > utf16_get() - ge

Re: [U-Boot] [PATCH 02/15] efi_loader: rename utf16_strlen, utf16_strnlen

2018-08-26 Thread Alexander Graf
On 11.08.18 17:28, Heinrich Schuchardt wrote: > The function names utf16_strlen() and utf16_strnlen() are misnomers. > The functions do not count utf-16 characters but non-zero words. > So let's rename them to u16_strlen and u16_strnlen(). > > In utf16_dup() avoid assignment in if clause. > > S

Re: [U-Boot] [PATCH 01/15] lib: build charset.o only if needed

2018-08-26 Thread Alexander Graf
On 11.08.18 17:28, Heinrich Schuchardt wrote: > charset.o is only needed for the EFI subsystem > > Signed-off-by: Heinrich Schuchardt > --- > lib/Makefile | 5 - > lib/vsprintf.c | 12 > 2 files changed, 12 insertions(+), 5 deletions(-) > > diff --git a/lib/Makefile b/lib/

Re: [U-Boot] [PATCH v9 00/18] efi: Enable sandbox support for EFI loader

2018-08-26 Thread Alexander Graf
On 08.08.18 11:54, Simon Glass wrote: > A limitation of the EFI loader at present is that it does not build with > sandbox. This makes it hard to write tests, since sandbox is used for most > testing in U-Boot. > > This series enables the EFI loader feature. It allows sandbox to build and > run

Re: [U-Boot] [U-Boot, v9, 17/18] efi_loader: Pass address to fs_read()

2018-08-26 Thread Alexander Graf
> From: Alexander Graf > > The fs_read() function wants to get an address rather than the > pointer to a buffer. > > So let's convert the passed buffer from pointer back a the address > to make efi_loader on sandbox happier. > > Signed-off-by: Alexander Graf > Reviewed-by: Simon Glass > Signe

Re: [U-Boot] [U-Boot, v9, 15/18] efi: sandbox: Tidy up copy_fdt() to work with sandbox

2018-08-26 Thread Alexander Graf
> At present this function takes a pointer as its argument, then passes this > to efi_allocate_pages(), which actually takes an address. It uses casts, > which are not supported on sandbox. > > Also the function calculates the FDT size rounded up to the neared EFI > page size, then its caller reca

Re: [U-Boot] [U-Boot, v9, 14/18] efi: Relocate FDT to 127MB instead of 128MB

2018-08-26 Thread Alexander Graf
> Sandbox only has 128MB of memory so we cannot relocate the device tree up > to start at 128MB. Use 127MB instead, which should be safe. > > Signed-off-by: Simon Glass Thanks, applied to efi-next Alex ___ U-Boot mailing list U-Boot@lists.denx.de htt

Re: [U-Boot] [PATCH v9 16/18] efi: Add more debugging for memory allocations

2018-08-26 Thread Alexander Graf
On 08.08.18 11:54, Simon Glass wrote: > Add some more verbose debugging when doing memory allocations. This might > help to find bugs later. > > Signed-off-by: Simon Glass I think I asked in some earlier version whether you double checked that this patch does not increase the binary size of a

Re: [U-Boot] [PATCH v9 13/18] efi: Add a call to exit() along with why we can't use it

2018-08-26 Thread Alexander Graf
On 23.08.18 22:37, Heinrich Schuchardt wrote: > On 08/08/2018 11:54 AM, Simon Glass wrote: >> The test should exit like any other EFI application, by calling exit() >> in the boot services API. But this crashes at present on sandbox. For now, >> add in the placeholder code. >> >> Signed-off-by: S

Re: [U-Boot] [PATCH v9 12/18] sandbox: Enhance map_to_sysmem() to handle foreign pointers

2018-08-26 Thread Alexander Graf
On 08.08.18 11:54, Simon Glass wrote: > At present map_sysmem() maps an address into the sandbox RAM buffer, > return a pointer, while map_to_sysmem() goes the other way. > > The mapping is currently just 1:1 since a case was not found where a more > flexible mapping was needed. PCI does have a

Re: [U-Boot] [PATCH v9 10/18] sandbox: Try to start the RAM buffer at a particular address

2018-08-26 Thread Alexander Graf
On 08.08.18 11:54, Simon Glass wrote: > Use a starting address of 256MB which should be available. This helps to > make sandbox RAM buffers pointers more recognisable. > > Signed-off-by: Simon Glass Is this really necessary still? Alex > --- > > Changes in v9: None > Changes in v8: None >

Re: [U-Boot] [PATCH v9 09/18] sandbox: Align RAM buffer to the machine page size

2018-08-26 Thread Alexander Graf
On 08.08.18 11:54, Simon Glass wrote: > At present the sandbox RAM buffer is not aligned to any particular > address boundary. This makes the internal pointers somewhat random with > respect to the associated RAM buffer addresses. > > Align the buffer to the page size of the machine to help with

Re: [U-Boot] [PATCH v9 06/18] efi: sandbox: Add a simple 'bootefi test' command

2018-08-26 Thread Alexander Graf
On 08.08.18 11:54, Simon Glass wrote: > This jumps to test code which can call directly into the EFI support. It > does not need a separate image so it is easy to write tests with it. > > This test can be executed without causing problems to the run-time > environment (e.g. U-Boot does not need

Re: [U-Boot] [PATCH v9 04/18] efi: sandbox: Enable EFI loader build for sandbox

2018-08-26 Thread Alexander Graf
On 08.08.18 11:54, Simon Glass wrote: > This allows this feature to build within sandbox. This is for testing > purposes only since it is not possible for sandbox to load native code. Last time I tried I successfully ran native code? Alex > > Signed-off-by: Simon Glass > --- > > Changes in

Re: [U-Boot] [PATCH v9 03/18] efi: sandbox: Add distroboot support

2018-08-26 Thread Alexander Graf
On 08.08.18 11:54, Simon Glass wrote: > With sandbox these values depend on the host system. Let's assume that it > is x86_64 for now. > > Signed-off-by: Simon Glass > --- > > Changes in v9: None > Changes in v8: None > Changes in v7: > - Drop an unnecessary comment > > Changes in v6: > - War

Re: [U-Boot] [PATCH v9 02/18] efi: Don't allow CMD_BOOTEFI_SELFTEST on sandbox

2018-08-26 Thread Alexander Graf
On 08.08.18 11:54, Simon Glass wrote: > This does not work at present and gives the following error: > > output: 'ld.bfd: read in flex scanner failed > scripts/Makefile.lib:390: recipe for target > 'lib/efi_selftest/efi_selftest_miniapp_return_efi.so' failed > > It may be possible to figure th

Re: [U-Boot] [PATCH] mtd: spi: Add DM support to SH QSPI driver

2018-08-26 Thread Marek Vasut
On 08/26/2018 08:45 AM, Jagan Teki wrote: > On Sat, Aug 25, 2018 at 11:04 PM, Marek Vasut wrote: >> Add DM support to the SH QSPI driver while retaining non-DM support. >> The later is required as this driver is used in SPL which has a size >> limitation of 16 kiB. >> >> Signed-off-by: Marek Vasut

[U-Boot] [PATCH 1/2] efi_loader: pass system table in loaded image protocol

2018-08-26 Thread Heinrich Schuchardt
The system table must be passed as a pointer in the loaded image protocol. Signed-off-by: Heinrich Schuchardt --- lib/efi_loader/efi_boottime.c | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/efi_loader/efi_boottime.c b/lib/efi_loader/efi_boottime.c index 3935e4f1ce..ef4495ea46 100644 --

[U-Boot] [PATCH 2/2] efi_selftest: test for loaded image protocol

2018-08-26 Thread Heinrich Schuchardt
Verify that the loaded image protocol is installed on the image handle. Verify that the loaded image protocol points to the system table. Signed-off-by: Heinrich Schuchardt --- lib/efi_selftest/Makefile| 1 + lib/efi_selftest/efi_selftest_loaded_image.c | 108 ++

[U-Boot] [PATCH 0/2] efi_loader: pass system table in loaded image protocol

2018-08-26 Thread Heinrich Schuchardt
The system table must be passed as a pointer in the loaded image protocol. A unit test is supplied. Heinrich Schuchardt (2): efi_loader: pass system table in loaded image protocol efi_selftest: test for loaded image protocol lib/efi_loader/efi_boottime.c| 1 + lib/efi_self

[U-Boot] [PATCH v2] drivers: esdhc: add support for ColdFire mcf5441x family

2018-08-26 Thread Angelo Dureghello
This patch has been tested on the mcf54415-based stmark2 board. The eSDHC driver works reliably using DMA mode. Signed-off-by: Angelo Dureghello --- Changes in v2: - add be to le 32-bit byte swap for ColdFire, - remove HS from host capabilities for ColdFire. --- drivers/mmc/fsl_esdhc.c | 43

[U-Boot] [PATCH 1/1] efi_selftest: memory leak testing manage protocols

2018-08-26 Thread Heinrich Schuchardt
Remove memory leak in efi_selftest_manageprotocols.c. Signed-off-by: Heinrich Schuchardt --- .../efi_selftest_manageprotocols.c| 21 +-- 1 file changed, 19 insertions(+), 2 deletions(-) diff --git a/lib/efi_selftest/efi_selftest_manageprotocols.c b/lib/efi_selftest/

Re: [U-Boot] [PATCH] drivers: esdhc: add support for coldfire mcf5441x family

2018-08-26 Thread Angelo Dureghello
Hi Antoniou, plese trash this patch, found additional points, will send a v2 in short. Many thanks, Angelo Dureghello ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH v4 16/17] sunxi: usb: Switch to Generic host controllers

2018-08-26 Thread Jagan Teki
Once of key blocker for using USB Generic host controller drivers in Allwinner are CLK and RESET drivers, now these available for USB usage. So switch to use EHCI and OHCI Generic controllers. Enabling USB is wisely a board choise, so Enable USB_OHCI_HCD where it already have USB_EHCI_HCD Tested-

[U-Boot] [PATCH v4 15/17] musb-new: sunxi: Use CLK and RESET support

2018-08-26 Thread Jagan Teki
Now clock and reset drivers are available for respective SoC's so use clk and reset ops on musb driver. Tested-by: Jagan Teki Acked-by: Maxime Ripard Signed-off-by: Jagan Teki --- drivers/usb/musb-new/sunxi.c | 87 +--- 1 file changed, 51 insertions(+), 36 delet

[U-Boot] [PATCH v4 07/17] clk: sunxi: Add Allwinner A31 CLK driver

2018-08-26 Thread Jagan Teki
Add initial clock driver for Allwinner A31. - Implement USB ahb1 and USB clocks via ccu_clk_map descriptor for A31, so it can accessed in common clk enable and disable functions from clk_sunxi.c - Implement USB ahb1 and USB resets via ccu_reset_map descriptor for A31, so it can accessed in c

[U-Boot] [PATCH v4 12/17] clk: sunxi: Add Allwinner V3S CLK driver

2018-08-26 Thread Jagan Teki
Add initial clock driver for Allwinner V3S. - Implement USB bus and USB clocks via ccu_clk_map descriptor for V3S, so it can accessed in common clk enable and disable functions from clk_sunxi.c - Implement USB bus and USB resets via ccu_reset_map descriptor for V3S, so it can accessed in com

[U-Boot] [PATCH v4 10/17] clk: sunxi: Add Allwinner A83T CLK driver

2018-08-26 Thread Jagan Teki
Add initial clock driver for Allwinner A83T. - Implement USB bus and USB clocks via ccu_clk_map descriptor for A83T, so it can accessed in common clk enable and disable functions from clk_sunxi.c - Implement USB bus and USB resets via ccu_reset_map descriptor for A83T, so it can accessed in

[U-Boot] [PATCH v4 13/17] sunxi: Enable CLK

2018-08-26 Thread Jagan Teki
CLK and DM_RESET drivers are now available for most of the Allwinner platforms, so enable in mach-sunxi/Kconfig Enabling CLK will select DM_RESET by default. Tested-by: Jagan Teki Signed-off-by: Jagan Teki --- arch/arm/mach-sunxi/Kconfig | 12 1 file changed, 12 insertions(+) dif

[U-Boot] [PATCH v4 08/17] clk: sunxi: Add Allwinner A23 CLK driver

2018-08-26 Thread Jagan Teki
Add initial clock driver for Allwinner A23. - Implement USB bus and USB clocks via ccu_clk_map descriptor for A23, so it can accessed in common clk enable and disable functions from clk_sunxi.c - Implement USB bus and USB resets via ccu_reset_map descriptor for A23, so it can accessed in com

[U-Boot] [PATCH v4 11/17] clk: sunxi: Add Allwinner R40 CLK driver

2018-08-26 Thread Jagan Teki
Add initial clock driver for Allwinner R40. - Implement USB bus and USB clocks via ccu_clk_map descriptor for R40, so it can accessed in common clk enable and disable functions from clk_sunxi.c - Implement USB bus and USB resets via ccu_reset_map descriptor for R40, so it can accessed in com

[U-Boot] [PATCH v4 09/17] clk: sunxi: a23: Add CLK support for A33

2018-08-26 Thread Jagan Teki
A33 has separate clock driver in Linux because of few clock differences wrt to A23 like audio etc,. these may not useful for U-Boot so added a33 ccu compatible on existing a23 clock driver. Signed-off-by: Jagan Teki --- drivers/clk/sunxi/Kconfig | 6 +++--- drivers/clk/sunxi/clk_a23.c | 2 ++

[U-Boot] [PATCH v4 05/17] clk: sunxi: Add Allwinner A10/A20 CLK driver

2018-08-26 Thread Jagan Teki
Add initial clock driver for Allwinner A10/A20. - Implement USB ahb and USB clocks via ccu_clk_map descriptor for A10/A20, so it can accessed in common clk enable and disable functions from clk_sunxi.c - Implement USB resets via ccu_reset_map descriptor for A10/A20, so it can accessed in com

[U-Boot] [PATCH v4 14/17] phy: sun4i-usb: Use CLK and RESET support

2018-08-26 Thread Jagan Teki
Now clock and reset drivers are available for respective SoC's so use clk and reset ops on phy driver. Tested-by: Jagan Teki Signed-off-by: Jagan Teki --- drivers/phy/allwinner/phy-sun4i-usb.c | 77 --- 1 file changed, 57 insertions(+), 20 deletions(-) diff --git a/driv

[U-Boot] [PATCH v4 17/17] usb: host: Drop [e-o]hci-sunxi drivers

2018-08-26 Thread Jagan Teki
Now Allwinner platform is all set to use Generic USB controller drivers, so remove the legacy sunxi drivers. Tested-by: Jagan Teki Signed-off-by: Jagan Teki --- drivers/usb/host/Makefile | 2 - drivers/usb/host/ehci-sunxi.c | 204 - drivers/usb/host/ohci-sunxi.

[U-Boot] [PATCH v4 02/17] reset: Return 0 if no request ops

2018-08-26 Thread Jagan Teki
Missing request ops from respective uclass driver generating "synchronous abort" in Allwinner platform, may be in arm. So return 0 if request ops is not used for those uclass drivers. Cc: Simon Glass Signed-off-by: Jagan Teki --- drivers/reset/reset-uclass.c | 3 +++ 1 file changed, 3 insertion

[U-Boot] [PATCH v4 04/17] clk: sunxi: Add Allwinner H3/H5 CLK driver

2018-08-26 Thread Jagan Teki
Add initial clock driver for Allwinner H3/H5. - Implement USB bus and USB clocks via ccu_clk_map descriptor for H3/H5, so it can accessed in common clk enable and disable functions from clk_sunxi.c - Implement USB bus and USB resets via ccu_reset_map descriptor for H3/H5, so it can accessed

[U-Boot] [PATCH v4 06/17] clk: sunxi: Add Allwinner A10s/A13 CLK driver

2018-08-26 Thread Jagan Teki
Add initial clock driver for Allwinner A10s/A13. - Implement USB ahb and USB clocks via ccu_clk_map descriptor for A10s/A13, so it can accessed in common clk enable and disable functions from clk_sunxi.c - Implement USB resets via ccu_reset_map descriptor for A10s/A13, so it can accessed in

[U-Boot] [PATCH v4 03/17] reset: Add Allwinner RESET driver

2018-08-26 Thread Jagan Teki
Add common reset driver for all Allwinner SoC's. Since CLK and RESET share common DT compatible, it is CLK driver job is to bind the reset driver. So add CLK bind call on respective SoC driver by passing ccu map descriptor so-that reset deassert, deassert operations held based on reset register ma

[U-Boot] [PATCH v4 01/17] clk: Add Allwinner A64 CLK driver

2018-08-26 Thread Jagan Teki
Add initial clock driver for Allwinner A64. Implement USB clock enable and disable functions for OHCI, EHCI, OTG and USBPHY gate and clock registers. Tested-by: Jagan Teki # BPI-M64 Signed-off-by: Jagan Teki --- arch/arm/include/asm/arch-sunxi/ccu.h | 47 drivers/clk/Kconf

[U-Boot] [PATCH v4 00/17] clk: Add Allwinner CLK, RESET support

2018-08-26 Thread Jagan Teki
This series is refined version of Allwinner CLK, RESET support for previous version[1] which implements CLK and RESET for USB. Tested on A64, H3, H5, R40, A83T, A20. Changes for v4: - Collect Acked-by and Tested-by - Rebase on master - Fixed checkpatch warnings - Fixed SPDX licence - return 0 if

Re: [U-Boot] [PATCH v2 0/7] ARM: dts: sunxi: Update dts(i) files from Linux-v4.18-rc3

2018-08-26 Thread Jagan Teki
On Thu, Aug 9, 2018 at 6:38 PM, Jagan Teki wrote: > This series is trying to update SoC dtsi and board dts files > for Allwinner sun4i, 5i, 6i, 8i-a23/a33/a83t, r40 SoC's > from Linux-4.18-rc3 > > These update is useful for supporting features > CLK, RESET, DM_MMC etc > > Since H3/H5/A64 is alread

[U-Boot] Regression on ESPRESSObin / status of upstreaming

2018-08-26 Thread Jan Kiszka
Hi all, I was playing with upstream u-boot on the ESPRESSObin - what an entertaining build process... Anyway, I noticed that, when using latest version of Marvell's ATF, a3700-utils and mv-ddr-marvell (all on 18.06), I can make u-boot 2018.05 at least boot. However, anything >= 2018.07 will c