[U-Boot] [PATCH v2 0/8] Sync and consolidate Linux-derived printk, BUILD_BUG, BUG, WARN, etc.

2017-09-15 Thread Masahiro Yamada
I tested this series with buildman. Masahiro Yamada (8): stdio.h: move printf() stuff from to printk: collect printk stuff into with loglevel support treewide: replace with error() with pr_err() common.h: remove error() vsprintf.h: include bug.h: sync BUILD_BUG stuff with

[U-Boot] [PATCH v2 2/8] printk: collect printk stuff into with loglevel support

2017-09-15 Thread Masahiro Yamada
When we import code from Linux, with regular re-sync planned, we want to use printk() and pr_*(). U-Boot does not support them in a clean way. So, people end up with local macros, or compat headers here and there, then we occasionally see build errors of definition conflicts. We have

[U-Boot] [PATCH v2 8/8] dm: define dev_*() log functions in DM header

2017-09-15 Thread Masahiro Yamada
Many drivers use dev_err, dev_info, etc. for logging. Currently, we are relying on , but I guess the best home is , taking into account that Linux defines them in . For now, I am keeping the ones in because lots of Linux-originated code uses dev_*() just for the purpose of syncing, but the

[U-Boot] [PATCH v2 1/8] stdio.h: move printf() stuff from to

2017-09-15 Thread Masahiro Yamada
pulls in a lot of headers. Including it from every .c file is a bad idea. We need to remove contents until it contains nothing. Move printf() and friends to . Signed-off-by: Masahiro Yamada --- Changes in v2: - newly added include/common.h | 41

[U-Boot] [PATCH v2 6/8] bug.h: sync BUILD_BUG stuff with Linux 4.13

2017-09-15 Thread Masahiro Yamada
As commit 84b8bf6d5d2a ("bug.h: move BUILD_BUG_* defines to include/linux/bug.h") noted, include/linux/bug.h was locally modified for U-Boot because the name conflict of error() caused build errors at that time. Now error() is gone, so we can fully sync BUILD_BUG* with Linux. These macros are

[U-Boot] [PATCH v2 4/8] common.h: remove error()

2017-09-15 Thread Masahiro Yamada
This macro prevents us from using compiletime_error/assert defined in . Now we can remove it, then we will be able to import more BUILD_BUG macros from Linux. Signed-off-by: Masahiro Yamada --- Changes in v2: None include/common.h | 5 - 1 file changed, 5

[U-Boot] [PATCH v2 5/8] vsprintf.h: include

2017-09-15 Thread Masahiro Yamada
This header uses ulong, size_t, loff_t. Include to make this header self-contained. Signed-off-by: Masahiro Yamada --- Changes in v2: None include/vsprintf.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/vsprintf.h b/include/vsprintf.h index

[U-Boot] [PATCH v2 7/8] bug.h: move runtime BUG/WARN macros into

2017-09-15 Thread Masahiro Yamada
Collect runtime BUG/WARN into a self-contained header to make these macros easier to use. Signed-off-by: Masahiro Yamada --- Changes in v2: None drivers/usb/dwc3/linux-compat.h | 1 - include/common.h| 9 + include/linux/bug.h

Re: [U-Boot] [PATCH 13/13] x86: Support Intel Cherry Hill board

2017-09-15 Thread Bin Meng
On Sat, Aug 26, 2017 at 9:39 PM, Simon Glass wrote: > On 15 August 2017 at 23:42, Bin Meng wrote: >> This adds support to Intel Cherry Hill board, a board based on >> Intel Braswell SoC. The following devices are validated: >> >> - serial port as the serial

Re: [U-Boot] [PATCH 11/13] x86: braswell: Add FSP configuration

2017-09-15 Thread Bin Meng
On Sat, Aug 26, 2017 at 9:39 PM, Simon Glass wrote: > On 15 August 2017 at 23:42, Bin Meng wrote: >> Add FSP related configuration for Braswell. >> >> Signed-off-by: Bin Meng >> --- >> >> arch/x86/cpu/braswell/Makefile

Re: [U-Boot] [PATCH 10/13] x86: braswell: Add microcode for B0/C0/D0 stepping SoC

2017-09-15 Thread Bin Meng
On Sat, Aug 26, 2017 at 9:39 PM, Simon Glass wrote: > On 15 August 2017 at 23:41, Bin Meng wrote: >> This adds microcode device tree fragment for Braswell B0 (406C2), >> C0 (406C3) and D0 (406C4) stepping SoC. >> >> Signed-off-by: Bin Meng

Re: [U-Boot] [PATCH 12/13] x86: braswell: Disable PUNIT power configuration for B0 stepping

2017-09-15 Thread Bin Meng
On Sat, Aug 26, 2017 at 9:39 PM, Simon Glass wrote: > On 15 August 2017 at 23:42, Bin Meng wrote: >> FSP's built-in UPD configuration enables PUNIT power configuration, >> but on B0 stepping, this causes CPU hangs in fsp_init(). Disable it. >> >>

Re: [U-Boot] [PATCH 09/13] x86: Add Intel Braswell SoC support

