Re: [U-Boot] [PATCH V2 3/4] ARM: mx6: Add PCI express driver

2013-12-14 Thread Jagan Teki
On Sat, Dec 14, 2013 at 11:03 AM, Marek Vasut ma...@denx.de wrote: On Saturday, December 14, 2013 at 06:23:30 AM, Jagan Teki wrote: [...] +/* + * Replace the original ARM DABT handler with a simple jump-back one. + * + * The problem here is that if we have a PCIe bridge attached to this

Re: [U-Boot] [PATCH V3 2/4] ARM: mx6: Add PCI express clock configuration

2013-12-14 Thread Jagan Teki
On Sat, Dec 14, 2013 at 10:57 AM, Marek Vasut ma...@denx.de wrote: Split the SATA clock enabling function and add PCI express clock enabling function. The SATA clock enabling function starts up the 100MHz SATA reference PLL in ENET_PLL register, but the code can be re-used to enable the 125MHz

Re: [U-Boot] [PATCH v3 1/4] mkexynosspl: fix uninitialized buffer

2013-12-14 Thread Albert ARIBAUD
Hi Minkyu, On Sat, 14 Dec 2013 14:07:57 +0900, Minkyu Kang mk7.k...@samsung.com wrote: Dear Albert, On 13/12/13 18:00, Albert ARIBAUD wrote: (adding Minkyu as the Samsung custodian) On Mon, 9 Dec 2013 18:09:18 +0100, Albert ARIBAUD albert.u.b...@aribaud.net wrote: mkexynos

Re: [U-Boot] [PATCH 4/4] arm: pxa: update LP-8x4x to boot DTS kernel

2013-12-14 Thread Marek Vasut
On Saturday, December 14, 2013 at 06:42:00 AM, Sergei Ianovich wrote: The device has only 256 kiB for U-Boot binary. Binary size exceeds 256 kiB after activating CONFIG_OF_LIBFDT, so we disable USB support which was not working anyway. Disabled USD frees more space than libfdt requires, so

Re: [U-Boot] [PATCH 2/4] arm: pxa: always init ethaddr for LP-8x4x

2013-12-14 Thread Marek Vasut
On Saturday, December 14, 2013 at 06:41:58 AM, Sergei Ianovich wrote: I always used tftp in my test, so the first dm9000 on LP-8x4x was always properly initialized. However, if the boot doesn't include network related commands, linux will not find a valid MAC and will complain. No longer.

Re: [U-Boot] [PATCH 1/4] ARM: pxa: prevent PXA270 occasional reboot freezes

2013-12-14 Thread Marek Vasut
On Saturday, December 14, 2013 at 06:41:57 AM, Sergei Ianovich wrote: Erratum 71 of PXA270M Processor Family Specification Update (April 19, 2010) explains that watchdog reset time is just 8us insead of 10ms in EMTS. If SDRAM is not reset, it causes memory bus congestion and the device

Re: [U-Boot] [PATCH V2 3/4] ARM: mx6: Add PCI express driver

2013-12-14 Thread Marek Vasut
On Saturday, December 14, 2013 at 09:52:21 AM, Jagan Teki wrote: On Sat, Dec 14, 2013 at 11:03 AM, Marek Vasut ma...@denx.de wrote: On Saturday, December 14, 2013 at 06:23:30 AM, Jagan Teki wrote: [...] +/* + * Replace the original ARM DABT handler with a simple jump-back one. + *

Re: [U-Boot] [PATCH 2/2] exynos5: arndale: Add network support

2013-12-14 Thread Marek Vasut
On Saturday, December 14, 2013 at 03:00:39 PM, Inderpal Singh wrote: Hi Marek, Thanks for the review. On 12 December 2013 19:03, Marek Vasut ma...@denx.de wrote: On Saturday, October 19, 2013 at 08:49:27 AM, Inderpal Singh wrote: From: Chander Kashyap chander.kash...@linaro.org

Re: [U-Boot] [PATCH 2/2] exynos5: arndale: Add network support

2013-12-14 Thread Inderpal Singh
Hi Marek, Thanks for the review. On 12 December 2013 19:03, Marek Vasut ma...@denx.de wrote: On Saturday, October 19, 2013 at 08:49:27 AM, Inderpal Singh wrote: From: Chander Kashyap chander.kash...@linaro.org Arndale board has AX88760, which is USB 2.0 Hub USB 2.0 Ethernet Combo

Re: [U-Boot] [PATCH 2/2] exynos5: arndale: Add network support

2013-12-14 Thread Inderpal Singh
On 14 December 2013 23:20, Marek Vasut ma...@denx.de wrote: On Saturday, December 14, 2013 at 03:00:39 PM, Inderpal Singh wrote: Hi Marek, Thanks for the review. On 12 December 2013 19:03, Marek Vasut ma...@denx.de wrote: On Saturday, October 19, 2013 at 08:49:27 AM, Inderpal Singh

Re: [U-Boot] [PATCH 1/4] ARM: pxa: prevent PXA270 occasional reboot freezes

2013-12-14 Thread Sergei Ianovich
On Sat, 2013-12-14 at 13:29 +0100, Marek Vasut wrote: On Saturday, December 14, 2013 at 06:41:57 AM, Sergei Ianovich wrote: Erratum 71 of PXA270M Processor Family Specification Update (April 19, 2010) explains that watchdog reset time is just 8us insead of 10ms in EMTS. If SDRAM is not

Re: [U-Boot] [PATCH 2/4] arm: pxa: always init ethaddr for LP-8x4x

2013-12-14 Thread Sergei Ianovich
On Sat, 2013-12-14 at 13:30 +0100, Marek Vasut wrote: On Saturday, December 14, 2013 at 06:41:58 AM, Sergei Ianovich wrote: I always used tftp in my test, so the first dm9000 on LP-8x4x was always properly initialized. However, if the boot doesn't include network related commands, linux

Re: [U-Boot] [PATCH 4/4] arm: pxa: update LP-8x4x to boot DTS kernel

2013-12-14 Thread Sergei Ianovich
On Sat, 2013-12-14 at 13:32 +0100, Marek Vasut wrote: On Saturday, December 14, 2013 at 06:42:00 AM, Sergei Ianovich wrote: The device has only 256 kiB for U-Boot binary. Binary size exceeds 256 kiB after activating CONFIG_OF_LIBFDT, so we disable USB support which was not working

[U-Boot] Improve upstream support for Snowball board

2013-12-14 Thread Roy Spliet
Patches are loosely based on the tree maintained by Calao. FDT confirmed working, LAN is successfully detected by u-boot (but lacking a proper tftp setup this is untested). Upstream U-Boot still has one bug that prevents booting from eMMC. Easiest workaround is to revert

[U-Boot] [PATCH 2/2] snowball: Enable FDT

2013-12-14 Thread Roy Spliet
Signed-off-by: Roy Spliet rspl...@eclipso.eu --- include/configs/snowball.h | 5 + 1 file changed, 5 insertions(+) diff --git a/include/configs/snowball.h b/include/configs/snowball.h index 6201b3f..4a19d2c 100644 --- a/include/configs/snowball.h +++ b/include/configs/snowball.h @@ -259,4

[U-Boot] [PATCH 1/2] snowball: Add networking support

2013-12-14 Thread Roy Spliet
Signed-off-by: Roy Spliet rspl...@eclipso.eu --- board/st-ericsson/snowball/snowball.c | 11 +++ drivers/net/smc911x.h | 14 ++ include/configs/snowball.h| 14 ++ 3 files changed, 35 insertions(+), 4 deletions(-) diff --git

Re: [U-Boot] [PATCH 1/2] snowball: Add networking support

2013-12-14 Thread Jagan Teki
Try to fix - checkpatch.pl issues. total: 2 errors, 4 warnings, 0 checks, 74 lines checked On Sat, Dec 14, 2013 at 10:09 PM, Roy Spliet rspl...@eclipso.eu wrote: Signed-off-by: Roy Spliet rspl...@eclipso.eu --- board/st-ericsson/snowball/snowball.c | 11 +++ drivers/net/smc911x.h

Re: [U-Boot] [PATCH 2/2] snowball: Enable FDT

2013-12-14 Thread Jagan Teki
On Sat, Dec 14, 2013 at 10:09 PM, Roy Spliet rspl...@eclipso.eu wrote: Signed-off-by: Roy Spliet rspl...@eclipso.eu --- include/configs/snowball.h | 5 + 1 file changed, 5 insertions(+) diff --git a/include/configs/snowball.h b/include/configs/snowball.h index 6201b3f..4a19d2c 100644

Re: [U-Boot] [PATCH 2/2] exynos5: arndale: Add network support

2013-12-14 Thread Marek Vasut
On Saturday, December 14, 2013 at 03:28:57 PM, Inderpal Singh wrote: On 14 December 2013 23:20, Marek Vasut ma...@denx.de wrote: On Saturday, December 14, 2013 at 03:00:39 PM, Inderpal Singh wrote: Hi Marek, Thanks for the review. On 12 December 2013 19:03, Marek Vasut

Re: [U-Boot] [PATCH 4/4] arm: pxa: update LP-8x4x to boot DTS kernel

2013-12-14 Thread Marek Vasut
On Saturday, December 14, 2013 at 04:41:30 PM, Sergei Ianovich wrote: On Sat, 2013-12-14 at 13:32 +0100, Marek Vasut wrote: On Saturday, December 14, 2013 at 06:42:00 AM, Sergei Ianovich wrote: The device has only 256 kiB for U-Boot binary. Binary size exceeds 256 kiB after activating

Re: [U-Boot] [PATCH 1/4] ARM: pxa: prevent PXA270 occasional reboot freezes

2013-12-14 Thread Marek Vasut
On Saturday, December 14, 2013 at 04:31:31 PM, Sergei Ianovich wrote: On Sat, 2013-12-14 at 13:29 +0100, Marek Vasut wrote: On Saturday, December 14, 2013 at 06:41:57 AM, Sergei Ianovich wrote: Erratum 71 of PXA270M Processor Family Specification Update (April 19, 2010) explains that

Re: [U-Boot] [PATCH 2/4] arm: pxa: always init ethaddr for LP-8x4x

2013-12-14 Thread Marek Vasut
On Saturday, December 14, 2013 at 04:39:00 PM, Sergei Ianovich wrote: On Sat, 2013-12-14 at 13:30 +0100, Marek Vasut wrote: On Saturday, December 14, 2013 at 06:41:58 AM, Sergei Ianovich wrote: I always used tftp in my test, so the first dm9000 on LP-8x4x was always properly initialized.

Re: [U-Boot] [PATCH 2/4] arm: pxa: always init ethaddr for LP-8x4x

2013-12-14 Thread Marek Vasut
On Saturday, December 14, 2013 at 09:53:48 PM, Sergei Ianovich wrote: On Sat, 2013-12-14 at 18:12 +0100, Marek Vasut wrote: On Saturday, December 14, 2013 at 04:39:00 PM, Sergei Ianovich wrote: I strongly believe the current way is easier. I disagree :) IF you set 'ethaddr'

Re: [U-Boot] [PATCH 1/4] ARM: pxa: prevent PXA270 occasional reboot freezes

2013-12-14 Thread Sergei Ianovich
On Sat, 2013-12-14 at 18:14 +0100, Marek Vasut wrote: On Saturday, December 14, 2013 at 04:31:31 PM, Sergei Ianovich wrote: On Sat, 2013-12-14 at 13:29 +0100, Marek Vasut wrote: Do you need to write this register in an endless loop ? I didn't think this way. We need to have at least 3,

Re: [U-Boot] [PATCH 2/4] arm: pxa: always init ethaddr for LP-8x4x

2013-12-14 Thread Sergei Ianovich
On Sat, 2013-12-14 at 21:57 +0100, Marek Vasut wrote: I don't think we need to support non-DT kernel. So we drop the patch. OK, that's nice :) It may even make sense to remove the code that initializes eth1, right? ___ U-Boot mailing list

Re: [U-Boot] [PATCH 1/4] ARM: pxa: prevent PXA270 occasional reboot freezes

2013-12-14 Thread Marek Vasut
On Sunday, December 15, 2013 at 12:51:44 AM, Sergei Ianovich wrote: On Sat, 2013-12-14 at 18:14 +0100, Marek Vasut wrote: On Saturday, December 14, 2013 at 04:31:31 PM, Sergei Ianovich wrote: On Sat, 2013-12-14 at 13:29 +0100, Marek Vasut wrote: Do you need to write this register in an

Re: [U-Boot] [PATCH 2/4] arm: pxa: always init ethaddr for LP-8x4x

2013-12-14 Thread Marek Vasut
On Sunday, December 15, 2013 at 12:53:18 AM, Sergei Ianovich wrote: On Sat, 2013-12-14 at 21:57 +0100, Marek Vasut wrote: I don't think we need to support non-DT kernel. So we drop the patch. OK, that's nice :) It may even make sense to remove the code that initializes eth1, right?