Re: [U-Boot] Sdcard boot for a generic allwinner q8 clone A33 tablet not work.

2016-09-12 Thread far5893
Hans de Goede wrote: Hi, On 09-09-16 14:10, far5893 wrote: Hi, i'm trying to boot a classic A33 q8 clone with uboot copied in sdcard but nothing happen (not backlight on) dd if=u-boot-sunxi-with-spl.bin of=/dev/sdb bs=1024 seek=8 on the partitioned sdcard for reference In my experience the

Re: [U-Boot] [PATCHv5 1/4] Revert "tegra: Add support for setting up a as3722 PMIC"

2016-09-12 Thread Julian Scheel
On 06.09.2016 20:49, Stephen Warren wrote: On 09/06/2016 12:04 PM, Julian Scheel wrote: On 06.09.16 18:50, Stephen Warren wrote: On 09/05/2016 07:29 AM, Julian Scheel wrote: This reverts commit 346451b5888c4663bccef4c5203357319aa41f99. The pmic shall be initialised in board specific code, as i

[U-Boot] [PATCH] sunxi: axp2xx: disable ldoio0/1 at boot

2016-09-12 Thread Hans de Goede
When cold-booting the ldoio0/1 regulators are always off / the gpios are always at tristate. But when re-booting from android these are sometimes on. Disable them at axp_init time (iow as early as possible) to remove this difference between a cold boot and a reboot. Signed-off-by: Hans de Goede -

Re: [U-Boot] [PATCH] sf: fix sf probe

2016-09-12 Thread Stefan Roese
Hi Tom, On 25.08.2016 13:13, Stefan Roese wrote: (Added Tom to Cc) On 17.08.2016 09:19, Wenyou Yang wrote: From: Cyrille Pitchen This patch fixes the "sf probe" command. The very first SPI flash probe passes, for instance when u-boot tries to read its environment settings from a (Q)SPI memor

[U-Boot] [PATCH] cmd: booti: fix the image runtime location

2016-09-12 Thread Peng Fan
We should not use "bi_dram[0].start + text_offset" as the image dst. The text_offset maybe 0 for some images, such as XEN. Then the dst is actually bi_dram[0].start, which maybe the location of spin table. Let's use "images->ep & ~(ih->text_offset)" as the dst address. Signed-off-by: Peng Fan Cc

Re: [U-Boot] [PATCH] cmd: booti: fix the image runtime location

2016-09-12 Thread Peng Fan
On Mon, Sep 12, 2016 at 05:07:58PM +0800, Peng Fan wrote: >We should not use "bi_dram[0].start + text_offset" as the image dst. >The text_offset maybe 0 for some images, such as XEN. Then the dst >is actually bi_dram[0].start, which maybe the location of spin table. > >Let's use "images->ep & ~(ih-

Re: [U-Boot] [PATCH 1/4] fs/fat: Remove two statements without effect

2016-09-12 Thread Lukasz Majewski
Hi Stefan, > fatlength is a local variable which is no more used after the > assignment. s_name is not used in the function, save the strncpy. > > Signed-off-by: Stefan Brüns > --- > fs/fat/fat_write.c | 5 + > 1 file changed, 1 insertion(+), 4 deletions(-) > > diff --git a/fs/fat/fat_writ

Re: [U-Boot] [PATCH 2/4] fs/fat: Do not write unmodified fat entries to disk

2016-09-12 Thread Lukasz Majewski
Hi Stefan, > The code caches 6 sectors of the FAT. On FAT traversal, the old > contents needs to be flushed to disk, but only if any FAT entries had > been modified. Explicitly flag the buffer on modification. > > Currently, creating a new file traverses the whole FAT up to the first > free clust

Re: [U-Boot] [PATCH 3/4] fs/fat: Correct description of determine_fatent function

2016-09-12 Thread Lukasz Majewski
Hi Stefan, > Current description does not match the function behaviour. > > Signed-off-by: Stefan Brüns > --- > fs/fat/fat_write.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/fs/fat/fat_write.c b/fs/fat/fat_write.c > index babe9c8..0583af3 100644 > --- a/fs/fat/

[U-Boot] question about himport_r() and possible separator

2016-09-12 Thread Robert P. J. Day
admittedly a simple question that i'll be able to test once i get into the lab later today, but if someone knows the answer, terrific. putting u-boot on a target board where there already exists, at a well-known address, a set of "var=val" settings, with the format: var1=val1 var2=val2 ...

Re: [U-Boot] [PATCH] cmd: booti: fix the image runtime location

2016-09-12 Thread Tom Rini
On Mon, Sep 12, 2016 at 05:18:53PM +0800, Peng Fan wrote: > On Mon, Sep 12, 2016 at 05:07:58PM +0800, Peng Fan wrote: > >We should not use "bi_dram[0].start + text_offset" as the image dst. > >The text_offset maybe 0 for some images, such as XEN. Then the dst > >is actually bi_dram[0].start, which

Re: [U-Boot] [PATCH] sunxi: axp2xx: disable ldoio0/1 at boot

