Re: [U-Boot] [PATCH] defconfig: taurus_defconfig: disable SPL

2015-12-09 Thread Yang, Wenyou
Hello Heiko, > -Original Message- > From: Heiko Schocher [mailto:h...@denx.de] > Sent: 2015年12月9日 16:46 > To: Yang, Wenyou > Cc: U-Boot Mailing List ; andreas.de...@googlemail.com; > Meier, Roger > Subject: Re: [PATCH]

Re: [U-Boot] [PATCH 00/24] usb: s3c-otg: Rename driver to dwc2-otg

2015-12-09 Thread Lukasz Majewski
Hi Marek, > The driver is for the dwc2 otg block , so rename it accordingly > finally. > > Marek Vasut (24): > usb: s3c-otg: Rename regs-otg.h to s3c_udc_otg_regs.h > usb: s3c-otg: Rename struct s3c_udc to dwc2_udc > usb: s3c-otg: Rename struct s3c_usbotg_reg to dwc2_usbotg_reg > usb:

Re: [U-Boot] [PATCH V2] ARM: imx: fsl_esdhc: fix usage of low 4 bits of sysctl register

2015-12-09 Thread Hector Palacios
On 12/04/2015 08:32 PM, Eric Nelson wrote: > The low four bits of the SYSCTL register are reserved on the USDHC > controller on i.MX6 and i.MX7 processors, but are used for clocking > operations on earlier models. > > Guard against their usage by hiding the bit mask macros on those > processors.

Re: [U-Boot] [PATCH] arm: zynq: Update ZYBO config options

2015-12-09 Thread Michal Simek
On 8.12.2015 15:44, Nathan Rossi wrote: > Update the ZYBO device tree and enable config options that relate to the > added devices in the device tree. > > Signed-off-by: Nathan Rossi > Cc: Albert Aribaud > Cc: Michal Simek >

[U-Boot] Please pull u-boot-x86, take 2

2015-12-09 Thread Bin Meng
Hi Tom, This includes fixes to PCI UART on CrownBay and Galileo, minor changes to x86 timer and some clean up to ivybridge support. The following changes since commit 8555dd88cd1a7b5dd1e76c111fe635a92223b98b: axs10x: add support of generic EHCI USB 2.0 controller (2015-12-08 17:29:56 +0300)

Re: [U-Boot] [PATCH] defconfig: taurus_defconfig: disable SPL

2015-12-09 Thread Yang, Wenyou
Hi > -Original Message- > From: Heiko Schocher [mailto:h...@denx.de] > Sent: 2015年12月9日 16:00 > To: Yang, Wenyou > Cc: U-Boot Mailing List ; andreas.de...@googlemail.com; > Meier, Roger > Subject: Re: [PATCH] defconfig:

[U-Boot] [PATCH v2 0/4] ARM: at91: add UTMI PLL handle functions

2015-12-09 Thread Wenyou Yang
To reduce the duplicated code, add UTMI PLL enable/disable functions, replace the UTMI PLL handle code with these functions. It is based on the following patch set. [PATCH v2 0/5] ARM: at91: improve peripheral and system clock handle functions Changes in v2: - remain asm/io.h include.

[U-Boot] [PATCH v2 4/4] board: atmel: siemens: clean up UTMI PLL code

2015-12-09 Thread Wenyou Yang
Due to introducing the new UTMI PLL clock handle functions, use the functions to reduce the duplicated code. Signed-off-by: Wenyou Yang --- Changes in v2: None board/siemens/corvus/board.c |4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git

[U-Boot] [PATCH v2 3/4] drivers: usb: atmel: clean up the UTMI PLL code

2015-12-09 Thread Wenyou Yang
Due to introducing the new UTMI PLL clock handle functions, use these function to reduce the duplicated code. Signed-off-by: Wenyou Yang --- Changes in v2: - remain asm/io.h include. drivers/usb/host/ehci-atmel.c | 38 --

[U-Boot] [PATCH v2 2/4] ARM: at91: armv7: clean up UTMI PLL handle code

2015-12-09 Thread Wenyou Yang
Due to introducing the UTMI PLL enable function, use this function to reduce the duplicated code. Signed-off-by: Wenyou Yang --- Changes in v2: None arch/arm/mach-at91/armv7/sama5d2_devices.c |5 + arch/arm/mach-at91/armv7/sama5d3_devices.c |5 +

[U-Boot] [PATCH v2 1/4] ARM: at91: clock: add UTMI PLL enable/disable function

2015-12-09 Thread Wenyou Yang
To avoid the duplicated code, add the UTMI PLL handle functions, and add PMC_USB init function too. Signed-off-by: Wenyou Yang --- Changes in v2: None arch/arm/mach-at91/clock.c| 48 + arch/arm/mach-at91/include/mach/clk.h |

Re: [U-Boot] [PATCH] usb: udc: Fix warnings on 64-bit builds

