Re: [U-Boot] [PATCH] patman: Don't barf if the word 'commit' starts a line

2013-03-02 Thread Simon Glass
Hi Doug, On Fri, Mar 1, 2013 at 1:11 PM, Doug Anderson wrote: > Patman's regular expression for detecting the start of a > commit in a git log was a little simplistic and could be > confused if the git log itself had the word "commit" as > the start of a line (as this commit does). Make patman >

Re: [U-Boot] [PATCH 3/4 V2] S5P: Serial: Add fdt support to driver

2013-03-02 Thread Simon Glass
Hi Rajeshwari, On Fri, Mar 1, 2013 at 2:16 AM, Rajeshwari Shinde wrote: > This patch adds FDT support to the serial s5p driver. > At present disabling the serial console (from the device tree) crashes > U-Boot. Add checks for this case, so that execution can continue without > a serial console. >

Re: [U-Boot] [PATCH 3/4 V2] S5P: Serial: Add fdt support to driver

2013-03-02 Thread Simon Glass
Hi Rajeshwari, On Fri, Mar 1, 2013 at 2:16 AM, Rajeshwari Shinde wrote: > This patch adds FDT support to the serial s5p driver. > At present disabling the serial console (from the device tree) crashes > U-Boot. Add checks for this case, so that execution can continue without > a serial console. >

Re: [U-Boot] [PATCH 2/4 V2] EXYNOS5: FDT: Add serial device node values

2013-03-02 Thread Simon Glass
Hi Rajeshwari, On Fri, Mar 1, 2013 at 2:16 AM, Rajeshwari Shinde wrote: > This patch adds the device node required for serial driver > > Signed-off-by: Abhilash Kesavan > Signed-off-by: Rajeshwari Shinde > --- > Changes in V2: > - Changed the compatible string to "samsung,exynos4210-uar

Re: [U-Boot] [PATCH 1/4 V2] EXYNOS5: FDT: Add compatible strings for Serial

2013-03-02 Thread Simon Glass
On Fri, Mar 1, 2013 at 2:16 AM, Rajeshwari Shinde wrote: > Add required compatible information for s5p serial driver > > Signed-off-by: Abhilash Kesavan > Signed-off-by: Rajeshwari Shinde Acked-by: Simon Glass > --- > Changes in V2: > - Changed the compatible string to "samsung,exynos

Re: [U-Boot] [PATCH v5 12/23] Add spl load feature

2013-03-02 Thread Simon Glass
Hi Scott, On Tue, Feb 12, 2013 at 2:34 PM, Scott Wood wrote: > On 02/12/2013 04:23:21 PM, Simon Glass wrote: >> >> +Albert >> >> Hi Scott, >> >> On Tue, Feb 12, 2013 at 12:29 PM, Scott Wood >> wrote: >> > On 02/08/2013 09:12:08 AM, Simon Glass wrote: >> >> >> >> This adds secondary program loade

Re: [U-Boot] [PATCH 1/3] mx6qsabresd: Fix card detection for invalid card id case

2013-03-02 Thread Otavio Salvador
On Sat, Mar 2, 2013 at 8:24 PM, Fabio Estevam wrote: > On Sat, Mar 2, 2013 at 8:22 PM, Otavio Salvador > wrote: > >> Well, we don't have ret here. > > Yes, exactly. My suggestion is to add a return value, just like > mx53loco and sabresd. Great; did it for v2. -- Otavio Salvador

Re: [U-Boot] [PATCH 1/3] mx6qsabresd: Fix card detection for invalid card id case

2013-03-02 Thread Fabio Estevam
On Sat, Mar 2, 2013 at 8:22 PM, Otavio Salvador wrote: > Well, we don't have ret here. Yes, exactly. My suggestion is to add a return value, just like mx53loco and sabresd. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listi

Re: [U-Boot] [PATCH 1/3] mx6qsabresd: Fix card detection for invalid card id case

