[U-Boot] [PATCH] spl: Add bootcount/bootlimit support

2013-09-29 Thread Stefan Roese
This patch adds bootcount/bootlimit support to SPL. Right now with SPL-NOR support only. But it should be easy to ad this feature to other SPL implementations as well. When the bootcounter exceeds the "bootlimit" environment variable SPL will boot into the full-blown U-Boot to handle the bootcmd s

Re: [U-Boot] [PATCH 1/2] SPL: Add CONFIG_SPL_BOOTCOUNT_SUPPORT

2013-09-29 Thread Stefan Roese
Hi Tom, On 27.09.2013 21:26, Tom Rini wrote: > Add a new symbol, CONFIG_SPL_BOOTCOUNT_SUPPORT, to make use of the > existing BOOTCOUNT_SUPPORT within SPL. It is strongly discouraged to > use bootcount in both SPL and full U-Boot, as they use the same counter. I just noticed that I missed sending

Re: [U-Boot] i.MX6 based board does not work if watchdog support is enabled

2013-09-29 Thread Stefano Babic
Hi Antolji, On 29/09/2013 20:08, Otavio Salvador wrote: >>> >>> I am booting the board from USB loader. May it be an issue? >> >> No, it shouldn't be an issue. Does the attached patch help? > > It does fix the issue! :-) > Can you resend the patch in the usual way to ML for including into main

[U-Boot] [PATCH] pmic: max77686: fix the wrong offset

2013-09-29 Thread Jaehoon Chung
0x1D is reserved. So BUCK3DVS1 is started from 0x1e. Signed-off-by: Jaehoon Chung --- include/power/max77686_pmic.h |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/power/max77686_pmic.h b/include/power/max77686_pmic.h index c98db1b..16e9016 100644 --- a/include/pow

[U-Boot] [PATCH 1/3][v3] mtd: move & update nand_ecclayout structure

2013-09-29 Thread Prabhakar Kushwaha
nand_ecclayout is present in mtd.h at Linux. Move this structure to mtd.h to comply with Linux. Also, increase the ecc placement locations to 640 to suport device having writesize/oobsize of 8KB/640B. This means that the maximum oobsize has gone up to 640 bytes and consequently the maximum ecc pla

[U-Boot] [PATCH 3/3][v3] board/c29xpcie: Add support of 8K page size NAND flash

2013-09-29 Thread Prabhakar Kushwaha
Defines constants required to support 8K page size NAND flash. Signed-off-by: Prabhakar Kushwaha --- Changes for v2:Sending as it is Changes for v3:Sending as it is include/configs/C29XPCIE.h | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/include/configs/C29X

[U-Boot] [PATCH 2/3][v3] mtd/ifc: Add support of 8K page size NAND flash

2013-09-29 Thread Prabhakar Kushwaha
Current IFC driver supports till 4K page size NAND flash. Add support of 8K NAND flash - Program Spare region size in csor_ext - Add nand_ecclayout for 4 bit & 8 bit ecc - Defines constants - Add support of 8K NAND boot. Signed-off-by: Prabhakar Kushwaha CC: Liu Po --- Changes for v2: s

Re: [U-Boot] [PATCH] buildman: Use env to pick the python from $PATH

2013-09-29 Thread Simon Glass
On Sat, Sep 28, 2013 at 11:38 AM, Jagannadha Sutradharudu Teki wrote: > python used in buildman doesn't need to be placed in > /usr/bin/python, So use env to ensure that the interpreter > will pick the python from environment. > > Usefull with several versions of python's installed on system. > >

Re: [U-Boot] [PATCH v3] PCIe:change the method to get the address of a requested capability in configuration space.

2013-09-29 Thread Zhao Qiang-B45475
Hi Tom, I'd like you to pay attention on this patch http://patchwork.ozlabs.org/patch/275173/ Regards Zhao Qiang > -Original Message- > From: Zhao Qiang-B45475 > Sent: Monday, September 16, 2013 5:28 PM > To: u-boot@lists.denx.de > Cc: Zhao Qiang-B45475 > Subject: [PATCH v3] PCIe:change

Re: [U-Boot] [PATCH] gpio: mxc_gpio: Fix gpio_get_value() when the GPIO is an output

2013-09-29 Thread Fabio Estevam
On Sun, Sep 29, 2013 at 7:24 PM, Otavio Salvador wrote: > I sent the patch to fix this adding the flag to the GPIO pins. > > I tested it and it works fine indeed. The patch is awaiting for > approval as it is a little big. The commitlog is below for reference: > > mx6: Add IOMUX_CONFIG_SION f

