Re: [U-Boot] [PATCHv2] Add support for LZ4 decompression algorithm

2015-10-07 Thread Julius Werner
Well then... a few hours and a significant reduction in sanity later, I found that I'm shadowing the new 'ret' variable from changing the API around because I forgot to remove the declaration part from the 'ret' that already existed in the else block. It would be nice if U-Boot compiled with -Wshad

[U-Boot] [PATCHv3] Add support for LZ4 decompression algorithm

2015-10-07 Thread Julius Werner
This patch adds support for LZ4-compressed FIT image contents. This algorithm has a slightly worse compression ration than LZO while being nearly twice as fast to decompress. When loading images from a fast storage medium this usually results in a boot time win. Sandbox-tested only since I don't h

Re: [U-Boot] [PATCH 1/1] imximage: header.length of 4 is valid

2015-10-07 Thread Stefano Babic
Hi Troy, On 06/10/2015 20:09, Troy Kisky wrote: > On 10/6/2015 4:30 AM, stefano babic wrote: >> Hi Troy, >> >> Am 06.10.2015 um 00:17 schrieb Troy Kisky: >>> On 9/21/2015 2:02 PM, Troy Kisky wrote: Signed-off-by: Troy Kisky --- tools/imximage.c | 4 ++-- 1 file changed, 2 ins

Re: [U-Boot] U-Boot v2015.04, x86 Intel Baytrail, and hsuart0 configuration

2015-10-07 Thread Bin Meng
Hi Tim, +Simon and U-Boot ML. On Wed, Oct 7, 2015 at 12:22 AM, Timothy Scott wrote: > Bin, > > I've seen your name associated with U-boot repo commits relating to the > ns16550 and the UARTs that are sitting on the PCI bus for the Intel > Baytrail. I'm working on setting up a board based on the

Re: [U-Boot] [PATCH 6/6] x86: Allow disabling IGD on Intel Queensbay

2015-10-07 Thread Bin Meng
Hi Simon, On Sat, Oct 3, 2015 at 10:29 PM, Simon Glass wrote: > Hi Bin, > > On 1 October 2015 at 08:36, Bin Meng wrote: >> Add a Kconfig option to disable the Integrated Graphics Device (IGD) >> so that it does not show in the PCI configuration space as a VGA >> disaplay controller. This gives a

Re: [U-Boot] [PATCH 2/2] armv8: ls2085a: Add support of random MAC address for QDS and RDB

2015-10-07 Thread Bin Meng
Hi, On Tue, Oct 6, 2015 at 8:43 PM, Prabhakar Kushwaha wrote: > Add support of setting RANDOM MAC address if env variable not available. > > Signed-off-by: Prabhakar Kushwaha > --- > include/configs/ls2085a_common.h | 4 > 1 file changed, 4 insertions(+) > > diff --git a/include/configs/ls

Re: [U-Boot] [PATCH 2/2] armv8: ls2085a: Add support of random MAC address for QDS and RDB

2015-10-07 Thread Kushwaha Prabhakar
Regards, Prabhakar > -Original Message- > From: Bin Meng [mailto:bmeng...@gmail.com] > Sent: Wednesday, October 07, 2015 1:56 PM > To: Kushwaha Prabhakar-B32579 > Cc: U-Boot Mailing List ; Sun York-R58495 > > Subject: Re: [U-Boot] [PATCH 2/2] armv8: ls2085a: Add support of random > MAC

[U-Boot] [PATCH] pci: Fix expansion ROM programming for multi-function devices

2015-10-07 Thread Bin Meng
PCI_HEADER_TYPE register (offset 0x0e) bit 7 is an indicator for multi-function devices. We should mask it off before using it as the header type. Signed-off-by: Bin Meng --- drivers/pci/pci_auto.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/pci/pci_auto.c b/drivers/pci/pci_auto

Re: [U-Boot] [PATCH 2/2] armv8: ls2085a: Add support of random MAC address for QDS and RDB

2015-10-07 Thread Bin Meng
On Wed, Oct 7, 2015 at 5:08 PM, Kushwaha Prabhakar wrote: > > > Regards, > Prabhakar > >> -Original Message- >> From: Bin Meng [mailto:bmeng...@gmail.com] >> Sent: Wednesday, October 07, 2015 1:56 PM >> To: Kushwaha Prabhakar-B32579 >> Cc: U-Boot Mailing List ; Sun York-R58495 >> >> Subj

Re: [U-Boot] [PATCH] pci: Fix expansion ROM programming for multi-function devices

2015-10-07 Thread Bin Meng
Hi Tom, On Wed, Oct 7, 2015 at 5:13 PM, Bin Meng wrote: > PCI_HEADER_TYPE register (offset 0x0e) bit 7 is an indicator > for multi-function devices. We should mask it off before using > it as the header type. > > Signed-off-by: Bin Meng > --- > > drivers/pci/pci_auto.c | 1 + > 1 file changed,

Re: [U-Boot] [PATCH 05/10] net: designware: Add a post-started hook