2015-12-09 Thread Michal Simek
On 9.12.2015 10:33, Lukasz Majewski wrote: > Hi Michal, > >> Hi Marek and Lukasz, >> >> On 3.11.2015 11:14, Lukasz Majewski wrote: >>> Hi Michal, >>> Cast u32 bit value to 64bit before recasting to 64bit pointer to avoid pointer from integer cast size mismatch warnings.

Re: [U-Boot] [PATCH] defconfig: taurus_defconfig: disable SPL

2015-12-09 Thread Heiko Schocher
Hello Wenyou, Am 09.12.2015 um 09:34 schrieb Yang, Wenyou: Hi -Original Message- From: Heiko Schocher [mailto:h...@denx.de] Sent: 2015年12月9日 16:00 To: Yang, Wenyou Cc: U-Boot Mailing List ; andreas.de...@googlemail.com; Meier, Roger

Re: [U-Boot] [PATCH] usb: udc: Fix warnings on 64-bit builds

2015-12-09 Thread Lukasz Majewski
Hi Michal, > Hi Marek and Lukasz, > > On 3.11.2015 11:14, Lukasz Majewski wrote: > > Hi Michal, > > > >> Cast u32 bit value to 64bit before recasting to 64bit pointer to > >> avoid pointer from integer cast size mismatch warnings. > >> > >> Warning log: > >>

Re: [U-Boot] [PATCH v2 0/5] ARM: at91: improve peripheral and system clock handle functions

2015-12-09 Thread Heiko Schocher
Hello Wenyou, Am 09.12.2015 um 08:48 schrieb Wenyou Yang: To reduce the duplicated code, rework the peripheral's and system's clock handle functions, use these functions to replace the clock handle code. Changes in v2: - fix checkpatch warning. Wenyou Yang (5): ARM: at91: asm/at91_pmc.h:

[U-Boot] ARMv7 MMU shareability issue (was: [PATCH] arm: Replace test for CONFIG_ARMV7 with CONFIG_CPU_V7)

2015-12-09 Thread Albert ARIBAUD
Hello Marek, On Tue, 8 Dec 2015 14:43:42 +0100, Marek Vasut wrote: > The arch/arm/lib/cache-cp15.c checks for CONFIG_ARMV7 and if this macro is > set, it configures TTBR0 register. This register must be configured for the > cache on ARMv7 to operate correctly. > > The problem is

Re: [U-Boot] [PATCH] spi: zynq_qspi: Add configuration to disable LQSPI feature

2015-12-09 Thread Michal Simek
On 8.12.2015 15:44, Nathan Rossi wrote: > When the Zynq Boot ROM code loads the payload from QSPI it uses the > LQSPI feature of the QSPI device, however it does not clean up its > configuration before handing over to the payload which leaves the device > confgured to by-pass the standard

Re: [U-Boot] [PATCH 1/3] ARM: at91: clock: add PMC_PLLICPR init function

2015-12-09 Thread Heiko Schocher
Hello Andreas, Am 09.12.2015 um 09:37 schrieb Andreas Bießmann: Hi Heiko, On 09.12.2015 08:14, Heiko Schocher wrote: Hello Wenyou, Am 09.12.2015 um 05:36 schrieb Wenyou Yang: To avoid the duplicated code, add the PMC_PLLICPR init function. Signed-off-by: Wenyou Yang

Re: [U-Boot] [PATCH] defconfig: taurus_defconfig: disable SPL

2015-12-09 Thread Heiko Schocher
Hello Wenyou, Am 09.12.2015 um 08:43 schrieb Yang, Wenyou: Hello Heiko, -Original Message- From: Heiko Schocher [mailto:h...@denx.de] Sent: 2015年12月9日 15:39 To: Yang, Wenyou Cc: U-Boot Mailing List ; andreas.de...@googlemail.com; Meier,

Re: [U-Boot] Secure Boot Mode on rk3288 Radxa Rock2

2015-12-09 Thread Sjoerd Simons
Hey Geoff On Mon, 2015-12-07 at 22:13 -0500, Geoff Cleary wrote: > We have successfully built u-boot loader for rk3288 from the rock_bsp > repository here: > https://github.com/radxa/rock-bsp.git The u-boot loader in the radxa/rockchip BSP is a vendor fork of an old mainline u-boot. This list is

Re: [U-Boot] Please pull u-boot-x86

2015-12-09 Thread Bin Meng
Hi Tom, On Wed, Dec 9, 2015 at 10:02 AM, Bin Meng wrote: > Hi Tom, > > This includes fixes to PCI UART on CrownBay and Galileo, and minor > changes to x86 timer. > > The following changes since commit 8555dd88cd1a7b5dd1e76c111fe635a92223b98b: > > axs10x: add support of

Re: [U-Boot] [PATCH] fdt: Change OF_BAD_ADDR to FDT_ADDR_T_NONE

