Re: [U-Boot] [PATCH] ARM: at91: sama5: change the environment address to 0x6000

2015-10-25 Thread Bo Shen
Hi Josh, On 10/23/2015 17:18 PM, Josh Wu wrote: As sama5 board has 32k sram size, so the at91bootstrap and spl for sama5 boards is bigger than 16k (0x4000). That will overlap the U-Boot Just curious about from which commit the spl binary size bigger than 16k? Best Regards, Bo Shen ___

[U-Boot] [PATCH] mpc85xx/t2081: enable parsing DDR ratio for T2081 rev1.1

2015-10-25 Thread Shengzhou Liu
T2081 rev 1.1 changes MEM_PLL_RAT in RCW which requires new parsing for ratio. Signed-off-by: Shengzhou Liu --- arch/powerpc/cpu/mpc85xx/speed.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/arch/powerpc/cpu/mpc85xx/speed.c b/arch/powerpc/cpu/mpc85xx/speed.c index 79202bb

[U-Boot] [PATCH] arm: ls102xa: Set fdt_high and initrd_high to the value of 0xffffffff

2015-10-25 Thread Alison Wang
As 3G/1G user/kernel memory split is used on LS1021A, the Linux kernel fails to access the device tree blob on boot. The reason is that u-boot relocates the device tree blob into high memory when booting the kernel and the kernel is unable to access the blob. To avoid this issue, fdt_high is set t

Re: [U-Boot] [PATCH v5 17/23] spi: cadence_qspi_apb: Use GENMASK

2015-10-25 Thread Stefan Roese
On 25.10.2015 00:25, Tom Rini wrote: On Sun, Oct 25, 2015 at 12:13:14AM +0200, Marek Vasut wrote: On Saturday, October 24, 2015 at 11:51:39 PM, Tom Rini wrote: On Sat, Oct 24, 2015 at 02:41:41PM +0200, Marek Vasut wrote: On Saturday, October 24, 2015 at 05:39:04 AM, Jagan Teki wrote: Replace

Re: [U-Boot] [PATCH v5 01/23] spi: zynq_[q]spi: Use BIT macro

2015-10-25 Thread Siva Durga Prasad Paladugu
> -Original Message- > From: Jagan Teki [mailto:jt...@openedev.com] > Sent: Saturday, October 24, 2015 9:09 AM > To: u-boot@lists.denx.de > Cc: Jagan Teki; Siva Durga Prasad Paladugu; Michal Simek > Subject: [PATCH v5 01/23] spi: zynq_[q]spi: Use BIT macro > > Used BIT macro on zynq_spi.

Re: [U-Boot] [PATCH] net/eth: fix a bug in on_ethaddr()

2015-10-25 Thread Hu Vincent
Hi Joe, Do you have any objection about this patch? If yes, then please share your comments then we can refine it. If no, would you please help to pick up the patch? Thanks in advance. Mingkai > -Original Message- > From: Hu Mingkai-B21284 > Sent: Wednesday, October 21, 2015 6:07 PM >

Re: [U-Boot] [PATCH 03/13] compat: Remove is_power_of_2() definition

2015-10-25 Thread Heiko Schocher
Hello Fabian, Am 24.10.2015 um 14:28 schrieb Fabio Estevam: From: Fabio Estevam Use the is_power_of_2() definition from log2.h to align with the kernel implementation. Signed-off-by: Fabio Estevam --- arch/arm/mach-mvebu/mbus.c | 2 +- drivers/mtd/mtdcore.c | 2 +- drivers/mtd/ubi/b

Re: [U-Boot] [PATCH] env_ubi.c: Correct pointer error in env load

2015-10-25 Thread Heiko Schocher
Hello Kevin, Am 23.10.2015 um 19:51 schrieb Kevin Smith: The variable "buf" in this function is a char array, and the function ubi_volume_read is expecting a char *. In the call, the address of the pointer is being taken, incorrectly passing a char **. The compiler warning was being silenced b

Re: [U-Boot] [PATCH v2] spi: zynq_qspi: Minor cleanups

2015-10-25 Thread Jagan Teki
On 26 October 2015 at 00:21, Tom Rini wrote: > On Sun, Oct 25, 2015 at 07:49:57PM +0530, Jagan Teki wrote: >> - Use __func__ on debug >> - Removed unnecessary comment >> - Fix function name in debug as zynq_qspi_xfer instead of spi_xfer >> >> Cc: Tom Rini >> Signed-off-by: Jagan Teki >> --- >> C

Re: [U-Boot] [PATCH v1 01/12] rockchip: define SUPPORT_SPL and SPL config in RK3288 config