2013-03-02 Thread Otavio Salvador
On Sat, Mar 2, 2013 at 8:19 PM, Otavio Salvador wrote: > On Sat, Mar 2, 2013 at 8:15 PM, Fabio Estevam wrote: >> On Sat, Mar 2, 2013 at 8:08 PM, Otavio Salvador >> wrote: >> > } > + > + return 0; ,but why do we need this? >>> >>> Fail by default? >> >> No, ju

Re: [U-Boot] [PATCH 1/3] mx6qsabresd: Fix card detection for invalid card id case

2013-03-02 Thread Otavio Salvador
On Sat, Mar 2, 2013 at 8:15 PM, Fabio Estevam wrote: > On Sat, Mar 2, 2013 at 8:08 PM, Otavio Salvador > wrote: > } + + return 0; >>> >>> ,but why do we need this? >> >> Fail by default? > > No, just do like mx53loco and mx6qsabrelite, where we always 'return ret' O

Re: [U-Boot] [PATCH 1/3] mx6qsabresd: Fix card detection for invalid card id case

2013-03-02 Thread Fabio Estevam
On Sat, Mar 2, 2013 at 8:08 PM, Otavio Salvador wrote: >>> } >>> + >>> + return 0; >> >> ,but why do we need this? > > Fail by default? No, just do like mx53loco and mx6qsabrelite, where we always 'return ret' ___ U-Boot mailing list U-Bo

Re: [U-Boot] [PATCH 1/3] mx6qsabresd: Fix card detection for invalid card id case

2013-03-02 Thread Otavio Salvador
On Sat, Mar 2, 2013 at 7:11 PM, Fabio Estevam wrote: > On Sat, Mar 2, 2013 at 6:49 PM, Otavio Salvador > wrote: > > Please provide a commit log. > >> Signed-off-by: Otavio Salvador >> --- >> board/freescale/mx6qsabresd/mx6qsabresd.c | 4 +++- >> 1 file changed, 3 insertions(+), 1 deletion(-) >

Re: [U-Boot] [PATCH 2/3] mx6qsabresd: Document the mapping of USDHC[2-4]

2013-03-02 Thread Otavio Salvador
On Sat, Mar 2, 2013 at 7:14 PM, Fabio Estevam wrote: > On Sat, Mar 2, 2013 at 6:49 PM, Otavio Salvador > wrote: >> Signed-off-by: Otavio Salvador > > No commit log. It is really hard for someone to understand why you > need this patch if you do not explain it. Right; I can extend it. >> --- >

Re: [U-Boot] [PATCH 0/2] ARM: mmu: Set domain permissions to client access - build warnings!

2013-03-02 Thread Albert ARIBAUD
Hi Sricharan, On Sat, 2 Mar 2013 10:08:20 +, "R, Sricharan" wrote: > Hi Tom/Albert, > > From: Rini, Tom > Sent: Friday, March 01, 2013 7:51 PM > To: Albert ARIBAUD > Cc: R, Sricharan; U-Boot; Stehle, Vincent > Subject: Re: [U-Boot] [PATCH 0/2] ARM: mm

Re: [U-Boot] [PATCH 2/3] mx6qsabresd: Document the mapping of USDHC[2-4]

2013-03-02 Thread Fabio Estevam
On Sat, Mar 2, 2013 at 6:49 PM, Otavio Salvador wrote: > Signed-off-by: Otavio Salvador No commit log. It is really hard for someone to understand why you need this patch if you do not explain it. > --- > board/freescale/mx6qsabresd/mx6qsabresd.c | 4 > 1 file changed, 4 insertions(+) > >

Re: [U-Boot] [PATCH 1/3] mx6qsabresd: Fix card detection for invalid card id case

2013-03-02 Thread Fabio Estevam
On Sat, Mar 2, 2013 at 6:49 PM, Otavio Salvador wrote: Please provide a commit log. > Signed-off-by: Otavio Salvador > --- > board/freescale/mx6qsabresd/mx6qsabresd.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/board/freescale/mx6qsabresd/mx6qsabresd.c > b/boar

[U-Boot] [PATCH 3/3] mx6qsabre{sd,auto}: Add boot mode select