2015-10-07 Thread Bin Meng
Hi, On Tue, Oct 6, 2015 at 12:39 AM, Joe Hershberger wrote: > Hi Sjoerd, > > On Mon, Oct 5, 2015 at 3:55 AM, Sjoerd Simons > wrote: >> >> On Sat, 2015-10-03 at 15:30 +0100, Simon Glass wrote: >> > Hi Sjoerd, >> >> > > static int designware_eth_send(struct udevice *dev, void *packet, >> > > int

[U-Boot] [PATCH 1/2][v2] driver: net: ldpaa_eth: Set MAC address during interface open

2015-10-07 Thread Prabhakar Kushwaha
Currently ldpaa ethernet driver rely on DPL file to statically configure mac address for the DPNIs. It is not a correct approach. Add support setting MAC address from env variable or Random MAC address. Signed-off-by: Prabhakar Kushwaha --- Changs for v2: Sending as it is drivers/net/ldpaa_eth

[U-Boot] [PATCH 2/2] armv8: ls2085a: Add support of random MAC address

2015-10-07 Thread Prabhakar Kushwaha
Add support of setting RANDOM MAC address if env variable not available. Signed-off-by: Prabhakar Kushwaha --- Changs for v2: Incorporated Bin Meng's comments - Moved defines to defconfig - updated subject configs/ls2085a_simu_defconfig| 2 ++ configs/ls2085aqds_defconfig | 2 ++ con

[U-Boot] zynq: mmc boot issue

2015-10-07 Thread Jagan Teki
Hi Michal/Siva, Looks like somethings broken with mmc boot, could you please check. dump1: U-Boot SPL 2015.10-rc4-00104-gafcdb02-dirty (Oct 07 2015 - 15:16:54) mmc boot reading system.dtb spl_load_image_fat_os: error reading image system.dtb, err - -1 reading u-boot.img spl_load_image_fat: error

Re: [U-Boot] [PATCH 2/2] armv8: ls2085a: Add support of random MAC address

2015-10-07 Thread Bin Meng
Hi Prabhakar, On Wed, Oct 7, 2015 at 6:00 PM, Prabhakar Kushwaha wrote: > Add support of setting RANDOM MAC address if env variable not available. > > Signed-off-by: Prabhakar Kushwaha > --- > Changs for v2: Incorporated Bin Meng's comments > - Moved defines to defconfig > - updated subject >

[U-Boot] [PATCH 1/2][v3] driver: net: ldpaa_eth: Set MAC address during interface open

2015-10-07 Thread Prabhakar Kushwaha
Currently ldpaa ethernet driver rely on DPL file to statically configure mac address for the DPNIs. It is not a correct approach. Add support setting MAC address from env variable or Random MAC address. Signed-off-by: Prabhakar Kushwaha --- Changs for v2: Sending as it is Changs for v3: Sending

[U-Boot] [PATCH 2/2][v3] armv8: ls2085a: Add support of random MAC address

2015-10-07 Thread Prabhakar Kushwaha
Add support of setting RANDOM MAC address if env variable not available. Signed-off-by: Prabhakar Kushwaha --- Changs for v2: Incorporated Bin Meng's comments - Moved defines to defconfig - updated subject Changes for v3: Incorporated Bin Meng's comments - Removed CONFIG_LIB_RAND - used make

Re: [U-Boot] [PATCH 2/2][v3] armv8: ls2085a: Add support of random MAC address

2015-10-07 Thread Bin Meng
On Wed, Oct 7, 2015 at 7:00 PM, Prabhakar Kushwaha wrote: > Add support of setting RANDOM MAC address if env variable not available. > > Signed-off-by: Prabhakar Kushwaha > --- > Changs for v2: Incorporated Bin Meng's comments > - Moved defines to defconfig > - updated subject > Changes for v3:

Re: [U-Boot] [PATCH 1/1] imximage: header.length of 4 is valid

2015-10-07 Thread Stefano Babic
On 21/09/2015 23:02, Troy Kisky wrote: > Signed-off-by: Troy Kisky > --- > tools/imximage.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/tools/imximage.c b/tools/imximage.c > index 97a6880..7c21922 100644 > --- a/tools/imximage.c > +++ b/tools/imximage.c > @@ -3

Re: [U-Boot] [PATCH] imximage: fix commands other than write_data

2015-10-07 Thread Stefano Babic
Hi Troy, On 15/09/2015 03:06, Troy Kisky wrote: > When CHECK_BITS_SET was added, they forgot to add > a new command table, and instead overwrote the > previous table. > > Signed-off-by: Troy Kisky > > --- Applied to u-boot-imx (fix), thanks ! Best regards, Stefano Babic -- =

[U-Boot] [PATCH v5 1/2] dm: implement a Miscellaneous uclass

2015-10-07 Thread Thomas Chou
Implement a Miscellaneous uclass with generic read or write operations. This class is used only for those do not fit other more general classes. Signed-off-by: Thomas Chou Acked-by: Simon Glass --- v2 add missing misc.h. v3 no change. v4 add ioctl and comments as Simon suggested. v5 no c

[U-Boot] [PATCH v5 2/2] nios2: convert altera sysid to driver model

2015-10-07 Thread Thomas Chou
Convert altera sysid to driver model with misc uclass. Signed-off-by: Thomas Chou Acked-by: Chin Liang See --- v2 fix coding style. v3 doc dts binding. v4 no change. v5 sort headers inclusion as Simon suggested. add display_sysid() to asm/system.h. arch/nios2/cpu/Makefile

Re: [U-Boot] zynq: mmc boot issue

2015-10-07 Thread Hannes Schmelzer
Hi Jagan, i tried this time most current mainline u-boot with default config. My ZC702 evalboard is still booting fine from MMC-card. best regars, Hannes On 10/07/2015 12:02 PM, Jagan Teki wrote: Hi Michal/Siva, Looks like somethings broken with mmc boot, could you please check. dump1: U-Boo

[U-Boot] SPL and DM on ZYNQ

2015-10-07 Thread Hannes Schmelzer
hi folks, i am not very familar with driver model until now and so i've a question. I'm trying to bring up my Zynq ZC702 evalboard using driver-model during SPL stage (because QSPI needs this). I simply switched on "Driver Model for SPL" within Kconfig and compiled. This ends up in: U-Boot S

[U-Boot] [PATCH] nios2: remove asm/psr.h

2015-10-07 Thread Thomas Chou
Remove asm/psr.h, which is not used. Also clean up asm/sections.h and unaligned.h. Signed-off-by: Thomas Chou --- arch/nios2/include/asm/psr.h | 12 arch/nios2/include/asm/sections.h | 10 -- arch/nios2/include/asm/unaligned.h | 5 - 3 files changed, 27 deletions

Re: [U-Boot] DWMMC crontroller in Exynos4412

2015-10-07 Thread Humberto López León
Hi Przemyslaw, Thanks for the documents suggested by you. Now I understand better the SDHCI controller implementations. The U-boot version I'm using the downloaded from https://github.com/hardkernel/u-boot/commits/odroid-v2010.12. From where I can download the version you're fixing? Thank y

[U-Boot] [PATCH] tools/proftool: fix use-after-free

2015-10-07 Thread Vincent Stehlé
The read_trace_config() can dereference the line pointer after freeing it on its error path. Avoid that. This was found by Coverity Scan. Signed-off-by: Vincent Stehlé Cc: Simon Glass --- tools/proftool.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/tools/proftool.c

[U-Boot] U-Boot malloc implementation on arm - problem after relocation

2015-10-07 Thread Marcin Krzemiński
Hello, In my board I have some problems with malloc functionality. In config I have settings for malloc: #define CONFIG_SYS_MALLOC_LEN (2 * 1024 * 1024) #define CONFIG_SYS_MALLOC_F_LEN (1024) I am running u-boot without SPL. Uboot starts from SRAM and then relocates to DRAM. When I run u-boot

Re: [U-Boot] [PATCH] tools/proftool: fix use-after-free

2015-10-07 Thread Tom Rini
On Wed, Oct 07, 2015 at 03:48:48PM +0200, Vincent Stehlé wrote: > The read_trace_config() can dereference the line pointer after freeing > it on its error path. Avoid that. > > This was found by Coverity Scan. > > Signed-off-by: Vincent Stehlé > Cc: Simon Glass Were you in the Coverity talk t

[U-Boot] [PATCH] board: axs10x - cap max SDIO clock value to bus/2

2015-10-07 Thread Alexey Brodkin
It turned out with some boards (FPGA firmwares?) and cards combos current clock settings doesn't work as expected leading to strange card freezes or corrupted data being read from the card. Especially this was seen with Transcend 2Gb cards shipped as a part of ARC SDP: ->8-

[U-Boot] Please pull ARC fixes

2015-10-07 Thread Alexey Brodkin
Hi Tom, The following changes since commit 996ec1dcc58a34b53891acde0ec5df9141b5fcc2: Merge branch 'master' of git://git.denx.de/u-boot-fdt (2015-10-03 10:48:06 -0400) are available in the git repository at: git://git.denx.de/u-boot-arc.git for you to fetch changes up to f6e27ba5b40a88613

Re: [U-Boot] [PATCH] tools/proftool: fix use-after-free

2015-10-07 Thread Vincent Stehlé
On 10/07/2015 04:19 PM, Tom Rini wrote: .. > Were you in the Coverity talk too? :) Hi Tom, No, I was not following that talk, sorry. .. > free(line); > - return regex_report_error(&line->regex, err, > "compile", > + err = regex

[U-Boot] [PATCH 2/2] x86: gpio: Add use-lvl-write-cache to fix baytrail

2015-10-07 Thread George McCollister
Add a device-tree property use-lvl-write-cache that will cause writes to lvl to be cached instead of read from lvl before each write. This is required on some platforms that have the register implemented as dual read/write (such as Baytrail). Prior to this fix the blue USB port on the Minnowboard

[U-Boot] [PATCH 1/2] x86: gpio: Make x86-pinctrl subnode of ich6-gpio

2015-10-07 Thread George McCollister
Instead of having x86-pinctrl work separately from ich6-gpio have it work underneath ich6-gpio. This removes redundant configuration and will allow the addition of shared bank settings in future commits. Signed-off-by: George McCollister --- arch/x86/dts/minnowmax.dts |

Re: [U-Boot] U-Boot v2015.04, x86 Intel Baytrail, and hsuart0 configuration

2015-10-07 Thread Timothy Scott
Bin, Thanks for the response. I've got things working with v2015.04, but it took some tweaking. Here's what I learned: 1) PCI is initialized AFTER the code was attempting to configure hsuart0. Since hsuart0 is on the PCI bus, the various registers being read were coming up 0x and therefore