2015-10-25 Thread Simon Glass
Hi, On Oct 22, 2015 7:22 PM, "hl" wrote: > > Hi Simon, > > > On 23/10/15 09:04, Simon Glass wrote: >> >> Hi Eddie, >> >> On 22 October 2015 at 18:54, Eddie Cai wrote: >>> >>> Hi Simon >>> >>> 2015-10-23 0:30 GMT+08:00 Simon Glass : Hi Eddie, On 22 October 2015 at 10:01, Eddie

Re: [U-Boot] [PATCH] gpio: atmel: Add the PIO4 driver support

2015-10-25 Thread Yang, Wenyou
Hi Andreas, Could you help review? Thanks a lot. > -Original Message- > From: Wenyou Yang [mailto:wenyou.y...@atmel.com] > Sent: 2015年9月8日 14:33 > To: U-Boot Mailing List > Cc: Yang, Wenyou; andreas.de...@googlemail.com > Subject: [PATCH] gpio: atmel: Add the PIO4 driver support > > The

Re: [U-Boot] [PATCH v2] mmc: atmel: Add atmel sdhci support

2015-10-25 Thread Yang, Wenyou
Hi Andreas, Thank you for your review. The new version is sent, please help review, thanks. > -Original Message- > From: Andreas Bießmann [mailto:andreas.de...@googlemail.com] > Sent: 2015年10月24日 5:27 > To: Yang, Wenyou; U-Boot Mailing List > Cc: Bo Shen; Pantelis Antoniou > Subject: Re:

Re: [U-Boot] [PATCH] arm: at91: clock: Add the generated clock support

2015-10-25 Thread Yang, Wenyou
Hi Andreas, Thank you for your review. The new version has been sent for review. > -Original Message- > From: Andreas Bießmann [mailto:andreas.de...@googlemail.com] > Sent: 2015年10月24日 5:01 > To: Yang, Wenyou; U-Boot Mailing List > Subject: Re: [PATCH] arm: at91: clock: Add the generate

[U-Boot] [PATCH v3] mmc: atmel: Add atmel sdhci support

2015-10-25 Thread Wenyou Yang
The SDHCI is introduced by sama5d2, named as Secure Digital Multimedia Card Controller(SDMMC). It supports the embedded MultiMedia Card (e.MMC) Specification V4.41, the SD Memory Card Specification V3.0, and the SDIO V3.0 specification. It is compliant with the SD Host Controller Standard V3.0 spec

[U-Boot] [PATCH v2] arm: at91: clock: Add the generated clock support

2015-10-25 Thread Wenyou Yang
Some peripherals may need a second clock source that may be different from the system clock. This second clock is the generated clock (GCK) and is managed by the PMC via PMC_PCR. For simplicity, the clock source of the GCK is fixed to PLLA_CLK. Signed-off-by: Wenyou Yang --- Hi Andreas, Bo Shen,

Re: [U-Boot] [PATCH V2 09/14] video: mxsfb: introduce lcdif_power_down

2015-10-25 Thread Peng Fan
Hi Stefano, On Tue, Oct 20, 2015 at 03:59:18PM +0200, Stefano Babic wrote: >Hi Peng, > >On 20/10/2015 13:39, Peng Fan wrote: >> 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 con

Re: [U-Boot] [PATCH V2 08/14] imx: mx6ul_14x14_evk: support lcdif display

2015-10-25 Thread Peng Fan
Hi Stefano, On Tue, Oct 20, 2015 at 03:41:58PM +0200, Stefano Babic wrote: >Hi Peng, > >On 20/10/2015 13:39, Peng Fan wrote: >> 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 heade

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

2015-10-25 Thread Peng Fan
Hi Stefano, On Tue, Oct 20, 2015 at 03:39:32PM +0200, Stefano Babic wrote: >Hi Peng, > >On 20/10/2015 13:39, Peng Fan wrote: >> 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. >> >>

Re: [U-Boot] [PATCH V2 01/14] mxs: add parameter base_addr for mxs_set_lcdclk

2015-10-25 Thread Peng Fan
Hi Stefano, Sorry for this late reply. On Tue, Oct 20, 2015 at 03:05:05PM +0200, Stefano Babic wrote: >Hi Peng, > > >On 20/10/2015 13:39, Peng Fan wrote: >> 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 L

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

2015-10-25 Thread Peng Fan
Hi Stefano, On Tue, Oct 20, 2015 at 03:15:19PM +0200, Stefano Babic wrote: >Hi Peng, > >On 20/10/2015 13:39, Peng Fan wrote: >> 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

Re: [U-Boot] [PATCH V2 03/14] xfi3: correct function name

2015-10-25 Thread Peng Fan
Hi Stefano, On Tue, Oct 20, 2015 at 03:10:38PM +0200, Stefano Babic wrote: >On 20/10/2015 13:39, Peng Fan wrote: >> board_mxsfb_system_setup shoule be named mxsfb_system_setup. > ^---should > >But really it is better *must*, else it is dead code as we currently have.

[U-Boot] [PATCH] nios2: add 10m50 devboard support