2015-12-09 Thread Bin Meng
On Wed, Dec 9, 2015 at 3:58 PM, Stefan Roese wrote: > Hi Bin, > > On 07.12.2015 10:39, Bin Meng wrote: >> >> Currently OF_BAD_ADDR is always -1ULL. When using OF_BAD_ADDR as the >> return value of dev_get_addr(), it creates potential size mismatch >> as dev_get_addr() uses

Re: [U-Boot] [PATCH] mmc: update MMC_ERASE argument to match Linux kernel.

2015-12-09 Thread Hector Palacios
Hi Eric and Fabio, On 12/07/2015 03:50 PM, Eric Nelson wrote: > Table 41 of the JEDEC standard for eMMC says that bit 31 of > the command argument is obsolete when issuing the ERASE > command (CMD38) on page 115 of this document: >

Re: [U-Boot] ARMv7 MMU shareability issue

2015-12-09 Thread Stefan Roese
Hi All! On 09.12.2015 09:09, Albert ARIBAUD wrote: On Tue, 8 Dec 2015 14:43:42 +0100, Marek Vasut wrote: The arch/arm/lib/cache-cp15.c checks for CONFIG_ARMV7 and if this macro is set, it configures TTBR0 register. This register must be configured for the cache on ARMv7 to

Re: [U-Boot] [PATCH] arm: socfpga: Fix cache configuration

2015-12-09 Thread Chin Liang See
On Tue, 2015-12-08 at 13:53 +0100, Marek Vasut wrote: > On Tuesday, December 08, 2015 at 12:13:23 PM, Pavel Machek wrote: > > [...] > > > > > Thanks for the pointers. > > > > > > > > I checked the source and enabled the debug message. Noticed my > > > > failure > > > > is due to small LEB and

[U-Boot] [PATCH v3 7/7] tools: env: update usage strings

2015-12-09 Thread Andreas Fenkart
Signed-off-by: Andreas Fenkart --- tools/env/fw_env_main.c | 117 +++- 1 file changed, 75 insertions(+), 42 deletions(-) diff --git a/tools/env/fw_env_main.c b/tools/env/fw_env_main.c index 39c8771..4bd4216 100644 ---

Re: [U-Boot] [PATCH 00/24] usb: s3c-otg: Rename driver to dwc2-otg

2015-12-09 Thread Lukasz Majewski
Hi Marek, > On Wednesday, December 09, 2015 at 10:37:42 AM, Lukasz Majewski wrote: > > Hi Marek, > > > > > The driver is for the dwc2 otg block , so rename it accordingly > > > finally. > > > > > > Marek Vasut (24): > > > usb: s3c-otg: Rename regs-otg.h to s3c_udc_otg_regs.h > > > usb:

Re: [U-Boot] [PATCH v8 14/14] sf: Rename sf_ops.c to spi-flash.c

2015-12-09 Thread Jagan Teki
On 9 December 2015 at 10:24, Simon Glass wrote: > On 6 December 2015 at 11:34, Jagan Teki wrote: >> Since all spi-flash core operations are moved into >> sf_ops.c then it's better to renamed as spi-flash.c >> >> Signed-off-by: Jagan Teki

[U-Boot] [PATCH v3 6/7] tools: env: factor out parse_common_args

2015-12-09 Thread Andreas Fenkart
Signed-off-by: Andreas Fenkart --- tools/env/fw_env_main.c | 62 + 1 file changed, 37 insertions(+), 25 deletions(-) diff --git a/tools/env/fw_env_main.c b/tools/env/fw_env_main.c index daf4688..39c8771 100644 ---

Re: [U-Boot] [PATCH 00/24] usb: s3c-otg: Rename driver to dwc2-otg

2015-12-09 Thread Marek Vasut
On Wednesday, December 09, 2015 at 04:02:58 PM, Lukasz Majewski wrote: > Hi Marek, > > > On Wednesday, December 09, 2015 at 10:37:42 AM, Lukasz Majewski wrote: > > > Hi Marek, > > > > > > > The driver is for the dwc2 otg block , so rename it accordingly > > > > finally. > > > > > > > > Marek

[U-Boot] [PATCH v3 1/7] tools: env validate: pass values as 0-based array

2015-12-09 Thread Andreas Fenkart
passing argv/argc can produce off-by-one errors Signed-off-by: Andreas Fenkart --- common/env_flags.c | 14 +++--- include/env_flags.h | 2 +- tools/env/fw_env.c | 11 +++ 3 files changed, 15 insertions(+), 12 deletions(-) diff --git

[U-Boot] [PATCH v3 0/7] tools: env: simplify argument parsing

2015-12-09 Thread Andreas Fenkart
Currently parameter parsing happens in two steps. First with getopt at the beginning of main to parse the common options. Then second is using adhoc parsing in printenv/setenv code path. When doing adhoc parsing, the order of arguments is important. This patch will parse arguments only in one