Re: [U-Boot] [PATCH] tools/proftool: fix use-after-free

2015-10-07 Thread Tom Rini
On Wed, Oct 07, 2015 at 04:35:53PM +0200, Vincent Stehlé wrote: > On 10/07/2015 04:19 PM, Tom Rini wrote: > .. > > Were you in the Coverity talk too? :) > > Hi Tom, > > No, I was not following that talk, sorry. Ah, coincidence then. > .. > > free(line); > > -

Re: [U-Boot] SPL and DM on ZYNQ

2015-10-07 Thread Simon Glass
Hi, On 7 October 2015 at 13:50, Hannes Schmelzer wrote: > hi folks, > > i am not very familar with driver model until now and so i've a question. > > I'm trying to bring up my Zynq ZC702 evalboard using driver-model during SPL > stage (because QSPI needs this). > I simply switched on "Driver Mode

Re: [U-Boot] U-Boot malloc implementation on arm - problem after relocation

2015-10-07 Thread Andreas Färber
Hi Marcin, Am 07.10.2015 um 15:58 schrieb Marcin Krzemiński: > Since I use qemu it > is very hard to debug with gdb u-boot after relocation( or I do not know > how to do it), so I am almost blind. QEMU has a built-in gdb stub that you can just connect to as gdb remote target, similar to how you w