2015-10-25 Thread Thomas Chou
Add 10m50 devboard support. It is based on the Golden Hardware Reference Design (GHRD), available at, http://rocketboards.org/foswiki/view/Documentation/ AlteraMAX1010M50RevCDevelopmentKitLinuxSetup Though we supported only one nios2-generic board in the past. Now, with the removal of the nios2-g

Re: [U-Boot] [PATCH] mtd: do not include cfi_flash.h if no flash

2015-10-25 Thread Marek Vasut
On Monday, October 26, 2015 at 12:15:15 AM, Thomas Chou wrote: > Hi Marek, Hi! > On 10/25/2015 09:30 PM, Marek Vasut wrote: > > btw. do you plan to send a PR with all the nios2 goodies (patches) any > > soon ? > > I did send it. And Tom applied them already. Excellent , thanks! > I am working

Re: [U-Boot] [PATCH] mtd: do not include cfi_flash.h if no flash

2015-10-25 Thread Thomas Chou
Hi Marek, On 10/25/2015 09:30 PM, Marek Vasut wrote: btw. do you plan to send a PR with all the nios2 goodies (patches) any soon ? I did send it. And Tom applied them already. I am working on max10 devboard. I am cleaning up the flash code and will add an altera quadspi driver. The altera ts

Re: [U-Boot] [PATCH 2/2] musb: sunxi: Implement dfu_usb_get_reset()

2015-10-25 Thread Albert ARIBAUD
Hello Ian, On Sun, 25 Oct 2015 19:22:00 +, Ian Campbell wrote: > On Sun, 2015-10-25 at 14:22 +0100, Albert ARIBAUD wrote: > > On Sun, 25 Oct 2015 12:40:45 +, Ian Campbell > > > Doesn't the bool return type already cause that to happen? (from the > > > PoV of the caller at least) > > > >

Re: [U-Boot] [PATCH 2/2] musb: sunxi: Implement dfu_usb_get_reset()

2015-10-25 Thread Albert ARIBAUD
Hello Siarhei, On Sun, 25 Oct 2015 16:55:43 +0200, Siarhei Siamashka wrote: > On Sun, 25 Oct 2015 12:00:09 +0100 > Marek Vasut wrote: > > > On Sunday, October 25, 2015 at 05:44:47 AM, Siarhei Siamashka wrote: > > > This is necessary to distinguish between the "dfu-util --detach" and > > > the "

Re: [U-Boot] [PATCH 2/2] musb: sunxi: Implement dfu_usb_get_reset()

2015-10-25 Thread Siarhei Siamashka
On Sun, 25 Oct 2015 19:24:04 + Ian Campbell wrote: > On Sun, 2015-10-25 at 16:46 +0200, Siarhei Siamashka wrote: > > replace "#include " with "typedef int bool;" and try it > > again. > > So don't do that then. Yes, I'm not going to do this. It was just an artificial example, demonstrating

Re: [U-Boot] [PATCH 2/2] musb: sunxi: Implement dfu_usb_get_reset()

2015-10-25 Thread Ian Campbell
On Sun, 2015-10-25 at 16:46 +0200, Siarhei Siamashka wrote: > replace "#include " with "typedef int bool;" and try it > again. So don't do that then. In u-boot the "bool" type comes from stdbool.h and it is therefore completely reasonable to assume that bool in u-boot has the properties which thi

Re: [U-Boot] [PATCH 2/2] musb: sunxi: Implement dfu_usb_get_reset()

2015-10-25 Thread Ian Campbell
On Sun, 2015-10-25 at 14:22 +0100, Albert ARIBAUD wrote: > On Sun, 25 Oct 2015 12:40:45 +, Ian Campbell > > Doesn't the bool return type already cause that to happen? (from the > > PoV of the caller at least) > > When all is said and done, a C bool is a C int, Not if it is a _Bool (via stdboo

Re: [U-Boot] [PATCH v2] spi: zynq_qspi: Minor cleanups

2015-10-25 Thread Tom Rini
On Sun, Oct 25, 2015 at 07:49:57PM +0530, Jagan Teki wrote: > - Use __func__ on debug > - Removed unnecessary comment > - Fix function name in debug as zynq_qspi_xfer instead of spi_xfer > > Cc: Tom Rini > Signed-off-by: Jagan Teki > --- > Changes for v2: > - Fix commit message - Use meani

[U-Boot] [PATCH v3 12/12] spi: Add SPI NOR protection mechanism

2015-10-25 Thread Fabio Estevam
From: Fabio Estevam Many SPI flashes have protection bits (BP2, BP1 and BP0) in the status register that can protect selected regions of the SPI NOR. Take these bits into account when performing erase operations, making sure that the protected areas are skipped. Tested on a mx6qsabresd: => sf

[U-Boot] [PATCH v3 11/12] spi: sf_ops: Add SPI protection mechanism from the kernel