2017-09-15 Thread Bin Meng
On Sat, Aug 26, 2017 at 9:55 PM, Bin Meng wrote: > Hi Simon, > > On Sat, Aug 26, 2017 at 9:39 PM, Simon Glass wrote: >> On 15 August 2017 at 23:41, Bin Meng wrote: >>> This adds initial Intel Braswell SoC support. It uses Intel FSP >>>

Re: [U-Boot] arm: dts: Add missing u-boot specific dtsi for dra7-evm

2017-09-15 Thread Tom Rini
On Fri, Sep 15, 2017 at 12:39:41PM +0200, Jean-Jacques Hiblot wrote: > commit b63b995 (arm: dts: Add u-boot specific compatibles) separated out > u-boot specific compatibles from dts files. > The u-boot specifics were moved in *-u-boot.dtsi files for all dra7 > platforms except dra7-evm. >

Re: [U-Boot] [U-Boot, v3, 7/9] fat/fs: move ls to generic implementation

2017-09-15 Thread Tom Rini
On Sat, Sep 09, 2017 at 01:15:58PM -0400, Rob Clark wrote: > Add a generic implementation of 'ls' using opendir/readdir/closedir, and > replace fat's custom implementation. Other filesystems should move to > the generic implementation after they add opendir/readdir/closedir > support. > >

Re: [U-Boot] spl: Fix compiling warning on gunzip argument

2017-09-15 Thread Tom Rini
On Thu, Sep 14, 2017 at 12:41:38PM -0700, York Sun wrote: > common/spl/spl_fit.c:201:12: warning: passing argument 4 of ‘gunzip’ > from incompatible pointer type [-Wincompatible-pointer-types] >src, )) > > Signed-off-by: York Sun > Reported-by: Heinrich Schuchardt

Re: [U-Boot] fs/fat: fix fatbuf leak

2017-09-15 Thread Tom Rini
On Tue, Sep 12, 2017 at 04:40:01PM -0400, Rob Clark wrote: > A new fatbuf was allocated by get_fs_info() (called by fat_itr_root()), > but not freed, resulting in eventually running out of memory. Spotted > by running 'ls -r' in a large FAT filesystem from Shell.efi. > > fatbuf is mainly used

Re: [U-Boot] [U-Boot,v3,8/9] fs/fat: fix case for FAT shortnames

2017-09-15 Thread Tom Rini
On Sat, Sep 09, 2017 at 01:15:59PM -0400, Rob Clark wrote: > Noticed when comparing our output to linux. There are some lcase bits > which control whether filename and/or extension should be downcase'd. > > Signed-off-by: Rob Clark > Reviewed-by: Łukasz Majewski

Re: [U-Boot] [U-Boot, v3, 9/9] fs/fat: Clean up open-coded sector <-> cluster conversions

2017-09-15 Thread Tom Rini
On Sat, Sep 09, 2017 at 01:16:00PM -0400, Rob Clark wrote: > Use the clust_to_sect() helper that was introduced earlier, and add an > inverse sect_to_clust(), plus update the various spots that open-coded > this conversion previously. > > Signed-off-by: Rob Clark >

Re: [U-Boot] spl: Fix compiling warning on gunzip argument

2017-09-15 Thread Tom Rini
On Fri, Sep 15, 2017 at 10:32:49PM -0400, Tom Rini wrote: > On Thu, Sep 14, 2017 at 12:41:38PM -0700, York Sun wrote: > > > common/spl/spl_fit.c:201:12: warning: passing argument 4 of ‘gunzip’ > > from incompatible pointer type [-Wincompatible-pointer-types] > >src, )) > > > >

Re: [U-Boot] ARM: davinci: Remove CONFIG_SOC_DA830

2017-09-15 Thread Tom Rini
On Fri, Sep 15, 2017 at 05:56:48AM -0500, Adam Ford wrote: > Nothing CONFIG_SOC_DA830 anymore, so let's remove it and > the one remaining source file for it. It appears to have been > abandoned a while ago. > > Signed-off-by: Adam Ford > > diff --git

Re: [U-Boot] [U-Boot,v3,4/9] fs: add fs_readdir()

2017-09-15 Thread Tom Rini
On Sat, Sep 09, 2017 at 01:15:55PM -0400, Rob Clark wrote: > Needed to support efi file protocol. The fallback.efi loader wants > to be able to read the contents of the /EFI directory to find an OS > to boot. > > Modelled after POSIX opendir()/readdir()/closedir(). Unlike the other > fs APIs,

Re: [U-Boot] arm: mach-omap2: Relax checks on OP-TEE location to allow pageable image

2017-09-15 Thread Tom Rini
On Wed, Sep 13, 2017 at 01:27:44PM -0500, Andrew F. Davis wrote: > From: Harinarayan Bhatta > > When the OP-TEE image is built for secure paging the load address may be > in SRAM, remove checks that prevent this. > > Signed-off-by: Harinarayan Bhatta >

Re: [U-Boot] [U-Boot, v3, 2/9] fs/fat: introduce new director iterators

2017-09-15 Thread Tom Rini
On Sat, Sep 09, 2017 at 01:15:53PM -0400, Rob Clark wrote: > Untangle directory traversal into a simple iterator, to replace the > existing multi-purpose do_fat_read_at() + get_dentfromdir(). > > Signed-off-by: Rob Clark > Reviewed-by: Łukasz Majewski