[U-Boot] [PATCH v2 3/4] arm: rmobile: kzm9g: Change clock definition of SCIF from CONFIG_SYS_CLK_FREQ to CONFIG_SH_SCIF_CLK_FREQ

2013-09-29 Thread Nobuhiro Iwamatsu
Signed-off-by: Nobuhiro Iwamatsu CC: Nobuhiro Iwamatsu CC: Albert Aribaud --- v2: no changes. re-send as further series. include/configs/kzm9g.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/configs/kzm9g.h b/include/configs/kzm9g.h index 222725c..6323050 100644 --- a/inclu

[U-Boot] [PATCH v2 2/4] sh: boards: Change clock definition of SCIF and TMU

2013-09-29 Thread Nobuhiro Iwamatsu
This changes clock definition of SCIF from CONFIG_SYS_CLK_FREQ to CONFIG_SH_SCIF_CLK_FREQ, and clock definition of TMU from CONFIG_SYS_CLK_FREQ to CONFIG_SH_TMU_CLK_FREQ for boards. Signed-off-by: Nobuhiro Iwamatsu Signed-off-by: Nobuhiro Iwamatsu CC: Albert Aribaud --- v2: no changes. re

[U-Boot] [PATCH v2 4/4] arm: rmobile: armadillo-800eva: Change clock definition of SCIF and TMU

2013-09-29 Thread Nobuhiro Iwamatsu
This changes clock definition of SCIF from CONFIG_SYS_CLK_FREQ to CONFIG_SH_SCIF_CLK_FREQ, and clock definition of TMU from CONFIG_SYS_CLK_FREQ to CONFIG_SH_TMU_CLK_FREQ, Signed-off-by: Nobuhiro Iwamatsu CC: Nobuhiro Iwamatsu CC: Albert Aribaud --- v2: no changes. re-send as further serie

[U-Boot] [PATCH v2 1/4] serial: sh: Change The definition of clock of SCIF

2013-09-29 Thread Nobuhiro Iwamatsu
The former SH/SCIF driver had calculated baudrate based on CONFIG_SYS_CLK_FREQ. The newest SH/SCIF needs calculation of the clock for SCIF. This patch defines clock CONFIG_SH_SCIF_CLK_FREQ for SCIF and changes it to CONFIG_SH_SCIF_CLK_FREQ from CONFIG_SYS_CLK_FREQ. Signed-off-by: Nobuhiro Iwamatsu

Re: [U-Boot] [PATCH v2] usb: Prevent using reserved registers on DM36x usb

2013-09-29 Thread Marek Vasut
Dear Andrew Murray, > The musb driver defines and uses MUSB_CSR0_H_DIS_PING, however this > bit is reserved on the DM36x. Thus this patch ensures that the > reserved bit is not accesssed. > > It has been observed that some USB devices will fail to enumerate > with errors such as 'error in inquiry

Re: [U-Boot] [PATCH] gpio: mxc_gpio: Fix gpio_get_value() when the GPIO is an output

2013-09-29 Thread Otavio Salvador
On Sun, Sep 29, 2013 at 4:49 PM, Otavio Salvador wrote: > On Sun, Sep 29, 2013 at 4:45 PM, Benoît Thébaudeau > wrote: >> On Sunday, September 29, 2013 9:42:44 PM, Otavio Salvador wrote: >>> On Sun, Sep 29, 2013 at 4:25 PM, Benoît Thébaudeau >>> wrote: >>> ... >>> > Hence, gpio_get_value() should

Re: [U-Boot] [PATCH] gpio: mxc_gpio: Fix gpio_get_value() when the GPIO is an output

2013-09-29 Thread Otavio Salvador
On Sun, Sep 29, 2013 at 4:45 PM, Benoît Thébaudeau wrote: > On Sunday, September 29, 2013 9:42:44 PM, Otavio Salvador wrote: >> On Sun, Sep 29, 2013 at 4:25 PM, Benoît Thébaudeau >> wrote: >> ... >> > Hence, gpio_get_value() should be left unchanged (using PSR in all cases), >> > and >> > SION sh

Re: [U-Boot] [PATCH] gpio: mxc_gpio: Fix gpio_get_value() when the GPIO is an output

2013-09-29 Thread Benoît Thébaudeau
Hi Otavio, On Sunday, September 29, 2013 9:42:44 PM, Otavio Salvador wrote: > On Sun, Sep 29, 2013 at 4:25 PM, Benoît Thébaudeau > wrote: > ... > > Hence, gpio_get_value() should be left unchanged (using PSR in all cases), > > and > > SION should be set for all GPIOs in the i.MX6 pin definition h

Re: [U-Boot] [PATCH] gpio: mxc_gpio: Fix gpio_get_value() when the GPIO is an output