2016-09-12 Thread Ian Campbell
On Mon, 2016-09-12 at 09:55 +0200, Hans de Goede wrote: > @@ -223,7 +223,14 @@ int axp_init(void) > >   if (!(axp_chip_id == 0x6 || axp_chip_id == 0x7 || axp_chip_id == 0x17)) > >   return -ENODEV; >   > > - return 0; > > + /* > > +  * Turn off LDOIO regulators / tri-state GPIO pi

Re: [U-Boot] sf: fix sf probe

2016-09-12 Thread Tom Rini
On Wed, Aug 17, 2016 at 03:19:39PM +0800, Wenyou Yang wrote: > From: Cyrille Pitchen > > This patch fixes the "sf probe" command. The very first SPI flash probe > passes, for instance when u-boot tries to read its environment settings > from a (Q)SPI memory but next "sf probe" commands fail beca

[U-Boot] [PATCH v2] sunxi: axp2xx: disable ldoio0/1 at boot

2016-09-12 Thread Hans de Goede
When cold-booting the ldoio0/1 regulators are always off / the gpios are always at tristate. But when re-booting from android these are sometimes on. Disable them at axp_init time (iow as early as possible) to remove this difference between a cold boot and a reboot. Signed-off-by: Hans de Goede -

Re: [U-Boot] [PATCH] sunxi: axp2xx: disable ldoio0/1 at boot

2016-09-12 Thread Hans de Goede
Hi, On 12-09-16 14:41, Ian Campbell wrote: On Mon, 2016-09-12 at 09:55 +0200, Hans de Goede wrote: @@ -223,7 +223,14 @@ int axp_init(void) if (!(axp_chip_id == 0x6 || axp_chip_id == 0x7 || axp_chip_id == 0x17)) return -ENODEV; - return 0; + /* +*

[U-Boot] [bug report] sunxi: booting from eMMC

2016-09-12 Thread Ciprian Manea
Hi, I'm using a SinA33 dev board (Allwinner a33 SoC) and at the moment I'm trying to boot from the eMMC. But it fails in the following way: *Got the following error at boot time:* *U-Boot SPL 2016.09-rc2-00125-g6e8b42f (Sep 08 2016 - 11:00:46) DRAM: 1024 MiB Trying to boot from MMC2 MMC Dev

Re: [U-Boot] [PATCH] Increase default of CONFIG_SYS_MALLOC_F_LEN for SPL_OF_CONTROL

2016-09-12 Thread Simon Glass
Hi Masahiro, On 11 September 2016 at 22:42, Masahiro Yamada wrote: > Hi Simon, > > > 2016-09-12 13:21 GMT+09:00 Simon Glass : >> Hi Masahiro, >> >> On 11 September 2016 at 06:50, Masahiro Yamada >> wrote: >>> Hi Simon, >>> >>> >>> 2016-09-06 21:46 GMT+09:00 Simon Glass : Hi Masahiro, >

Re: [U-Boot] [bug report] sunxi: booting from eMMC

2016-09-12 Thread Maxime Ripard
Hi, On Mon, Sep 12, 2016 at 01:53:24PM +0200, Ciprian Manea wrote: > I'm using a SinA33 dev board (Allwinner a33 SoC) and at the moment > I'm trying to boot from the eMMC. > > But it fails in the following way: > > > *Got the following error at boot time:* > *U-Boot SPL 2016.09-rc2-00125-g6e8b4

Re: [U-Boot] [PATCHv5 4/4] Add support the Avionic Design Meerkat COM and Kein Baseboard

2016-09-12 Thread Julian Scheel
On 06.09.2016 19:15, Stephen Warren wrote: diff --git a/board/avionic-design/common/meerkat.c b/board/avionic-design/common/meerkat.c +void pinmux_init(void) +{ +pinmux_set_tristate_input_clamping(); That should be pinmux_clear_tristate_input_clamping(); gpio_config_table() is missing h

Re: [U-Boot] [PATCH v2 21/44] Convert CONFIG_SPL_GPIO_SUPPORT to Kconfig

2016-09-12 Thread Tom Rini
On Mon, Sep 12, 2016 at 01:32:54PM +0900, Masahiro Yamada wrote: > Hi Simon, > > > 2016-09-12 13:16 GMT+09:00 Simon Glass : > > Hi, > > > > On 6 September 2016 at 09:54, Tom Rini wrote: > >> On Mon, Sep 05, 2016 at 07:04:45PM -0600, Simon Glass wrote: > >>> Hi Masahiro, > >>> > >>> On 4 Septembe

Re: [U-Boot] [PATCH v2] sunxi: axp2xx: disable ldoio0/1 at boot

2016-09-12 Thread Ian Campbell
On Mon, 2016-09-12 at 15:24 +0200, Hans de Goede wrote: > When cold-booting the ldoio0/1 regulators are always off / the > gpios are always at tristate. But when re-booting from android these > are sometimes on. Disable them at axp_init time (iow as early as > possible) > to remove this difference

Re: [U-Boot] [PATCH 1/2] configs: Resync with savedefconfig

2016-09-12 Thread Tom Rini
On Mon, Sep 12, 2016 at 11:24:35AM +0900, Masahiro Yamada wrote: > 2016-09-10 7:25 GMT+09:00 Stephen Warren : > > On 09/09/2016 03:14 PM, Tom Rini wrote: > >> > >> On Fri, Sep 09, 2016 at 03:06:01PM -0600, Stephen Warren wrote: > >>> > >>> On 09/09/2016 01:11 PM, Tom Rini wrote: > > On Fr

[U-Boot] [PATCH] wandboard: Fix hang when going into low frequency

2016-09-12 Thread Fabio Estevam
From: Fabio Estevam A kernel hang is observed when running wandboard 3.14 kernel and going to the lowest operational point of cpufreq: # ifconfig eth0 down # echo 1 > /sys/class/graphics/fb0/blank The problem is caused by incorrect set

Re: [U-Boot] [bug report] sunxi: booting from eMMC

2016-09-12 Thread Hans de Goede
Hi, On 12-09-16 15:56, Maxime Ripard wrote: Hi, On Mon, Sep 12, 2016 at 01:53:24PM +0200, Ciprian Manea wrote: I'm using a SinA33 dev board (Allwinner a33 SoC) and at the moment I'm trying to boot from the eMMC. But it fails in the following way: *Got the following error at boot time:* *U-B

[U-Boot] [PATCH] wandboard: Remove videoargs script

2016-09-12 Thread Fabio Estevam
From: Fabio Estevam The videoargs script is kernel version dependent and since wandboard uses distro config, there is no need to handle videoargs locally. In case such video related settings are needed, then the proper location would be the distro extlinux.conf or boot.scr files. So remove 'vid

[U-Boot] [ANN] U-Boot v2016.09 is released

2016-09-12 Thread Tom Rini
Hey all, I've released v2016.09 and it's now live on git and FTP and ACD (along with PGP sig file). To repeat some of the highlights from the rc releases: - More DM work (MMC, of-platdata for size constrained instances, etc) - Lots and lots of architecture / SoC / Platform updates: x86, rockchip,

Re: [U-Boot] [PATCH] efi: console: Correctly report modes

2016-09-12 Thread Emmanuel Vadot
Hello, Any comments on this patch ? Cheers, On Fri, 19 Aug 2016 18:12:50 +0200 Emmanuel Vadot wrote: > Hello Alexander, > > The UEFI spec say that mode 0 will always be 80x25, mode 1 80x50 and other > size will be mode >=2. > This patch : > - set the default size to 80x25. > - returns EF

Re: [U-Boot] [ANN] U-Boot v2016.09 is released

2016-09-12 Thread Michal Simek
Hi Tom, 2016-09-12 18:21 GMT+02:00 Tom Rini : > Hey all, > > I've released v2016.09 and it's now live on git and FTP and ACD (along > with PGP sig file). > > To repeat some of the highlights from the rc releases: > - More DM work (MMC, of-platdata for size constrained instances, etc) > - Lots and

Re: [U-Boot] [PATCH v7 01/12] clk: Use dummy clk_get_by_* functions when CONFIG_CLK is disabled

2016-09-12 Thread Stephen Warren
On 09/11/2016 07:25 AM, Masahiro Yamada wrote: Hi Paul, 2016-09-09 17:01 GMT+09:00 Paul Burton : On 09/09/16 04:15, Masahiro Yamada wrote: 2016-09-08 15:47 GMT+09:00 Paul Burton : The implementations of clk_get_by_index & clk_get_by_name are only available when CONFIG_CLK is enabled. Unless

Re: [U-Boot] [PATCHv5 4/4] Add support the Avionic Design Meerkat COM and Kein Baseboard

2016-09-12 Thread Stephen Warren
On 09/12/2016 08:03 AM, Julian Scheel wrote: On 06.09.2016 19:15, Stephen Warren wrote: diff --git a/board/avionic-design/common/meerkat.c b/board/avionic-design/common/meerkat.c +void pinmux_init(void) +{ +pinmux_set_tristate_input_clamping(); That should be pinmux_clear_tristate_input

Re: [U-Boot] [PATCHv5 1/4] Revert "tegra: Add support for setting up a as3722 PMIC"

2016-09-12 Thread Stephen Warren
On 09/12/2016 01:33 AM, Julian Scheel wrote: On 06.09.2016 20:49, Stephen Warren wrote: On 09/06/2016 12:04 PM, Julian Scheel wrote: On 06.09.16 18:50, Stephen Warren wrote: On 09/05/2016 07:29 AM, Julian Scheel wrote: This reverts commit 346451b5888c4663bccef4c5203357319aa41f99. The pmic sha

Re: [U-Boot] [PATCHv5 1/4] Revert "tegra: Add support for setting up a as3722 PMIC"

2016-09-12 Thread Julian Scheel
On 12.09.16 19:02, Stephen Warren wrote: On 09/12/2016 01:33 AM, Julian Scheel wrote: On 06.09.2016 20:49, Stephen Warren wrote: On 09/06/2016 12:04 PM, Julian Scheel wrote: On 06.09.16 18:50, Stephen Warren wrote: On 09/05/2016 07:29 AM, Julian Scheel wrote: This reverts commit 346451b5888c

Re: [U-Boot] [ANN] U-Boot v2016.09 is released

2016-09-12 Thread Lukasz Majewski
Hi Tom, > Hey all, > > I've released v2016.09 and it's now live on git and FTP and ACD (along > with PGP sig file). > > To repeat some of the highlights from the rc releases: > - More DM work (MMC, of-platdata for size constrained instances, etc) > - Lots and lots of architecture / SoC / Platfor

Re: [U-Boot] [PATCH v2 1/4] disk: part: implement generic function part_get_info_by_name()

2016-09-12 Thread Steve Rae
Hi Petr, On Fri, Sep 9, 2016 at 1:27 AM, Petr Kulhavy wrote: > So far partition search by name has been supported only on the EFI partition > table. This patch extends the search to all partition tables. > > Rename part_get_info_efi_by_name() to part_get_info_by_name(), move it from > part_efi.c

Re: [U-Boot] [PATCH v2 2/4] fastboot: add support for writing MBR

2016-09-12 Thread Steve Rae
On Fri, Sep 9, 2016 at 1:27 AM, Petr Kulhavy wrote: > Add special target "mbr" (otherwise configurable via CONFIG_FASTBOOT_MBR_NAME) > to write MBR partition table. > Partitions are now searched using the generic function which finds any > partiiton by name. For MBR the partition names hda1, sda1,

Re: [U-Boot] [PATCH v2 3/4] disk: part: refactor generic name creation for DOS and ISO

2016-09-12 Thread Steve Rae
On Fri, Sep 9, 2016 at 1:27 AM, Petr Kulhavy wrote: > In both DOS and ISO partition tables the same code to create partition name > like "hda1" was repeated. > > Code moved to into a new function part_set_generic_name() in part.c and > optimized. > Added recognition of MMC and SD types, name is l

Re: [U-Boot] [PATCH v2 4/4] fastboot: move FASTBOOT_FLASH options into Kconfig

2016-09-12 Thread Steve Rae
On Fri, Sep 9, 2016 at 6:56 AM, Tom Rini wrote: > On Fri, Sep 09, 2016 at 10:27:18AM +0200, Petr Kulhavy wrote: > >> Move FASTBOOT_MBR_NAME and FASTBOOT_GPT_NAME into Kconfig. >> Add dependency on the FASTBOOT_FLASH setting (also for FASTBOOT_MBR_NAME). >> Remove the now redundant GPT_ENTRY_NAME.

Re: [U-Boot] [PATCH v2 0/4] Fastboot MBR and generic partition support

2016-09-12 Thread Steve Rae
On Fri, Sep 9, 2016 at 1:27 AM, Petr Kulhavy wrote: > This set extends the Fastboot implementation from GPT-only to any partition > support. Further it adds a special target "mbr" (configurable) to write the > DOS MBR. > > Version 2: > Add a fourth patch into the set to move CONFIG_FASTBOOT_GPT_N

[U-Boot] [PATCH 2/2] net: add driver for Synopsys Ethernet QoS device

2016-09-12 Thread Stephen Warren
From: Stephen Warren This driver supports the Synopsys Designware Ethernet QoS (Quality of Service) a/k/a eqos IP block, which is a different design than the HW supported by the existing designware.c driver. The IP supports many options for bus type, clocking/reset structure, and feature list. Th

[U-Boot] [PATCH 1/2] dt: net: add DWC EQoS binding

2016-09-12 Thread Stephen Warren
Tegra186, which is applied for next-20160912. So far, my changes have been acked by Lars Persson, the original author of the binding. Signed-off-by: Stephen Warren --- .../net/snps,dwc-qos-ethernet.txt | 166 + 1 file changed, 166 insertions(+) create mode

[U-Boot] [PATCH 1/4] ARM: tegra: add SoC-level hook for board_late_init()

2016-09-12 Thread Stephen Warren
From: Stephen Warren Extend the Tegra186 implementation of board_late_init() to call a per-SoC "hook" function. This will allow SoC-specific (rather than Tegra-wide) functionality to be implemented without the core Tegra code needing to be aware of the details. While board186.c is currently only

[U-Boot] [PATCH 4/4] ARM: tegra: enable Ethernet on p2771-0000

2016-09-12 Thread Stephen Warren
From: Stephen Warren Enable the Ethernet device in DT, provide board-specific configuration, and enable the driver in Kconfig. Signed-off-by: Stephen Warren --- arch/arm/dts/tegra186-p2771-.dtsi | 5 + configs/p2771--000_defconfig | 1 + configs/p2771--500_defconfig |

[U-Boot] [PATCH 3/4] ARM: tegra: add DWC EQoS (ethernet) to Tegra186 DT

2016-09-12 Thread Stephen Warren
From: Stephen Warren Tegra186 includes a Synopsys DWC EQoS (Ethernet) device. Add this to the Tegra186 SoC DT so that boards can make use of it. Signed-off-by: Stephen Warren --- arch/arm/dts/tegra186.dtsi | 20 1 file changed, 20 insertions(+) diff --git a/arch/arm/dts/t

[U-Boot] [PATCH 2/4] ARM: tegra: configure Ethernet address on Tegra186

2016-09-12 Thread Stephen Warren
From: Stephen Warren On Tegra186, the bootloader which runs before U-Boot passes the Ethernet MAC address to U-Boot using device tree. Extract this value and write it to the environment, so that the Ethernet uclass picks it up and uses it for the built-in Ethernet device. Signed-off-by: Stephen

Re: [U-Boot] [PATCH v1 0/5] various fixes mainly for colibri_t20

2016-09-12 Thread Stephen Warren
On 09/09/2016 10:10 AM, Marcel Ziswiler wrote: This series addresses various issues as seen on Colibri T20. Please note that for successful Ethernet operation not only on Colibri T20 but also on Colibri T30 the following patch will still need applying as well: [PATCH] net: asix: Fix ASIX 88772B

Re: [U-Boot] [PATCH v1 1/5] tegra: usb gadget: fix ci udc operation if not hostpc capable

2016-09-12 Thread Stephen Warren
On 09/09/2016 10:10 AM, Marcel Ziswiler wrote: The Tegra 2 aka T20 is not host PC capable. Therefore move the define CONFIG_CI_UDC_HAS_HOSTPC from the generic tegra-common-usb-gadget.h header file into resp. SoC type specific ones. This is OK, but ... diff --git a/include/configs/tegra-common

Re: [U-Boot] [PATCH v1 3/5] colibri_t20: fix display configuration

2016-09-12 Thread Stephen Warren
On 09/09/2016 10:10 AM, Marcel Ziswiler wrote: Without this patch the following error will be shown: stdio_add_devices: Video device failed (ret=-22) As commit ec5507707a1d1e84056a6c864338f95f6118d3ca (video: tegra: Move to using simple-panel and pwm-backlight) states the Colibri T20 needs upda

Re: [U-Boot] [PATCH v1 4/5] colibri_t20: fix usb operation and controller order

2016-09-12 Thread Stephen Warren
On 09/09/2016 10:10 AM, Marcel Ziswiler wrote: Without this patch the following error will be shown: Colibri T20 # usb start starting USB... No controllers found This change seems fine, but I'm concerned that the alias order causes failures; arbitrary orders should be allowed. Have you determ

Re: [U-Boot] [PATCH v1 5/5] colibri_t20: enable dfu also for nand

2016-09-12 Thread Stephen Warren
On 09/09/2016 10:10 AM, Marcel Ziswiler wrote: Enable USB gadget DFU functionality for NAND as well. diff --git a/include/configs/colibri_t20.h b/include/configs/colibri_t20.h +/* USB DFU */ +#define CONFIG_DFU_NAND Oh, I see this file already includes tegra-common-usb-gadget.h, so USB d

