Re: [U-Boot] [PATCH 3/8] ARM: hikey: Use linux/sizes.h for malloc size

2015-09-10 Thread Peter Griffin
Hi Fabio, On 9 September 2015 at 22:22, Fabio Estevam wrote: > On Wed, Sep 9, 2015 at 6:13 PM, Peter Griffin > wrote: > > > /* Size of malloc() pool */ > > -#define CONFIG_SYS_MALLOC_LEN (CONFIG_ENV_SIZE + (8 << 20)) > > +#define

[U-Boot] dm eth rotation is broken under some circumstances

2015-09-10 Thread Bin Meng
Hi Joe, I found under some circumstances the eth rotation functionality is broken. Test scenario: 1). Say we have two ethernet devices, eth0 and eth1. Both devices are not probed yet (or probe failed due to ethaddr and eth1addr not set) Now we set the eth1addr to a valid MAC address, set

[U-Boot] [PATCH v2 2/3] dm: test: Add a new test case against dm eth codes for NULL pointer access

2015-09-10 Thread Bin Meng
U-Boot crashes when doing a 'ping' with the following test scenario: - All ethernet devices are not probed - "ethaddr" for all ethernet devices are not set - "ethact" is set to a valid ethernet device name Add a new test case 'dm_test_eth_act' to hit such scenario. Signed-off-by: Bin Meng

[U-Boot] [PATCH v2 3/3] net: eth: Check return value in various places

2015-09-10 Thread Bin Meng
eth_get_dev() can return NULL which means device_probe() fails for that ethernet device. Add return value check in various places or U-Boot will crash due to NULL pointer access. With this commit, 'dm_test_eth_act' test case passes. Signed-off-by: Bin Meng --- Changes in

[U-Boot] [PATCH v2 1/3] net: eth: Clear MAC address in eth_pre_remove()

2015-09-10 Thread Bin Meng
When removing an Ethernet device, the MAC address saved in the dev's platdata should be cleared. Signed-off-by: Bin Meng --- Changes in v2: - New patch to clear MAC address in eth_pre_remove() net/eth.c | 5 + 1 file changed, 5 insertions(+) diff --git a/net/eth.c

[U-Boot] [PATCH 07/11] imx: mx6: add clock api for lcdif

2015-09-10 Thread Peng Fan
Implement mxs_set_lcdclk, enable_lcdif_clock and enable_pll_video. The three API can be used to configure lcdif related clock when CONFIG_VIDEO_MXS enabled. Signed-off-by: Peng Fan Cc: Stefano Babic --- arch/arm/cpu/armv7/mx6/clock.c| 239

[U-Boot] [PATCH 04/11] imx: imx-common: move lcdif structure and macro definition to imx-common

2015-09-10 Thread Peng Fan
Move 'struct mxs_lcdif_regs' and lcdif related macro definitions to arch/arm/include/asm/imx-common/regs-lcdif.h. i.MX6SX/UL/28 have similar register layout and bit definitions of registers. Signed-off-by: Peng Fan Cc: Stefano Babic ---

[U-Boot] [PATCH 10/11] imx: mx6: implement reset_misc

2015-09-10 Thread Peng Fan
We need to power down lcdif before uboot reset to make reset can pass stress test. Or system may hang. Signed-off-by: Peng Fan Cc: Stefano Babic --- arch/arm/cpu/armv7/mx6/soc.c | 8 1 file changed, 8 insertions(+) diff --git

[U-Boot] [PATCH 06/11] imx: mx6: crm_reg: add LCDIF related macros

2015-09-10 Thread Peng Fan
Add i.MX6UL/SX LCDIF related macros. Discard uneccessary '#ifdef xxx'. Signed-off-by: Peng Fan Cc: Stefano Babic --- arch/arm/include/asm/arch-mx6/crm_regs.h | 34 1 file changed, 30 insertions(+), 4 deletions(-) diff

[U-Boot] [PATCH 05/11] imx: mx6: fix register address

2015-09-10 Thread Peng Fan
1. Move WDOG3_BASE_ADDR to '#if !(defined(__ASSEMBLY__))'. 2. Add i.MX6UL LCDIF register base address. And Introduce LCDIF1_BASE_ADDR to support runtime check. 3. include for imx-regs.h to avoid building error for mxsfb.c, since mxsfb.c use imx-regs.h. Signed-off-by: Peng Fan

Re: [U-Boot] [PATCH 1/5] net: mdio: Add mdio_free() and mdio_unregister() API

2015-09-10 Thread Bin Meng
Hi Simon, On Thu, Sep 10, 2015 at 2:07 AM, Simon Glass wrote: > Hi Bin, > > On Friday, 4 September 2015, Bin Meng wrote: >> >> Currently there is no API to uninitialize mdio. Add two APIs for this. >> >> Signed-off-by: Bin Meng >> ---