2013-09-29 Thread Otavio Salvador
On Sun, Sep 29, 2013 at 4:25 PM, Benoît Thébaudeau wrote: ... > Hence, gpio_get_value() should be left unchanged (using PSR in all cases), and > SION should be set for all GPIOs in the i.MX6 pin definition header files. I just does not follow why this preferred against Fabio's proposed patch to r

Re: [U-Boot] [PATCH] gpio: mxc_gpio: Fix gpio_get_value() when the GPIO is an output

2013-09-29 Thread Benoît Thébaudeau
Hi Fabio, On Sunday, September 29, 2013 8:58:09 PM, Fabio Estevam wrote: > Hi Benoît, > > On Sun, Sep 29, 2013 at 3:50 PM, Benoît Thébaudeau > wrote: > > > Can you test again without any GPIO patch, but with SION set for this pin > > in the > > IOMUXC? According to the reference manual, SION no

Re: [U-Boot] [PATCH] gpio: mxc_gpio: Fix gpio_get_value() when the GPIO is an output

2013-09-29 Thread Fabio Estevam
Hi Benoît, On Sun, Sep 29, 2013 at 3:50 PM, Benoît Thébaudeau wrote: > Can you test again without any GPIO patch, but with SION set for this pin in > the > IOMUXC? According to the reference manual, SION not being set in the IOMUXC is > the only reason that would prevent PSR from reading the pi

Re: [U-Boot] [PATCH] gpio: mxc_gpio: Fix gpio_get_value() when the GPIO is an output

2013-09-29 Thread Benoît Thébaudeau
Hi Fabio, On Sunday, September 29, 2013 8:48:46 PM, Fabio Estevam wrote: > On Sun, Sep 29, 2013 at 3:26 PM, Eric Bénard wrote: > > Hi Fabio, > > > > Le Sun, 29 Sep 2013 15:22:36 -0300, > > Fabio Estevam a écrit : > > > >> On Sun, Sep 29, 2013 at 3:19 PM, Eric Bénard wrote: > >> > >> > on which

Re: [U-Boot] [PATCH] gpio: mxc_gpio: Fix gpio_get_value() when the GPIO is an output

2013-09-29 Thread Fabio Estevam
On Sun, Sep 29, 2013 at 3:26 PM, Eric Bénard wrote: > Hi Fabio, > > Le Sun, 29 Sep 2013 15:22:36 -0300, > Fabio Estevam a écrit : > >> On Sun, Sep 29, 2013 at 3:19 PM, Eric Bénard wrote: >> >> > on which CPU is that ? >> >> Otavio tested it on mx6. >> >> > It's strange reading PSR works in the k

Re: [U-Boot] [PATCH] gpio: mxc_gpio: Fix gpio_get_value() when the GPIO is an output

2013-09-29 Thread Eric Bénard
Hi Fabio, Le Sun, 29 Sep 2013 15:22:36 -0300, Fabio Estevam a écrit : > On Sun, Sep 29, 2013 at 3:19 PM, Eric Bénard wrote: > > > on which CPU is that ? > > Otavio tested it on mx6. > > > It's strange reading PSR works in the kernel and not in u-boot. > > The patch I sent aligns with the ke

Re: [U-Boot] [PATCH] gpio: mxc_gpio: Fix gpio_get_value() when the GPIO is an output

2013-09-29 Thread Fabio Estevam
On Sun, Sep 29, 2013 at 3:19 PM, Eric Bénard wrote: > on which CPU is that ? Otavio tested it on mx6. > It's strange reading PSR works in the kernel and not in u-boot. The patch I sent aligns with the kernel behaviour as well: http://git.freescale.com/git/cgit.cgi/imx/linux-2.6-imx.git/commit

Re: [U-Boot] [PATCH] gpio: mxc_gpio: Fix gpio_get_value() when the GPIO is an output

2013-09-29 Thread Eric Bénard
Le Sun, 29 Sep 2013 14:48:32 -0300, Otavio Salvador a écrit : > On Sun, Sep 29, 2013 at 2:09 PM, Eric Bénard wrote: > > Hi Benoît, > > > > Le Sun, 29 Sep 2013 15:21:52 +0200 (CEST), > > Benoît Thébaudeau a écrit : > >> Why is this required? Is it because there is a different behavior of the >

Re: [U-Boot] i.MX6 based board does not work if watchdog support is enabled

2013-09-29 Thread Otavio Salvador
On Sun, Sep 29, 2013 at 1:57 PM, Anatolij Gustschin wrote: > On Sat, 28 Sep 2013 21:19:19 -0300 > Otavio Salvador wrote: > >> I am trying to add support for watchdog in one board and it is not >> working as expected. >> >> I did add the: >> >> #define CONFIG_HW_WATCHDOG >> #define CONFIG_IMX_WATC