2015-10-25 Thread Fabio Estevam
From: Fabio Estevam Add the SPI NOR protection mechanism from the kernel. This code is based on the work from Brian Norris https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/commit/drivers/mtd/spi-nor/spi-nor.c?id=62593cf40b23b523b9fc9334ca61ba6c595ebb09 Signed-off-by: Fabio Est

[U-Boot] [PATCH v3 10/12] sandbox: Use the generic bitops headers

2015-10-25 Thread Fabio Estevam
From: Fabio Estevam The generic bitops headers are required when calling logarithimic functions, such as ilog2(). Signed-off-by: Fabio Estevam Reviewed-by: Tom Rini --- Changes since v2: - None arch/sandbox/include/asm/bitops.h | 4 1 file changed, 4 insertions(+) diff --git a/arch/san

[U-Boot] [PATCH v3 09/12] sh: bitops: Use the generic bitops headers

2015-10-25 Thread Fabio Estevam
From: Fabio Estevam The generic bitops headers are required when calling logarithimic functions, such as ilog2(). Signed-off-by: Fabio Estevam Reviewed-by: Tom Rini --- Changes since v2: - None arch/microblaze/include/asm/bitops.h | 4 arch/sh/include/asm/bitops.h | 5 + 2 f

[U-Boot] [PATCH v3 07/12] m68k: bitops: Use the generic bitops headers

2015-10-25 Thread Fabio Estevam
From: Fabio Estevam The generic bitops headers are required when calling logarithimic functions, such as ilog2(). Signed-off-by: Fabio Estevam Reviewed-by: Tom Rini --- Changes since v2: - None arch/m68k/include/asm/bitops.h | 4 1 file changed, 4 insertions(+) diff --git a/arch/m68k/i

[U-Boot] [PATCH v3 08/12] blackfin: bitops: Use the generic bitops headers

2015-10-25 Thread Fabio Estevam
From: Fabio Estevam The generic bitops headers are required when calling logarithimic functions, such as ilog2(). Signed-off-by: Fabio Estevam Reviewed-by: Tom Rini --- Changes since v2: - None arch/blackfin/include/asm/bitops.h | 4 1 file changed, 4 insertions(+) diff --git a/arch/bl

[U-Boot] [PATCH v3 06/12] x86: bitops: Use the generic bitops headers

2015-10-25 Thread Fabio Estevam
From: Fabio Estevam Use the generic bitops and also add custom __ffs() implementation as per the kernel. Also align the ffs() implementation with the kernel. Signed-off-by: Fabio Estevam Reviewed-by: Tom Rini --- Changes since v2: - None arch/x86/include/asm/bitops.h | 21 ++

[U-Boot] [PATCH v3 05/12] ARM: bitops: Use the generic bitops headers

2015-10-25 Thread Fabio Estevam
From: Fabio Estevam The generic bitops headers are required when calling logarithimic functions, such as ilog2(). Signed-off-by: Fabio Estevam Reviewed-by: Tom Rini --- Changes since v2: - None arch/arm/include/asm/bitops.h | 5 + 1 file changed, 5 insertions(+) diff --git a/arch/arm/in

[U-Boot] [PATCH v3 03/12] compat: Remove is_power_of_2() definition

2015-10-25 Thread Fabio Estevam
From: Fabio Estevam Use the is_power_of_2() definition from log2.h to align with the kernel implementation. Signed-off-by: Fabio Estevam Reviewed-by: Tom Rini --- Changes since v2: - None arch/arm/mach-mvebu/mbus.c | 2 +- drivers/mtd/mtdcore.c | 2 +- drivers/mtd/ubi/build.c| 2 +-

[U-Boot] [PATCH v3 04/12] powerpc: Remove __ilog2_u64 and ffs4 from bitops

2015-10-25 Thread Fabio Estevam
From: Fabio Estevam Remove __ilog2_u64 and ffs4 from powerpc bitops to align with the kernel implementation. Use the generic __ffs64 instead of a custom powerpc implementation. Cc: York Sun Signed-off-by: Fabio Estevam Reviewed-by: Tom Rini --- Changes since v2: - None arch/powerpc/cpu/mpc

[U-Boot] [PATCH v3 01/12] include: Add log2 header from the kernel

2015-10-25 Thread Fabio Estevam
From: Fabio Estevam Use the log2 header files from the kernel. Imported from kernel 4.2.3. Signed-off-by: Fabio Estevam Reviewed-by: Tom Rini Reviewed-by: Jagan Teki --- Changes since v2: - None include/linux/log2.h | 205 +++ 1 file changed,

[U-Boot] [PATCH v3 2/12] include: Add generic bitops headers

2015-10-25 Thread Fabio Estevam
From: Fabio Estevam Use the generic bitops header files from the kernel. Imported from kernel 4.2.3. Signed-off-by: Fabio Estevam --- Changes since v2: - Use the original __always_inline (Jagan) include/asm-generic/bitops/__ffs.h | 43 ++ include/asm-gener