Re: [U-Boot] [PATCHv5 1/4] Revert "tegra: Add support for setting up a as3722 PMIC"

2016-09-12 Thread Stephen Warren
On 09/12/2016 11:10 AM, Julian Scheel wrote: On 12.09.16 19:02, Stephen Warren wrote: On 09/12/2016 01:33 AM, Julian Scheel wrote: On 06.09.2016 20:49, Stephen Warren wrote: On 09/06/2016 12:04 PM, Julian Scheel wrote: On 06.09.16 18:50, Stephen Warren wrote: On 09/05/2016 07:29 AM, Julian S

Re: [U-Boot] [PATCH 1/2] configs: Resync with savedefconfig

2016-09-12 Thread Stephen Warren
On 09/11/2016 08:24 PM, Masahiro Yamada wrote: 2016-09-10 7:25 GMT+09:00 Stephen Warren : On 09/09/2016 03:14 PM, Tom Rini wrote: On Fri, Sep 09, 2016 at 03:06:01PM -0600, Stephen Warren wrote: On 09/09/2016 01:11 PM, Tom Rini wrote: On Fri, Sep 09, 2016 at 01:09:43PM -0600, Stephen Warren

Re: [U-Boot] [PATCH 1/2] configs: Resync with savedefconfig

2016-09-12 Thread Stephen Warren
On 09/12/2016 08:23 AM, Tom Rini wrote: On Mon, Sep 12, 2016 at 11:24:35AM +0900, Masahiro Yamada wrote: 2016-09-10 7:25 GMT+09:00 Stephen Warren : On 09/09/2016 03:14 PM, Tom Rini wrote: On Fri, Sep 09, 2016 at 03:06:01PM -0600, Stephen Warren wrote: On 09/09/2016 01:11 PM, Tom Rini wrote:

Re: [U-Boot] [PATCH 1/3] test/fs: Restructure file path specification to allow some flexibility

2016-09-12 Thread Stephen Warren
On 09/11/2016 02:46 PM, Stefan Brüns wrote: Instead of providing the full path, specify directory and filename separately. This allows to specify intermediate directories, required for some additional tests. diff --git a/test/fs/fs-test.sh b/test/fs/fs-test.sh @@ -166,11 +150,14 @@ function

Re: [U-Boot] [PATCH 2/3] test/fs: Check ext4 behaviour if dirent is first entry in directory block

2016-09-12 Thread Stephen Warren
On 09/11/2016 02:46 PM, Stefan Brüns wrote: This is a regression test for a crash happening if the first dirent in the block matches. Code tried to access a predecessor entry which does not exist. The crash happened for any block, but "." is always the first entry in the first directory block and

Re: [U-Boot] [PATCH 3/3] test/fs: Check writes using "." (same dir) relative path

2016-09-12 Thread Stephen Warren
On 09/11/2016 02:46 PM, Stefan Brüns wrote: / and /./ should reference the same file. diff --git a/test/fs/fs-test.sh b/test/fs/fs-test.sh +# Read 1MB from small file +${PREFIX}load host${SUFFIX} $addr ${FPATH}$FILE_SMALL I think the same issue with $FPATH ending/not-ending in / applies h