Re: [U-Boot] [U-Boot, v3, 5/9] fs/fat: implement opendir/readdir/closedir

2017-09-15 Thread Tom Rini
On Sat, Sep 09, 2017 at 01:15:56PM -0400, Rob Clark wrote: > Implement the readdir interface using the directory iterators. > > Signed-off-by: Rob Clark > Reviewed-by: Łukasz Majewski > Reviewed-by: Simon Glass Applied to u-boot/master,

Re: [U-Boot] [U-Boot, v3, 3/9] fat/fs: convert to directory iterators

2017-09-15 Thread Tom Rini
On Sat, Sep 09, 2017 at 01:15:54PM -0400, Rob Clark wrote: > And drop a whole lot of ugly code! > > Signed-off-by: Rob Clark > Reviewed-by: Łukasz Majewski > Reviewed-by: Simon Glass Applied to u-boot/master, thanks! -- Tom

Re: [U-Boot] [U-Boot, v3, 1/9] fs/fat: split out helper to init fsdata

2017-09-15 Thread Tom Rini
On Sat, Sep 09, 2017 at 01:15:52PM -0400, Rob Clark wrote: > Want to re-use this in fat dirent iterator in next patch. > > Signed-off-by: Rob Clark > Reviewed-by: Łukasz Majewski > Reviewed-by: Simon Glass Applied to u-boot/master,

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

2017-09-15 Thread Tom Rini
On Fri, Sep 15, 2017 at 07:50:19AM +0200, Heiko Schocher wrote: > Hello Tom, > > please pull from u-boot-ubi.git master > > The following changes since commit c98ac3487e413c71e5d36322ef3324b21c6f60f9: > > Prepare v2017.09 (2017-09-11 14:10:40 -0400) > > are available in the git repository

Re: [U-Boot] Please pull from u-boot-i2c

2017-09-15 Thread Tom Rini
On Fri, Sep 15, 2017 at 07:44:00AM +0200, Heiko Schocher wrote: > Hello Tom, > > please pull from u-boot-i2c master > > The following changes since commit ba39608147c797cffb266579b5791649f0f8e60c: > > ARM: DRA72x: Add support for detection of DRA71x SR 2.1 (2017-09-12 > 18:02:29 -0400) > >

Re: [U-Boot] [PATCH 1/1] x86: ivybridge: remove unused variables

2017-09-15 Thread Bin Meng
On Tue, Sep 12, 2017 at 9:40 AM, Heinrich Schuchardt wrote: > legacy_hole_base_k and legacy_hole_size_k are defined but > not used. > > Signed-off-by: Heinrich Schuchardt > --- > arch/x86/cpu/ivybridge/northbridge.c | 10 -- > 1 file changed, 10

Re: [U-Boot] [PATCH v2] net/tftp: fix build if CMD_BOOTEFI is not set

2017-09-15 Thread Bin Meng
On Sat, Sep 16, 2017 at 4:16 AM, Jörg Krause wrote: > Fixes: > net/tftp.c:811: undefined reference to `efi_set_bootdev' > > Signed-off-by: Jörg Krause > --- > v2: > * remove ifdef for efi header file (suggested by Bin Weng) > --- >

[U-Boot] [PATCH] gpio: Kconfig: DM_GPIO depends on OF_CONTROL

2017-09-15 Thread Jörg Krause
Building U-Boot with DM_GPIO set, but with OF_CONTROL unset fails. Fixes: drivers/gpio/built-in.o: In function `gpio_request_tail': drivers/gpio/gpio-uclass.c:666: undefined reference to `ofnode_get_name' drivers/gpio/built-in.o: In function `_gpio_request_by_name_nodev':

[U-Boot] [PATCH v2] net/tftp: fix build if CMD_BOOTEFI is not set

2017-09-15 Thread Jörg Krause
Fixes: net/tftp.c:811: undefined reference to `efi_set_bootdev' Signed-off-by: Jörg Krause --- v2: * remove ifdef for efi header file (suggested by Bin Weng) --- net/tftp.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/net/tftp.c b/net/tftp.c index

Re: [U-Boot] [PATCH] MIPS: Drop unused PTR_COUNT_SHIFT from u-boot.lds

2017-09-15 Thread Daniel Schwierzeck
Am 15.09.2017 um 20:35 schrieb Paul Burton: > The u-boot.lds linker script for MIPS defines a PTR_COUNT_SHIFT macro to > 2 or 3 for 32 bit or 64 bit builds respectively. This macro is never > actually used though, so remove the dead code. > > Signed-off-by: Paul Burton

Re: [U-Boot] [PATCH] boston: Drop unused return value

2017-09-15 Thread Daniel Schwierzeck
Am 15.09.2017 um 20:33 schrieb Paul Burton: > The boston lowlevel_init() function zeroes the return register v0, > despite the function not being expected to return a value & that value > never being used. > > Remove the redundant assignment to v0. > > Signed-off-by: Paul Burton

Re: [U-Boot] [PATCH V2] test/py: gpt: make use of infra-structure

2017-09-15 Thread Tom Rini
On Fri, Sep 15, 2017 at 12:19:38PM -0600, Stephen Warren wrote: > From: Stephen Warren > > Make various changes to the GPT test: > > 1) Reference the disk image using an absolute path in all cases. This > allows test/py to operate correctly if it's run from a directory