Re: [U-Boot] [PATCH 07/13] fastboot: Implement NAND backend

2015-09-10 Thread Boris Brezillon
Hi Maxime, On Mon, 31 Aug 2015 16:46:07 +0200 Maxime Ripard wrote: > So far the fastboot code was only supporting MMC-backed devices for its > flashing operations (flash and erase). > > Add a storage backend for NAND-backed devices. > > Signed-off-by: Maxime

Re: [U-Boot] [PATCH 4/5] dm: core: Remove the call to device_free()

2015-09-10 Thread Bin Meng
Hi Simon, On Thu, Sep 10, 2015 at 2:07 AM, Simon Glass wrote: > Hi Bin, > > On Friday, 4 September 2015, Bin Meng wrote: >> >> When something goes wrong during device_probe(), we just need do >> device_remove() which calls device_free() internally. >> >>

Re: [U-Boot] Fastboot is not detected

2015-09-10 Thread Paul Kocialkowski
Hi, Le mercredi 09 septembre 2015 à 18:29 -0300, Fabio Estevam a écrit : > On Wed, Sep 9, 2015 at 6:21 PM, Marek Vasut wrote: > > > Don't you need this patch from Peng? > > [PATCH] usb: gadget: ci_udc: implement usb_ep_ops dequeue callback > >

[U-Boot] [PATCH 02/11] sandisk: sfp: correct function name

2015-09-10 Thread Peng Fan
board_mxsfb_system_setup shoule be named mxsfb_system_setup. Signed-off-by: Peng Fan Cc: Marek Vasut Cc: Stefano Babic --- board/sandisk/sansa_fuze_plus/sfp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[U-Boot] [PATCH 00/11] imx: mx6: support lcdif

2015-09-10 Thread Peng Fan
This patch set is to introduce lcdif support for i.MX6. Now only i.MX6UL support is in this patchset. Patchset tested on mx6ul_14x14_evk and mx6ul_9x9_evk (needs 9x9 SPL patch applied). 1/11 There are two LCD interface for i.MX6SX and one interface for i.MX6UL, so change the prototype

[U-Boot] [PATCH 03/11] xfi3: correct function name

2015-09-10 Thread Peng Fan
board_mxsfb_system_setup shoule be named mxsfb_system_setup. Signed-off-by: Peng Fan Cc: Marek Vasut Cc: Stefano Babic --- board/creative/xfi3/xfi3.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[U-Boot] [PATCH 01/11] mxs: add parameter base_addr for mxs_set_lcdclk

2015-09-10 Thread Peng Fan
Change mxs_set_lcdclk prototype to add a new parameter base_addr. There are two LCD interfaces for i.MX6SX, we may support LCDIF1 or LCDIF2. Signed-off-by: Peng Fan Cc: Stefano Babic Cc: Anatolij Gustschin ---

[U-Boot] [PATCH 08/11] imx: mx6ul_14x14_evk: support lcdif display

2015-09-10 Thread Peng Fan
Support lcdif display: 1. Add pinmux and pad settings for LCDIF 2. Introduce setup_lcd to do the settings for LCDIF 3. Enable VIDEO related macros in board header files 4. Add a new env videomode which is needed by mxsfb.c. The settings for videomode in this patch is for TFT43AB. 5. Tested on

[U-Boot] [PATCH 11/11] imx: imx-common: power down lcdif before boot os

2015-09-10 Thread Peng Fan
Need to call lcdif_power_down to make lcdif in initial state before kernel boot. Similar issue for uboot reset with lcdif enabled, system will hang after serveral times resetting. Need to let lcdif initial state to make all go well. Signed-off-by: Peng Fan Cc: Stefano

[U-Boot] [PATCH 09/11] video: mxsfb: introduce lcdif_power_down

2015-09-10 Thread Peng Fan
Introudce a new function lcdif_power_down. 1. Waits for a VSYNC interrupt to guarantee the reset is done at the VSYNC edge, which somehow makes the LCDIF consume the display FIFO(?) and helps the LCDIF work normally at the kernel stage. 2. Add power down function to stop lcdif. The reason

Re: [U-Boot] [PATCH 4/5] dm: usb: Add support for USB keyboards with driver model

2015-09-10 Thread Marek Vasut
On Thursday, September 10, 2015 at 04:45:53 AM, Simon Glass wrote: > Hi Marek, > > On 8 September 2015 at 12:33, Marek Vasut wrote: > > On Tuesday, September 08, 2015 at 07:15:11 PM, Simon Glass wrote: > >> Switch USB keyboards over to use driver model instead of scanning with >

Re: [U-Boot] [PATCH 2/4] spi: cadence_qspi: Fix fdt read of spi-max-frequency