Re: [U-Boot] [PATCH 1/4] fs/fat: Remove two statements without effect

2016-09-12 Thread Stephen Warren
On 09/11/2016 02:51 PM, Stefan Brüns wrote: fatlength is a local variable which is no more used after the assignment. s_name is not used in the function, save the strncpy. The series, Acked-by: Stephen Warren ___ U-Boot mailing list U-Boot@lists.den

[U-Boot] [PATCH] cros_ec: Fix issue with cros_ec_flash_write command

2016-09-12 Thread Moritz Fischer
This commit fixes an issue where data is written to an invalid memory location. The issue has been introduced in commit 88364387 cros: add cros_ec_driver Signed-off-by: Moritz Fischer Cc: u-boot@lists.denx.de --- drivers/misc/cros_ec.c | 25 ++--- 1 file changed, 18 insertion

Re: [U-Boot] [PATCH] cros_ec: Fix issue with cros_ec_flash_write command

2016-09-12 Thread Moritz Fischer
Hi Simon, On Mon, Sep 12, 2016 at 1:48 PM, Moritz Fischer wrote: > - return ec_command_inptr(dev, EC_CMD_FLASH_WRITE, 0, > - &p, sizeof(p), NULL, 0) >= 0 ? 0 : -1; > + ret = ec_command_inptr(dev, EC_CMD_FLASH_WRITE, 1, > + p, sizeof(*p)

[U-Boot] [PATCH v2] cros_ec: Fix issue with cros_ec_flash_write command

2016-09-12 Thread Moritz Fischer
This commit fixes an issue where data is written to an invalid memory location. The issue has been introduced in commit (88364387 cros: add cros_ec_driver) Cc: Simon Glass Cc: u-boot@lists.denx.de Signed-off-by: Moritz Fischer --- Changes from v1: - Fixed accidential change of command version

Re: [U-Boot] [PATCH 1/2] armv8/fsl-lsch2: refactor the clock system initialization

2016-09-12 Thread york sun
Prabhakar, On 09/11/2016 09:20 PM, Zhiqiang Hou wrote: > From: Hou Zhiqiang > > Up to now, there are 3 kinds of SoC under Layerscape Chassis 2, > such as LS1043A, LS1046A and LS1012A. But the clocks tree has a > lot of difference, for instance the IP modules have different > divisors to get clock

Re: [U-Boot] [PATCH 1/4] fs/fat: Remove two statements without effect

2016-09-12 Thread Benoît Thébaudeau
Hi Stefan, On Sun, Sep 11, 2016 at 10:51 PM, Stefan Brüns wrote: > fatlength is a local variable which is no more used after the assignment. > s_name is not used in the function, save the strncpy. > > Signed-off-by: Stefan Brüns [...] For the series: Reviewed-by: Benoît Thébaudeau Best regar

Re: [U-Boot] [PATCH 2/3] test/fs: Check ext4 behaviour if dirent is first entry in directory block

2016-09-12 Thread Stefan Bruens
On Montag, 12. September 2016 12:39:35 CEST you wrote: > On 09/11/2016 02:46 PM, Stefan Brüns wrote: > > This is a regression test for a crash happening if the first dirent > > in the block matches. Code tried to access a predecessor entry which > > does not exist. > > The crash happened for any bl

Re: [U-Boot] [v2, 1/5] mmc: fsl_esdhc: don't set XFERTYP_RSPTYP_48_BUSY for CMD with busy response

2016-09-12 Thread york sun
Panto, On 08/02/2016 02:32 AM, Yangbo Lu wrote: > For CMD with busy response, the eSDHC driver would poll DAT0 until > CMD completion rather than polling IRQSTAT. So, don't set > XFERTYP_RSPTYP_48_BUSY to avoid interrupts (DTOE or TC) in IRQSTAT. > > Signed-off-by: Yangbo Lu > --- > Changes for v

Re: [U-Boot] [PATCH 3/3] test/fs: Check writes using "." (same dir) relative path

2016-09-12 Thread Stefan Bruens
On Montag, 12. September 2016 12:44:08 CEST you wrote: > On 09/11/2016 02:46 PM, Stefan Brüns wrote: > > / and /./ should reference the same file. > > > > diff --git a/test/fs/fs-test.sh b/test/fs/fs-test.sh > > > > +# Read 1MB from small file > > +${PREFIX}load host${SUFFIX} $addr ${FPATH}$FILE_

[U-Boot] [PATCH v1 0/7] clk: at91: Improve the clock implementation

2016-09-12 Thread Wenyou Yang
Add the clock ops for such as spi0_clk, which is the real clock provider, instead of periph32ck, which only recursively bind its children as clk devices. Also update the clocks called in the drivers. Wenyou Yang (7): clk: clk-uclass: Assign clk->dev before call .of_xlate clk: at91: Improve th

[U-Boot] [PATCH v1 1/7] clk: clk-uclass: Assign clk->dev before call .of_xlate

2016-09-12 Thread Wenyou Yang
In order to make clk->dev available in ops->of_xlate() to get the clock ID from the 'reg' property of the clock node, assign the clk->dev before calling ops->of_xlate(). Signed-off-by: Wenyou Yang --- drivers/clk/clk-uclass.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/clk/cl

[U-Boot] [PATCH v1 3/7] gpio: atmel_pio4: Remove unneccessary clock calling

2016-09-12 Thread Wenyou Yang
Due to the peripheral clock driver improvement, remove the unneccessary clock calling. Signed-off-by: Wenyou Yang --- drivers/gpio/atmel_pio4.c | 12 1 file changed, 12 deletions(-) diff --git a/drivers/gpio/atmel_pio4.c b/drivers/gpio/atmel_pio4.c index 7adea88..cb90b02 100644 --

[U-Boot] [PATCH v1 2/7] clk: at91: Improve the clock implementation

2016-09-12 Thread Wenyou Yang
For the peripheral clock, provide the clock ops for the clock provider, such as spi0_clk. The .of_xlate is to get the clk->id, the .enable is to enable the spi0 peripheral clock, the .get_rate is to get the clock frequency. The driver for periph32ck node is responsible for recursively binding its

[U-Boot] [PATCH v1 4/7] i2c: at91_i2c: Remove unneccessary clock calling

2016-09-12 Thread Wenyou Yang
Due to the peripheral clock driver improvement, remove the unneccessary clock calling. Signed-off-by: Wenyou Yang --- drivers/i2c/at91_i2c.c | 16 1 file changed, 16 deletions(-) diff --git a/drivers/i2c/at91_i2c.c b/drivers/i2c/at91_i2c.c index 8e9c3ad..73f29e3 100644 --- a/d

[U-Boot] [PATCH v1 5/7] i2c: at91_i2c: Change error return -ENODEV to -EINVAL

2016-09-12 Thread Wenyou Yang
Change the error return value -ENODEV from to -EINVAL for more reasonable. Signed-off-by: Wenyou Yang --- drivers/i2c/at91_i2c.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/i2c/at91_i2c.c b/drivers/i2c/at91_i2c.c index 73f29e3..472420b 100644 --- a/drivers/i2c/at

[U-Boot] [PATCH v1 6/7] usb: ehci-atmel: Remove unneccessary clock calling

2016-09-12 Thread Wenyou Yang
Due to the peripheral clock driver improvement, remove the unneccessary clock calling. Signed-off-by: Wenyou Yang --- drivers/usb/host/ehci-atmel.c | 15 --- 1 file changed, 15 deletions(-) diff --git a/drivers/usb/host/ehci-atmel.c b/drivers/usb/host/ehci-atmel.c index d65bbe9..6c

[U-Boot] [PATCH v1 7/7] mmc: atmel_sdhci: Remove unneccessary clock calling

2016-09-12 Thread Wenyou Yang
Due to the peripheral and generated clock driver improvement, remove the unneccessary clock calling. Signed-off-by: Wenyou Yang --- drivers/mmc/atmel_sdhci.c | 27 ++- 1 file changed, 2 insertions(+), 25 deletions(-) diff --git a/drivers/mmc/atmel_sdhci.c b/drivers/mmc/

Re: [U-Boot] [PATCH v1 2/2] clk: at91: Add .ops callback for clk_generic

2016-09-12 Thread Wenyou.Yang
Hi Stephen, > -Original Message- > From: Stephen Warren [mailto:swar...@wwwdotorg.org] > Sent: 2016年9月3日 1:38 > To: Wenyou Yang - A41535 > Cc: swar...@nvidia.com; u-boot@lists.denx.de > Subject: Re: [U-Boot] [PATCH v1 2/2] clk: at91: Add .ops callback for > clk_generic > > On 09/01/2016

[U-Boot] [PATCH v9] dm: at91: Add driver model support for the spi driver

2016-09-12 Thread Wenyou Yang
Add driver model support while retaining the existing legacy code. This allows the driver to support boards that have converted to driver model as well as those that have not. Signed-off-by: Wenyou Yang Reviewed-by: Simon Glass --- Changes in v9: - Due to the peripheral clock driver improvemen

[U-Boot] [PATCH v2] clk: at91: Fix at91-pmc and at91-sckc's class ID

2016-09-12 Thread Wenyou Yang
The at91-pmc and at91-sckc aren't the clock providers, change their class ID from UCLASS_CLK to UCLASS_SIMPLE_BUS, they also don't need to bind the child nodes explicitly, the .post_bind callback of simple_bus uclass will do it for them. Signed-off-by: Wenyou Yang Acked-by: Stephen Warren Review

[U-Boot] [PATCH v2] mmc: mmc-uclass: Add mmc_init() in mmc preinit phrase

2016-09-12 Thread Wenyou Yang
Add mmc_init() in the mmc preinit phrase to get the device's information. Otherwise, if the environment variables are from the mmc device, it will fail with "** Bad device size - mmc 1 **". Signed-off-by: Wenyou Yang --- Changes in v2: - Add the bracket for #if defined. drivers/mmc/mmc-uclass

[U-Boot] [PATCH v2 1/2] serial: Kconfig: Add ATMEL_USART option

2016-09-12 Thread Wenyou Yang
Add ATMEL_USART option to support to enable the Atmel usart driver from Kconfig. Signed-off-by: Wenyou Yang --- Changes in v2: None drivers/serial/Kconfig | 7 +++ 1 file changed, 7 insertions(+) diff --git a/drivers/serial/Kconfig b/drivers/serial/Kconfig index ab5df70..a6035dc 100644 --

[U-Boot] [PATCH v2 0/2] serial: atmel_usart: Support to enable an early debug UART

2016-09-12 Thread Wenyou Yang
The patch set is to add support to enable an early debug UART for debugging. And add ATMEL_USART option to enable the Atmel usartdriver from Kconfig. Changes in v2: - Collect the Reviewed-by tag. Wenyou Yang (2): serial: Kconfig: Add ATMEL_USART option serial: atmel_usart: Support enable an

[U-Boot] [PATCH v2 2/2] serial: atmel_usart: Support enable an early debug UART

2016-09-12 Thread Wenyou Yang
Add support to enable an early debug UART for debugging. Signed-off-by: Wenyou Yang Reviewed-by: Simon Glass --- Changes in v2: - Collect the Reviewed-by tag. drivers/serial/Kconfig | 7 +++ drivers/serial/atmel_usart.c | 22 ++ 2 files changed, 29 insertions(+

[U-Boot] [PATCH v2] i2c: at91_i2c: Fix the wrong include file

2016-09-12 Thread Wenyou Yang
Since the 'clk_client.h' doesn't exist, it should be 'clk.h'. Signed-off-by: Wenyou Yang Reviewed-by: Simon Glass --- Changes in v2: - Collect Reviewed-by tag. drivers/i2c/at91_i2c.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/i2c/at91_i2c.c b/drivers/i2c/at91

[U-Boot] [PATCH v2 0/4] board: sama5d2_xplained: Convert to use device tree and drivers with DM

2016-09-12 Thread Wenyou Yang
Convert to use the drivers which support the driver model and use the device tree. And add the support to enable an early debug UART for debugging. It is based on the following patches: - [PATCH v2] i2c: at91_i2c: Fix the wrong include file http://lists.denx.de/pipermail/u-boot/2016-Sept

[U-Boot] [PATCH v2 1/4] board: sama5d2_xplained: Move config options to defconfigs

2016-09-12 Thread Wenyou Yang
Move the config options from the include/configs/sama5d2_xplained.h to configs/sama5d2_xplained_*_defconfig. Signed-off-by: Wenyou Yang --- Changes in v2: None board/atmel/sama5d2_xplained/sama5d2_xplained.c | 2 ++ configs/sama5d2_xplained_mmc_defconfig | 31 +++-

[U-Boot] [PATCH v2 4/4] board: sama5d2_xplained: Enable an early debug UART

2016-09-12 Thread Wenyou Yang
Enable an early debug UART to debug problems when an ICE or other debug mechanism is not available. Signed-off-by: Wenyou Yang Reviewed-by: Simon Glass --- Changes in v2: - Collect Reviewed-by tags. board/atmel/sama5d2_xplained/sama5d2_xplained.c | 14 ++ configs/sama5d2_xplained

[U-Boot] [PATCH v2 2/4] board: sama5d2_xplained: Clean up code

2016-09-12 Thread Wenyou Yang
Since the introduction of pinctrl and clk driver, and the dts file, remove unneeded the pin configurations and the clock enabling code. Signed-off-by: Wenyou Yang Reviewed-by: Simon Glass --- Changes in v2: None board/atmel/sama5d2_xplained/sama5d2_xplained.c | 104 1

[U-Boot] [PATCH v2 3/4] board: sama5d2_xplained: Set 'ethaddr' got from AT24MAC

2016-09-12 Thread Wenyou Yang
If 'ethaddr' is not set, we will get the ethernet address from AT24MAC, and set it to 'ethaddr' variable. Signed-off-by: Wenyou Yang Signed-off-by: Songjun Wu --- Changes in v2: None board/atmel/sama5d2_xplained/sama5d2_xplained.c | 51 + include/configs/sama5d2_xplain

[U-Boot] [PATCH v3 06/45] Drop CONFIG_SPL_RAM_SUPPORT

2016-09-12 Thread Simon Glass
This option does not exist in U-Boot. Drop it. Signed-off-by: Simon Glass --- Changes in v3: None Changes in v2: None include/configs/rk3288_common.h | 1 - 1 file changed, 1 deletion(-) diff --git a/include/configs/rk3288_common.h b/include/configs/rk3288_common.h index 601186c..49f35a6 1006

[U-Boot] [PATCH v3 01/45] Correct defconfigs using savedefconfig

2016-09-12 Thread Simon Glass
Update the defconfig files to match their canonical form, as produced by 'make safedefconfig'. This is the result of running 'tools/moveconfig.py -s' on the tree. Signed-off-by: Simon Glass --- Changes in v3: None Changes in v2: None configs/am43xx_hs_evm_defconfig | 2 +- configs/gwventana_d

[U-Boot] [PATCH v3 00/45] Kconfig: Move CONFIG_SPL_..._SUPPORT to Kconfig

2016-09-12 Thread Simon Glass
This series moves all the CONFIG_SPL_..._SUPPORT options to Kconfig and fixes up existing boards to continue to build. It also adds a few small but useful features to moveconfig. There is existing work going on in this area, so some of these patches may be superseded. It has taken me a while to g

[U-Boot] [PATCH v3 03/45] moveconfig: Add an option to commit changes

2016-09-12 Thread Simon Glass
The moveconfig tool is quite clever and generally produces results that are suitable for sending as a patch without further work. The main required step is to add the changes to a commit. Add an option to do this automatically. This allows moveconfig to be used from a script to convert multiple CO

[U-Boot] [PATCH v3 05/45] arm: fsl: Adjust ordering of #ifndef CONFIG_SPL_BUILD

2016-09-12 Thread Simon Glass
The secure boot header files incorrectly define SPL options only if CONFIG_SPL_BUILD is defined. This means that the options are only enabled in an SPL build, and not with a normal 'make xxx_defconfig'. This means that moveconfig.py cannot work, since it sees the options as disabled even when they

[U-Boot] [PATCH v3 02/45] moveconfig: Add an option to skip prompts

2016-09-12 Thread Simon Glass
At present it is not easy to use moveconfig from a script since it asks for user input a few times. Add a -y option to skip this and assume that 'y' was entered. Signed-off-by: Simon Glass --- Changes in v3: None Changes in v2: - Add some notes on this option to moveconfig.py tools/moveconfig.

[U-Boot] [PATCH v3 09/45] Kconfig: tpl: Add some TPL support options to Kconfig

2016-09-12 Thread Simon Glass
Some of the SPL options have TPL equivalents. Add these to Kconfig so that we can convert these options over to work from Kconfig. Signed-off-by: Simon Glass --- Changes in v3: None Changes in v2: None common/spl/Kconfig | 67 ++ 1 file chang

[U-Boot] [PATCH v3 07/45] Use separate options for TPL support

2016-09-12 Thread Simon Glass
At present TPL uses the same options as SPL support. In a few cases the board config enables or disables the SPL options depending on whether CONFIG_TPL_BUILD is defined. With the move to Kconfig, options are determined for the whole build and (without a hack like an #undef in a header file) canno

[U-Boot] [PATCH v3 12/45] spear: Use upper case for CONFIG options

2016-09-12 Thread Simon Glass
There are a few options which use lower case. We should use upper case for all CONFIG options. Signed-off-by: Simon Glass --- Changes in v3: None Changes in v2: None arch/arm/cpu/arm926ejs/spear/timer.c | 2 +- configs/spear300_defconfig | 2 +- configs/spear300_nand_defconfig

[U-Boot] [PATCH v3 11/45] Move existing use of CONFIG_SPL_RSA to Kconfig

2016-09-12 Thread Simon Glass
A few boards define this in a header file which is incorrect. It means that Kconfig options that rely on this cannot be used. Move it. Signed-off-by: Simon Glass --- Changes in v3: None Changes in v2: None arch/arm/include/asm/fsl_secure_boot.h | 1 - arch/powerpc/include/asm/f

[U-Boot] [PATCH v3 16/45] Convert CONFIG_SPL_DRIVERS_MISC_SUPPORT to Kconfig

2016-09-12 Thread Simon Glass
Move this option to Kconfig and tidy up existing uses. Signed-off-by: Simon Glass --- Changes in v3: None Changes in v2: None arch/arm/include/asm/fsl_secure_boot.h | 1 - arch/arm/mach-rockchip/rk3288/Kconfig | 3 +++ arch/powerpc/include/asm/fsl_secure_boot.h

[U-Boot] [PATCH v3 08/45] Kconfig: spl: Add SPL support options to Kconfig

2016-09-12 Thread Simon Glass
There are a lot of SPL options in U-Boot to enable various features and drivers. Currently these do not use Kconfig. Add them to Kconfig along with suitable help, and drop them from the README. Signed-off-by: Simon Glass --- Changes in v3: - Move SPL_NET_VCI_STRING into the SPL Kconfig file also

[U-Boot] [PATCH v3 14/45] Convert CONFIG_SPL_HASH_SUPPORT to Kconfig

2016-09-12 Thread Simon Glass
Move this option to Kconfig and tidy up existing uses. Signed-off-by: Simon Glass --- Changes in v3: None Changes in v2: None arch/arm/include/asm/fsl_secure_boot.h | 1 - arch/powerpc/include/asm/fsl_secure_boot.h | 1 - configs/T1042RDB_PI_NAND_SECURE_BOOT_defconfig

[U-Boot] [PATCH v3 13/45] Convert CONFIG_SPL_CRYPTO_SUPPORT to Kconfig

2016-09-12 Thread Simon Glass
Move this option to Kconfig and tidy up existing uses. Signed-off-by: Simon Glass --- Changes in v3: None Changes in v2: None arch/arm/include/asm/fsl_secure_boot.h | 1 - arch/powerpc/include/asm/fsl_secure_boot.h | 1 - configs/T1042RDB_PI_NAND_SECURE_BOOT_defconfig

[U-Boot] [PATCH v3 26/45] Convert CONFIG_SPL_MMC_SUPPORT to Kconfig

2016-09-12 Thread Simon Glass
Move this option to Kconfig and tidy up existing uses. Signed-off-by: Simon Glass --- Changes in v3: None Changes in v2: None arch/arm/cpu/armv7/omap3/Kconfig| 3 +++ arch/arm/cpu/armv7/omap4/Kconfig| 3 +++ arch/arm/cpu/armv7/omap5/Kconfig

  1   2   >