Re: [U-Boot] [PATCH] imximage: fix commands other than write_data

2015-10-07 Thread Troy Kisky
On 10/7/2015 5:05 AM, Stefano Babic wrote: > Hi Troy, > > On 15/09/2015 03:06, Troy Kisky wrote: >> When CHECK_BITS_SET was added, they forgot to add >> a new command table, and instead overwrote the >> previous table. >> >> Signed-off-by: Troy Kisky >> >> --- > > Applied to u-boot-imx (fix), th

Re: [U-Boot] [PATCH] Fix variation in timestamps caused by timezone differences.

2015-10-07 Thread Paul Kocialkowski
Le vendredi 02 octobre 2015 à 09:11 -0700, Vagrant Cascadian a écrit : > When building with SOURCE_DATE_EPOCH set, avoid use of mktime in > default_image.c, which converts the timestamp into localtime. This > causes variation based on timezone when building u-boot.img and > u-boot-sunxi-with-spl.bi

Re: [U-Boot] SPL and DM on ZYNQ

2015-10-07 Thread Jagan Teki
On 7 October 2015 at 21:29, Simon Glass wrote: > Hi, > > On 7 October 2015 at 13:50, Hannes Schmelzer wrote: >> hi folks, >> >> i am not very familar with driver model until now and so i've a question. >> >> I'm trying to bring up my Zynq ZC702 evalboard using driver-model during SPL >> stage (be

Re: [U-Boot] [PATCH] common/image.c: Make boot_get_ramdisk() perform a check for Android images

2015-10-07 Thread Paul Kocialkowski
Hi, Le lundi 05 octobre 2015 à 14:23 -0500, Rob Herring a écrit : > On Tue, Sep 1, 2015 at 8:50 AM, Paul Kocialkowski > wrote: > > Le jeudi 27 août 2015 à 15:42 -0400, Tom Rini a écrit : > > > In 2dd4632 the check for where a ramdisk is found on an Android > > > image > > > was got moved into the

Re: [U-Boot] [PATCH] imximage: fix commands other than write_data

2015-10-07 Thread Fabio Estevam
On Wed, Oct 7, 2015 at 2:54 PM, Troy Kisky wrote: > Could someone verify that one of the following boards still boot > with this patch? > > > > colibri_vf_dtb > mx6sabresd_spl Tested with mx6sabresd_spl and it still boots. ___ U-Boot mailing list U-Boo

[U-Boot] PCI: Cannot read bus configuration

2015-10-07 Thread Fabio Estevam
Hi Simon, Since commit ff3e077bd2 ("dm: pci: Add a uclass for PCI") I see the following error message after running the 'pci' command: => pci 0 Scanning PCI devices on bus 0 BusDevFun VendorId DeviceId Device Class Sub-Class _

Re: [U-Boot] [PATCH 2/6] video: vesa_fb: Fix wrong return value check of pci_find_class()

2015-10-07 Thread Anatolij Gustschin
On Thu, 1 Oct 2015 00:36:00 -0700 Bin Meng wrote: > When pci_find_class() fails to find a device, it returns -ENODEV. > But now we check the return value against -1. Fix it. > > Signed-off-by: Bin Meng > --- > > drivers/video/vesa_fb.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-)

Re: [U-Boot] U-Boot - Download URL and certificates