2015-09-10 Thread Marek Vasut
On Thursday, September 10, 2015 at 06:58:10 AM, Chin Liang See wrote: > Hi, > > On Tue, 2015-09-08 at 13:19 +0200, ma...@denx.de wrote: > > On Tuesday, September 08, 2015 at 03:18:24 AM, Chin Liang See wrote: > > > Fix the fdt read for spi-max-frequency as it's contained > > > in the child node.

Re: [U-Boot] [PATCH v3 2/5] spi: cadence_qspi: Fix fdt read of spi-max-frequency

2015-09-10 Thread Marek Vasut
On Thursday, September 10, 2015 at 07:01:28 AM, Chin Liang See wrote: > Fix the fdt read for spi-max-frequency as it's contained > in the child node. Current state of code is always > returning default value. > > Signed-off-by: Chin Liang See > Cc: Dinh Nguyen

Re: [U-Boot] [PATCH v3 1/5] spi: cadence_qspi: Ensure spi_calibration is run when sclk change

2015-09-10 Thread Marek Vasut
On Thursday, September 10, 2015 at 07:01:12 AM, Chin Liang See wrote: > Ensuring spi_calibration is run when there is a change of sclk > frequency. This will ensure the qspi flash access works for high > sclk frequency > > Signed-off-by: Chin Liang See > Cc: Dinh Nguyen

Re: [U-Boot] [PATCH v3 5/5] common: Enable fdt_support in SPL build

2015-09-10 Thread Marek Vasut
On Thursday, September 10, 2015 at 07:02:16 AM, Chin Liang See wrote: > Enable more device tree function support such as > fdt_getprop_u32_default_node in SPL. > > Signed-off-by: Chin Liang See > Cc: Dinh Nguyen > Cc: Dinh Nguyen

Re: [U-Boot] [PATCH 2/2] ARM: tegra114: Clear IDDQ when enabling PLLC

