Re: [U-Boot] [PATCH v3 12/12] tests: Introduce DT overlay tests

2016-06-26 Thread David Gibson
On Mon, Jun 27, 2016 at 08:30:28AM +0200, Maxime Ripard wrote: > Hi David, > > On Mon, Jun 27, 2016 at 03:26:58PM +1000, David Gibson wrote: > > On Fri, Jun 24, 2016 at 04:27:57PM +0200, Maxime Ripard wrote: > > > This adds a bunch of unit tests for the "fdt apply" command. > > > > > > They've al

Re: [U-Boot] [PATCH v3 12/12] tests: Introduce DT overlay tests

2016-06-26 Thread Maxime Ripard
Hi David, On Mon, Jun 27, 2016 at 03:26:58PM +1000, David Gibson wrote: > On Fri, Jun 24, 2016 at 04:27:57PM +0200, Maxime Ripard wrote: > > This adds a bunch of unit tests for the "fdt apply" command. > > > > They've all been run successfully in the sandbox. However, as you still > > require an

Re: [U-Boot] [PATCH v3 00/12] cmd: fdt: Add device tree overlays support

2016-06-26 Thread Maxime Ripard
Hi David, On Mon, Jun 27, 2016 at 01:37:19AM +1000, David Gibson wrote: > On Fri, Jun 24, 2016 at 04:27:45PM +0200, Maxime Ripard wrote: > > Hi, > > > > The device tree overlays are a great solution to the issue raised by > > the bunch expandable boards we find everywhere these days, like the > >

[U-Boot] [PATCH v2 5/5] tools: patman: Handle missing 'END' in non-last commit of a series

2016-06-26 Thread Bin Meng
The following python error: Traceback (most recent call last): File "./tools/patman/patman", line 144, in series = patchstream.FixPatches(series, args) File "./tools/patman/patchstream.py", line 477, in FixPatches commit = series.commits[count] IndexError: list index out of range is

[U-Boot] [PATCH v2 2/5] tools: patman: Handle tag sections without an 'END'

2016-06-26 Thread Bin Meng
'Cover-letter', 'Series-notes' and 'Commit-notes' tags require an 'END' to be put at the end of its section. If we forget to put an 'END' in those sections, and these sections are followed by another patman tag, patman generates incorrect patches. This adds codes to handle such scenario. Signed-of

[U-Boot] [PATCH v2 4/5] tools: patman: Handle missing blank line for 'Series-changes'

2016-06-26 Thread Bin Meng
'Series-changes' uses blank line to indicate its end. If that is missing, series internal state variable 'in_change' may be wrong. Correct its state. Signed-off-by: Bin Meng Acked-by: Simon Glass --- Changes in v2: - Geneate warning when blank line is not found in 'Series-changes' tools/patm

[U-Boot] [PATCH v2 1/5] tools: patman: Use cover_match for 'Cover-letter'

2016-06-26 Thread Bin Meng
Like other patman tags, use a new variable cover_match to indicate a match for 'Cover-letter'. Signed-off-by: Bin Meng --- Changes in v2: - New patch to use cover_match for 'Cover-letter' tools/patman/patchstream.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tools/p

[U-Boot] [PATCH v2 3/5] tools: patman: Generate cover letter correctly when 'END' is missing

2016-06-26 Thread Bin Meng
If 'END' is missing in a 'Cover-letter' section, and that section happens to show up at the very end of the commit message, and the commit is the last commit of the series, patman fails to generate cover letter for us. Handle this in CloseCommit of patchstream. Signed-off-by: Bin Meng --- Chang

[U-Boot] [PATCH v2 0/5] tools: patman: Fixes to improve handling of various wrong usage

2016-06-26 Thread Bin Meng
This fixes several issues when abnormal tags are given to patman. For example, when we forget to put an 'END' after 'Cover-letter', patman does not handle such cases very well. This adds several additional checking to let patman be a little more tolerant on handling such exceptional paths. This se

[U-Boot] Bug in cbfs