Re: [U-Boot] i.MX6 based board does not work if watchdog support is enabled

2013-09-29 Thread Otavio Salvador
On Sun, Sep 29, 2013 at 5:23 AM, Michael Trimarchi wrote: > On Sun, Sep 29, 2013 at 2:19 AM, Otavio Salvador > wrote: >> I am trying to add support for watchdog in one board and it is not >> working as expected. >> >> I did add the: >> >> #define CONFIG_HW_WATCHDOG >> #define CONFIG_IMX_WATCHDOG

Re: [U-Boot] [PATCH] gpio: mxc_gpio: Fix gpio_get_value() when the GPIO is an output

2013-09-29 Thread Otavio Salvador
On Sun, Sep 29, 2013 at 2:09 PM, Eric Bénard wrote: > Hi Benoît, > > Le Sun, 29 Sep 2013 15:21:52 +0200 (CEST), > Benoît Thébaudeau a écrit : >> Why is this required? Is it because there is a different behavior of the PSR >> register on one of the i.MXs? >> >> See my commit message here: >> http:

Re: [U-Boot] [PATCH] gpio: mxc_gpio: Fix gpio_get_value() when the GPIO is an output

2013-09-29 Thread Eric Bénard
Hi Benoît, Le Sun, 29 Sep 2013 15:21:52 +0200 (CEST), Benoît Thébaudeau a écrit : > Why is this required? Is it because there is a different behavior of the PSR > register on one of the i.MXs? > > See my commit message here: > http://git.denx.de/cgi-bin/gitweb.cgi?p=u-boot.git;a=commitdiff;h=5da

Re: [U-Boot] [PATCH RFC] usb: Prevent using reserved registers on DM36x usb

2013-09-29 Thread Andrew Murray
On 27 September 2013 19:10, Marek Vasut wrote: >> It would be much nicer if you avoided using this bit in musb_hcd.c instead >> of hacking it like this. > > Bump? Will we get a V2 here ? I've posted a v2 on the list, apologies for the slow response. Andrew Murray

[U-Boot] [PATCH v2] usb: Prevent using reserved registers on DM36x usb

2013-09-29 Thread Andrew Murray
The musb driver defines and uses MUSB_CSR0_H_DIS_PING, however this bit is reserved on the DM36x. Thus this patch ensures that the reserved bit is not accesssed. It has been observed that some USB devices will fail to enumerate with errors such as 'error in inquiry' without this patch. See http:/

Re: [U-Boot] i.MX6 based board does not work if watchdog support is enabled

2013-09-29 Thread Anatolij Gustschin
Hi, On Sat, 28 Sep 2013 21:19:19 -0300 Otavio Salvador wrote: > I am trying to add support for watchdog in one board and it is not > working as expected. > > I did add the: > > #define CONFIG_HW_WATCHDOG > #define CONFIG_IMX_WATCHDOG > > into the board config file. > > Into the board .c file

Re: [U-Boot] [PATCH 6/7] status_led: Add support for inverted LEDs

2013-09-29 Thread Benoît Thébaudeau
Hi Otavio, On Saturday, September 28, 2013 9:08:48 PM, Otavio Salvador wrote: > On Sat, Sep 28, 2013 at 1:49 PM, Fabio Estevam wrote: > > On Sat, Sep 28, 2013 at 11:17 AM, Benoît Thébaudeau > > wrote: > >> Dear Otavio Salvador, > >> > >> On Saturday, September 28, 2013 5:24:17 AM, Otavio Salvado

Re: [U-Boot] [PATCH] gpio: mxc_gpio: Fix gpio_get_value() when the GPIO is an output

2013-09-29 Thread Benoît Thébaudeau
Hi Fabio, On Saturday, September 28, 2013 7:22:44 PM, Fabio Estevam wrote: > From: Fabio Estevam > > When the GPIO is configured as an output, we should return the value from the > DR > register. > > This implements the same fix as in the following kernel commit from FSL BSP: > http://git.frees

Re: [U-Boot] i.MX6 based board does not work if watchdog support is enabled

2013-09-29 Thread Michael Trimarchi
Hi On Sun, Sep 29, 2013 at 2:19 AM, Otavio Salvador wrote: > Hello, > > I am trying to add support for watchdog in one board and it is not > working as expected. > > I did add the: > > #define CONFIG_HW_WATCHDOG > #define CONFIG_IMX_WATCHDOG > > into the board config file. > > Into the board .c f