Re: [U-Boot] [PATCH v2 03/16] dtoc: Add a 64-bit type and a way to convert cells into 64 bits

2017-09-15 Thread sjg
When dealing with multi-cell values we need a type that can hold this value. Add this and a function to process it from a list of cell values. Signed-off-by: Simon Glass Reviewed-by: Philipp Tomsich Tested-by: Philipp Tomsich

Re: [U-Boot] [PATCH v2 04/16] dtoc: Avoid very long lines in output

2017-09-15 Thread sjg
Large arrays can result in lines with hundreds or thousands of characters which is not very editor-friendly. To avoid this, addjust the tool to group values 8 per line. Signed-off-by: Simon Glass --- Changes in v2: None tools/dtoc/dtb_platdata.py | 7 ++-

Re: [U-Boot] [PATCH v2 13/16] dtoc: Put phandle args in an array

2017-09-15 Thread sjg
We want to support more than one phandle argument. It makes sense to use an array for this rather than discrete struct members. Adjust the code to support this. Rename the member to 'arg' instead of 'id'. Signed-off-by: Simon Glass --- Changes in v2: None

Re: [U-Boot] [PATCH v2 05/16] dtoc: Add support for 32 or 64-bit addresses

2017-09-15 Thread sjg
On 29 August 2017 at 14:23, Dr. Philipp Tomsich wrote: > >> On 29 Aug 2017, at 22:15, Simon Glass wrote: >> >> When using 32-bit addresses dtoc works correctly. For 64-bit addresses it >> does not since it ignores the #address-cells and

Re: [U-Boot] [PATCH v2 06/16] dtoc: Handle 'reg' properties with unusual sizes

2017-09-15 Thread sjg
At present dtoc assumes that all 'reg' properties have both an address and a size. For I2C devices we do not have this. Adjust dtoc to cope. Reported-by: Philipp Tomsich Signed-off-by: Simon Glass --- Changes in v2: None

Re: [U-Boot] [PATCH v2 01/16] fdt: Sync libfdt up to upstream

2017-09-15 Thread sjg
Add upstream changes to U-Boot: - new pylibfdt functions - fdt_setprop_placeholder() Signed-off-by: Simon Glass --- Changes in v2: None lib/libfdt/fdt_rw.c | 20 --- lib/libfdt/libfdt.h | 31 +++

Re: [U-Boot] [PATCH v2 12/16] dtoc: Put each phandle on a separate line

2017-09-15 Thread sjg
When writing values from properties which contain phandles, dtoc currently writes 8 phandles per line. Change this to write one phandle per line. This helps reduce line length, since phandles are generally longer and may have arguments. Signed-off-by: Simon Glass --- Changes

Re: [U-Boot] [PATCH v2 08/16] dtoc: Use the Fdt's class's phandle map

2017-09-15 Thread sjg
Now that the Fdt class can map phandles to the associated nodes, use that instead of a separate implementation. Signed-off-by: Simon Glass --- Changes in v2: None tools/dtoc/dtb_platdata.py | 16 +++- 1 file changed, 3 insertions(+), 13 deletions(-) Applied to

Re: [U-Boot] [PATCH v3 01/14] Introduce fdt_setprop_placeholder() method