Re: [U-Boot] [PATCH v2 02/13] include: Add generic bitops headers

2015-10-25 Thread Fabio Estevam
Hi Jagan, On Sun, Oct 25, 2015 at 2:28 PM, Fabio Estevam wrote: > On Sun, Oct 25, 2015 at 4:50 AM, Jagan Teki wrote: >> Hi Febio, >> >> You replaced __always_inline to inline any specific reason? found >> __always_inline at some parts of header in code. But my compilation is >> not-smooth with t

Re: [U-Boot] [PATCH v2 02/13] include: Add generic bitops headers

2015-10-25 Thread Fabio Estevam
On Sun, Oct 25, 2015 at 4:50 AM, Jagan Teki wrote: > Hi Febio, > > You replaced __always_inline to inline any specific reason? found > __always_inline at some parts of header in code. But my compilation is > not-smooth with this. Yes, if I use the original __always_inline I get the following when

Re: [U-Boot] [PATCH 2/2] musb: sunxi: Implement dfu_usb_get_reset()

2015-10-25 Thread Marek Vasut
On Sunday, October 25, 2015 at 03:55:43 PM, Siarhei Siamashka wrote: > On Sun, 25 Oct 2015 12:00:09 +0100 > > Marek Vasut wrote: > > On Sunday, October 25, 2015 at 05:44:47 AM, Siarhei Siamashka wrote: > > > This is necessary to distinguish between the "dfu-util --detach" and > > > the "dfu-util

Re: [U-Boot] [PATCH 2/2] musb: sunxi: Implement dfu_usb_get_reset()

2015-10-25 Thread Marek Vasut
On Sunday, October 25, 2015 at 03:46:15 PM, Siarhei Siamashka wrote: > On Sun, 25 Oct 2015 14:29:59 +0100 > > Marek Vasut wrote: > > On Sunday, October 25, 2015 at 02:22:53 PM, Albert ARIBAUD wrote: > > > Hello Ian, > > > > Hi! > > > > > On Sun, 25 Oct 2015 12:40:45 +, Ian Campbell > > > >

Re: [U-Boot] [PATCH 2/2] musb: sunxi: Implement dfu_usb_get_reset()