[U-Boot] [PATCH v3 2/7] tools: env: make parse_aes_key stateless

2015-12-09 Thread Andreas Fenkart
Signed-off-by: Andreas Fenkart --- tools/env/fw_env.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/tools/env/fw_env.c b/tools/env/fw_env.c index 22507f6..5b76b74 100644 --- a/tools/env/fw_env.c +++ b/tools/env/fw_env.c @@

Re: [U-Boot] [PATCH] eth: dtsec: fix TBI ANA setting bug in dtsec_configure_serdes()

2015-12-09 Thread 李远正
The value TBIANA_SETTINGS is also work on the p2041rdb board, so it can work on the both phy and phyless connections. At 2015-11-26 12:17:42, "Shaohui Xie" wrote: >> -Original Message- >> From: York Sun [mailto:york...@freescale.com] >> Sent: Thursday,

Re: [U-Boot] [PATCH] arm: socfpga: Fix cache configuration

2015-12-09 Thread Chin Liang See
On Tue, 2015-12-08 at 13:54 +0100, Marek Vasut wrote: > On Tuesday, December 08, 2015 at 01:04:29 PM, Stefan Roese wrote: > > On 08.12.2015 12:13, Pavel Machek wrote: > > > > > > > Usage: > > > > > > > ubifsmount > > > > > > > > > > > > > > - mount 'volume-name' volume > > > > > > > > > >

[U-Boot] [PATCH v3 4/7] tools: env: parse aes key / suppress flag into argument struct

2015-12-09 Thread Andreas Fenkart
disabled original parsing, but not yet removed since the argument indexing needs to be fixed Signed-off-by: Andreas Fenkart --- tools/env/fw_env.c | 64 ++--- tools/env/fw_env.h | 15

[U-Boot] [PATCH v3 3/7] tools: env: introduce setenv/printenv argument structs

2015-12-09 Thread Andreas Fenkart
goal is to use getopt for all argument parsing instead of adhoc parsing in fw_getenv/fw_setenv functions Signed-off-by: Andreas Fenkart --- tools/env/fw_env.h | 9 tools/env/fw_env_main.c | 113 2

[U-Boot] [PATCH v3 5/7] tools: env: shift optind arguments and fix argument indices

2015-12-09 Thread Andreas Fenkart
Signed-off-by: Andreas Fenkart --- tools/env/fw_env.c | 54 ++--- tools/env/fw_env_main.c | 4 2 files changed, 15 insertions(+), 43 deletions(-) diff --git a/tools/env/fw_env.c b/tools/env/fw_env.c index

Re: [U-Boot] [PATCH 5/6] arm: socfpga: Introduce common board code

2015-12-09 Thread Chin Liang See
On Tue, 2015-12-08 at 13:57 +0100, Marek Vasut wrote: > On Tuesday, December 08, 2015 at 01:34:53 PM, Chin Liang See wrote: > > On Mon, 2015-12-07 at 18:46 +0100, Marek Vasut wrote: > > > On Monday, December 07, 2015 at 03:56:01 PM, Chin Liang See > > > wrote: > > > [...] > > > > > > > > The

Re: [U-Boot] [PATCH] arm: socfpga: Fix cache configuration

2015-12-09 Thread Marek Vasut
On Wednesday, December 09, 2015 at 02:48:49 PM, Chin Liang See wrote: > On Tue, 2015-12-08 at 13:54 +0100, Marek Vasut wrote: > > On Tuesday, December 08, 2015 at 01:04:29 PM, Stefan Roese wrote: > > > On 08.12.2015 12:13, Pavel Machek wrote: > > > > > > > > Usage: > > > > > > > > ubifsmount > >

Re: [U-Boot] [PATCH 1/3] ARM: at91: clock: add PMC_PLLICPR init function

2015-12-09 Thread Andreas Bießmann
Hi Heiko, On 09.12.2015 08:14, Heiko Schocher wrote: > Hello Wenyou, > > Am 09.12.2015 um 05:36 schrieb Wenyou Yang: >> To avoid the duplicated code, add the PMC_PLLICPR init function. >> >> Signed-off-by: Wenyou Yang >> --- >> >> arch/arm/mach-at91/armv7/clock.c

Re: [U-Boot] ARMv7 MMU shareability issue

2015-12-09 Thread Marek Vasut
On Wednesday, December 09, 2015 at 02:02:03 PM, Stefan Roese wrote: > Hi All! > > On 09.12.2015 09:09, Albert ARIBAUD wrote: > > On Tue, 8 Dec 2015 14:43:42 +0100, Marek Vasut wrote: > >> The arch/arm/lib/cache-cp15.c checks for CONFIG_ARMV7 and if this macro > >> is set, it

Re: [U-Boot] [PATCH 00/24] usb: s3c-otg: Rename driver to dwc2-otg