2015-10-07 Thread Wolfgang Denk
Dear Markus, In message <56122fa6.8010...@tqsc.de> you wrote: > > we are facing problems with downloading U-Boot tarballs and > automated build systems. Tarballs? But that should be trivial. Tarballs are easily available on the FTP server. They are also eavailable on ACD, but this is not exac

Re: [U-Boot] [PATCH 1/5] fs/fat/fat_write: Fix buffer alignments

2015-10-07 Thread Benoît Thébaudeau
Hi Tom, On Mon, Sep 28, 2015 at 5:22 PM, Tom Rini wrote: > On Mon, Sep 28, 2015 at 03:45:28PM +0200, Benoît Thébaudeau wrote: > >> set_cluster() was using a temporary buffer without enforcing its >> alignment for DMA and cache. Moreover, it did not check the alignment of >> the passed buffer, whi

Re: [U-Boot] DWMMC crontroller in Exynos4412

2015-10-07 Thread Humberto López León
Hi Przemyslaw, the version of the U-boot on which you made the corrections in the SDHCI controller is at https://github.com/hardkernel/u-boot/commits/odroid-v2015.10? Regards, Humberto On 10/01/2015 07:24 AM, Przemyslaw Marczak wrote: Hello Humberto, On 09/28/2015 03:27 PM, Humberto López

Re: [U-Boot] [PATCH] imximage: fix commands other than write_data

2015-10-07 Thread Troy Kisky
On 10/7/2015 11:25 AM, Fabio Estevam wrote: > On Wed, Oct 7, 2015 at 2:54 PM, Troy Kisky > wrote: > >> Could someone verify that one of the following boards still boot >> with this patch? >> >> >> >> colibri_vf_dtb >> mx6sabresd_spl > > Tested with mx6sabresd_spl and it still boots. > Thanks Fa

Re: [U-Boot] [PATCH] ls102xa: Fix reset hang

2015-10-07 Thread Fabio Estevam
On Sat, Oct 3, 2015 at 12:18 PM, Tom Rini wrote: > Fabio, can you do a v2 that makes the commit message a bit clearer that > this is a temporary work-around and that a proper solution to the > underlying problem is coming? I think everyone that's reading this > thread knows this but we should ma

Re: [U-Boot] [PATCH] ls102xa: Fix reset hang

2015-10-07 Thread York Sun
On 10/07/2015 02:43 PM, Fabio Estevam wrote: > On Sat, Oct 3, 2015 at 12:18 PM, Tom Rini wrote: > >> Fabio, can you do a v2 that makes the commit message a bit clearer that >> this is a temporary work-around and that a proper solution to the >> underlying problem is coming? I think everyone th

Re: [U-Boot] [PATCH] ls102xa: Fix reset hang

2015-10-07 Thread Fabio Estevam
On Wed, Oct 7, 2015 at 6:45 PM, York Sun wrote: > Please use master, or Tom's tree. My tree is behind and I only update before > requesting a pull. I generated the patches against master, so Tom should be able to apply them cleanly. ___ U-Boot mailing

[U-Boot] [PATCH] net: phy: micrel: disable NAND-tree for KSZ8051

2015-10-07 Thread Sylvain Rochet
NAND-tree is used to check wiring between MAC and PHY using NAND gates on the PHY side, hence the name. NAND-tree initial status is latched at reset by probing the IRQ pin. However some devices are sharing the PHY IRQ pin with other peripherals such as Atmel SAMA5D[34]x-EK boards when using the op

Re: [U-Boot] U-Boot malloc implementation on arm - problem after relocation

2015-10-07 Thread mar.krzeminski
W dniu 07.10.2015 o 19:38, Andreas Färber pisze: Hi Marcin, Am 07.10.2015 um 15:58 schrieb Marcin Krzemiński: Since I use qemu it is very hard to debug with gdb u-boot after relocation( or I do not know how to do it), so I am almost blind. QEMU has a built-in gdb stub that you can just conne

Re: [U-Boot] SPL and DM on ZYNQ

2015-10-07 Thread Simon Glass
Hi Jagan, On 7 October 2015 at 19:01, Jagan Teki wrote: > On 7 October 2015 at 21:29, Simon Glass wrote: >> Hi, >> >> On 7 October 2015 at 13:50, Hannes Schmelzer wrote: >>> hi folks, >>> >>> i am not very familar with driver model until now and so i've a question. >>> >>> I'm trying to bring u

Re: [U-Boot] PCI: Cannot read bus configuration

2015-10-07 Thread Simon Glass
Hi Fabio, On 7 October 2015 at 19:34, Fabio Estevam wrote: > Hi Simon, > > Since commit ff3e077bd2 ("dm: pci: Add a uclass for PCI") I see the following > error message after running the 'pci' command: > > => pci 0 > Scanning PCI devices on bus 0 > BusDevFun VendorId DeviceId Device Class

Re: [U-Boot] [PATCH v5 2/2] nios2: convert altera sysid to driver model