2013-03-02 Thread Otavio Salvador
Adds support for 'bmode' command which let user to choose where to boot from; this allows U-Boot to load system from another storage without messing with jumpers. Signed-off-by: Otavio Salvador --- board/freescale/mx6qsabreauto/mx6qsabreauto.c | 17 + board/freescale/mx6qsabresd/

[U-Boot] [PATCH 2/3] mx6qsabresd: Document the mapping of USDHC[2-4]

2013-03-02 Thread Otavio Salvador
Signed-off-by: Otavio Salvador --- board/freescale/mx6qsabresd/mx6qsabresd.c | 4 1 file changed, 4 insertions(+) diff --git a/board/freescale/mx6qsabresd/mx6qsabresd.c b/board/freescale/mx6qsabresd/mx6qsabresd.c index 3c00023..c334403 100644 --- a/board/freescale/mx6qsabresd/mx6qsabresd.c

[U-Boot] [PATCH 1/3] mx6qsabresd: Fix card detection for invalid card id case

2013-03-02 Thread Otavio Salvador
Signed-off-by: Otavio Salvador --- board/freescale/mx6qsabresd/mx6qsabresd.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/board/freescale/mx6qsabresd/mx6qsabresd.c b/board/freescale/mx6qsabresd/mx6qsabresd.c index 65c4a1a..3c00023 100644 --- a/board/freescale/mx6qsabres

[U-Boot] [PATCH 0/3] i.MX6 SabreSD and SabreAUTO bmode support