2017-09-15 Thread sjg
On 4 September 2017 at 14:12, Pantelis Antoniou wrote: > In some cases you need to add a property but the contents of it > are not known at creation time, merely the extend of it. > > This method allows you to create a property of a given size (filled > with

[U-Boot] Please pull u-boot-fdt

2017-09-15 Thread Simon Glass
Hi Tom, Here are the overlay and dtoc series. The following changes since commit 5541543f686b43210fb92181003ff7175d4ab036: configs: at91: Remove CONFIG_SYS_EXTRA_OPTIONS assignment (2017-09-14 16:02:48 -0400) are available in the git repository at: git://git.denx.de/u-boot-fdt.git for

Re: [U-Boot] [PATCH v2 16/16] dtoc: Add a header to the generated files

2017-09-15 Thread sjg
Add a header that indicates that the files generated by dtoc should not be modified. Signed-off-by: Simon Glass --- Changes in v2: None tools/dtoc/dtb_platdata.py | 12 1 file changed, 12 insertions(+) Applied to u-boot-fdt thanks!

Re: [U-Boot] [PATCH v3 02/14] fdt: Introduce helper method fdt_overlay_apply_verbose()

2017-09-15 Thread sjg
On 4 September 2017 at 14:12, Pantelis Antoniou wrote: > Introduce fdt_overlay_apply_verbose, a method that applies an > overlay but in the case of an error produces a helpful message. > > In addition if a base tree is found to be missing the __symbols__ > node the

Re: [U-Boot] [PATCH v2 11/16] dtoc: Rename the phandle struct

2017-09-15 Thread sjg
Rather than naming the phandle struct according to the number of cells it uses (e.g. struct phandle_2_cell) name it according to the number of arguments it has (e.g. struct phandle_1_arg). This is a more intuitive naming. Signed-off-by: Simon Glass --- Changes in v2: None

Re: [U-Boot] [PATCH v3 03/14] fdt: Switch to using the verbose overlay application method

2017-09-15 Thread Simon Glass
On 8 September 2017 at 22:53, Simon Glass wrote: > On 4 September 2017 at 14:12, Pantelis Antoniou > wrote: >> The verbose overlay application method prints out more helpful >> messages, so switch to it. >> >> Signed-off-by: Pantelis Antoniou

Re: [U-Boot] [PATCH v1] cmd: usb: add blk devices to ignore list in tree graph

2017-09-15 Thread Suneel Garapati
Hi Bin, On Thu, Sep 14, 2017 at 11:27 PM, Bin Meng wrote: > Hi Suneel, > > On Mon, Sep 11, 2017 at 2:17 PM, Simon Glass wrote: >> On 6 September 2017 at 11:01, Suneel Garapati wrote: >>> add blk child devices to ignore list while

Re: [U-Boot] [PATCH v2 07/16] dtoc: Update the Fdt class to record phandles

2017-09-15 Thread sjg
Add a map from phandles to nodes. This can be used by clients of the the class instead of maintaining this themselves. Signed-off-by: Simon Glass --- Changes in v2: None tools/dtoc/fdt.py | 5 + 1 file changed, 5 insertions(+) Applied to u-boot-fdt thanks!

Re: [U-Boot] [PATCH v2 10/16] dtoc: Rename is_phandle() and adjust it to return more detail

2017-09-15 Thread sjg
Update this function to return more detail about a property that contains phandles. This will allow (in a future commit) more accurate handling of these properties. Signed-off-by: Simon Glass --- Changes in v2: None tools/dtoc/dtb_platdata.py | 79

Re: [U-Boot] [PATCH v2 15/16] dtoc: Rename the auto-generated dt-structs.h file

2017-09-15 Thread sjg
The filename of the auto-generated file is the same as the file that includes it. Even though the form is in the generated/ subdirectory, this could be confused. Rename the generated file to something that makes it clear it is auto-generated. Signed-off-by: Simon Glass ---

Re: [U-Boot] [PATCH v2 02/16] dtoc: Adjust Node to record its parent

2017-09-15 Thread sjg
We need to be able to search back up the tree for #address-cells and #size-cells. Record the parent of each node to make this easier. Signed-off-by: Simon Glass Reviewed-by: Philipp Tomsich Tested-by: Philipp Tomsich

Re: [U-Boot] [PATCH v2 14/16] dtoc: Support properties containing multiple phandle values

2017-09-15 Thread sjg
At present dtoc has a very simplistic view of phandles. It assumes that a property has only a single phandle with a single argument (i.e. two cells per property). This is not true in many cases. Enhance the implementation to scan all phandles in a property and to use the correct number of

Re: [U-Boot] [PATCH v2 09/16] dtoc: Make is_phandle() a member function

2017-09-15 Thread sjg
This function will need to have access to class members once we enhance it to support multiple phandle values. In preparation for that, move it into the class. Signed-off-by: Simon Glass --- Changes in v2: None tools/dtoc/dtb_platdata.py | 36

Re: [U-Boot] efi_loader: queued patches

2017-09-15 Thread Rob Clark
On Fri, Sep 15, 2017 at 12:27 AM, Heinrich Schuchardt wrote: > Hello Alex, > > this is the sequence of patches I am currently working with. Maybe Rob > could also send a similar list. That way you know in which sequence you > can do reviewing. > > # EFI event services > # This

Re: [U-Boot] [PATCH] libfdt: Initialize the stack variable

2017-09-15 Thread sjg
On 9/4/2017 8:41 PM, Chee, Tien Fong wrote: > On Rab, 2017-08-30 at 06:31 -0700, J. William Campbell wrote: >> On 8/29/2017 10:15 PM, tien.fong.c...@intel.com wrote: >>> From: Tien Fong Chee >>> >>> Report Coverity log: >>> The code uses a variable that has not >>> been

Re: [U-Boot] [PATCH v3 10/14] doc: overlays: Tweak documentation regarding overlays

2017-09-15 Thread sjg
On 4 September 2017 at 14:12, Pantelis Antoniou wrote: > From: Franklin S Cooper Jr > > Pull some information regarding overlays from commit messages and > put them directly within the documentation. Also add some information > regarding required

Re: [U-Boot] [PATCH v3 12/14] config: sandbox: Add unit tests

2017-09-15 Thread sjg
On 4 September 2017 at 14:12, Pantelis Antoniou wrote: > Add unit tests for sandbox default config. > > Signed-off-by: Pantelis Antoniou > --- > configs/sandbox_defconfig | 2 ++ > 1 file changed, 2 insertions(+) > Acked-by: Simon

Re: [U-Boot] [PATCH v3 11/14] dtbo: make dtbos special

2017-09-15 Thread sjg
On Fri, Sep 15, 2017 at 10:21:28AM -0600, Stephen Warren wrote: > On 09/15/2017 10:13 AM, Tom Rini wrote: > >On Fri, Sep 15, 2017 at 09:52:41AM -0600, Stephen Warren wrote: > >>On 09/15/2017 09:50 AM, Stephen Warren wrote: > >>>On 09/04/2017 02:12 PM, Pantelis Antoniou wrote: > Special rule

Re: [U-Boot] [PATCH v3 04/14] fdt: Allow stacked overlays phandle references

2017-09-15 Thread sjg
On 4 September 2017 at 14:12, Pantelis Antoniou wrote: > This patch enables an overlay to refer to a previous overlay's > labels by performing a merge of symbol information at application > time. > > In a nutshell it allows an overlay to refer to a symbol that a

Re: [U-Boot] [PATCH v3 13/14] ut: fix fdt_getprop_u32() change

2017-09-15 Thread sjg
On 4 September 2017 at 14:12, Pantelis Antoniou wrote: > fdt_getprop_u32 is not exported and it's different than what the > unit test uses. Rename u32 prop access methods to something that's > unit test specific. > > Signed-off-by: Pantelis Antoniou

Re: [U-Boot] [PATCH v3 07/14] fit: Introduce methods for applying overlays on fit-load

2017-09-15 Thread sjg
On 4 September 2017 at 14:12, Pantelis Antoniou wrote: > Introduce an overlay based method for constructing a base DT blob > to pass to the kernel. > > It is based on a specific method now to get the FDT from a FIT image > named boot_get_fdt_fit(). > >

Re: [U-Boot] [PATCH v3 06/14] fit: Do not throw away extra configuration on fit_image_load()

2017-09-15 Thread sjg
Hi, On 4 September 2017 at 14:12, Pantelis Antoniou wrote: > fit_image_load() threw away the extra configuration parts when > loading. We need them around for applying extra overlays for > building the boot fdt. > > Signed-off-by: Pantelis Antoniou

Re: [U-Boot] [PATCH v3 05/14] fit: Allow multiple images per property

2017-09-15 Thread sjg
On 4 September 2017 at 14:12, Pantelis Antoniou wrote: > As part of the fdt overlay support which need it, allow > a list of configurations per property. > > Signed-off-by: Pantelis Antoniou > --- > common/image-fit.c | 18

Re: [U-Boot] [PATCH v3 08/14] fit: fdt overlays doc

2017-09-15 Thread sjg
On 4 September 2017 at 14:12, Pantelis Antoniou wrote: > Signed-off-by: Pantelis Antoniou > --- > doc/uImage.FIT/command_syntax_extensions.txt | 12 +- > doc/uImage.FIT/overlay-fdt-boot.txt | 221 >

Re: [U-Boot] [PATCH v3 09/14] doc: Document how to apply fdt overlays

2017-09-15 Thread sjg
On 4 September 2017 at 14:12, Pantelis Antoniou wrote: > We have the capability to apply overlays on the command line but > we didn't have a document explaining how. > > Signed-off-by: Pantelis Antoniou > --- >

Re: [U-Boot] [PATCH v3 14/14] test: overlay: Add unit test for stacked overlay

2017-09-15 Thread sjg
On 4 September 2017 at 14:12, Pantelis Antoniou wrote: > Verify that stacked overlays work. > > Signed-off-by: Pantelis Antoniou > --- > test/overlay/Makefile | 1 + > test/overlay/cmd_ut_overlay.c

[U-Boot] [PATCH 4/5] ARM: rmobile: Zap ad-hoc PFC and GPIO setup in board files

2017-09-15 Thread Marek Vasut
At long last, nuke all this ad-hoc setup in board files in favor of letting PFC pinmux and GPIO drivers do the same job, but based on DT description of the hardware rather than this board-file ugliness. Signed-off-by: Marek Vasut Cc: Nobuhiro Iwamatsu

[U-Boot] [PATCH] ARM: rmobile: Enable GPIO command

2017-09-15 Thread Marek Vasut
This command is convenient for manipulating the GPIOs, so enable it. Signed-off-by: Marek Vasut Cc: Nobuhiro Iwamatsu --- configs/r8a7795_salvator-x_defconfig | 1 + configs/r8a7795_ulcb_defconfig | 1 +

[U-Boot] [PATCH 3/5] ARM: rmobile: Switch to DM PFC pinmux and GPIO driver

2017-09-15 Thread Marek Vasut
Enable the PFC pinmux and GPIO drivers and disable the SH GPIO combo driver. This allows the drivers to obtain pinmux and GPIO configuration from DT rather than hard-coding it in board files. Signed-off-by: Marek Vasut Cc: Nobuhiro Iwamatsu

[U-Boot] [PATCH 3/3] ARM: rmobile: Remove SCIF clock configuration from board files

2017-09-15 Thread Marek Vasut
The configuration is now fully performed by the SCIF and clk drivers, so remove it from the board file. Signed-off-by: Marek Vasut Cc: Nobuhiro Iwamatsu --- board/renesas/salvator-x/salvator-x.c | 2 -- board/renesas/ulcb/ulcb.c

[U-Boot] [PATCH 2/3] ARM: rmobile: Remove AVB clock configuration from board files

2017-09-15 Thread Marek Vasut
The configuration is now fully performed by the AVB and clk drivers, so remove it from the board file. Signed-off-by: Marek Vasut Cc: Nobuhiro Iwamatsu --- board/renesas/salvator-x/salvator-x.c | 3 --- board/renesas/ulcb/ulcb.c

[U-Boot] [PATCH 1/3] ARM: rmobile: Remove SD clock configuration from board files

2017-09-15 Thread Marek Vasut
The configuration is now fully performed by the SD and clk drivers, so remove it from the board file. Signed-off-by: Marek Vasut Cc: Nobuhiro Iwamatsu --- board/renesas/salvator-x/salvator-x.c | 18 --

[U-Boot] [PATCH] serial: sh: Enable clock if available

2017-09-15 Thread Marek Vasut
If the clock framework provides the driver with valid clock, enable them, otherwise the SCIF might not work if the clock are not enabled prior to the driver probe. Signed-off-by: Marek Vasut Cc: Nobuhiro Iwamatsu ---

[U-Boot] [PATCH] ARM: dts: rmobile: Add PHY reset GPIO

2017-09-15 Thread Marek Vasut
The PHY reset GPIOs were missing from the DT, add them. Signed-off-by: Marek Vasut Cc: Nobuhiro Iwamatsu --- arch/arm/dts/salvator-common.dtsi | 1 + arch/arm/dts/ulcb.dtsi| 1 + 2 files changed, 2 insertions(+) diff --git

[U-Boot] [PATCH] net: ravb: Add PHY reset GPIO support

2017-09-15 Thread Marek Vasut
Add support for obtaining PHY reset GPIO from DT and toggling it before configuring the PHY to put the PHY into defined state. Signed-off-by: Marek Vasut Cc: Nobuhiro Iwamatsu Cc: Joe Hershberger ---

[U-Boot] [PATCH] mmc: uniphier-sd: Add vqmmc regulator support

2017-09-15 Thread Marek Vasut
Add initial support for setting the vqmmc regulator. Since we do not support 1V8 modes, set the regulator to 3V3 and enable it. Signed-off-by: Marek Vasut Cc: Masahiro Yamada Cc: Jaehoon Chung ---

[U-Boot] [PATCH] clk: rmobile: Add support for setting SDxCKCR

2017-09-15 Thread Marek Vasut
The SDxCKCR must be configured correctly, otherwise the SDIF can be unstable. This is done in board files thus far, but those are going away, so move the setting of SDxCKCR into the correct place. Signed-off-by: Marek Vasut Cc: Nobuhiro Iwamatsu

[U-Boot] [PATCH] ARM: rmobile: Place initial stack right below U-Boot

2017-09-15 Thread Marek Vasut
Place the stack right below U-Boot so it's not in the way in case U-Boot grows too much. Signed-off-by: Marek Vasut Cc: Nobuhiro Iwamatsu --- include/configs/rcar-gen3-common.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

Re: [U-Boot] [U-Boot,v3,03/21] efi: add some missing __packed

2017-09-15 Thread Rob Clark
On Fri, Sep 15, 2017 at 2:53 PM, Heinrich Schuchardt wrote: > On 09/14/2017 12:05 AM, Rob Clark wrote: >> All of the device-path related structures should be packed. UEFI >> defines the device-path as a byte-aligned data structure. >> >> Signed-off-by: Rob Clark

Re: [U-Boot] [U-Boot,v3,03/21] efi: add some missing __packed

2017-09-15 Thread Heinrich Schuchardt
On 09/14/2017 12:05 AM, Rob Clark wrote: > All of the device-path related structures should be packed. UEFI > defines the device-path as a byte-aligned data structure. > > Signed-off-by: Rob Clark > --- > include/efi_api.h | 8 > 1 file changed, 4 insertions(+), 4

[U-Boot] [PATCH] MIPS: Drop unused PTR_COUNT_SHIFT from u-boot.lds

2017-09-15 Thread Paul Burton
The u-boot.lds linker script for MIPS defines a PTR_COUNT_SHIFT macro to 2 or 3 for 32 bit or 64 bit builds respectively. This macro is never actually used though, so remove the dead code. Signed-off-by: Paul Burton Cc: Daniel Schwierzeck

[U-Boot] [PATCH] boston: Remove unused label in lowlevel_display

2017-09-15 Thread Paul Burton
The lowlevel_display() function includes a "1:" label which is never used. Remove it. Signed-off-by: Paul Burton Cc: Daniel Schwierzeck --- board/imgtec/boston/lowlevel_init.S | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

Re: [U-Boot] [U-Boot,v3,01/21] part: move efi_guid_t

2017-09-15 Thread Heinrich Schuchardt
Hello Alex, I guess it makes most sense if you pick up this patch in your efi-next branch as it touches include/efi.h and otherwise we would have to load this onto Tom. Best regards Heinrich On 09/15/2017 08:19 PM, Heinrich Schuchardt wrote: > On 09/14/2017 12:05 AM, Rob Clark wrote: >> Prep

[U-Boot] [PATCH] boston: Drop unused return value

2017-09-15 Thread Paul Burton
The boston lowlevel_init() function zeroes the return register v0, despite the function not being expected to return a value & that value never being used. Remove the redundant assignment to v0. Signed-off-by: Paul Burton Cc: Daniel Schwierzeck

Re: [U-Boot] [U-Boot,v3,01/21] part: move efi_guid_t

2017-09-15 Thread Heinrich Schuchardt
On 09/14/2017 12:05 AM, Rob Clark wrote: > Prep work for next patch. > > Signed-off-by: Rob Clark > --- > include/efi.h | 4 > include/part.h | 3 ++- > include/part_efi.h | 4 > 3 files changed, 6 insertions(+), 5 deletions(-) > > diff --git

[U-Boot] [PATCH V2] test/py: gpt: make use of infra-structure

2017-09-15 Thread Stephen Warren
From: Stephen Warren Make various changes to the GPT test: 1) Reference the disk image using an absolute path in all cases. This allows test/py to operate correctly if it's run from a directory other than the root of the U-Boot source tree. 2) Store the disk image in the