2015-10-07 Thread Simon Glass
On 7 October 2015 at 13:29, Thomas Chou wrote: > Convert altera sysid to driver model with misc uclass. > > Signed-off-by: Thomas Chou > Acked-by: Chin Liang See > --- > v2 > fix coding style. > v3 > doc dts binding. > v4 > no change. > v5 > sort headers inclusion as Simon suggested. >

Re: [U-Boot] PCI: Cannot read bus configuration

2015-10-07 Thread Fabio Estevam
Hi Simon, On Wed, Oct 7, 2015 at 7:30 PM, Simon Glass wrote: > This added error checking to pci_read_config_word(). It might be worth > looking at why the access is failing. I noticed that when VendorID !=0x then pci_read_config_word() returns 0. In the case when VendorID ==0x then it

Re: [U-Boot] PCI: Cannot read bus configuration

2015-10-07 Thread Fabio Estevam
On Wed, Oct 7, 2015 at 8:07 PM, Fabio Estevam wrote: > Hi Simon, > > On Wed, Oct 7, 2015 at 7:30 PM, Simon Glass wrote: > >> This added error checking to pci_read_config_word(). It might be worth >> looking at why the access is failing. > > I noticed that when VendorID !=0x then pci_read_con

Re: [U-Boot] PCI: Cannot read bus configuration

2015-10-07 Thread Fabio Estevam
On Wed, Oct 7, 2015 at 8:24 PM, Fabio Estevam wrote: > On Wed, Oct 7, 2015 at 8:07 PM, Fabio Estevam wrote: >> Hi Simon, >> >> On Wed, Oct 7, 2015 at 7:30 PM, Simon Glass wrote: >> >>> This added error checking to pci_read_config_word(). It might be worth >>> looking at why the access is failing

[U-Boot] [PATCH] cmd_pci: Check for VendorID earlier

2015-10-07 Thread Fabio Estevam
From: Fabio Estevam Since commit ff3e077bd2 ("dm: pci: Add a uclass for PCI") the following error message is seen: => pci 0 Scanning PCI devices on bus 0 BusDevFun VendorId DeviceId Device Class Sub-Class _ 00.01.00 0x16c3

Re: [U-Boot] [PATCH v4 1/2] dm: implement a Timer uclass

2015-10-07 Thread Thomas Chou
Hi Simon, On 10/06/2015 10:14 PM, Simon Glass wrote: Hi Thomas, On 4 October 2015 at 14:56, Thomas Chou wrote: Implement a Timer uclass to work with lib/time.c. Signed-off-by: Thomas Chou Reviewed-by: Simon Glass Sorry, after a lot of consideration I'd like to retract that :-( Please se

[U-Boot] [PATCH v5 3/3] nios2: convert altera timer to driver model

2015-10-07 Thread Thomas Chou
Convert altera timer to driver model. Signed-off-by: Thomas Chou Acked-by: Chin Liang See --- v2 fix coding style. v3 doc dts binding. v4 no change. v5 revert to get_rate and use uclass priv to store the clock_rate. arch/nios2/cpu/Makefile | 2 +- arch/nios2/c

[U-Boot] [PATCH v5 1/3] dm: implement a Timer uclass

2015-10-07 Thread Thomas Chou
Implement a Timer uclass to work with lib/time.c. Signed-off-by: Thomas Chou Reviewed-by: Simon Glass --- v2 fix coding style. v3 add description to Kconfig as Simon suggested. move timer.c code to lib/time.c. add dm_timer dev to global data. remove timer_init(). change API name get_

[U-Boot] [PATCH v5 2/3] timer: start a new dm_timer after relocation

2015-10-07 Thread Thomas Chou
Start a new dm_timer after relocation, just in case the timer has been used in per-relocation. Signed-off-by: Thomas Chou --- common/board_r.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/common/board_r.c b/common/board_r.c index f8c1baa..aaf390e 100644 --- a/common/board_r.c +++ b/com

Re: [U-Boot] U-Boot v2015.04, x86 Intel Baytrail, and hsuart0 configuration

2015-10-07 Thread Bin Meng
Hi Tim, On Wed, Oct 7, 2015 at 11:26 PM, Timothy Scott wrote: > Bin, > > Thanks for the response. I've got things working with v2015.04, but it took > some tweaking. Here's what I learned: > > 1) PCI is initialized AFTER the code was attempting to configure hsuart0. > Since hsuart0 is on the PC

[U-Boot] [PATCH v3 01/12] x86: Initialize GDT entry 1 to be the 32-bit CS as well

2015-10-07 Thread Bin Meng
Some OS (like VxWorks) requires GDT entry 1 to be the 32-bit CS. Signed-off-by: Bin Meng Acked-by: Simon Glass Tested-by: Jian Luo --- Changes in v3: None Changes in v2: None arch/x86/cpu/cpu.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/arch/x86/cpu/cpu.c b/arc

[U-Boot] [PATCH v3 02/12] x86: Move install_e820_map() out of zimage.c

2015-10-07 Thread Bin Meng
install_e820_map() has nothing to do with zimage related codes. Move it to a dedicated place. Signed-off-by: Bin Meng Acked-by: Simon Glass --- Changes in v3: None Changes in v2: - New patch to move install_e820_map() out of zimage.c arch/x86/include/asm/e820.h | 3 +++ arch/x86/include/a