2013-03-02 Thread Otavio Salvador
This adds the bmode support for i.MX6 SabreSD and SabreAUTO boards. This allows user to choose the boot mode at runtime making it easy to boot from USB or other media. Otavio Salvador (3): mx6qsabresd: Fix card detection for invalid card id case mx6qsabresd: Document the mapping of USDHC[2-4

Re: [U-Boot] [PATCH v5 0/3] mx23/mxs pending patches

2013-03-02 Thread Otavio Salvador
On Sat, Mar 2, 2013 at 6:12 PM, Fabio Estevam wrote: > On Sat, Mar 2, 2013 at 5:07 PM, Otavio Salvador > wrote: > >>> What issues does this patch series fix? I see that it adds USB >>> support, but I don't see any bug fix. >> >> This is the missing patches, which has not been included in Marek s

Re: [U-Boot] [PATCH v5 0/3] mx23/mxs pending patches

2013-03-02 Thread Fabio Estevam
On Sat, Mar 2, 2013 at 5:07 PM, Otavio Salvador wrote: >> What issues does this patch series fix? I see that it adds USB >> support, but I don't see any bug fix. > > This is the missing patches, which has not been included in Marek series. He > has included most patches in his pull request but th

Re: [U-Boot] [PATCH v5 0/3] mx23/mxs pending patches

2013-03-02 Thread Otavio Salvador
On Sat, Mar 2, 2013 at 5:41 PM, Marek Vasut wrote: > Dear Otavio Salvador, > >> This patchset includes the pending patches we have in our tree. It >> fixes issues for mx23evk and mx23_olinuxino boards. This depends on >> Marek's patchset which has included most of the previous patches in >> this p

Re: [U-Boot] [PATCH v5 0/3] mx23/mxs pending patches

2013-03-02 Thread Marek Vasut
Dear Otavio Salvador, > This patchset includes the pending patches we have in our tree. It > fixes issues for mx23evk and mx23_olinuxino boards. This depends on > Marek's patchset which has included most of the previous patches in > this patchset. > > Changes in v5: > - Adapt code to newer multip

Re: [U-Boot] [PATCH V2 4/4] ARM: tegra: enable a common set of disk-related commands everywhere

2013-03-02 Thread Simon Glass
On Thu, Feb 28, 2013 at 5:03 PM, Stephen Warren wrote: > From: Stephen Warren > > Enable a common set of partition types, filesystems, and related > commands in tegra-common.h, so that they are available on all Tegra > boards. This allows boot.scr (loaded and executed by the default > built-in en

Re: [U-Boot] [PATCH V2 3/4] disk: define HAVE_BLOCK_DEVICE if CONFIG_CMD_PART

2013-03-02 Thread Simon Glass
On Thu, Feb 28, 2013 at 5:03 PM, Stephen Warren wrote: > From: Stephen Warren > > Various code that is conditional upon HAVE_BLOCK_DEVICE is required by > code conditional upon CONFIG_CMD_PART. So, enable HAVE_BLOCK_DEVICE if > CONFIG_CMD_PART is enabled. > > Signed-off-by: Stephen Warren Acked

Re: [U-Boot] [PATCH V2 1/4] ARM: tegra: make bounce buffer option common

2013-03-02 Thread Simon Glass
On Thu, Feb 28, 2013 at 5:03 PM, Stephen Warren wrote: > From: Stephen Warren > > All Tegra devices will need CONFIG_BOUNCE_BUFFER. Move it to > tegra-common.h to ensure it's always set. > > Signed-off-by: Stephen Warren Acked-by: Simon Glass > --- > v2: Add missing word to commit subject. >

Re: [U-Boot] [PATCH V2 2/4] disk: define HAVE_BLOCK_DEVICE in a common place

2013-03-02 Thread Simon Glass
On Thu, Feb 28, 2013 at 5:03 PM, Stephen Warren wrote: > From: Stephen Warren > > This set of ifdefs is used in a number of places. Move its definition > somewhere common so it doesn't have to be repeated. > > Signed-off-by: Stephen Warren Acked-by: Simon Glass

Re: [U-Boot] [PATCH v5 0/3] mx23/mxs pending patches

2013-03-02 Thread Fabio Estevam
On Sat, Mar 2, 2013 at 12:17 PM, Otavio Salvador wrote: > > This patchset includes the pending patches we have in our tree. It > fixes issues for mx23evk and mx23_olinuxino boards. This depends on What issues does this patch series fix? I see that it adds USB support, but I don't see any bug fix.

[U-Boot] [PATCH v3 3/9] am33xx: refactor am33xx clocks and add ti814x support

2013-03-02 Thread Matt Porter
Split clock.c for am335x and ti814x and add ti814x specific clock support. Signed-off-by: Matt Porter --- v3: * Fix EMIF/L3F clock enable ordering issue (Exposed on UART boot with hang at check for L3F enabled) v2: * remove unused dmtimer support * add register

[U-Boot] [PATCH v5 1/3] mx23evk: Enable USB support

2013-03-02 Thread Otavio Salvador
This enabled USB support for the mx23evk board. Signed-off-by: Otavio Salvador --- Changes in v5: - Adapt code to newer multiport support Changes in v2: None include/configs/mx23evk.h | 11 +++ 1 file changed, 11 insertions(+) diff --git a/include/configs/mx23evk.h b/include/configs/m

[U-Boot] [PATCH v5 0/3] mx23/mxs pending patches

2013-03-02 Thread Otavio Salvador
This patchset includes the pending patches we have in our tree. It fixes issues for mx23evk and mx23_olinuxino boards. This depends on Marek's patchset which has included most of the previous patches in this patchset. Changes in v5: - Adapt code to newer multiport support - Adapt code to newer mu

[U-Boot] [PATCH v5 2/3] mx23_olinuxino: Enable USB support

2013-03-02 Thread Otavio Salvador
This enabled USB support for the mx23_olinuxino board. Signed-off-by: Otavio Salvador --- Changes in v5: - Adapt code to newer multiport support Changes in v2: - Add changes for allow use of the USB hub board/olimex/mx23_olinuxino/mx23_olinuxino.c | 5 + board/olimex/mx23_olinuxino/spl_bo

[U-Boot] [PATCH v5 3/3] mx23_olinuxino: Add ethernet support

2013-03-02 Thread Otavio Salvador
This adds support to the LAN9512 chip included in the board and extend the environment to easy netboot use. Signed-off-by: Otavio Salvador --- Changes in v5: None Changes in v2: - Remove extra newline include/configs/mx23_olinuxino.h | 39 --- 1 file changed,

Re: [U-Boot] FDT howto

2013-03-02 Thread Simon Glass
Hi Jagan, On Fri, Mar 1, 2013 at 11:44 PM, Jagan Teki wrote: > Hi Simon, > > On Sat, Mar 2, 2013 at 6:30 AM, Simon Glass wrote: >> Hi Jagan, >> >> On Fri, Mar 1, 2013 at 1:21 PM, Jagan Teki wrote: >>> Hi Simon, >>> >>> On Sat, Mar 2, 2013 at 1:54 AM, Simon Glass wrote: Hi Jagan,

Re: [U-Boot] [PATCH v6] mtd: cfi_flash: Fix CFI flash driver for 8-bit bus support

2013-03-02 Thread Luka Perkov
Hi Jagannadha, On Fri, Mar 01, 2013 at 06:19:01PM +0530, Jagannadha Sutradharudu Teki wrote: > Tested-by: Luka Perkov If you are going to use me as a tester then please put me in CC of the email next time (and use l...@openwrt.org address). And I didn't test your patch I have tested the old one

Re: [U-Boot] [PATCH v8 31/31] arm: Remove duplicated start.S code

2013-03-02 Thread Benoît Thébaudeau
Hi Albert, On Saturday, March 2, 2013 7:45:06 AM, Albert ARIBAUD wrote: > Hi Benoît, > > On Fri, 1 Mar 2013 23:54:26 +0100 (CET), Benoît Thébaudeau > wrote: > > Re: assembler error messages: > > > > > And in case you ask, with relocate_code() as a function in its own > > > > file instead of a

Re: [U-Boot] Remounting UBI image leads to ECC errors

2013-03-02 Thread Mark Jackson
On 26/02/13 15:11, Mark Jackson wrote: We have a custom made AM335x board I'm working on and I have encountered a problem when mounting an ubifs image from nand. If I ... (1) Write an ubifs image to nand (2) Mount the ubifs image (3) Repeat step (2) ... this then corrupts the ubifs image. Bel

Re: [U-Boot] [PATCH 0/2] ARM: mmu: Set domain permissions to client access - build warnings!

2013-03-02 Thread R, Sricharan
Hi Tom/Albert, From: Rini, Tom Sent: Friday, March 01, 2013 7:51 PM To: Albert ARIBAUD Cc: R, Sricharan; U-Boot; Stehle, Vincent Subject: Re: [U-Boot] [PATCH 0/2] ARM: mmu: Set domain permissions to client access - build warnings! -BEGIN PGP SIGNED MES

[U-Boot] [PATCH] mtd: nand: fix the written length when nand_write_skip_bad failed

2013-03-02 Thread Tao Hou
When the data has been partially written into the NAND Flash, returning the written length instead of 0. The written length may be useful when the upper level decides to continue the writing after skipping the block causing the write failure. Signed-off-by: Tao Hou Cc: Scott Wood Cc: Ben Gardine

[U-Boot] [PATCH] mtd: nand: use ssize_t instead of size_t to prevent infinite loop

2013-03-02 Thread Tao Hou
When a all 0xFF buffer is passed to drop_ffs, the no-0xFF check loop will loop forever. After the fix, If ssize_t i = -1 and size_t l = i + 1, the value of l will still be 0 as expected. Signed-off-by: Tao Hou Cc: Ben Gardiner Cc: Scott Wood --- drivers/mtd/nand/nand_util.c |3 ++- 1 file

[U-Boot] [PATCH] mtd: nand: fix the partial page write condition

2013-03-02 Thread Tao Hou
When writelen is mtd->writesize - 1, it is still a partial page write Signed-off-by: Tao Hou Cc: Scott Wood --- drivers/mtd/nand/nand_base.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mtd/nand/nand_base.c b/drivers/mtd/nand/nand_base.c index a2d06be..3d84659

Re: [U-Boot] [PATCH 00/12] cmd_sf: Add support for read and write instructions

2013-03-02 Thread Jagan Teki
Hi All, On Fri, Jan 11, 2013 at 7:46 AM, Simon Glass wrote: > Hi Jagannadha, > > On Mon, Dec 31, 2012 at 3:13 AM, Jagannadha Sutradharudu Teki > wrote: >> All these patches are added a support for read and write instruction >> for programming/reading SPI flash. >> > > I think this is all great a