Re: [U-Boot] [PATCH] test/py: gpt: make use of infra-structure

2017-09-15 Thread Tom Rini
On Fri, Sep 15, 2017 at 11:25:51AM -0600, Stephen Warren wrote: > From: Stephen Warren > > Make various changes to the GPT test: > > 1) Reference the disk image using an absolute path in all cases. This > allows test/py to operate correctly if it's run from a directory

Re: [U-Boot] [PATCH] Makefile: Update minimum dtc version to 1.4.3

2017-09-15 Thread Tom Rini
On Fri, Sep 15, 2017 at 11:33:23AM -0600, Stephen Warren wrote: > On 09/15/2017 11:15 AM, Tom Rini wrote: > >With support for overlays and calling the -@ flag to dtc we need to have > >at least 1.4.3 available now. > > >diff --git a/Makefile b/Makefile > > > checkdtc: > >-@if test $(call

Re: [U-Boot] [PATCH] Makefile: Update minimum dtc version to 1.4.3

2017-09-15 Thread Stephen Warren
On 09/15/2017 11:15 AM, Tom Rini wrote: With support for overlays and calling the -@ flag to dtc we need to have at least 1.4.3 available now. diff --git a/Makefile b/Makefile checkdtc: - @if test $(call dtc-version) -lt 0104; then \ - echo '*** Your dtc is too old,

[U-Boot] [PATCH] test/py: gpt: make use of infra-structure

2017-09-15 Thread Stephen Warren
From: Stephen Warren Make various changes to the GPT test: 1) Reference the disk image using an absolute path in all cases. This allows test/py to operate correctly if it's run from a directory other than the root of the U-Boot source tree. 2) Store the disk image in the