2016-06-26 Thread Yaroslav K.
Hello. Current implementation of cbfs prints incorrect file names. In the file fs/cbfs/cbfs.c there are lines swap_file_header(&header, fileHeader); if (header.offset < sizeof(const struct cbfs_cachenode *) || header.offset > header.

Re: [U-Boot] [PATCH v2] rockchip: rk3288: Change method of loading u-boot

2016-06-26 Thread Ziyuan Xu
On 2016年06月17日 11:50, Simon Glass wrote: Hi, On 12 June 2016 at 01:39, Ziyuan Xu wrote: If we would like to boot from SD card, we have to implement mmc driver in SPL stage, and get a slightly large spl binrary. RK3288's bootrom code has the ability to load spl and u-boot. This patch tasks bo

Re: [U-Boot] [PATCH v3 12/12] tests: Introduce DT overlay tests

2016-06-26 Thread David Gibson
On Fri, Jun 24, 2016 at 04:27:57PM +0200, Maxime Ripard wrote: > This adds a bunch of unit tests for the "fdt apply" command. > > They've all been run successfully in the sandbox. However, as you still > require an out-of-tree dtc with overlay support, this is disabled by > default. > > Acked-by:

Re: [U-Boot] [PATCH v3 10/12] libfdt: Add overlay application function

2016-06-26 Thread David Gibson
On Fri, Jun 24, 2016 at 04:27:55PM +0200, Maxime Ripard wrote: > The device tree overlays are a good way to deal with user-modifyable > boards or boards with some kind of an expansion mechanism where we can > easily plug new board in (like the BBB, the Raspberry Pi or the CHIP). > > Add a new func

[U-Boot] [PATCH v6 4/5] atmel: Bring in at91 pio4 device tree file and bindings

2016-06-26 Thread Wenyou Yang
Bring in required device tree file and bindings from Linux. Signed-off-by: Wenyou Yang Reviewed-by: Andreas Bießmann Reviewed-by: Simon Glass --- Changes in v6: None Changes in v5: None Changes in v4: None Changes in v3: None Changes in v2: - add detailed example to show how to configure pinc

[U-Boot] [PATCH v6 5/5] configs: sama5d2_xplained: Add #ifndef before CONFIG_ATMEL_PIO4

2016-06-26 Thread Wenyou Yang
Since CONFIG_ATMEL_PIO4 options is added to Kconfig, to avoid compilation warnings when the option is enabled, add #ifndef CONFIG_DM_GPIO before #define CONFIG_ATMEL_PIO4 to avoid compilation warnings. Signed-off-by: Wenyou Yang Reviewed-by: Simon Glass --- Changes in v6: None Changes in v5: -

[U-Boot] [PATCH v6 3/5] pinctrl: at91-pio4: Add pinctrl driver

2016-06-26 Thread Wenyou Yang
AT91 PIO4 controller is a combined gpio-controller, pin-mux and pin-config module. The peripheral's pins are assigned through per-pin based muxing logic. The pin configuration is performed on specific registers which are shared along with the gpio controller. So regard the pinctrl device as a chil

[U-Boot] [PATCH v6 1/5] gpio: atmel_pio4: Move PIO4 definitions to head file

2016-06-26 Thread Wenyou Yang
In order to make these PIO4 definitions shared with AT91 PIO4 pinctrl driver, move them from the existing gpio driver to the head file, and rephrase them. Signed-off-by: Wenyou Yang Reviewed-by: Simon Glass --- Changes in v6: None Changes in v5: None Changes in v4: None Changes in v3: None Chan

[U-Boot] [PATCH v6 2/5] gpio: atmel_pio4: Rework to support DM & DT

2016-06-26 Thread Wenyou Yang
Rework the driver to support driver model and device tree, and support to regard the pio4 pinctrl device as a child of atmel_pio4 device. Signed-off-by: Wenyou Yang Reviewed-by: Simon Glass --- Changes in v6: - Add Reviewed-by tag. - Fixed the return value, -EINVAL -> ret. Changes in v5: -

[U-Boot] [PATCH v6 0/5] ARM: AT91: Add AT91 PIO4 pinctrl driver and gpio driver with DM

2016-06-26 Thread Wenyou Yang
AT91 PIO4 controller is a combined gpio-controller, pin-mux and pin-config module. This patch is to add the pinctrl driver, and rework the atmel-pio4 gpio driver to support driver model and device tree. Changes in v6: - Add Reviewed-by tag. - Fixed the return value, -EINVAL -> ret. Changes in

[U-Boot] [PATCH v4] clk: at91: Add clock driver

2016-06-26 Thread Wenyou Yang
The patch is referred to at91 clock driver of Linux, to make the clock node descriptions in DT aligned with the Linux's. Signed-off-by: Wenyou Yang Reviewed-by: Simon Glass --- Changes in v4: - Add Reviewed-by tag. - Add more information in Kconfig help. - Use u32 for num_parents variable, n

Re: [U-Boot] [PATCH v4] mmc: atmel_sdhci: Convert to the driver model support

2016-06-26 Thread Yang, Wenyou
Hi Simon, > -Original Message- > From: s...@google.com [mailto:s...@google.com] On Behalf Of Simon Glass > Sent: 2016年6月26日 10:53 > To: Yang, Wenyou > Cc: U-Boot Mailing List ; Andreas Bießmann > > Subject: Re: [PATCH v4] mmc: atmel_sdhci: Convert to the driver model support > > Hi Weny

Re: [U-Boot] [PATCH] mmc: dw_mmc: fix the wrong AND operation

2016-06-26 Thread Minkyu Kang
Hi, 2016년 5월 13일 금요일, Jaehoon Chung님이 작성한 메시지: > These condition checking are wrong. > Original Author's intention might be "&" instead of "&&". > It can know whether receive or transmit data request with > BIT[4]/BIT[5] of RINTSTS register. > > Signed-off-by: Jaehoon Chung > > --- > drivers/mmc

[U-Boot] [PATCH] mmc: msm_sdhci: Set mmc->dev pointer in msm_sdc_probe()

2016-06-26 Thread Mateusz Kulikowski
MMC core expects (now) valid mmc->dev pointer. During conversion in commit cffe5d86 not every driver was updated. This patch fixes crash while accessing MMC on boards using Qualcomm SDHCI controller. Signed-off-by: Mateusz Kulikowski --- This patch fixes MMC support on dragonboard. Without it a

[U-Boot] [PULL] u-boot-atmel/master -> u-boot/master

2016-06-26 Thread Andreas Bießmann
Hi Tom, please pull the following into u-boot/master for 2016.07. It contains the spl_boot_mode() change from Marek which was delegated partly to you and partly to me on patchwork. The rest is DT support for some Siemens boards which where sent before MW close but missed in my last pull request.

Re: [U-Boot] ARM: at91: Don't invoke spl_boot_device() twice

2016-06-26 Thread Andreas Bießmann
Dear Marek Vasut, Marek Vasut writes: >Since the spl_boot_mode() is now passed the boot device to boot from, >make use of it instead of inquiring for the boot device again. This >allows board_boot_order() to function correctly. > >Signed-off-by: Marek Vasut >Cc: Andreas Bießmann >Reviewed-by: A

Re: [U-Boot] [U-Boot,v1,1/5] arm: at91: add CONFIG_AT91SAM9M10G45

2016-06-26 Thread Andreas Bießmann
Dear Heiko Schocher, Heiko Schocher writes: >add support for CONFIG_AT91SAM9M10G45. > >Signed-off-by: Heiko Schocher >Reviewed-by: Andreas Bießmann >--- > > arch/arm/mach-at91/include/mach/at91_matrix.h | 26 ++ > 1 file changed, 14 insertions(+), 12 deletions(-) applied

Re: [U-Boot] [U-Boot, v1, 3/5] arm: at91: dts: Bring in dts files for AT91SAM9G20 and SAM9260

2016-06-26 Thread Andreas Bießmann
Dear Heiko Schocher, Heiko Schocher writes: >Add this files from Linux v4.6-rc5 > >66b8a424d: [workqueue: fix ghost PENDING flag while doing MQ IO] > >Signed-off-by: Heiko Schocher >Acked-by: Andreas Bießmann >--- > > arch/arm/dts/at91sam9260.dtsi | 1034

Re: [U-Boot] [U-Boot, v1, 4/5] arm: at91: smartweb: add DM and DTS support

2016-06-26 Thread Andreas Bießmann
Dear Heiko Schocher, Heiko Schocher writes: >Signed-off-by: Heiko Schocher >Reviewed-by: Andreas Bießmann >[rebased on current ToT] >Signed-off-by: Andreas Bießmann >--- > arch/arm/dts/Makefile | 1 + > arch/arm/dts/at91sam9260-smartweb.dts | 110 ++

Re: [U-Boot] [U-Boot, v1, 5/5] arm: at91: taurus/axm: add DM and DTS support

2016-06-26 Thread Andreas Bießmann
Dear Heiko Schocher, Heiko Schocher writes: >add DM and DTS support for the at91 based siemens >boards. > >Signed-off-by: Heiko Schocher >Reviewed-by: Andreas Bießmann >[rebased on current ToT] >Signed-off-by: Andreas Bießmann >--- > arch/arm/dts/Makefile | 1 + > arch/arm/dts/a

Re: [U-Boot] [U-Boot,v1,2/5] corvus DTS / DM support

2016-06-26 Thread Andreas Bießmann
Dear Heiko Schocher, Heiko Schocher writes: >Signed-off-by: Heiko Schocher >Reviewed-by: Andreas Bießmann >[rebase on current ToT, don't delete gurnard DTB creation] >Signed-off-by: Andreas Bießmann >--- > arch/arm/dts/Makefile | 4 +- > arch/arm/dts/at91sam9g45-corvus.dts | 108

Re: [U-Boot] common: Pass the boot device into spl_boot_mode()

2016-06-26 Thread Andreas Bießmann
Dear Marek Vasut, Marek Vasut writes: >The SPL code already knows which boot device it calls the spl_boot_mode() >on, so pass that information into the function. This allows the code of >spl_boot_mode() avoid invoking spl_boot_device() again, but it also lets >board_boot_order() correctly alter t

Re: [U-Boot] [PATCH v2 6/9] spl: fit: add support for post-processing of images

2016-06-26 Thread Andreas Dannenberg
On Sat, Jun 25, 2016 at 08:54:19PM -0600, Simon Glass wrote: > Hi Andreas, > > On 24 June 2016 at 09:57, Andreas Dannenberg wrote: > > From: Daniel Allred > > > > The next stage boot loader image and the selected FDT can be post- > > processed by board/platform/device-specific code, which can in

Re: [U-Boot] [PATCH v3 00/12] cmd: fdt: Add device tree overlays support

2016-06-26 Thread David Gibson
On Fri, Jun 24, 2016 at 04:27:45PM +0200, Maxime Ripard wrote: > Hi, > > The device tree overlays are a great solution to the issue raised by > the bunch expandable boards we find everywhere these days, like the > Beaglebone, Raspberry Pi or CHIP. > > However, most of the time, the overlays are a

Re: [U-Boot] [PATCH v3 04/12] libfdt: Add new headers and defines

2016-06-26 Thread David Gibson
On Fri, Jun 24, 2016 at 04:27:49PM +0200, Maxime Ripard wrote: > The libfdt overlay support introduces a bunch of new includes and > functions. > > Make sure we are able to build it by adding the needed glue. > > Signed-off-by: Maxime Ripard Is this intended only for downstream u-boot only, or

Re: [U-Boot] [PATCH v3 07/12] libfdt: Add fdt_setprop_inplace_by_index

2016-06-26 Thread David Gibson
On Fri, Jun 24, 2016 at 04:27:52PM +0200, Maxime Ripard wrote: > Add a function to modify inplace a property starting from a given index. > > This is especially useful when the property is an array of values, and you > want to update one of them without changing the DT size. > > Signed-off-by: Ma

Re: [U-Boot] [PATCH v3 08/12] libfdt: Add fdt_path_offset_namelen

2016-06-26 Thread David Gibson
On Fri, Jun 24, 2016 at 04:27:53PM +0200, Maxime Ripard wrote: > Add a namelen variant of fdt_path_offset to retrieve the node offset using > only a fixed number of characters. > > Signed-off-by: Maxime Ripard Looks good for upstream. > --- > include/libfdt.h| 16 +++- > lib/li

Re: [U-Boot] [PATCH v3 09/12] libfdt: Add fdt_getprop_namelen_w

2016-06-26 Thread David Gibson
On Fri, Jun 24, 2016 at 04:27:54PM +0200, Maxime Ripard wrote: > Add a function to retrieve a writeable property only by the first > characters of its name. > > Signed-off-by: Maxime Ripard > --- > include/libfdt.h | 7 +++ > 1 file changed, 7 insertions(+) > > diff --git a/include/libfdt.h

[U-Boot] [PATCH] sunxi: spl: Fix DRAM info printing

2016-06-26 Thread Hans de Goede
The switch to simple_printf was causing the SPL dram info to show as: DRAM: u MiB This fixes this by switching from %lu to %d for printing the DRAM size. Signed-off-by: Hans de Goede --- board/sunxi/board.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/board/sunxi/board.c

[U-Boot] [PATCH] sunxi: Add support for eth1addr

2016-06-26 Thread Hans de Goede
Currently we will already fill ethaddr with a fixed unique address based on the SoCs serial (from the sid) to make sure that boards which use the integrated emac / gmac get a fixed mac rather then a random one. On some boards (observed on 2 tablets using sdio rtl8703as wifi chips) the wifi does no

[U-Boot] [PATCH] MMC: Do not write to mmc_uclass_priv if it was not allocated

2016-06-26 Thread Matt Corallo
Signed-off-by: Matt Corallo --- drivers/mmc/mmc.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/mmc/mmc.c b/drivers/mmc/mmc.c index aabfc71..eba20f0 100644 --- a/drivers/mmc/mmc.c +++ b/drivers/mmc/mmc.c @@ -1734,7 +1734,8 @@ int mmc_init(struct mmc *mmc) #ifdef C

Re: [U-Boot] [PATCH v3 09/12] libfdt: Add fdt_getprop_namelen_w

2016-06-26 Thread Maxime Ripard
Hi Simon, On Sat, Jun 25, 2016 at 08:53:55PM -0600, Simon Glass wrote: > Hi Maxime, > > On 24 June 2016 at 08:27, Maxime Ripard > wrote: > > Add a function to retrieve a writeable property only by the first > > characters of its name. > > > > Signed-off-by: Maxime Ripard > > --- > > include/li

Re: [U-Boot] [PATCH v3 08/12] libfdt: Add fdt_path_offset_namelen

2016-06-26 Thread Maxime Ripard
Hi, On Sat, Jun 25, 2016 at 08:53:53PM -0600, Simon Glass wrote: > On 24 June 2016 at 08:27, Maxime Ripard > wrote: > > Add a namelen variant of fdt_path_offset to retrieve the node offset using > > only a fixed number of characters. > > > > Signed-off-by: Maxime Ripard > > --- > > include/libf

Re: [U-Boot] [PATCH v3 07/12] libfdt: Add fdt_setprop_inplace_by_index

2016-06-26 Thread Maxime Ripard
Hi Simon, On Sat, Jun 25, 2016 at 08:53:50PM -0600, Simon Glass wrote: > On 24 June 2016 at 08:27, Maxime Ripard > wrote: > > Add a function to modify inplace a property starting from a given index. > > > > This is especially useful when the property is an array of values, and you > > want to upd