2015-10-25 Thread Siarhei Siamashka
On Sun, 25 Oct 2015 12:00:09 +0100 Marek Vasut wrote: > On Sunday, October 25, 2015 at 05:44:47 AM, Siarhei Siamashka wrote: > > This is necessary to distinguish between the "dfu-util --detach" and > > the "dfu-util --reset" requests. > > > > The default weak implementation of dfu_usb_get_reset(

Re: [U-Boot] [PATCH 2/2] musb: sunxi: Implement dfu_usb_get_reset()

2015-10-25 Thread Siarhei Siamashka
On Sun, 25 Oct 2015 14:29:59 +0100 Marek Vasut wrote: > On Sunday, October 25, 2015 at 02:22:53 PM, Albert ARIBAUD wrote: > > Hello Ian, > > Hi! > > > On Sun, 25 Oct 2015 12:40:45 +, Ian Campbell > > > > wrote: > > > On Sun, 2015-10-25 at 12:46 +0100, Albert ARIBAUD wrote: > > > > > > +st

[U-Boot] [PATCH v2] spi: zynq_qspi: Minor cleanups

2015-10-25 Thread Jagan Teki
- Use __func__ on debug - Removed unnecessary comment - Fix function name in debug as zynq_qspi_xfer instead of spi_xfer Cc: Tom Rini Signed-off-by: Jagan Teki --- Changes for v2: - Fix commit message - Use meaningful commit header, add proper details for all fixes. - F

Re: [U-Boot] [PATCH] spi: zynq_qspi: Use __func__ on debug

2015-10-25 Thread Jagan Teki
On 25 October 2015 at 19:34, Tom Rini wrote: > On Sun, Oct 25, 2015 at 07:22:24PM +0530, Jagan Teki wrote: >> On 25 October 2015 at 19:17, Tom Rini wrote: >> > On Sun, Oct 25, 2015 at 09:36:59AM +0530, Jagan Teki wrote: >> > >> >> - Use __func__ on debug >> >> - Removed unnecessary comment >> >>

Re: [U-Boot] [PATCH] spi: zynq_qspi: Use __func__ on debug

2015-10-25 Thread Tom Rini
On Sun, Oct 25, 2015 at 07:22:24PM +0530, Jagan Teki wrote: > On 25 October 2015 at 19:17, Tom Rini wrote: > > On Sun, Oct 25, 2015 at 09:36:59AM +0530, Jagan Teki wrote: > > > >> - Use __func__ on debug > >> - Removed unnecessary comment > >> > >> Signed-off-by: Jagan Teki > >> --- > >> drivers

Re: [U-Boot] [PATCH] spi: zynq_qspi: Use __func__ on debug

2015-10-25 Thread Jagan Teki
On 25 October 2015 at 19:17, Tom Rini wrote: > On Sun, Oct 25, 2015 at 09:36:59AM +0530, Jagan Teki wrote: > >> - Use __func__ on debug >> - Removed unnecessary comment >> >> Signed-off-by: Jagan Teki >> --- >> drivers/spi/zynq_qspi.c | 8 +++- >> 1 file changed, 3 insertions(+), 5 deletions

Re: [U-Boot] [PATCH] spi: zynq_qspi: Use __func__ on debug

2015-10-25 Thread Tom Rini
On Sun, Oct 25, 2015 at 09:36:59AM +0530, Jagan Teki wrote: > - Use __func__ on debug > - Removed unnecessary comment > > Signed-off-by: Jagan Teki > --- > drivers/spi/zynq_qspi.c | 8 +++- > 1 file changed, 3 insertions(+), 5 deletions(-) > > diff --git a/drivers/spi/zynq_qspi.c b/drivers

Re: [U-Boot] [PATCH] u-boot/mpc85xx/u-boot*.lds: remove _GLOBAL_OFFSET_TABLE_ definition

2015-10-25 Thread Tom Rini
On Sun, Oct 25, 2015 at 09:28:50AM +0530, Prabhakar Kushwaha wrote: > From: Zhenhua Luo > > In binutils-2.25, the _GLOBAL_OFFSET_TABLE_ symbols defined by PROVIDE in > u-boot.lds overrides the linker built-in symbols > (https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;a=commitdiff;h=b893

Re: [U-Boot] [PATCH] mtd: do not include cfi_flash.h if no flash

2015-10-25 Thread Marek Vasut
On Sunday, October 25, 2015 at 02:29:06 PM, Thomas Chou wrote: > Hi Marek, > > On 10/25/2015 12:00 PM, Marek Vasut wrote: > > On Sunday, October 25, 2015 at 04:02:58 AM, Thomas Chou wrote: > >> Do not include cfi_flash.h if CONFIG_SYS_NO_FLASH. Because > >> the flash_info is undefined in flash.h i

Re: [U-Boot] [PATCH 2/2] musb: sunxi: Implement dfu_usb_get_reset()

2015-10-25 Thread Marek Vasut
On Sunday, October 25, 2015 at 02:22:53 PM, Albert ARIBAUD wrote: > Hello Ian, Hi! > On Sun, 25 Oct 2015 12:40:45 +, Ian Campbell > > wrote: > > On Sun, 2015-10-25 at 12:46 +0100, Albert ARIBAUD wrote: > > > > > +static u8 last_int_usb; > > > > > + > > > > > +bool dfu_usb_get_reset(void) >

Re: [U-Boot] [PATCH] mtd: do not include cfi_flash.h if no flash

2015-10-25 Thread Thomas Chou
Hi Marek, On 10/25/2015 12:00 PM, Marek Vasut wrote: On Sunday, October 25, 2015 at 04:02:58 AM, Thomas Chou wrote: Do not include cfi_flash.h if CONFIG_SYS_NO_FLASH. Because the flash_info is undefined in flash.h if CONFIG_SYS_NO_FLASH. Signed-off-by: Thomas Chou --- include/mtd/cfi_flash.

Re: [U-Boot] [PATCH 2/2] musb: sunxi: Implement dfu_usb_get_reset()

2015-10-25 Thread Albert ARIBAUD
Hello Ian, On Sun, 25 Oct 2015 12:40:45 +, Ian Campbell wrote: > On Sun, 2015-10-25 at 12:46 +0100, Albert ARIBAUD wrote: > > > > +static u8 last_int_usb; > > > > + > > > > +bool dfu_usb_get_reset(void) > > > > +{ > > > > + return !!(last_int_usb & MUSB_INTR_RESET); > > > > > > The !!

Re: [U-Boot] [PATCH v2 03/12] compat: Remove is_power_of_2() definition

2015-10-25 Thread Fabio Estevam
On Sun, Oct 25, 2015 at 4:53 AM, Jagan Teki wrote: > Hi Febio, > > On 24 October 2015 at 20:56, Fabio Estevam wrote: >> From: Fabio Estevam >> >> Use the is_power_of_2() definition from log2.h to align with the >> kernel implementation. >> >> Signed-off-by: Fabio Estevam >> Reviewed-by: Tom Rin

Re: [U-Boot] [PATCH v2 01/13] include: Add log2 header from the kernel

2015-10-25 Thread Fabio Estevam
On Sun, Oct 25, 2015 at 4:44 AM, Jagan Teki wrote: >> +++ b/include/linux/log2.h >> @@ -0,0 +1,205 @@ >> +/* Integer base 2 logarithm calculation > > Of-course Linux look same, but the multi-line comment should be as > > /** > * Integer base 2 logarithm calculation > * > */ > > Probably we can

Re: [U-Boot] [PATCH 2/2] musb: sunxi: Implement dfu_usb_get_reset()

2015-10-25 Thread Ian Campbell
On Sun, 2015-10-25 at 12:46 +0100, Albert ARIBAUD wrote: > > > +static u8 last_int_usb; > > > + > > > +bool dfu_usb_get_reset(void) > > > +{ > > > + return !!(last_int_usb & MUSB_INTR_RESET); > > > > The !! is not needed. > > Except if you want to be sure that you return 0 or 1 rather than 0 or >

Re: [U-Boot] [PATCH 2/2] musb: sunxi: Implement dfu_usb_get_reset()

2015-10-25 Thread Albert ARIBAUD
Hello Marek, On Sun, 25 Oct 2015 12:00:09 +0100, Marek Vasut wrote: > On Sunday, October 25, 2015 at 05:44:47 AM, Siarhei Siamashka wrote: > > This is necessary to distinguish between the "dfu-util --detach" and > > the "dfu-util --reset" requests. > > > > The default weak implementation of dfu_

Re: [U-Boot] [PATCH v2 1/2] arm: stm32f4: fix a bug when only first sector gets erased

2015-10-25 Thread Albert ARIBAUD
Hello Vadzim, On Fri, 23 Oct 2015 21:14:06 +0300, Vadzim Dambrouski wrote: > flash_lock call is inside a for loop, so after the first iteration flash > is locked and no more sectors can be erased. > Move flash_lock out of the loop. > > Signed-off-by: Vadzim Dambrouski > --- The history wrt v1

Re: [U-Boot] [PATCH 2/2] musb: sunxi: Implement dfu_usb_get_reset()

2015-10-25 Thread Marek Vasut
On Sunday, October 25, 2015 at 05:44:47 AM, Siarhei Siamashka wrote: > This is necessary to distinguish between the "dfu-util --detach" and > the "dfu-util --reset" requests. > > The default weak implementation of dfu_usb_get_reset() unconditionally > reboots the device, but we want to be able to

Re: [U-Boot] [PATCH v4] net: TFTP: variables cleanup and addition

2015-10-25 Thread Albert ARIBAUD
On Mon, 12 Oct 2015 00:02:57 +0200, Albert ARIBAUD (3ADEV) wrote: > TFTP source and destination port variable names are > 'tftpsrcp' and 'tftpdstp' in the code, but 'tftpsrcport' > and 'tftpdstport' in the README file. Fix the README. > > Add environment variable 'tftptimeoutcountmax'. As per the

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

2015-10-25 Thread Jagan Teki
Hi Tom, This PR will also include zynq changes which I sent before the release. And the zynq SPI SPL build issue still there, as Michal need to send related patches. thanks! Jagan. The following changes since commit 677f970bc62a661690b3431543d5a5d5e682ba70: common/board_f.c: modify the macro

[U-Boot] [PATCH 5/5] driver: USB: Fix pointer conversion warnings for xilinx_zynqmp_ep

2015-10-25 Thread Prabhakar Kushwaha
Fix below warnings happening for xilinx_zynqmp_ep_defconfig drivers/usb/dwc3/core.c: In function ‘dwc3_setup_scratch_buffers’: drivers/usb/dwc3/core.c:284:19: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] dma_unmap_single((void *)dwc->scratch_addr, dwc->nr_scrat

[U-Boot] [PATCH 3/5] driver: usb: Fix pointer conversion warnings for hikey

2015-10-25 Thread Prabhakar Kushwaha
Fix below compilation warings happening for hikey_defconfig drivers/usb/eth/smsc95xx.c:698:56: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast] debug("** %s(), len %d, buf %#x\n", __func__, length, (int)msg);

[U-Boot] [PATCH 4/5] driver: net: Fix pointer conversion warnings for xilinx_zynqmp_ep

2015-10-25 Thread Prabhakar Kushwaha
Fix below warnings happening for xilinx_zynqmp_ep_defconfig drivers/net/zynq_gem.c: In function ‘zynq_gem_init’: drivers/net/zynq_gem.c:330:7: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast] ((u32)(priv->rxbuffers) + ^ In file included from drivers/net/

[U-Boot] [PATCH 2/5] driver: dwmmc: Fix pointer conversion warnings for hikey

2015-10-25 Thread Prabhakar Kushwaha
Fix below compilation warings happening for hikey_defconfig drivers/mmc/dw_mmc.c: In function ‘dwmci_set_idma_desc’: drivers/mmc/dw_mmc.c:43:20: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast] desc->next_addr = (unsigned int)desc + sizeof(struct dwmci_idmac);

[U-Boot] [PATCH 0/5] driver: Fix pointer conversion warnings for hikey and xilinx_zynqmp_ep

2015-10-25 Thread Prabhakar Kushwaha
ARMv8 platforms hikey and xilinx_zynqmp_ep showing compilation warning related to pointer conversion. These warnings are coming with aarch64-linux-gnu-gcc (crosstool-NG linaro-1.13.1-4.8-2013.12 - Linaro GCC 2013.11) 4.8.3 20131202 Prabhakar Kushwaha (5): driver: gpio: hikey: Fix pointer convers

[U-Boot] [PATCH 1/5] driver: gpio: hikey: Fix pointer conversion warnings for hikey

2015-10-25 Thread Prabhakar Kushwaha
Fix below compilation warnings- drivers/gpio/hi6220_gpio.c: In function ‘hi6220_gpio_probe’: drivers/gpio/hi6220_gpio.c:82:15: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast] bank->base = (u8 *)plat->base; Signed-off-by: Prabhakar Kushwaha --- arch/arm/include/a

Re: [U-Boot] [PATCH] spi: zynq_qspi: Use __func__ on debug

2015-10-25 Thread Jagan Teki
On 25 October 2015 at 09:36, Jagan Teki wrote: > - Use __func__ on debug > - Removed unnecessary comment > > Signed-off-by: Jagan Teki Applied to u-boot-spi/master -- Jagan | openedev. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de

Re: [U-Boot] [PATCH v4 07/21] sf: Add FSR support to spi_flash_cmd_wait_ready

2015-10-25 Thread Jagan Teki
On 12 October 2015 at 20:31, Jagan Teki wrote: > This patch adds flag status register reading support to > spi_flash_cmd_wait_ready. > > Signed-off-by: Jagan Teki > Cc: Simon Glass > Cc: Marek Vasut > Cc: Michal Simek > Cc: Siva Durga Prasad Paladugu > Cc: Stefan Roese > Cc: Tom Warren > Cc

Re: [U-Boot] [PATCH v4 06/21] sf: Update status reg check in spi_flash_cmd_wait_ready

2015-10-25 Thread Jagan Teki
On 12 October 2015 at 20:30, Jagan Teki wrote: > Current flash wait_ready logic is not modular to add new > register status check, hence updated the status check for > adding few more register checks in future. > > Below are the sf speed runs with 'sf update' on whole flash, 16MiB. > > => sf updat

Re: [U-Boot] [PATCH v4 05/21] sf: Make flash->flags use for generic usage

2015-10-25 Thread Jagan Teki
On 12 October 2015 at 20:30, Jagan Teki wrote: > Use the flash->flags for generic usage, not only for dm-spi-flash, > this will be used for future flag additions. > > Signed-off-by: Jagan Teki > [Correct the spi flash flags detect logic] > Signed-off-by: Bin Meng > Tested-by: Bin Meng > --- Ap

Re: [U-Boot] [PATCH v4 04/21] sf: Optimize BAR write code

2015-10-25 Thread Jagan Teki
On 12 October 2015 at 20:30, Jagan Teki wrote: > Optimized spi-flash bar writing code and also removed > unnecessary bank_sel in read_ops. > > Signed-off-by: Jagan Teki > Cc: Simon Glass > Cc: Michal Simek > Cc: Siva Durga Prasad Paladugu > --- Applied to u-boot-spi/master -- Jagan | opened

Re: [U-Boot] [PATCH v4 03/21] sf: Add spi_flash_read_bar

2015-10-25 Thread Jagan Teki
On 12 October 2015 at 20:30, Jagan Teki wrote: > Add spi_flash_read_bar function for reading bar and > discovering bar commands at probe time. > > Signed-off-by: Jagan Teki > Cc: Michal Simek > Cc: Siva Durga Prasad Paladugu > --- Applied to u-boot-spi/master -- Jagan | openedev. ___

Re: [U-Boot] [PATCH v4 02/21] sf: Return bank_sel, if flash->bank_curr == bank_sel

2015-10-25 Thread Jagan Teki
On 12 October 2015 at 20:30, Jagan Teki wrote: > If computed bank_sel is same as flash->bank_curr which is > computed at probe time, then return the bank_sel instead of zero. > > Signed-off-by: Jagan Teki > Cc: Michal Simek > Cc: Siva Durga Prasad Paladugu Applied to u-boot-spi/master -- Jag

Re: [U-Boot] [PATCH v4 01/21] spi: zynq_spi: Remove unneeded headers

2015-10-25 Thread Jagan Teki
On 12 October 2015 at 20:30, Jagan Teki wrote: > - Removed unneeded inclusion of header files > - Add "Xilinx" on license text > > Signed-off-by: Jagan Teki > Cc: Michal Simek > Cc: Siva Durga Prasad Paladugu Applied to u-boot-spi/master -- Jagan | openedev. _