2015-09-10 Thread Thierry Reding
On Tue, Sep 08, 2015 at 03:58:38PM +, Tom Warren wrote: > Thierry, > > > -Original Message- > > From: Thierry Reding [mailto:thierry.red...@gmail.com] > > Sent: Tuesday, September 08, 2015 2:38 AM > > To: Tom Warren > > Cc: Nicolas Chauvet; u-boot@lists.denx.de > > Subject: [PATCH

Re: [U-Boot] [PATCH 1/2] ARM: tegra124: Clear IDDQ when enabling PLLC

2015-09-10 Thread Thierry Reding
On Wed, Sep 09, 2015 at 08:37:34PM -0700, Stephen Warren wrote: > On 09/08/2015 02:38 AM, Thierry Reding wrote: > > From: Thierry Reding > > > > Enabling a PLL while IDDQ is high. The Linux kernel checks for this > > Is there some word missing in/at-the-end-of that first

Re: [U-Boot] Fastboot is not detected

2015-09-10 Thread Fabio Estevam
On Thu, Sep 10, 2015 at 5:00 AM, Paul Kocialkowski wrote: > Hmm, does the device show up at all on the host (in dmesg)? Yes, it does. > I.e. is it somewhat detected by the host usb driver, does it fail early, > etc? The issue is solved now. I had to apply Peng's patch for

[U-Boot] [PATCH] zynq-common: Define spi args offset and size

2015-09-10 Thread Siva Durga Prasad Paladugu
Define CONFIG_SYS_SPI_ARGS_OFFS, CONFIG_SYS_SPI_ARGS_SIZE and CONFIG_SYS_SPI_KERNEL_OFFS which will be used in SPL spi mode. Signed-off-by: Siva Durga Prasad Paladugu --- - This patch is based on u-boot-spi/master branch --- include/configs/zynq-common.h | 4 1 file

Re: [U-Boot] [PATCH] fat: handle paths that include ../

2015-09-10 Thread Stephen Warren
On 07/28/2015 08:55 PM, Stephen Warren wrote: > The FAT code contains a special case to parse the root directory. This > is needed since the root directory location/layout on disk is special > cased for FAT12/16. In particular, the location and size of the FAT12/16 > root directory is hard-coded

Re: [U-Boot] [PATCH] sunxi: Fix wrong serial console setup in Forfun Q88DB tablet

2015-09-10 Thread Hans de Goede
Hi, On 08-09-15 16:49, Siarhei Siamashka wrote: The Forfun Q88DB tablet was unbootable since commit b6006baf9c2553543e3384983d23d95efbf24fa6 ("sunxi: Move all boards to the driver-model"). Appears that this is caused by the wrong serial console setup in the SPL. The serial console should use

Re: [U-Boot] [PATCH] sunxi_nand_spl: Add config parameter for 4KiB page sized NAND devices

2015-09-10 Thread Hans de Goede
Hi, On 04-09-15 14:43, Stefan Roese wrote: This patch adds support for NAND chips with 4KiB page size and 24/1024 ECC strength. Like the Micron MT29F32G08CBACAWP which is used on the ICnova-A20 SoM. Signed-off-by: Stefan Roese Cc: Hans de Goede Cc: Ian

Re: [U-Boot] [PATCH] sunxi: Ensure that 'mksunxiboot' tool produces deterministic output

2015-09-10 Thread Hans de Goede
Hi, On 03-09-15 01:36, Siarhei Siamashka wrote: Currently some uninitialized padding bytes are written to the output file, as can be confirmed with valgrind: $ valgrind tools/mksunxiboot spl/u-boot-spl.bin spl/sunxi-spl.bin ==5581== Syscall param write(buf) points to uninitialised byte(s)

Re: [U-Boot] [RFC PATCH 1/2] sunxi: retrieve FEL-provided values to environment variables

2015-09-10 Thread Hans de Goede
Hi, On 03-09-15 16:11, Bernhard Nortmann wrote: This patch extends the misc_init_r() function on sunxi boards to test for the presence of a suitable "sunxi" SPL header. If found, and the loader ("fel" utility) provided non-zero values for either data address or size, then corresponding

Re: [U-Boot] [PATCH 12/13] sunxi: A13-Olinuxino: Enable the USB OTG controller

2015-09-10 Thread Hans de Goede
Hi, On 03-09-15 23:41, Maxime Ripard wrote: Hi Hans, On Tue, Sep 01, 2015 at 11:01:06AM +0200, Hans de Goede wrote: Hi, On 31-08-15 16:46, Maxime Ripard wrote: The A13-Olinuxino has a mini-USB connector that can be used to power up the board and as an OTG connector. Since we have already

Re: [U-Boot] [PATCH] sunxi: (mksunxiboot) signature to indicate "sunxi" SPL variant

2015-09-10 Thread Hans de Goede
Hi, On 02-09-15 15:17, Bernhard Nortmann wrote: This patch follows up on a discussion of ways to improve support for the sunxi FEL ("USB boot") mechanism, especially with regard to boot scripts, see: https://groups.google.com/d/msg/linux-sunxi/wBEGUoLNRro/rHGq6nSYCQAJ The idea is to convert

Re: [U-Boot] [PATCH v3 5/5] common: Enable fdt_support in SPL build

2015-09-10 Thread Marek Vasut
On Friday, September 11, 2015 at 02:14:17 AM, Simon Glass wrote: > Hi, > > On 10 September 2015 at 04:49, Marek Vasut wrote: > > On Thursday, September 10, 2015 at 07:02:16 AM, Chin Liang See wrote: > >> Enable more device tree function support such as > >>

Re: [U-Boot] [PATCH 1/1] net: phy: micrel: add support for KSZ8021RNL & KSZ8031RNL

2015-09-10 Thread Joe Hershberger
Hi Sylvain, On Wed, Sep 9, 2015 at 3:29 PM, wrote: > From: Sylvain Lemieux > > This patch adds support for Micrel KSZ8021RNL & KSZ8031RNL. > > Signed-off-by: Sylvain Lemieux Acked-by: Joe Hershberger

Re: [U-Boot] [PATCH v2 2/3] dm: test: Add a new test case against dm eth codes for NULL pointer access

2015-09-10 Thread Joe Hershberger
Hi Bin, On Thu, Sep 10, 2015 at 4:29 AM, Bin Meng wrote: > U-Boot crashes when doing a 'ping' with the following test scenario: > > - All ethernet devices are not probed > - "ethaddr" for all ethernet devices are not set > - "ethact" is set to a valid ethernet device

[U-Boot] Please pull u-boot-x86.git

2015-09-10 Thread Simon Glass
Hi Tom, Here are a few more driver model PCI fixes, the tftp revert and some driver model conversions for x86, as well as a few other minor fixes. There are a few more patches in flight but I could not apply them due to problems. I thought it best to get this out in the interim since the hour is

Re: [U-Boot] [PATCH v2] pcie_imx: Use 'ms' for milliseconds

2015-09-10 Thread Marek Vasut
On Friday, September 11, 2015 at 01:45:25 AM, Fabio Estevam wrote: > From: Fabio Estevam > > milliseconds should be written as 'ms' instead of 'mS'. > > Signed-off-by: Fabio Estevam Tested-by: Marek Vasut My hardware

Re: [U-Boot] [PATCH] fat: handle paths that include ../

2015-09-10 Thread Tom Rini
On Thu, Sep 10, 2015 at 10:22:35AM -0700, Stephen Warren wrote: > On 07/28/2015 08:55 PM, Stephen Warren wrote: > > The FAT code contains a special case to parse the root directory. This > > is needed since the root directory location/layout on disk is special > > cased for FAT12/16. In

Re: [U-Boot] [PATCH 4/5] dm: core: Remove the call to device_free()

2015-09-10 Thread Simon Glass
Hi Bin, On 10 September 2015 at 00:07, Bin Meng wrote: > Hi Simon, > > On Thu, Sep 10, 2015 at 2:07 AM, Simon Glass wrote: >> Hi Bin, >> >> On Friday, 4 September 2015, Bin Meng wrote: >>> >>> When something goes wrong during

Re: [U-Boot] [PATCH 4/5] dm: core: Remove the call to device_free()

2015-09-10 Thread Simon Glass
Hi Bin, On Sep 10, 2015 5:47 PM, "Bin Meng" wrote: > > Hi Simon, > > On Fri, Sep 11, 2015 at 8:38 AM, Simon Glass wrote: > > Hi Bin, > > > > On 10 September 2015 at 00:07, Bin Meng wrote: > >> Hi Simon, > >> > >> On Thu, Sep 10, 2015

Re: [U-Boot] [PATCH v2 1/3] net: eth: Clear MAC address in eth_pre_remove()

2015-09-10 Thread Joe Hershberger
Hi Bin, On Thu, Sep 10, 2015 at 4:29 AM, Bin Meng wrote: > When removing an Ethernet device, the MAC address saved in the dev's > platdata should be cleared. Why is this important to do? Test case? > Signed-off-by: Bin Meng > > --- > > Changes in v2: >

Re: [U-Boot] [PATCH] net: eth: Do sanity test on eth dev before eth_get_ops(dev)->start

2015-09-10 Thread Joe Hershberger
Hi Bin, On Wed, Sep 9, 2015 at 1:14 AM, Bin Meng wrote: > Hi Joe, > > On Wed, Sep 9, 2015 at 11:19 AM, Bin Meng wrote: >> Hi Joe, >> >> On Wed, Sep 9, 2015 at 1:23 AM, Joe Hershberger >> wrote: >>> Hi Bin, >>> >>> On Tue, Sep

Re: [U-Boot] commit 620776d causes TFTP error: 'Unsupported option(s) requested' (8)

2015-09-10 Thread Joe Hershberger
Hi Alexey, On Thu, Sep 10, 2015 at 5:26 PM, Alexey Brodkin wrote: > Hi Pavel, > > On Tue, 2015-08-25 at 14:19 +0200, Pavel Machek wrote: >> On Tue 2015-08-25 14:13:17, Hannes Schmelzer wrote: >> > Hi, >> > >> > i've same problem. >> > >> > Running tftp-hpa on Linux

Re: [U-Boot] [PATCH v2 1/3] net: eth: Clear MAC address in eth_pre_remove()

2015-09-10 Thread Joe Hershberger
Hi Bin, On Thu, Sep 10, 2015 at 7:49 PM, Bin Meng wrote: > Hi Joe, > > On Fri, Sep 11, 2015 at 6:32 AM, Joe Hershberger > wrote: >> Hi Bin, >> >> On Thu, Sep 10, 2015 at 4:29 AM, Bin Meng wrote: >>> When removing an Ethernet

Re: [U-Boot] commit 620776d causes TFTP error: 'Unsupported option(s) requested' (8)

2015-09-10 Thread Alexey Brodkin
Hi Pavel, On Tue, 2015-08-25 at 14:19 +0200, Pavel Machek wrote: > On Tue 2015-08-25 14:13:17, Hannes Schmelzer wrote: > > Hi, > > > > i've same problem. > > > > Running tftp-hpa on Linux Mint 17. > > > > cat /etc/default/tftpd-hpa > > # /etc/default/tftpd-hpa > > RUN_DAEMON="yes" > >

Re: [U-Boot] [PATCH 1/4 v2] mtd: nand: fsmc: Add BCH4 SW ECC support for SPEAr600

2015-09-10 Thread Scott Wood
On Wed, 2015-09-02 at 14:29 +0200, Stefan Roese wrote: > This patch adds support for 4-bit ECC BCH4 for the SPEAr600 SoC. This can > be used by boards equipped with a NAND chip that requires 4-bit ECC > strength. > The SPEAr600 HW ECC only supports 1-bit ECC strength. > > To enable SW BCH4, you

Re: [U-Boot] commit 620776d causes TFTP error: 'Unsupported option(s) requested' (8)

2015-09-10 Thread Simon Glass
Hi, On 10 September 2015 at 15:26, Alexey Brodkin wrote: > Hi Pavel, > > On Tue, 2015-08-25 at 14:19 +0200, Pavel Machek wrote: >> On Tue 2015-08-25 14:13:17, Hannes Schmelzer wrote: >> > Hi, >> > >> > i've same problem. >> > >> > Running tftp-hpa on Linux Mint 17.

Re: [U-Boot] [PATCH 4/5] dm: usb: Add support for USB keyboards with driver model

2015-09-10 Thread Simon Glass
Hi Marek, On 10 September 2015 at 04:40, Marek Vasut wrote: > On Thursday, September 10, 2015 at 04:45:53 AM, Simon Glass wrote: >> Hi Marek, >> >> On 8 September 2015 at 12:33, Marek Vasut wrote: >> > On Tuesday, September 08, 2015 at 07:15:11 PM, Simon Glass

Re: [U-Boot] dm eth rotation is broken under some circumstances

2015-09-10 Thread Joe Hershberger
Hi Bin, On Thu, Sep 10, 2015 at 4:46 AM, Bin Meng wrote: > Hi Joe, > > I found under some circumstances the eth rotation functionality is broken. > > Test scenario: > > 1). Say we have two ethernet devices, eth0 and eth1. Both devices are > not probed yet (or probe failed due

[U-Boot] [PATCH] pcie_imx: Use 'ms' for miliseconds

2015-09-10 Thread Fabio Estevam
From: Fabio Estevam miliseconds should be written as 'ms' instead of 'mS'. Signed-off-by: Fabio Estevam --- drivers/pci/pcie_imx.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/pci/pcie_imx.c

Re: [U-Boot] [PATCH 1/5] net: mdio: Add mdio_free() and mdio_unregister() API

2015-09-10 Thread Joe Hershberger
Hi Bin, On Fri, Sep 4, 2015 at 9:53 AM, Bin Meng wrote: > Currently there is no API to uninitialize mdio. Add two APIs for this. Is this causing some failure in the short term? The plan is to move eth phy support to driver model in the next year. If it is not needed for a

Re: [U-Boot] [PATCH v2 3/3] net: eth: Check return value in various places

2015-09-10 Thread Joe Hershberger
On Thu, Sep 10, 2015 at 4:29 AM, Bin Meng wrote: > eth_get_dev() can return NULL which means device_probe() fails for > that ethernet device. Add return value check in various places or > U-Boot will crash due to NULL pointer access. > > With this commit, 'dm_test_eth_act'

Re: [U-Boot] [PATCH v3 5/5] common: Enable fdt_support in SPL build

2015-09-10 Thread Simon Glass
Hi, On 10 September 2015 at 04:49, Marek Vasut wrote: > On Thursday, September 10, 2015 at 07:02:16 AM, Chin Liang See wrote: >> Enable more device tree function support such as >> fdt_getprop_u32_default_node in SPL. >> >> Signed-off-by: Chin Liang See >> Cc:

[U-Boot] [PATCH v2] pcie_imx: Use 'ms' for milliseconds

2015-09-10 Thread Fabio Estevam
From: Fabio Estevam milliseconds should be written as 'ms' instead of 'mS'. Signed-off-by: Fabio Estevam --- Changes since v1: - Fixed typo drivers/pci/pcie_imx.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git

Re: [U-Boot] [PATCH] dm: core: Enable optional use of fdt_translate_address()

2015-09-10 Thread Simon Glass
Hi Stefan, On 9 September 2015 at 22:54, Stefan Roese wrote: > Hi Simon, > > On 09.09.2015 20:07, Simon Glass wrote: >> >> On Thursday, 3 September 2015, Stefan Roese wrote: >>> >>> >>> The current "simple" address translation simple_bus_translate() is not >>>

Re: [U-Boot] [PATCH v2 1/3] net: eth: Clear MAC address in eth_pre_remove()

2015-09-10 Thread Bin Meng
Hi Joe, On Fri, Sep 11, 2015 at 6:32 AM, Joe Hershberger wrote: > Hi Bin, > > On Thu, Sep 10, 2015 at 4:29 AM, Bin Meng wrote: >> When removing an Ethernet device, the MAC address saved in the dev's >> platdata should be cleared. > > Why is this

Re: [U-Boot] [PATCH 4/5] dm: core: Remove the call to device_free()

2015-09-10 Thread Bin Meng
Hi Simon, On Fri, Sep 11, 2015 at 8:38 AM, Simon Glass wrote: > Hi Bin, > > On 10 September 2015 at 00:07, Bin Meng wrote: >> Hi Simon, >> >> On Thu, Sep 10, 2015 at 2:07 AM, Simon Glass wrote: >>> Hi Bin, >>> >>> On Friday, 4 September

[U-Boot] [PATCH] mx6sabre_common: Add Fastboot support

2015-09-10 Thread Fabio Estevam
From: Fabio Estevam Tested basic fastboot commands, such as: On the mx6qsabresd U-boot prompt: => fastboot 0 On the host PC: $ fastboot getvar bootloader-version -i 0x0525 bootloader-version: U-Boot 2015.10-rc2-23960-g2462cce-dirty finished. total time: 0.000s $

Re: [U-Boot] [PATCH 7/8] ARM: hikey: hi6220: Migrate over to DM_SERIAL and UART3 for serial.

2015-09-10 Thread Peter Griffin
Hi Tom, On 9 September 2015 at 22:49, Tom Rini wrote: > On Wed, Sep 09, 2015 at 10:13:54PM +0100, Peter Griffin wrote: > > > Use DM for the pl01x serial driver on hikey. Also migrate over to UART3 > > as the latest version of ATF outputs to this UART. UART3 comes out on the

Re: [U-Boot] [PATCH v3 2/5] spi: cadence_qspi: Fix fdt read of spi-max-frequency

2015-09-10 Thread Pavel Machek
On Thu 2015-09-10 00:01:28, Chin Liang See wrote: > Fix the fdt read for spi-max-frequency as it's contained > in the child node. Current state of code is always > returning default value. > > Signed-off-by: Chin Liang See > Cc: Dinh Nguyen >

[U-Boot] [PATCH v2 3/8] ARM: hikey: Use linux/sizes.h for malloc size

2015-09-10 Thread Peter Griffin
Use the #defines in linux/sizes for malloc size as it is more readable. Signed-off-by: Peter Griffin --- include/configs/hikey.h | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/include/configs/hikey.h b/include/configs/hikey.h index

[U-Boot] [PATCH v2 4/8] ARM: hi6220: Add UART0 and UART3 base addresses

2015-09-10 Thread Peter Griffin
Signed-off-by: Peter Griffin --- arch/arm/include/asm/arch-hi6220/hi6220.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/arch/arm/include/asm/arch-hi6220/hi6220.h b/arch/arm/include/asm/arch-hi6220/hi6220.h index 4b987c2..3a12c75 100644 ---

[U-Boot] [PATCH v2 7/8] ARM: hikey: hi6220: Migrate over to DM_SERIAL and use UART3 by default.

2015-09-10 Thread Peter Griffin
Use DM for the pl01x serial driver on hikey. Also allow UART0 or UART3 to be chosen via Kconfig. By default we now output to UART3 as the latest version of ATF outputs to this UART. Also UART3 comes out on the LS connector, as opposed to UART0 which goes to a unpopulated header. As part of this

Re: [U-Boot] [PATCH v4 08/11] dm: pci: Add an inline API to test if a device is on a PCI bus

2015-09-10 Thread Simon Glass
Hi Bin, On 9 September 2015 at 21:58, Simon Glass wrote: > On 3 September 2015 at 06:37, Bin Meng wrote: >> Introduce device_is_on_pci_bus() which can be utilized by driver >> to test if a device is on a PCI bus. >> >> Signed-off-by: Bin Meng

Re: [U-Boot] [PATCH v3 1/5] spi: cadence_qspi: Ensure spi_calibration is run when sclk change

2015-09-10 Thread Pavel Machek
On Thu 2015-09-10 00:01:12, Chin Liang See wrote: > Ensuring spi_calibration is run when there is a change of sclk > frequency. This will ensure the qspi flash access works for high > sclk frequency > > Signed-off-by: Chin Liang See > Cc: Dinh Nguyen

[U-Boot] [PATCH v2 6/8] ARM: hikey: Select DM, DM_GPIO from Kconfig

2015-09-10 Thread Peter Griffin
Most platforms enable these options from Kconfig rather than the configs header file. Signed-off-by: Peter Griffin --- arch/arm/Kconfig| 2 ++ include/configs/hikey.h | 2 -- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arm/Kconfig

[U-Boot] [PATCH v2 5/8] ARM: hikey: Remove resetting gd->flags in board_init()

2015-09-10 Thread Peter Griffin
This causes exceptions and other strange behaviour when enabling CONFIG_SYS_MALLOC_F_LEN which is required to migrate the serial driver over to DM_SERIAL. As GD_FLG_FULL_MALLOC_INIT flag gets reset, after relocation we don't end up using the full malloc which ultimately ends up causing a

[U-Boot] [PATCH v2 0/8] arm: hikey: hi6220: Various fixups

2015-09-10 Thread Peter Griffin
Hi folks, This series fixes a few issues with the recently merged 96boards hikey u-boot port. It updates the README with some issues Simon found whilst attempting to follow my instructions. Also it now includes the ATF makefile referenced from the README to avoid an external URL as per Simons

[U-Boot] [PATCH v2 2/8] ARM: hikey: Add ATF makefile referenced by README

2015-09-10 Thread Peter Griffin
Rather than relying on an external URL in the README include the Makefile in the hikey directory. Signed-off-by: Peter Griffin --- board/hisilicon/hikey/build-tf.mak | 42 ++ 1 file changed, 42 insertions(+) create mode 100644

[U-Boot] [PATCH v2 8/8] ARM: hikey: Adjust SDRAM_1_SIZE to 0x3EFFFFFF

2015-09-10 Thread Peter Griffin
DRAM region 0x3f00 - 0x3fff is reserved for OP-TEE. Touching 0x3f00 memory location from unsecure world causes the board to hang. Signed-off-by: Peter Griffin --- include/configs/hikey.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

[U-Boot] [PATCH v2 1/8] ARM: hikey: Update README with various corrections

2015-09-10 Thread Peter Griffin
The README had a few mistakes, and one of the URL's had changed. Also update the boot log with the latest boot trace from ATF, which now includes the mcuimage.bin. Signed-off-by: Peter Griffin --- board/hisilicon/hikey/README | 197

[U-Boot] [PATCH v2 3/8] x86: quark: Convert to use clrbits, setbits, clrsetbits macros

2015-09-10 Thread Bin Meng
Change existing codes to use clrbits, setbits, clrsetbits macros. Signed-off-by: Bin Meng Acked-by: Simon Glass --- Changes in v2: None arch/x86/cpu/quark/quark.c | 72 ++ 1 file changed, 21 insertions(+), 51

[U-Boot] [PATCH v2 5/8] x86: quark: Initialize thermal sensor properly

2015-09-10 Thread Bin Meng
Thermal sensor on Quark SoC needs to be properly initialized per Quark firmware writer guide, otherwise when booting Linux kernel, it triggers system shutdown because of wrong temperature in the thermal sensor is detected by the kernel driver (see below): [5.119819] thermal_sys: Critical

[U-Boot] [PATCH v2 1/8] x86: galileo: Enable random mac address for Quark

2015-09-10 Thread Bin Meng
Not like other Intel Ethernet controllers (e.g.: E1000), Intel Quark SoC integrated designware Ethernet controller does not have a chipset defined way to store/restore mac address. Enable random mac address so that we can use Ethernet even without 'ethaddr'. Signed-off-by: Bin Meng

[U-Boot] [PATCH v2 6/8] x86: galileo: Add PCIe root port IRQ routing

2015-09-10 Thread Bin Meng
Now we have enabled PCIe root port on Quark SoC, add its PIRQ routing information in the device tree as well. Signed-off-by: Bin Meng Acked-by: Simon Glass --- Changes in v2: None arch/x86/dts/galileo.dts | 12 1 file changed, 12

[U-Boot] [PATCH v2 2/8] x86: quark: Add clrbits, setbits, clrsetbits macros for message port access

2015-09-10 Thread Bin Meng
On Intel Quark, lots of registers on the message port need be programmed. Add handy clrbits, setbits, clrsetbits macros for message port access. Signed-off-by: Bin Meng Reviewed-by: Simon Glass --- Changes in v2: - Change all macros to use a single

[U-Boot] [PATCH v2 0/8] x86: Support booting Linux kernel on Intel Galileo

2015-09-10 Thread Bin Meng
Currently U-Boot works great on Intel Galileo board, but it still has some issues when booting a Linux kernel. The issues are mainly with Isolated Memory Region (IMR) and thermal sensor support in the Linux kernel for Intel Quark SoC. For example, kernel just hangs after kernel IMR driver is

[U-Boot] [PATCH v2 7/8] x86: doc: Document some porting hints about Intel Quark

2015-09-10 Thread Bin Meng
Document porting considerations for Intel Quark based board, including MRC parameters and PCIe initialization. Signed-off-by: Bin Meng Acked-by: Simon Glass --- Changes in v2: - Update per review comments from Simon doc/README.x86 | 24

[U-Boot] [PATCH v2 8/8] x86: doc: Add DMI to the TODO list

2015-09-10 Thread Bin Meng
Desktop Management Interface (DMI) is not supported by U-Boot now. Add it to the TODO list. Signed-off-by: Bin Meng Acked-by: Simon Glass --- Changes in v2: None doc/README.x86 | 2 ++ 1 file changed, 2 insertions(+) diff --git a/doc/README.x86

[U-Boot] [PATCH v2 4/8] x86: quark: Lock HMBOUND register before jumping to kernel

2015-09-10 Thread Bin Meng
When Linux kernel boots, it hangs at: [0.829408] Intel Quark side-band driver registered This happens when Quark kernel Isolated Memory Region (IMR) driver tries to lock an IMR register to protect kernel's text and rodata sections. However in order to have IMR function correctly, HMBOUND

Re: [U-Boot] [RFC PATCH 2/2] sunxi: add "fel" boot target

2015-09-10 Thread Hans de Goede
Hi, On 03-09-15 16:12, Bernhard Nortmann wrote: This patch makes use of the previous changes to add a new "fel" boot target for sunxi boards. When booting via FEL, it's often desirable to work around the absence of other (usable) boot devices - or to be able to override them, deviating from