2015-12-09 Thread Marek Vasut
On Wednesday, December 09, 2015 at 10:37:42 AM, Lukasz Majewski wrote: > Hi Marek, > > > The driver is for the dwc2 otg block , so rename it accordingly > > finally. > > > > Marek Vasut (24): > > usb: s3c-otg: Rename regs-otg.h to s3c_udc_otg_regs.h > > usb: s3c-otg: Rename struct s3c_udc to

Re: [U-Boot] [PATCH 00/24] usb: s3c-otg: Rename driver to dwc2-otg

2015-12-09 Thread Lukasz Majewski
Hi Marek, > On Wednesday, December 09, 2015 at 04:02:58 PM, Lukasz Majewski wrote: > > Hi Marek, > > > > > On Wednesday, December 09, 2015 at 10:37:42 AM, Lukasz Majewski > > > wrote: > > > > Hi Marek, > > > > > > > > > The driver is for the dwc2 otg block , so rename it > > > > > accordingly

Re: [U-Boot] [PATCH 00/24] usb: s3c-otg: Rename driver to dwc2-otg

2015-12-09 Thread Marek Vasut
On Wednesday, December 09, 2015 at 05:06:52 PM, Lukasz Majewski wrote: > Hi Marek, > > > On Wednesday, December 09, 2015 at 04:02:58 PM, Lukasz Majewski wrote: > > > Hi Marek, > > > > > > > On Wednesday, December 09, 2015 at 10:37:42 AM, Lukasz Majewski > > > > > > > > wrote: > > > > > Hi

Re: [U-Boot] [PATCH V2 1/7] test/py: Implement pytest infrastructure

2015-12-09 Thread Stephen Warren
On 12/02/2015 03:18 PM, Stephen Warren wrote: This tool aims to test U-Boot by executing U-Boot shell commands using the console interface. A single top-level script exists to execute or attach to the U-Boot console, run the entire script of tests against it, and summarize the results.

[U-Boot] [PATCH] part: fix "part list ... -bootable varname" to use hex

2015-12-09 Thread Stephen Warren
From: Stephen Warren Unfortunately U-Boot assumes that almost all numbers are in hex, including partition numbers passed to e.g. "load". So, the command "part list mmc 0 -bootable devplist" should use hex when writing partition numbers into $devplist, so they'll be correctly

[U-Boot] [PATCH] net: phy: do not read configuration register on reset

2015-12-09 Thread Stefan Agner
When doing a software reset, the reset flag should be written without other bits set. Writing the current state will lead to restoring the state of the PHY (e.g. Powerdown), which is not what is expected from a software reset. Signed-off-by: Stefan Agner --- This lead to the PHY

Re: [U-Boot] [PATCH] net: phy: do not read configuration register on reset

2015-12-09 Thread Michael Welling
On Wed, Dec 09, 2015 at 11:21:25AM -0800, Stefan Agner wrote: > When doing a software reset, the reset flag should be written without > other bits set. Writing the current state will lead to restoring the > state of the PHY (e.g. Powerdown), which is not what is expected from > a software reset. >

Re: [U-Boot] [PATCH] ARM: mxs: allow boards to select DC-DC switching clock source

2015-12-09 Thread Fabio Estevam
Hi Michael, On Wed, Dec 9, 2015 at 7:30 PM, Michael Heimpold wrote: > For some board designs, it might be useful to switch the DC-DC > clock source to something else rather the default 24 MHz, e.g. > for EMI reasons. > > For this, use something like this in your board

[U-Boot] [PATCH] ARM: mxs: allow boards to select DC-DC switching clock source

2015-12-09 Thread Michael Heimpold
For some board designs, it might be useful to switch the DC-DC clock source to something else rather the default 24 MHz, e.g. for EMI reasons. For this, use something like this in your board configuration: Signed-off-by: Michael Heimpold Cc: Marek Vasut Cc:

Re: [U-Boot] Please pull u-boot-x86, take 2

2015-12-09 Thread Tom Rini
On Wed, Dec 09, 2015 at 05:55:05PM +0800, Bin Meng wrote: > Hi Tom, > > This includes fixes to PCI UART on CrownBay and Galileo, minor changes > to x86 timer and some clean up to ivybridge support. > > The following changes since commit 8555dd88cd1a7b5dd1e76c111fe635a92223b98b: > > axs10x:

Re: [U-Boot] [PATCH] ARM: mxs: allow boards to select DC-DC switching clock source

2015-12-09 Thread Marek Vasut
On Wednesday, December 09, 2015 at 10:30:46 PM, Michael Heimpold wrote: > For some board designs, it might be useful to switch the DC-DC > clock source to something else rather the default 24 MHz, e.g. > for EMI reasons. > > For this, use something like this in your board configuration: > >

Re: [U-Boot] [PATCH 3/3] sandbox: sandbox_flash.c: Ensure NUL-termination on product/vendor strings