[U-Boot] [PATCH] Makefile: Update minimum dtc version to 1.4.3

2017-09-15 Thread Tom Rini
With support for overlays and calling the -@ flag to dtc we need to have at least 1.4.3 available now. Cc: Simon Glass Reported-by: Stephen Warren Signed-off-by: Tom Rini --- Simon, please make sure to grab this one too with

Re: [U-Boot] [PATCH v3 11/14] dtbo: make dtbos special

2017-09-15 Thread Tom Rini
On Fri, Sep 15, 2017 at 10:21:28AM -0600, Stephen Warren wrote: > On 09/15/2017 10:13 AM, Tom Rini wrote: > >On Fri, Sep 15, 2017 at 09:52:41AM -0600, Stephen Warren wrote: > >>On 09/15/2017 09:50 AM, Stephen Warren wrote: > >>>On 09/04/2017 02:12 PM, Pantelis Antoniou wrote: > Special rule

Re: [U-Boot] [U-Boot, 1/1] add pytests for 'gpt rename' and 'gpt swap'

2017-09-15 Thread Stephen Warren
On 09/15/2017 10:16 AM, Tom Rini wrote: On Fri, Sep 15, 2017 at 09:40:29AM -0600, Stephen Warren wrote: On 09/15/2017 06:29 AM, Tom Rini wrote: On Sat, Sep 09, 2017 at 11:54:51PM -0700, ali...@peloton-tech.com wrote: From: Alison Chaiken Add unit tests for the 'gpt