[U-Boot] [PATCH v3 05/12] cmd: Clean up cmd_elf a little bit

2015-10-07 Thread Bin Meng
This commit cleans up cmd_elf.c per U-Boot coding convention, and removes the unnecessary DECLARE_GLOBAL_DATA_PTR and out-of-date powerpc comments (it actually supports not only powerpc targets). Signed-off-by: Bin Meng Reviewed-by: Tom Rini --- Changes in v3: None Changes in v2: None common/

[U-Boot] [PATCH v3 04/12] cmd: Convert CONFIG_CMD_ELF to Kconfig

2015-10-07 Thread Bin Meng
Convert CONFIG_CMD_ELF to Kconfig and tidy up affected boards. Signed-off-by: Bin Meng --- Changes in v3: None Changes in v2: - Split the defconfig reorder to another patch, making this patch as CONFIG_CMD_ELF mods only - Update sparc and avr32 boards to unset CONFIG_CMD_ELF common/Kconfig

[U-Boot] [PATCH v3 00/12] Better support of booting VxWorks via 'bootvx'

2015-10-07 Thread Bin Meng
This series adds better support of booting VxWorks using 'bootvx'. Tested by booting a VxWorks 6.9.4 kernel on Intel Crown Bay, and a VxWorks 7 kernel on Intel Galileo. Rebased on u-boot/master branch and dropped the #1 and #2 patches in the v2 series as they were already applied in the main stre

[U-Boot] [PATCH v3 03/12] x86: Remove quotation mark in CONFIG_HOSTNAME

2015-10-07 Thread Bin Meng
CONFIG_HOSTNAME is an environment varible, so that quotation mark is not needed. Signed-off-by: Bin Meng Acked-by: Simon Glass --- Changes in v3: None Changes in v2: - New patch to remove quotation mark in CONFIG_HOSTNAME include/configs/x86-common.h | 2 +- 1 file changed, 1 insertion(+), 1

[U-Boot] [PATCH v3 08/12] cmd: bootvx: Pass netmask and gatewayip to VxWorks bootline

2015-10-07 Thread Bin Meng
There are fields in VxWorks bootline for netmask and gatewayip. We can get these from U-Boot environment variables and pass them to VxWorks, just like ipaddr and serverip. Signed-off-by: Bin Meng --- Changes in v3: - Avoid calls to strlen() Changes in v2: - Fix the endian issue for netmask c

[U-Boot] [PATCH v3 06/12] cmd: elf: Reorder load_elf_image_phdr() and load_elf_image_shdr()

2015-10-07 Thread Bin Meng
Move load_elf_image_phdr() and load_elf_image_shdr() to the beginning of the cmd_elf.c so that forward declaration is not needed. Signed-off-by: Bin Meng Reviewed-by: Tom Rini --- Changes in v3: None Changes in v2: None common/cmd_elf.c | 161 +++---

[U-Boot] [PATCH v3 11/12] cmd: bootvx: Add asmlinkage to the VxWorks x86 entry

2015-10-07 Thread Bin Meng
VxWorks on x86 uses stack to pass parameters. Reported-by: Jian Luo Signed-off-by: Bin Meng Acked-by: Simon Glass --- Changes in v3: None Changes in v2: - New patch to add asmlinkage to the VxWorks x86 entry common/cmd_elf.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/common/c

[U-Boot] [PATCH v3 09/12] cmd: bootvx: Always get VxWorks bootline from env

2015-10-07 Thread Bin Meng
So far VxWorks bootline can be contructed from various environment variables, but when these variables do not exist we get these from corresponding config macros. This is not helpful as it requires rebuilding U-Boot, and to make sure these config macros take effect we should not have these environm

[U-Boot] [PATCH v3 07/12] cmd: bootvx: Avoid strlen() calls when constructing VxWorks bootline

2015-10-07 Thread Bin Meng
Remember the position in the VxWorks bootline buffer to avoid the call to strlen() each time. Signed-off-by: Bin Meng --- Changes in v3: - New patch to avoid strlen() calls when constructing VxWorks bootline Changes in v2: None common/cmd_elf.c | 20 ++-- 1 file changed, 10 i

[U-Boot] [PATCH v3 10/12] cmd: bootvx: Pass E820 information to an x86 VxWorks kernel

2015-10-07 Thread Bin Meng
E820 is critical to the kernel as it provides system memory map information. Pass it to an x86 VxWorks kernel. Signed-off-by: Bin Meng Acked-by: Simon Glass Reviewed-by: Tom Rini Tested-by: Jian Luo --- Changes in v3: None Changes in v2: None common/cmd_elf.c | 30 +

[U-Boot] [PATCH v3 12/12] doc: Complement document about booting VxWorks

2015-10-07 Thread Bin Meng
Current document about how to boot VxWorks is limited. Add several chapters in README.vxworks to document this. Signed-off-by: Bin Meng Reviewed-by: Tom Rini --- Changes in v3: None Changes in v2: - Describe typical values for bootaddr, e820data, e820info. doc/README.vxworks | 82 +++