2015-12-09 Thread Simon Glass
Hi Tom, On 8 December 2015 at 16:58, Tom Rini wrote: > > On Tue, Dec 08, 2015 at 04:42:17PM -0700, Simon Glass wrote: > > Hi Tom, > > > > On Dec 8, 2015 4:28 PM, "Tom Rini" wrote: > > > > > > On Tue, Dec 08, 2015 at 12:35:23PM -0700, Simon Glass wrote: >

Re: [U-Boot] [PATCH] arm: Add and select CONFIG_ARMV7

2015-12-09 Thread Tom Rini
On Tue, Dec 08, 2015 at 05:20:49PM +0100, Wolfgang Denk wrote: > Dear Marek, > > In message <201512081439.50797.ma...@denx.de> you wrote: > > > > Please ignore, after discussing this on IRC, we decided on doing > > s/CONFIG_ARMV7/CONFIG_CPU_V7/g instead. > > As IRC si (to the best or my

[U-Boot] [PATCH] fs: ext4: Remove ext4fs_free_node()

2015-12-09 Thread Tom Rini
The function ext4fs_free_node() exists for dealing with "dirnode" structures that we allocate. However, we do not allocate these dynamically as needed but rather as a single instance in ext4fs_mount() that we zalloc(). Coverity scan notes that in two places we're doing what it calls a "Free of

Re: [U-Boot] [PATCH 3/3] sandbox: sandbox_flash.c: Ensure NUL-termination on product/vendor strings

2015-12-09 Thread Tom Rini
On Wed, Dec 09, 2015 at 04:58:12PM -0700, Simon Glass wrote: > Hi Tom, > > On 8 December 2015 at 16:58, Tom Rini wrote: > > > > On Tue, Dec 08, 2015 at 04:42:17PM -0700, Simon Glass wrote: > > > Hi Tom, > > > > > > On Dec 8, 2015 4:28 PM, "Tom Rini"

Re: [U-Boot] [PATCH v8 14/14] sf: Rename sf_ops.c to spi-flash.c

2015-12-09 Thread Bin Meng
On Wed, Dec 9, 2015 at 12:54 PM, Simon Glass wrote: > On 6 December 2015 at 11:34, Jagan Teki wrote: >> Since all spi-flash core operations are moved into >> sf_ops.c then it's better to renamed as spi-flash.c >> >> Signed-off-by: Jagan Teki

Re: [U-Boot] [PATCH v8 08/14] sf: Flash power up read-only based on idcode0

2015-12-09 Thread Bin Meng
On Mon, Dec 7, 2015 at 2:34 AM, Jagan Teki wrote: > Using macro's for flash power up read-only access code > leads wrong behaviour hence use idcode0 for runtime > detection, hence the flash which require this functionality > gets detected at runtime. > > Reviewed-by: Simon

Re: [U-Boot] [PATCH v8 04/14] sf: probe: Code cleanup

2015-12-09 Thread Bin Meng
On Mon, Dec 7, 2015 at 2:34 AM, Jagan Teki wrote: > - Move bar read code below the bar write hance both > at once place, hence it easy for #ifdef macro only > once and readable. > - Move read_cmd_array at top > > Reviewed-by: Simon Glass >

Re: [U-Boot] [PATCH v8 07/14] sf: Use simple name for register access functions

2015-12-09 Thread Bin Meng
On Mon, Dec 7, 2015 at 2:34 AM, Jagan Teki wrote: > Most of the register access function are static, > so used simple name to represent each. > > Reviewed-by: Simon Glass > Signed-off-by: Jagan Teki > --- > drivers/mtd/spi/sf_ops.c |

Re: [U-Boot] [PATCH v8 10/14] sf: Remove unneeded SST_BP and SST_WP

2015-12-09 Thread Bin Meng
On Mon, Dec 7, 2015 at 2:34 AM, Jagan Teki wrote: > SST parts added on sf_params.c supports both SST_WR which consits > of both BP and WP and there is a spi controller ich which supports > only BP so the relevent _write hook set based on "slave->op_mode_tx" > hence there is no

Re: [U-Boot] [PATCH v8 12/14] sf: sf_probe: Remove spi_slave pointer argument

2015-12-09 Thread Bin Meng
On Mon, Dec 7, 2015 at 2:34 AM, Jagan Teki wrote: > Since spi_slave is a spi pointer in spi_flash{} then assign > spi_slave{} pointer to flash->spi and remove spi_slave > pointer argument to > - spi_flash_probe_slave > - spi_flash_scan > > Cc: Simon Glass >

Re: [U-Boot] [PATCH v8 09/14] sf: Remove unneeded header includes

2015-12-09 Thread Bin Meng
On Mon, Dec 7, 2015 at 2:34 AM, Jagan Teki wrote: > Removed unneeded header includes in sf_ops and sf_probe > > Reviewed-by: Simon Glass > Signed-off-by: Jagan Teki > --- > drivers/mtd/spi/sf_ops.c | 2 -- >

Re: [U-Boot] [PATCH v2 0/5] ARM: at91: improve peripheral and system clock handle functions

2015-12-09 Thread Yang, Wenyou
Hi Heiko, > -Original Message- > From: Heiko Schocher [mailto:h...@denx.de] > Sent: 2015年12月9日 18:01 > To: Yang, Wenyou > Cc: U-Boot Mailing List ; andreas.de...@googlemail.com > Subject: Re: [PATCH v2 0/5] ARM: at91: improve peripheral and

Re: [U-Boot] ARMv7 MMU shareability issue

2015-12-09 Thread Tom Rini
On Wed, Dec 09, 2015 at 03:09:13PM +0100, Marek Vasut wrote: > On Wednesday, December 09, 2015 at 02:02:03 PM, Stefan Roese wrote: > > Hi All! > > > > On 09.12.2015 09:09, Albert ARIBAUD wrote: > > > On Tue, 8 Dec 2015 14:43:42 +0100, Marek Vasut wrote: > > >> The

Re: [U-Boot] [PATCH] net: phy: do not read configuration register on reset

2015-12-09 Thread Bin Meng
On Thu, Dec 10, 2015 at 3:21 AM, Stefan Agner wrote: > When doing a software reset, the reset flag should be written without > other bits set. Writing the current state will lead to restoring the > state of the PHY (e.g. Powerdown), which is not what is expected from > a software

Re: [U-Boot] [PATCH 02/19] dm: timer: uclass: add timer init to add timer device

2015-12-09 Thread Bin Meng
Hi Mugunthan, On Wed, Dec 2, 2015 at 5:50 PM, Mugunthan V N wrote: > On Wednesday 02 December 2015 02:59 PM, Bin Meng wrote: >> Hi Mugunthan, >> >> On Tue, Dec 1, 2015 at 10:39 AM, Bin Meng wrote: >>> Hi Mugunthan, >>> >>> On Sun, Nov 29, 2015 at 9:16

Re: [U-Boot] [PATCH v8 13/14] sf: Use static for file-scope functions

2015-12-09 Thread Bin Meng
On Mon, Dec 7, 2015 at 2:34 AM, Jagan Teki wrote: > Used static for file-scope functions in sf_probe.c > > Cc: Simon Glass > Signed-off-by: Jagan Teki > --- > drivers/mtd/spi/sf_probe.c | 10 +- > 1 file changed, 5

Re: [U-Boot] [PATCH v8 03/14] sf: Move read_id code to sf_ops

2015-12-09 Thread Bin Meng
On Mon, Dec 7, 2015 at 2:34 AM, Jagan Teki wrote: > read_id code is related to spi_flash stuff > hence moved to sf_ops. > > Reviewed-by: Simon Glass > Signed-off-by: Jagan Teki > --- > drivers/mtd/spi/sf_internal.h | 3 +-- >

Re: [U-Boot] [PATCH v8 01/14] sf: spi_flash_validate_params => spi_flash_scan

2015-12-09 Thread Bin Meng
On Mon, Dec 7, 2015 at 2:34 AM, Jagan Teki wrote: > Rename spi_flash_validate_params to spi_flash_scan > as this code not only deals with params setup but > also configure all spi_flash attributes. > > And also moved all flash related code into > spi_flash_scan for future

Re: [U-Boot] [PATCH v8 05/14] sf: Use static for file-scope functions

2015-12-09 Thread Bin Meng
On Mon, Dec 7, 2015 at 2:34 AM, Jagan Teki wrote: > Use static for file-scope functions and removed > them from header files. > > Signed-off-by: Jagan Teki > --- > drivers/mtd/spi/sf_internal.h | 18 -- > drivers/mtd/spi/sf_ops.c | 11

Re: [U-Boot] [PATCH v8 02/14] sf: Move spi_flash_scan code to sf_ops

2015-12-09 Thread Bin Meng
On Mon, Dec 7, 2015 at 2:34 AM, Jagan Teki wrote: > Intension is that sf_ops should deals all spi_flash > related stuff and sf_probe (which should renamed future) > should be an interface layer for spi_flash versus spi drivers. > > sf_ops => spi_flash interface > sf_probe =>

[U-Boot] [PATCH v3 0/5] ARM: at91: improve peripheral and system clock handle functions

2015-12-09 Thread Wenyou Yang
To reduce the duplicated code, rework the peripheral's and system's clock handle functions, use these functions to replace the clock handle code. Changes in v3: - fix incorrectly used the indicator '&' to achieve the variable pointer of pmc->pcer and pmc->pcdr. Changes in v2: - fix

[U-Boot] [PATCH v3 1/5] ARM: at91: asm/at91_pmc.h: fix trival register offset

2015-12-09 Thread Wenyou Yang
Remove unnecessary #ifdef CPU_HAS_PCR. Signed-off-by: Wenyou Yang --- Changes in v3: None Changes in v2: None arch/arm/mach-at91/include/mach/at91_pmc.h |8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git

[U-Boot] [PATCH v3 2/5] ARM: at91: clock: add a new file to handle clock

2015-12-09 Thread Wenyou Yang
To reduce the duplicated code, add a new file to accommodate the peripheral's and system's clock handle code, shared with the SoCs with different ARM core. Signed-off-by: Wenyou Yang --- Changes in v3: - fix incorrectly used the indicator '&' to achieve the variable

[U-Boot] [PATCH v3 3/5] ARM: cpu: at91: clean up peripheral clock code

2015-12-09 Thread Wenyou Yang
Due to introducing the new peripheral clock handle functions, use these functions to clean up the duplicated code. Meanwhile, remove unneeded header file include, at91_pmc.h. Signed-off-by: Wenyou Yang --- Changes in v3: None Changes in v2: None

[U-Boot] [PATCH v3 4/5] board: atmel: clean up peripheral clock code

2015-12-09 Thread Wenyou Yang
Due to introducing the new peripheral clock handle functions, use these functions to reduce duplicated code. Signed-off-by: Wenyou Yang --- Changes in v3: None Changes in v2: - fix checkpatch warning. board/atmel/at91rm9200ek/at91rm9200ek.c |1 -

[U-Boot] [PATCH v3 5/5] drivers: at91: clean up peripheral clock code

2015-12-09 Thread Wenyou Yang
Due to introducing the new peripheral clock handle functions, use these functions to reduce the duplicated code. Signed-off-by: Wenyou Yang --- Changes in v3: None Changes in v2: None drivers/net/at91_emac.c |9 - drivers/usb/host/ehci-atmel.c |8

Re: [U-Boot] [PATCH v2 0/5] ARM: at91: improve peripheral and system clock handle functions

2015-12-09 Thread Yang, Wenyou
Hi Heiko, > -Original Message- > From: Heiko Schocher [mailto:h...@denx.de] > Sent: 2015年12月10日 14:02 > To: Yang, Wenyou > Cc: U-Boot Mailing List ; andreas.de...@googlemail.com; > Meier, Roger > Subject: Re: [PATCH v2

Re: [U-Boot] [PATCH v2 0/5] ARM: at91: improve peripheral and system clock handle functions

2015-12-09 Thread Heiko Schocher
Hello Wenyou, Am 10.12.2015 um 03:15 schrieb Yang, Wenyou: Hi Heiko, -Original Message- From: Heiko Schocher [mailto:h...@denx.de] Sent: 2015年12月9日 18:01 To: Yang, Wenyou Cc: U-Boot Mailing List ; andreas.de...@googlemail.com Subject: Re:

Re: [U-Boot] [PATCH] dm: core: Add platform specific bus translation function

2015-12-09 Thread Stefan Roese
Hi Simon, On 08.12.2015 03:46, Simon Glass wrote: Hi Stefan, On 4 December 2015 at 00:45, Stefan Roese wrote: Hi Simon, On 03.12.2015 18:21, Simon Glass wrote: Hi Stefan, On 3 December 2015 at 04:31, Stefan Roese wrote: Hi Simon, On 02.12.2015 18:45, Simon

Re: [U-Boot] ARMv7 MMU shareability issue

2015-12-09 Thread Marek Vasut
On Thursday, December 10, 2015 at 03:27:08 AM, Tom Rini wrote: > On Wed, Dec 09, 2015 at 03:09:13PM +0100, Marek Vasut wrote: > > On Wednesday, December 09, 2015 at 02:02:03 PM, Stefan Roese wrote: > > > Hi All! > > > > > > On 09.12.2015 09:09, Albert ARIBAUD wrote: > > > > On Tue, 8 Dec 2015

[U-Boot] [PATCH] board/t4240rdb: VID support

2015-12-09 Thread ying.zhang
From: Ying Zhang The fuse status register provides the values from on-chip voltage ID efuses programmed at the factory. These values define the voltage requirements for the chip. u-boot reads FUSESR and translates the values into the appropriate commands to set the voltage

[U-Boot] [PATCH] ARM: Disable "DISCARD" for secure section if CONFIG_ARMV7_SECURE_BASE isn't defined

2015-12-09 Thread Dongsheng Wang
From: Wang Dongsheng Fix PSCI hang up without CONFIG_ARMV7_SECURE_BASE define. "DISCARD" will remove ._secure.text relocate, but PSCI framework has already used some absolute address those need to relocate. Use readelf -t -r u-boot show us: .__secure_start addr:

[U-Boot] [PATCH] armv8: fsl-layerscape: fixes lsch2 serdes registers define

2015-12-09 Thread shh.xie
From: Shaohui Xie Fixes lsch2 SerDes registers define according to LS1043A RM Rev D. Signed-off-by: Shaohui Xie --- .../include/asm/arch-fsl-layerscape/immap_lsch2.h | 67 +++--- 1 file changed, 47 insertions(+), 20