Re: [U-Boot] [PATCH v3 11/14] dtbo: make dtbos special

2017-09-15 Thread Stephen Warren
On 09/15/2017 10:13 AM, Tom Rini wrote: On Fri, Sep 15, 2017 at 09:52:41AM -0600, Stephen Warren wrote: On 09/15/2017 09:50 AM, Stephen Warren wrote: On 09/04/2017 02:12 PM, Pantelis Antoniou wrote: Special rule for dtbo generation diff --git a/scripts/Makefile.lib b/scripts/Makefile.lib

Re: [U-Boot] [U-Boot, 1/1] add pytests for 'gpt rename' and 'gpt swap'

2017-09-15 Thread Tom Rini
On Fri, Sep 15, 2017 at 09:40:29AM -0600, Stephen Warren wrote: > On 09/15/2017 06:29 AM, Tom Rini wrote: > >On Sat, Sep 09, 2017 at 11:54:51PM -0700, ali...@peloton-tech.com wrote: > >>From: Alison Chaiken > >> > >>Add unit tests for the 'gpt rename' and 'gpt swap'

Re: [U-Boot] [PATCH v3 11/14] dtbo: make dtbos special

2017-09-15 Thread Tom Rini
On Fri, Sep 15, 2017 at 09:52:41AM -0600, Stephen Warren wrote: > On 09/15/2017 09:50 AM, Stephen Warren wrote: > >On 09/04/2017 02:12 PM, Pantelis Antoniou wrote: > >>Special rule for dtbo generation > > > >>diff --git a/scripts/Makefile.lib b/scripts/Makefile.lib > > > >>+quiet_cmd_dtco = DTCO

  1   2   >