[U-Boot] [PATCH v2 1/3] net: phy: Don't create phy device when there is no phy

2015-10-07 Thread Bin Meng
In get_phy_device_by_mask(), when no phy is found, we should not create any phy device. Signed-off-by: Bin Meng Acked-by: Joe Hershberger --- Changes in v2: - Rebase on u-boot/master drivers/net/phy/phy.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/net/phy/p

[U-Boot] [PATCH v2 3/3] net: phy: Test previous phydev->dev against new mac dev

2015-10-07 Thread Bin Meng
In phy_connect_dev(), if the phy device has an accociated mac device before, a warning message will be printed. But we should test the old device against the new one, if they are actually the same one, don't print the warning message. Signed-off-by: Bin Meng Acked-by: Joe Hershberger --- Chang

[U-Boot] [PATCH v2 2/3] net: phy: Change to print all phys that are not found

2015-10-07 Thread Bin Meng
In get_phy_device_by_mask(), when no phy is found, currently we only print a message to show the first phy address that is not found. But this is not always the case as multiple phys can be specified by phy_mask. Change to print all phys that are not found, and to reduce the console boot log, chang

[U-Boot] [PATCH v2 1/4] net: mdio: Add mdio_free() and mdio_unregister() API

2015-10-07 Thread Bin Meng
Currently there is no API to uninitialize mdio. Add two APIs for this. Signed-off-by: Bin Meng Acked-by: Joe Hershberger --- Changes in v2: None common/miiphyutil.c | 19 +++ include/miiphy.h| 2 ++ 2 files changed, 21 insertions(+) diff --git a/common/miiphyutil.c b/com

[U-Boot] [PATCH v2 4/4] dm: core: Remove unnecessary codes in uclass_pre_remove_device()

2015-10-07 Thread Bin Meng
dev->uclass->uc_drv->per_device_auto_alloc_size is to be freed in device_free(), so is dev->seq. Remove these unnecessary codes. Signed-off-by: Bin Meng Acked-by: Simon Glass --- Changes in v2: - Drop the v1 patch: "dm: core: Remove the call to device_free()" drivers/core/uclass.c | 7 --

[U-Boot] [PATCH v2 2/4] net: designware: Add driver remove support

2015-10-07 Thread Bin Meng
In designware_eth_probe(), some additional resources are allocated (eg: mdio, phy). We should free these in the driver remove phase. Add designware_eth_remove() to clean it up. Signed-off-by: Bin Meng Acked-by: Simon Glass --- Changes in v2: None drivers/net/designware.c | 12 1

[U-Boot] [PATCH v2 3/4] net: pch_gbe: Add driver remove support

2015-10-07 Thread Bin Meng
In pch_gbe_probe(), some additional resources are allocated (eg: mdio, phy). We should free these in the driver remove phase. Add pch_gbe_remove() to clean it up. Signed-off-by: Bin Meng Reviewed-by: Simon Glass --- Changes in v2: None drivers/net/pch_gbe.c | 12 1 file changed,

Re: [U-Boot] Inconsistencies in commands regarding load_addr

2015-10-07 Thread Wolfgang Denk
Dear Benoît, In message <5613e20f.8060...@wsystem.com> you wrote: > > I've just noticed that before the commit > 045fa1e1142552799ad3203e9e0bc22a11e866ea, ext2load and ext4load were setting > the > load_addr global variable, but not fatload. Since then, none of these commands > set load_addr (in

[U-Boot] [PATCH v3 1/3] net: eth: Clear MAC address in eth_pre_remove()

2015-10-07 Thread Bin Meng
platdata->enetaddr was assigned to a value in dev_probe() last time. If we don't clear it, for dev_probe() at the second time, dm eth will end up treating it as a MAC address from ROM no matter where it came from originally (maybe env, ROM, or even random). Fix this by clearing platdata->enetaddr w

[U-Boot] [PATCH v3 2/3] dm: test: Add a new test case against dm eth codes for NULL pointer access

2015-10-07 Thread Bin Meng
U-Boot crashes when doing a 'ping' with the following test scenario: - All ethernet devices are not probed - "ethaddr" for all ethernet devices are not set - "ethact" is set to a valid ethernet device name Add a new test case 'dm_test_eth_act' to hit such scenario. Signed-off-by: Bin Meng

[U-Boot] [PATCH v3 3/3] net: eth: Check return value in various places

2015-10-07 Thread Bin Meng
eth_get_dev() can return NULL which means device_probe() fails for that ethernet device. Add return value check in various places or U-Boot will crash due to NULL pointer access. With this commit, 'dm_test_eth_act' test case passes. Signed-off-by: Bin Meng Acked-by: Joe Hershberger --- Change

[U-Boot] [PATCH] ARM: k2e/l: Apply WA for selecting PA clock source

2015-10-07 Thread Lokesh Vutla
On keystone2 Lamarr and Edison platforms, the PA clocksource mux in PLL REG1, can be changed only after enabling its clock domain. So selecting the output of PASS PLL as input to PA only after enabling the clockdomain. This is as per the debug done by "Vitaly Andrianov " and based on the previous w