[U-Boot] [GIT PULL] Zynq patches

2014-05-13 Thread Michal Simek
Hi Albert,

here is the set of patches I have capture for Xilinx Zynq.
I am sending it now because this patchset was based on
fpga changes I have done and which were merged yesterday.

I have also checked your arm custodian repo and you don't have
pending patches that's why I believe it shouldn't be
any problem to do a merge with Tom's tree.

Buildman doesn't show any problem around zynq patches I am sending.

Thanks,
Michal


[u-boot]$ ./tools/buildman/buildman -b zynq zynq -sSed
Summary of 21 commits for 6 boards (4 threads, 1 job per thread)
01: Merge branch 'master' of git://git.denx.de/u-boot-mpc85xx
02: ARM: zynq: Fix sparse warnings in slcr.c
03: ARM: zynq: Fix sparse warning in ddrc.c
04: ARM: zynq: Remove sparse warnings
   arm: (for 6/6 boards)  data -168.0  rodata +168.0
zynq_zc770_xm010:  data -168  rodata +168
zynq_zc770_xm013:  data -168  rodata +168
zynq_zc770_xm012:  data -168  rodata +168
zynq_zed   :  data -168  rodata +168
zynq_zc70x :  data -168  rodata +168
zynq_microzed  :  data -168  rodata +168
05: ARM: zynq: Added efuse status register base address
06: ARM: zynq: Do not use half memory size for ECC case
   arm: (for 6/6 boards)  all -32.0  text -32.0
zynq_zc770_xm010:  all -32  text -32
zynq_zc770_xm013:  all -32  text -32
zynq_zc770_xm012:  all -32  text -32
zynq_zed   :  all -32  text -32
zynq_zc70x :  all -32  text -32
zynq_microzed  :  all -32  text -32
07: ARM: zynq: Call zynq board_init() in SPL
   arm: (for 6/6 boards)  spl/u-boot-spl:all +16.0  spl/u-boot-spl:text 
+16.0
zynq_zc770_xm010:  spl/u-boot-spl:all +16  spl/u-boot-spl:text +16
zynq_zc770_xm013:  spl/u-boot-spl:all +16  spl/u-boot-spl:text +16
zynq_zc770_xm012:  spl/u-boot-spl:all +16  spl/u-boot-spl:text +16
zynq_zed   :  spl/u-boot-spl:all +16  spl/u-boot-spl:text +16
zynq_zc70x :  spl/u-boot-spl:all +16  spl/u-boot-spl:text +16
zynq_microzed  :  spl/u-boot-spl:all +16  spl/u-boot-spl:text +16
08: ARM: zynq: slcr: Fix incorrect commentary
09: ARM: zynq: Setup correct slcr_lock value
   arm: (for 6/6 boards)  all +102.0  bss +2.7  data +4.0  
spl/u-boot-spl:all +72.0  spl/u-boot-spl:data +4.0  spl/u-boot-spl:text +68.0  
text +95.3
zynq_zc770_xm010:  all +112  bss +12  data +4  spl/u-boot-spl:all 
+72  spl/u-boot-spl:data +4  spl/u-boot-spl:text +68  text +96
zynq_zed   :  all +112  bss +12  data +4  spl/u-boot-spl:all 
+72  spl/u-boot-spl:data +4  spl/u-boot-spl:text +68  text +96
zynq_microzed  :  all +112  bss +12  data +4  spl/u-boot-spl:all 
+72  spl/u-boot-spl:data +4  spl/u-boot-spl:text +68  text +96
zynq_zc770_xm013:  all +100  data +4  spl/u-boot-spl:all +72  
spl/u-boot-spl:data +4  spl/u-boot-spl:text +68  text +96
zynq_zc770_xm012:  all +96  data +4  spl/u-boot-spl:all +72  
spl/u-boot-spl:data +4  spl/u-boot-spl:text +68  text +92
zynq_zc70x :  all +80  bss -20  data +4  spl/u-boot-spl:all +72 
 spl/u-boot-spl:data +4  spl/u-boot-spl:text +68  text +96
10: ARM: zynq: Fix bootmode mask
11: ARM: zynq: Fix building SPL without FPGA support
12: ARM: zynq: Extend kernel image size to 60MB
13: ARM: zynq: Add MIO detection code
14: ARM: zynq: ehci: Added USB host driver support
   arm: (for 6/6 boards)  all +9.3  bss -0.7  rodata +10.0  
spl/u-boot-spl:all +10.0  spl/u-boot-spl:rodata +10.0
zynq_zc70x :  all +34  bss +24  rodata +10  spl/u-boot-spl:all 
+10  spl/u-boot-spl:rodata +10
zynq_zc770_xm013:  all +10  rodata +10  spl/u-boot-spl:all +10  
spl/u-boot-spl:rodata +10
zynq_zc770_xm012:  all +10  rodata +10  spl/u-boot-spl:all +10  
spl/u-boot-spl:rodata +10
zynq_zc770_xm010:  all +2  bss -8  rodata +10  spl/u-boot-spl:all 
+10  spl/u-boot-spl:rodata +10
zynq_microzed  :  all +2  bss -8  rodata +10  spl/u-boot-spl:all 
+10  spl/u-boot-spl:rodata +10
zynq_zed   :  all -2  bss -12  rodata +10  spl/u-boot-spl:all 
+10  spl/u-boot-spl:rodata +10
15: ARM: zynq: Added USB host support for zynq boards
   arm: (for 6/6 boards)  all +36453.7  bss +26203.3  data +38.7  rodata 
+2047.7  spl/u-boot-spl:all +1.3  spl/u-boot-spl:rodata +1.3  text +8164.0
zynq_zed   :  all +109121  bss +78632  data +116  rodata +5881  
spl/u-boot-spl:all +4  spl/u-boot-spl:rodata +4  text +24492
zynq_zc70x :  all +109085  bss +78596  data +116  rodata +5881  
spl/u-boot-spl:all +4  spl/u-boot-spl:rodata +4  text +24492
zynq_zc770_xm013:  all +131  rodata +131
zynq_zc770_xm012:  all +131  rodata +131
zynq_zc770_xm010:  all +127  bss -4  rodata +131
zynq_microzed  :  all +127  bss -4  rodata +131
16: ARM: zynq: Enable the FAT write capability

Re: [U-Boot] [PATCH 4/4] cm-t54: add cm-t54 board support

2014-05-13 Thread Igor Grinberg
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Hi Tom,

On 05/06/14 20:45, Tom Rini wrote:
> On Sun, Apr 27, 2014 at 01:17:28PM +0300, Dmitry Lifshitz wrote:
> 
>> Add cm-t54 board directory, config file. Enable build.
> [snip]
>> diff --git a/board/compulab/cm_t54/Makefile b/board/compulab/cm_t54/Makefile
>> new file mode 100644
>> index 000..bd8bc51
>> --- /dev/null
>> +++ b/board/compulab/cm_t54/Makefile
>> @@ -0,0 +1,10 @@
>> +#
>> +# Copyright (C) 2014 Compulab Ltd - http://compulab.co.il/
>> +#
>> +# Author: Dmitry Lifshitz 
>> +#
>> +# SPDX-License-Identifier:  GPL-2.0+
>> +#
>> +
>> +obj-y += $(BOARD).o
> 
> Please don't use $(BOARD) here, just went and changed the other users of
> this convention.
> 
>> +/*
>> + * Routine: board_init
>> + * Description: hardware init.
>> + */
>> +int board_init(void)
>> +{
>> +gd->bd->bi_boot_params = (0x8000 + 0x100); /* boot param addr */
>> +
>> +return 0;
>> +}
> 
> CONFIG_SYS_SDRAM_BASE + 0x100
> 
>> +#define CONFIG_EXTRA_ENV_SETTINGS \
>> +"loadaddr=0x8200\0" \
>> +"fdtaddr=0x80F8\0" \
>> +"fdt_high=0x\0" \
>> +"rdaddr=0x8100\0" \
> 
> Can you use DEFAULT_LINUX_BOOT_ENV here (see u-boot-ti) ?  I really
> don't like disabling fdt relocation with it being below 128MB (I'd
> prefer what we do with DEFAULT_LINUX_BOOT_ENV and bootm_size, please see
> the various ML threads about that).

Thanks Tom,

We will fix the comments and repost v2 in a couple of days.

- -- 
Regards,
Igor.
-BEGIN PGP SIGNATURE-
Version: GnuPG v2.0.22 (GNU/Linux)

iQIcBAEBAgAGBQJTcwroAAoJEBDE8YO64EfaxQoP/2GKp8kJErMdiMZs8TdIhkGp
De1T6qw+qoTsA/91nFQUq4AMv5lfApghTgjmU/zx5oI4Uo8IPE8uMtZw9VSwTucu
Zfy1BrhGj8kbVe9u6oOf2uZiqvzyaEqdi9a/EwmB69cYu8QcQFP+nbPPMqmxgA2i
B9aUP/+54PluWcBDN0Z5btPfeRBgodyHLJlOtNWCwy4SmzgoARYbS+wrrKqiZApJ
NPYdH1wtgVcrsqiBFGILCktgVkUl4ncC7oa0MTzQfzAPKaKvbhbuwT98lhgUNcsZ
xf7QS3KSGD6/H8xZS0VTAmC5frf7/IAHzSFJsb25OMfi8gwrL8a1Nf9XA+UaAB0I
cpv53xxEoTvdAP/gmBz0sKdC0YKLvYin/OqY0PeFl1itPrKJ1CiKBXL60l0u7wkT
J4HtxsUUfl/nbZc/KbB9cnYCf57V1nnDw3KesRR2nwWaBf0dTWFiPIKCHpZjyUKg
OUinqCjRIbL6KTLEkcipPYXHuFGVNqcE3a+uRDn7jLW0+XcU2zKi04Djpn7F84TM
LXUhnJX8UC59AZuseMqi3yK26OOmTFcIOMcRDi45Zri3qsaYIi7TrD9offwlxb6/
+JEP3YROi0fm8jrC//0GUjx8idmjeZYlx0U4EDjLxKBqnABVlTjrBoiheB+rr+f0
t2HoJHgnV89Dtu26mZi5
=C1e2
-END PGP SIGNATURE-
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] powerpc/mpc85xx: SECURE BOOT- corrected CSPR settings for BSC9132QDS NAND

2014-05-13 Thread Aneesh Bansal
In case of secure boot from NAND, CSPR and FTIM settings are
same as non-secure NAND boot. CSPR0 is configured as NAND and
CSPR1 is configured as NOR.

Signed-off-by: Aneesh Bansal 
---
 include/configs/BSC9132QDS.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/configs/BSC9132QDS.h b/include/configs/BSC9132QDS.h
index 37e8725..2234874 100644
--- a/include/configs/BSC9132QDS.h
+++ b/include/configs/BSC9132QDS.h
@@ -361,7 +361,7 @@ combinations. this should be removed later
 #endif
 
 /* Set up IFC registers for boot location NOR/NAND */
-#if defined(CONFIG_NAND)
+#if defined(CONFIG_NAND) || defined(CONFIG_NAND_SECBOOT)
 #define CONFIG_SYS_CSPR0   CONFIG_SYS_NAND_CSPR
 #define CONFIG_SYS_AMASK0  CONFIG_SYS_NAND_AMASK
 #define CONFIG_SYS_CSOR0   CONFIG_SYS_NAND_CSOR
-- 
1.8.1.4

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2 6/7] mtd: nand: omap_gpmc: minor cleanup of omap_correct_data_bch

2014-05-13 Thread Stefan Roese

On 11.04.2014 09:25, Pekon Gupta wrote:

This patch tries to avoid some local pointer dereferences, by using common
local variables in omap_correct_data_bch()

Signed-off-by: Pekon Gupta 


Reviewed-by: Stefan Roese 

Thanks,
Stefan

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2 7/7] mtd: nand: omap: fix error-codes returned from omap-elm driver

2014-05-13 Thread Stefan Roese

On 11.04.2014 09:25, Pekon Gupta wrote:

This patch
  omap-elm.c: replaces -ve integer value returned during errorneous condition,
  with proper error-codes.
  omap-gpmc.c: updates omap-gpmc driver to pass error-codes returned from
  omap-elm driver to upper layers

Signed-off-by: Pekon Gupta 


Reviewed-by: Stefan Roese 

Thanks,
Stefan

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2 4/7] mtd: nand: omap_gpmc: remove unused members of 'struct nand_bch_priv'

2014-05-13 Thread Stefan Roese

On 11.04.2014 09:25, Pekon Gupta wrote:

This patch prepares to refactor 'struct nand_bch_priv' -> 'struct 
omap_nand_info'
And thus performs following clean-ups:
  - remove nand_bch_priv.type: use nand_bch_priv.ecc_scheme instead
  - remove nand_bch_priv.mode: 

Signed-off-by: Pekon Gupta 


Reviewed-by: Stefan Roese 

Thanks,
Stefan

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2 0/12] Enable LCD display on snow

2014-05-13 Thread Minkyu Kang
Dear Simon Glass,

On 13/05/14 07:45, Simon Glass wrote:
> Hi,
> 
> On 2 April 2014 17:24, Simon Glass  wrote:
>> This series adds a driver for TPS65090 and plumbs it in to get the LCD
>> working correctly on snow.
>>
>> The display driver is already present, but needs information about the
>> display to be provided in the device tree.
>>
>> The backlight also needs to be enabled - it is controlled by a FET on
>> the TPS65090. Note that the TPS65090 is controlled by the device tree
>> so will only be present if the board's device tree file specifies it.
>> At present this is only the case for snow, but other exynos5 boards will
>> use it (e.g. pit).
>>
>> Note: the TPS65090 driver was sent to the list around the middle of last
>> year but was never applied.
> 
> Is there any news on this series please?
> 

I'll check them soon.

Thanks,
Minkyu Kang.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 12/12] imx: ventana: switch to SPL

2014-05-13 Thread Tim Harvey
On Tue, May 13, 2014 at 9:58 PM, Tim Harvey  wrote:
> On Wed, May 7, 2014 at 2:29 AM, Stefano Babic  wrote:
>> Hi Tim,
>>
>> On 06/05/2014 20:18, Tim Harvey wrote:
>>>
>>> Stefano / York,
>>>
>>> While preparing for a v3 patch series of my IMX6 SPL bootloader, I
>>> find that commit dec1861be90c948ea9fb771927d3d26a994d2e20 [1] breaks
>>> the above code because gd is now needed within setup_i2c.
>>>
>>> I've always been a bit fuzzy on the order of the above calls so I dug
>>> through the code and I think I understand things better. Please
>>> correct any wrong assumptions I'm making below:
>>>  - assignment to gd should 'always' be first (before anything needs
>>> it, so why not do it first)
>>>  - arch_cpu_init() should go next as this sets up very low level
>>> CPU/SoC resources (in this case AIPS config and watchdog disable)
>>>  - board_early_init_f() should be next as that sets up any board-specific 
>>> iomux
>>>  - any additional iomux necessary for SPL should go next (I take care
>>> of i2c iomux and setup here)
>>>  - timer_init() next as you need a timer for UART and mxc i2c (for
>>> delays and busy checks)
>>>  - preloader_console_init() next as we are now able to send something
>>> over the UART (this gives me early debug for sdram config now too!)
>>>  - sdram setup goes next
>>>  - after sdram is setup, the bss can be cleared
>>>  - board_init_r - pass over to generic SPL code which will load/call
>>> an image based on boot device
>>
>> I think your analyses is correct.
>>
>>>
>>> So, if the above is correct, I should rework the above function as follows:
>>>
>>> void board_init_f(ulong dummy)
>>> {
>>> struct ventana_board_info ventana_info;
>>> int board_model;
>>>
>>> /* Set global data pointer. */
>>> gd = &gdata;
>>>
>>> /* setup AIPS and disable watchdog */
>>> arch_cpu_init();
>>>
>>> /* iomux and setup of i2c */
>>> board_early_init_f();
>>> i2c_setup_iomux();
>>>
>>> /* setup GP timer */
>>> timer_init();
>>>
>>> /* UART clocks enabled and gd valid - init serial console */
>>> preloader_console_init();
>>>
>>> /* read/validate EEPROM info to determine board model and SDRAM cfg 
>>> */
>>> board_model = read_eeprom(I2C_GSC, &ventana_info);
>>>
>>> /* provide some some default: 32bit 128MB */
>>> if (GW_UNKNOWN == board_model) {
>>> ventana_info.sdram_width = 2;
>>> ventana_info.sdram_size = 3;
>>> }
>>>
>>> /* configure MMDC for SDRAM width/size and per-model calibration */
>>> spl_dram_init(8 << ventana_info.sdram_width,
>>>   16 << ventana_info.sdram_size,
>>>   board_model);
>>>
>>> /* Clear the BSS. */
>>> memset(__bss_start, 0, __bss_end - __bss_start);
>>>
>>> /* load/boot image from boot device */
>>> board_init_r(NULL, 0);
>>> }
>>
>> It seems reasonable, go on this way.
>>
>> Regards,
>> Stefano
>>
>
> Stefano,
>
> I've just found that one of my boards fails with the above re-org.
> Strangely a board which has the same mem layout, mem width/size, CPU
> and nand does not fail.
>
> If I make the following change:

(sorry, accidentally fat fingered and sent instead of pasting).  I'll continue:

--- a/board/gateworks/gw_ventana/gw_ventana_spl.c
+++ b/board/gateworks/gw_ventana/gw_ventana_spl.c
@@ -380,9 +380,6 @@ void board_init_f(ulong dummy)
 */
memset((void *)gd, 0, sizeof(struct global_data));

-   /* setup AIPS and disable watchdog */
-   arch_cpu_init();
-
/* iomux and setup of i2c */
board_early_init_f();
i2c_setup_iomux();
@@ -407,6 +404,9 @@ void board_init_f(ulong dummy)
  16 << ventana_info.sdram_size,
  board_model);

+   /* setup AIPS and disable watchdog */
+   arch_cpu_init();
+
/* Clear the BSS. */
memset(__bss_start, 0, __bss_end - __bss_start);

Things start to work properly. I took a look at arch_cpu_init() and
the call that really needs to be moved (or just duplicated here) is
mxs_dma_init() to start APBH DMA. The failure mode of the one board is
that apparently it hangs while loading u-boot.img from NAND (which of
course uses mxs_nand and thus APBH DMA).

Anyone know what's going on here and why mxs_dma_init() needs to be
called after MMDC setup, and before clearing the BSS? I don't like
having magic placement of functions without understanding why.

Thanks,

Tim
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 12/12] imx: ventana: switch to SPL

2014-05-13 Thread Tim Harvey
On Wed, May 7, 2014 at 2:29 AM, Stefano Babic  wrote:
> Hi Tim,
>
> On 06/05/2014 20:18, Tim Harvey wrote:
>>
>> Stefano / York,
>>
>> While preparing for a v3 patch series of my IMX6 SPL bootloader, I
>> find that commit dec1861be90c948ea9fb771927d3d26a994d2e20 [1] breaks
>> the above code because gd is now needed within setup_i2c.
>>
>> I've always been a bit fuzzy on the order of the above calls so I dug
>> through the code and I think I understand things better. Please
>> correct any wrong assumptions I'm making below:
>>  - assignment to gd should 'always' be first (before anything needs
>> it, so why not do it first)
>>  - arch_cpu_init() should go next as this sets up very low level
>> CPU/SoC resources (in this case AIPS config and watchdog disable)
>>  - board_early_init_f() should be next as that sets up any board-specific 
>> iomux
>>  - any additional iomux necessary for SPL should go next (I take care
>> of i2c iomux and setup here)
>>  - timer_init() next as you need a timer for UART and mxc i2c (for
>> delays and busy checks)
>>  - preloader_console_init() next as we are now able to send something
>> over the UART (this gives me early debug for sdram config now too!)
>>  - sdram setup goes next
>>  - after sdram is setup, the bss can be cleared
>>  - board_init_r - pass over to generic SPL code which will load/call
>> an image based on boot device
>
> I think your analyses is correct.
>
>>
>> So, if the above is correct, I should rework the above function as follows:
>>
>> void board_init_f(ulong dummy)
>> {
>> struct ventana_board_info ventana_info;
>> int board_model;
>>
>> /* Set global data pointer. */
>> gd = &gdata;
>>
>> /* setup AIPS and disable watchdog */
>> arch_cpu_init();
>>
>> /* iomux and setup of i2c */
>> board_early_init_f();
>> i2c_setup_iomux();
>>
>> /* setup GP timer */
>> timer_init();
>>
>> /* UART clocks enabled and gd valid - init serial console */
>> preloader_console_init();
>>
>> /* read/validate EEPROM info to determine board model and SDRAM cfg 
>> */
>> board_model = read_eeprom(I2C_GSC, &ventana_info);
>>
>> /* provide some some default: 32bit 128MB */
>> if (GW_UNKNOWN == board_model) {
>> ventana_info.sdram_width = 2;
>> ventana_info.sdram_size = 3;
>> }
>>
>> /* configure MMDC for SDRAM width/size and per-model calibration */
>> spl_dram_init(8 << ventana_info.sdram_width,
>>   16 << ventana_info.sdram_size,
>>   board_model);
>>
>> /* Clear the BSS. */
>> memset(__bss_start, 0, __bss_end - __bss_start);
>>
>> /* load/boot image from boot device */
>> board_init_r(NULL, 0);
>> }
>
> It seems reasonable, go on this way.
>
> Regards,
> Stefano
>

Stefano,

I've just found that one of my boards fails with the above re-org.
Strangely a board which has the same mem layout, mem width/size, CPU
and nand does not fail.

If I make the following change:
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2 0/7] mtd: nand: omap: clean up of omap_elm and omap_gpmc driver

2014-05-13 Thread Gupta, Pekon
>From: Gupta, Pekon
>
>*changes v1->v2*
>[PATCH 2/7] fix for BCH4 and BCH16 ECC scheme
>[PATCH 7/7] 
> - white space-clean up
>
>
>*original v1*
>This patch series
> - removes un-wanted & redundant code in omap_elm and omap_gpmc drivers
> - refactors some private struct in omap_gpmc to make them generic across 
> ecc-schemes
> - minor code fixes and clean-ups
>This series can be as-it-is applied on u-boot 'v2014.04-rc3'.
>
>*Testing Procedure*
>Boot tested on am335x_evm (MLO -> U-boot -> kernel with filesystem flashed 
>from U-boot)
>using OMAP_ECC_BCH8_CODE_HW and OMAP_ECC_BCH8_CODE_HW_DETECTION_SW ecc-schemes
>
Ping .. 
Please review and accept this. These are simple driver clean-ups.


with regards, pekon
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [Bug Report] Patman removes author's Signed-off-by credit

2014-05-13 Thread Masahiro Yamada
Hi Simon,

I suspect there is something wrong with Patman
since commit 102061bd (patman: Avoid duplicate sign-offs).


For example, my commit log is like follows

sandbox: set sandbox's vendor to null

Because sandbox is not a real hardware, setting vendor=sandbox is
almost meaningless.

This commit sets sandbox's vendor field to '-'.

It is a good thing that it decreases one level directory hierarchy.
The files board/sandbox/sandbox/* have been moved to board/sandbox/*.

Signed-off-by: Masahiro Yamada 
Acked-by: Simon Glass 
Series-to: uboot
Series-cc: sjg
Series-version: 2
Series-changes: 2
  - Rebase


When I run "tools/patman/patman -c 1",
Patman removes all Signed-off-by credits and checkpatch.pl
reports missing Signed-off-by credit error.



1 errors, 0 warnings, 0 checks for 
0001-sandbox-set-sandbox-s-vendor-to-null.patch:
error: ,0: Missing Signed-off-by: line(s)

checkpatch.pl found 1 error(s), 0 warning(s), 0 checks(s)


I think this is worse than dupulicated signed-off-by.


Best Regards
Masahiro Yamada

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH v2] sandbox: set sandbox's vendor to null

2014-05-13 Thread Masahiro Yamada
Because sandbox is not a real hardware, setting vendor=sandbox is
almost meaningless.

This commit sets sandbox's vendor field to '-'.

It is a good thing that it decreases one level directory hierarchy.
The files board/sandbox/sandbox/* have been moved to board/sandbox/*.

Signed-off-by: Masahiro Yamada 
Acked-by: Simon Glass 
---

Changes in v2:
  - Rebase

 board/sandbox/{sandbox => }/Makefile   | 0
 board/sandbox/{sandbox => }/README.sandbox | 0
 board/sandbox/{sandbox => }/sandbox.c  | 0
 boards.cfg | 2 +-
 4 files changed, 1 insertion(+), 1 deletion(-)
 rename board/sandbox/{sandbox => }/Makefile (100%)
 rename board/sandbox/{sandbox => }/README.sandbox (100%)
 rename board/sandbox/{sandbox => }/sandbox.c (100%)

diff --git a/board/sandbox/sandbox/Makefile b/board/sandbox/Makefile
similarity index 100%
rename from board/sandbox/sandbox/Makefile
rename to board/sandbox/Makefile
diff --git a/board/sandbox/sandbox/README.sandbox b/board/sandbox/README.sandbox
similarity index 100%
rename from board/sandbox/sandbox/README.sandbox
rename to board/sandbox/README.sandbox
diff --git a/board/sandbox/sandbox/sandbox.c b/board/sandbox/sandbox.c
similarity index 100%
rename from board/sandbox/sandbox/sandbox.c
rename to board/sandbox/sandbox.c
diff --git a/boards.cfg b/boards.cfg
index 6aea684..59b5c4c 100644
--- a/boards.cfg
+++ b/boards.cfg
@@ -1182,7 +1182,7 @@ Active  powerpc ppc4xx -   xilinx 
 ppc405-generic
 Active  powerpc ppc4xx -   xilinx  ppc405-generic  
xilinx-ppc405-generic_flash   
xilinx-ppc405-generic:SYS_TEXT_BASE=0xF7F6,RESET_VECTOR_ADDRESS=0xF7FC  
  Ricardo Ribalda 

 Active  powerpc ppc4xx -   xilinx  ppc440-generic  
xilinx-ppc440-generic 
xilinx-ppc440-generic:SYS_TEXT_BASE=0x0400,RESET_VECTOR_ADDRESS=0x0410,BOOT_FROM_XMD=1
Ricardo Ribalda 
 Active  powerpc ppc4xx -   xilinx  ppc440-generic  
xilinx-ppc440-generic_flash   
xilinx-ppc440-generic:SYS_TEXT_BASE=0xF7F6,RESET_VECTOR_ADDRESS=0xF7FC  
  Ricardo Ribalda 

-Active  sandbox sandbox-   sandbox sandbox 
sandbox   - 

Simon Glass 
+Active  sandbox sandbox-   -   sandbox 
sandbox   - 

Simon Glass 
 Active  sh  sh2-   renesas rsk7203 
rsk7203   - 

Nobuhiro Iwamatsu :Nobuhiro Iwamatsu 

 Active  sh  sh2-   renesas rsk7264 
rsk7264   - 

Phil Edworthy 
 Active  sh  sh2-   renesas rsk7269 
rsk7269   - 

-
-- 
1.8.3.2

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2] Because sandbox is not a real hardware, setting vendor=sandbox is almost meaningless.

2014-05-13 Thread Masahiro Yamada
Hi Simon,

On Tue, 13 May 2014 12:13:48 -0600
Simon Glass  wrote:

> From: Masahiro Yamada 
> 
> This commit sets sandbox's vendor field to '-'.
> 
> It is a good thing that it reduces one level directory hierarchy.
> The files board/sandbox/sandbox/* have been moved to board/sandbox/*.
> 
> Signed-off-by: Masahiro Yamada 
> Signed-off-by: Simon Glass 
> (modified to apply to latest upstream)


I am not happy about this fix-up approach because the commit description
does not reflect the code any more.



> It is a good thing that it reduces one level directory hierarchy.

This sounds odd.
This patch moves arch/sandbox/lib/sandbox.c
to board/sandbox/sandbox.c,
creating board/sandbox/ directory.


We should revert the wrong commit first,
and then apply the right one.

I will send them.

Best Regards
Masahiro Yamada

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] Revert "sandbox: move source files from board/ to arch/sandbox/"

2014-05-13 Thread Masahiro Yamada
This reverts commit 258060905e04fe2eb509756ef3b37e23e220a2d6.

Conflicts:
boards.cfg

Wrong patch 25806090 was applied by accident. Revert it.

Signed-off-by: Masahiro Yamada 
Cc: Simon Glass 
---
 arch/sandbox/lib/Makefile | 2 +-
 board/sandbox/sandbox/Makefile| 7 +++
 {doc => board/sandbox/sandbox}/README.sandbox | 0
 {arch/sandbox/lib => board/sandbox/sandbox}/sandbox.c | 0
 boards.cfg| 2 +-
 5 files changed, 9 insertions(+), 2 deletions(-)
 create mode 100644 board/sandbox/sandbox/Makefile
 rename {doc => board/sandbox/sandbox}/README.sandbox (100%)
 rename {arch/sandbox/lib => board/sandbox/sandbox}/sandbox.c (100%)

diff --git a/arch/sandbox/lib/Makefile b/arch/sandbox/lib/Makefile
index 6480ebf..4c1a38d 100644
--- a/arch/sandbox/lib/Makefile
+++ b/arch/sandbox/lib/Makefile
@@ -8,4 +8,4 @@
 #
 
 
-obj-y  += interrupts.o sandbox.o
+obj-y  += interrupts.o
diff --git a/board/sandbox/sandbox/Makefile b/board/sandbox/sandbox/Makefile
new file mode 100644
index 000..a0b9880
--- /dev/null
+++ b/board/sandbox/sandbox/Makefile
@@ -0,0 +1,7 @@
+#
+# Copyright (c) 2011 The Chromium OS Authors.
+#
+# SPDX-License-Identifier: GPL-2.0+
+#
+
+obj-y  := sandbox.o
diff --git a/doc/README.sandbox b/board/sandbox/sandbox/README.sandbox
similarity index 100%
rename from doc/README.sandbox
rename to board/sandbox/sandbox/README.sandbox
diff --git a/arch/sandbox/lib/sandbox.c b/board/sandbox/sandbox/sandbox.c
similarity index 100%
rename from arch/sandbox/lib/sandbox.c
rename to board/sandbox/sandbox/sandbox.c
diff --git a/boards.cfg b/boards.cfg
index 9d90550..6aea684 100644
--- a/boards.cfg
+++ b/boards.cfg
@@ -1182,7 +1182,7 @@ Active  powerpc ppc4xx -   xilinx 
 ppc405-generic
 Active  powerpc ppc4xx -   xilinx  ppc405-generic  
xilinx-ppc405-generic_flash   
xilinx-ppc405-generic:SYS_TEXT_BASE=0xF7F6,RESET_VECTOR_ADDRESS=0xF7FC  
  Ricardo Ribalda 

 Active  powerpc ppc4xx -   xilinx  ppc440-generic  
xilinx-ppc440-generic 
xilinx-ppc440-generic:SYS_TEXT_BASE=0x0400,RESET_VECTOR_ADDRESS=0x0410,BOOT_FROM_XMD=1
Ricardo Ribalda 
 Active  powerpc ppc4xx -   xilinx  ppc440-generic  
xilinx-ppc440-generic_flash   
xilinx-ppc440-generic:SYS_TEXT_BASE=0xF7F6,RESET_VECTOR_ADDRESS=0xF7FC  
  Ricardo Ribalda 

-Active  sandbox sandbox-   - 
sandbox   - 

Simon Glass 
+Active  sandbox sandbox-   sandbox sandbox 
sandbox   - 

Simon Glass 
 Active  sh  sh2-   renesas rsk7203 
rsk7203   - 

Nobuhiro Iwamatsu :Nobuhiro Iwamatsu 

 Active  sh  sh2-   renesas rsk7264 
rsk7264   - 

Phil Edworthy 
 Active  sh  sh2-   renesas rsk7269 
rsk7269   - 

-
-- 
1.8.3.2

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH v2 1/2] mkimage : Split out and clean pbl_crc32 for use by other image types

2014-05-13 Thread Charles Manning
The crc32 used by pblimgae is NOT the same as zlib crc32.

The pbl_crc32 is useful for other purposes in mkimage so split it out.

While we are about it, clean up redundant and confusing code.

Signed-off-by: Charles Manning 
---

Unchanged from V1.

 tools/Makefile|1 +
 tools/pbl_crc32.c |   56 +
 tools/pbl_crc32.h |   13 +
 tools/pblimage.c  |   47 +---
 4 files changed, 71 insertions(+), 46 deletions(-)
 create mode 100644 tools/pbl_crc32.c
 create mode 100644 tools/pbl_crc32.h

diff --git a/tools/Makefile b/tools/Makefile
index c34df4f..2f30749 100644
--- a/tools/Makefile
+++ b/tools/Makefile
@@ -86,6 +86,7 @@ dumpimage-mkimage-objs := aisimage.o \
omapimage.o \
os_support.o \
pblimage.o \
+   pbl_crc32.o \
sha1.o \
sha256.o \
ublimage.o \
diff --git a/tools/pbl_crc32.c b/tools/pbl_crc32.c
new file mode 100644
index 000..6e6735a
--- /dev/null
+++ b/tools/pbl_crc32.c
@@ -0,0 +1,56 @@
+/*
+ * Copyright 2012 Freescale Semiconductor, Inc.
+ *
+ * Cleaned up and refactored by Charles Manning.
+ *
+ * SPDX-License-Identifier:GPL-2.0+
+ */
+#include "pblimage.h"
+
+static uint32_t crc_table[256];
+static int crc_table_valid;
+
+static void make_crc_table(void)
+{
+   uint32_t mask;
+   int i, j;
+   uint32_t poly; /* polynomial exclusive-or pattern */
+
+   if (crc_table_valid)
+   return;
+
+   /*
+* the polynomial used by PBL is 1 + x1 + x2 + x4 + x5 + x7 + x8 + x10
+* + x11 + x12 + x16 + x22 + x23 + x26 + x32.
+*/
+   poly = 0x04c11db7;
+
+   for (i = 0; i < 256; i++) {
+   mask = i << 24;
+   for (j = 0; j < 8; j++) {
+   if (mask & 0x8000)
+   mask = (mask << 1) ^ poly;
+   else
+   mask <<= 1;
+   }
+   crc_table[i] = mask;
+   }
+
+   crc_table_valid = 1;
+}
+
+uint32_t pbl_crc32(uint32_t in_crc, const char *buf, uint32_t len)
+{
+   uint32_t crc32_val;
+   int i;
+
+   make_crc_table();
+
+   crc32_val = ~in_crc;
+
+   for (i = 0; i < len; i++)
+   crc32_val = (crc32_val << 8) ^
+   crc_table[(crc32_val >> 24) ^ (*buf++ & 0xff)];
+
+   return crc32_val;
+}
diff --git a/tools/pbl_crc32.h b/tools/pbl_crc32.h
new file mode 100644
index 000..4ab55ee
--- /dev/null
+++ b/tools/pbl_crc32.h
@@ -0,0 +1,13 @@
+/*
+ * Copyright 2012 Freescale Semiconductor, Inc.
+ *
+ * SPDX-License-Identifier:GPL-2.0+
+ */
+
+#ifndef PBLCRC32_H
+#define PBLCRC32_H
+
+#include 
+uint32_t pbl_crc32(uint32_t in_crc, const char *buf, uint32_t len);
+
+#endif
diff --git a/tools/pblimage.c b/tools/pblimage.c
index ef3d7f6..6e6e801 100644
--- a/tools/pblimage.c
+++ b/tools/pblimage.c
@@ -6,6 +6,7 @@
 #include "imagetool.h"
 #include 
 #include "pblimage.h"
+#include "pbl_crc32.h"
 
 /*
  * Initialize to an invalid value.
@@ -137,52 +138,6 @@ static void pbl_parser(char *name)
fclose(fd);
 }
 
-static uint32_t crc_table[256];
-
-static void make_crc_table(void)
-{
-   uint32_t mask;
-   int i, j;
-   uint32_t poly; /* polynomial exclusive-or pattern */
-
-   /*
-* the polynomial used by PBL is 1 + x1 + x2 + x4 + x5 + x7 + x8 + x10
-* + x11 + x12 + x16 + x22 + x23 + x26 + x32.
-*/
-   poly = 0x04c11db7;
-
-   for (i = 0; i < 256; i++) {
-   mask = i << 24;
-   for (j = 0; j < 8; j++) {
-   if (mask & 0x8000)
-   mask = (mask << 1) ^ poly;
-   else
-   mask <<= 1;
-   }
-   crc_table[i] = mask;
-   }
-}
-
-unsigned long pbl_crc32(unsigned long crc, const char *buf, uint32_t len)
-{
-   uint32_t crc32_val = 0x;
-   uint32_t xor = 0x0;
-   int i;
-
-   make_crc_table();
-
-   for (i = 0; i < len; i++)
-   crc32_val = (crc32_val << 8) ^
-   crc_table[(crc32_val >> 24) ^ (*buf++ & 0xff)];
-
-   crc32_val = crc32_val ^ xor;
-   if (crc32_val < 0) {
-   crc32_val += 0x;
-   crc32_val += 1;
-   }
-   return crc32_val;
-}
-
 static uint32_t reverse_byte(uint32_t val)
 {
uint32_t temp;
-- 
1.7.9.5

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH v2 2/2] tools: Refactor mxsimage to use pbl_crc32

2014-05-13 Thread Charles Manning
Both pblimage and mxsimage use the same crc algorithm, so refactor.

Signed-off-by: Charles Manning 
---
Changes for V2:
  - Cast buffer pointers when calling pbl_crc32().

 tools/mxsimage.c |   32 +++-
 1 file changed, 7 insertions(+), 25 deletions(-)

diff --git a/tools/mxsimage.c b/tools/mxsimage.c
index 045b35a..81c7f2d 100644
--- a/tools/mxsimage.c
+++ b/tools/mxsimage.c
@@ -19,6 +19,7 @@
 
 #include "imagetool.h"
 #include "mxsimage.h"
+#include "pbl_crc32.h"
 #include 
 
 
@@ -231,29 +232,6 @@ static int sb_aes_reinit(struct sb_image_ctx *ictx, int 
enc)
 }
 
 /*
- * CRC32
- */
-static uint32_t crc32(uint8_t *data, uint32_t len)
-{
-   const uint32_t poly = 0x04c11db7;
-   uint32_t crc32 = 0x;
-   unsigned int byte, bit;
-
-   for (byte = 0; byte < len; byte++) {
-   crc32 ^= data[byte] << 24;
-
-   for (bit = 8; bit > 0; bit--) {
-   if (crc32 & (1UL << 31))
-   crc32 = (crc32 << 1) ^ poly;
-   else
-   crc32 = (crc32 << 1);
-   }
-   }
-
-   return crc32;
-}
-
-/*
  * Debug
  */
 static void soprintf(struct sb_image_ctx *ictx, const char *fmt, ...)
@@ -998,7 +976,9 @@ static int sb_build_command_load(struct sb_image_ctx *ictx,
 
ccmd->load.address  = dest;
ccmd->load.count= cctx->length;
-   ccmd->load.crc32= crc32(cctx->data, cctx->length);
+   ccmd->load.crc32= pbl_crc32(0,
+   (const char *)cctx->data,
+   cctx->length);
 
cctx->size = sizeof(*ccmd) + cctx->length;
 
@@ -1834,7 +1814,9 @@ static int sb_verify_command(struct sb_image_ctx *ictx,
EVP_DigestUpdate(&ictx->md_ctx, cctx->data, asize);
sb_aes_crypt(ictx, cctx->data, cctx->data, asize);
 
-   if (ccmd->load.crc32 != crc32(cctx->data, asize)) {
+   if (ccmd->load.crc32 != pbl_crc32(0,
+ (const char *)cctx->data,
+ asize)) {
fprintf(stderr,
"ERR: SB LOAD command payload CRC32 
invalid!\n");
return -EINVAL;
-- 
1.7.9.5

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH v2 0/2] mkimage: Refactor and clean up 32-bit crcs

2014-05-13 Thread Charles Manning
As a step towards adding an image signer for socfpga, this set of patches
breaks out the common crc code that is used in both pblimage and mxsimage.

Not that this is NOT the same algorithm as the crc generated by zlib.

The new function has been named pbl_crc32.

This patch set just changes patch 1/2: casting pointers in mxsimage.c

Charles Manning (2):
  mkimage : Split out and clean pbl_crc32 for use by other image types
  tools: Refactor mxsimage to use pbl_crc32

 tools/Makefile|1 +
 tools/mxsimage.c  |   32 +++---
 tools/pbl_crc32.c |   56 +
 tools/pbl_crc32.h |   13 +
 tools/pblimage.c  |   47 +---
 5 files changed, 78 insertions(+), 71 deletions(-)
 create mode 100644 tools/pbl_crc32.c
 create mode 100644 tools/pbl_crc32.h

-- 
1.7.9.5

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] sandbox: move source files from board/ to arch/sandbox/

2014-05-13 Thread Masahiro Yamada
Hi Simon,

On Tue, 13 May 2014 09:18:22 -0600
Simon Glass  wrote:

> Hi Masahiro,
> 
> On 29 April 2014 22:53, Masahiro Yamada  wrote:
> >
> > Hi Simon,
> >
> > > > With this change, you can decrease one level hierarchy.
> > > >
> > > > board/sandbox/sandbox/sandbox.c
> > > > ->  board/sandbox/sandbox.c
> > >
> > > That seems better to me. Will you update your patch?
> > >
> >
> > Sure.
> > Could you check this?
> > http://patchwork.ozlabs.org/patch/344051/
> 
> 
> Sorry I applied the wrong patch - the other one is not showing up in
> patchwork for me. I'll see if I can figure it out.

After u-boot-x86/tom was merged,
I cleared those patches out.

Please feel free to revert this patch and apply the other one.

Best Regards
Masahiro Yamada

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [RFC PATCH v0 2/4] cmd_bootm.c: Add 'booti' for ARM64 Linux kernel Images

2014-05-13 Thread Rob Herring
On Mon, May 5, 2014 at 10:26 AM, Tom Rini  wrote:
> The default format for arm64 Linux kernels is the "Image" format,
> described in Documentation/arm64/booting.txt.  This, along with an
> optional gzip compression on top is all that is generated by default.
> The Image format has a magic number within the header for verification,
> a text_offset where the Image must be run from and reserved fields.
>
> This does not support automatic detection of a gzip compressed image.
>
> Signed-off-by: Tom Rini 
> ---
>  README |1 +
>  common/cmd_bootm.c |  139 
> 
>  2 files changed, 140 insertions(+)
>
> diff --git a/README b/README
> index 12758dc..e7c8b2e 100644
> --- a/README
> +++ b/README
> @@ -946,6 +946,7 @@ The following options need to be configured:
> CONFIG_CMD_BMP  * BMP support
> CONFIG_CMD_BSP  * Board specific commands
> CONFIG_CMD_BOOTD  bootd
> +   CONFIG_CMD_BOOTI* ARM64 Linux kernel Image support

As I mentioned at ELC, I would prefer to see bootz and booti combined
so we have a single "boot a plain kernel image" command. I'd rather
see the command be smart enough to figure what the image is rather
than the user have to figure out the right command. Of course with
this argument, then everything should be a bootm command. IIRC, there
would be some challenges to add bootz or booti functionality into
bootm.

> CONFIG_CMD_CACHE* icache, dcache
> CONFIG_CMD_CLK  * clock command support
> CONFIG_CMD_CONSOLEconinfo
> diff --git a/common/cmd_bootm.c b/common/cmd_bootm.c
> index c243a5b..3b9635c 100644
> --- a/common/cmd_bootm.c
> +++ b/common/cmd_bootm.c
> @@ -1929,3 +1929,142 @@ U_BOOT_CMD(
> "boot Linux zImage image from memory", bootz_help_text
>  );
>  #endif /* CONFIG_CMD_BOOTZ */
> +
> +#ifdef CONFIG_CMD_BOOTI
> +/* See Documentation/arm64/booting.txt in the Linux kernel */
> +struct Image_header {
> +   uint32_tcode0;  /* Executable code */
> +   uint32_tcode1;  /* Executable code */
> +   uint64_ttext_offset;/* Image load offset */
> +   uint64_tres0;   /* reserved */
> +   uint64_tres1;   /* reserved */
> +   uint64_tres2;   /* reserved */
> +   uint64_tres3;   /* reserved */
> +   uint64_tres4;   /* reserved */
> +   uint32_tmagic;  /* Magic number */
> +   uint32_tres5;
> +};
> +
> +#define LINUX_ARM64_IMAGE_MAGIC0x644d5241
> +/* XXX: Hack 16MB image size for now */
> +#define HACK_ARM64_IMAGE_SIZE  (16 << 20)
> +
> +static int booti_setup(bootm_headers_t *images)
> +{
> +   struct Image_header *ih;
> +   uint64_t dst;
> +
> +   ih = (struct Image_header *)map_sysmem(images->ep, 0);
> +
> +   if (ih->magic != LINUX_ARM64_IMAGE_MAGIC) {
> +   puts("Bad Linux ARM64 Image magic!\n");
> +   return 1;
> +   }
> +
> +   /*
> +* If we are not at the correct run-time location, set the new
> +* correct location and then move the image there.
> +*/
> +   dst = gd->bd->bi_dram[0].start + ih->text_offset;
> +   if (images->ep != dst) {
> +   void *src;
> +
> +   debug("Moving Image from 0x%lx to 0x%llx\n", images->ep, dst);
> +
> +   src = (void *)images->ep;
> +   images->ep = dst;
> +   memmove((void *)dst, src, HACK_ARM64_IMAGE_SIZE);

Don't you at least know the load size or the decompressed size? Only
the .bss size is missing and you only need that to have some checks
for overlapping images.

Rob
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [RFC PATCH v0 3/4] vexpress_aemv8a.h: Clean up the config

2014-05-13 Thread Rob Herring
On Mon, May 5, 2014 at 10:26 AM, Tom Rini  wrote:
> - Drop DEBUG
> - Drop defines we can use the default of.
> - Drop CONFIG_GICV3 and related defines, we aren't using interrupts and
>   do not need to configure the interrupt controller.
> - Provide a larger malloc pool.
> - Correct default locations for kernel / initrd / device tree
>
> Signed-off-by: Tom Rini 
> ---

[...]

> -#define CONFIG_GICV3
> -
>  /*#define CONFIG_ARMV8_SWITCH_TO_EL1*/
>
>  /*#define CONFIG_SYS_GENERIC_BOARD*/
> @@ -94,20 +90,11 @@
>  /* Generic Timer Definitions */
>  #define COUNTER_FREQUENCY  (0x180) /* 24MHz */
>
> -/* Generic Interrupt Controller Definitions */
> -#ifdef CONFIG_GICV3
> -#define GICD_BASE  (0x2f00)
> -#define GICR_BASE  (0x2f10)
> -#else
> -#define GICD_BASE  (0x2C001000)
> -#define GICC_BASE  (0x2C002000)
> -#endif

IIRC, these should be needed if secure to non-secure switching is done.

>  #define CONFIG_SYS_MEMTEST_START   V2M_BASE
>  #define CONFIG_SYS_MEMTEST_END (V2M_BASE + 0x8000)
>
>  /* Size of malloc() pool */
> -#define CONFIG_SYS_MALLOC_LEN  (CONFIG_ENV_SIZE + 128 * 1024)
> +#define CONFIG_SYS_MALLOC_LEN  (CONFIG_ENV_SIZE + (8 << 20))

This seems like another candidate for some sane default value.

>  /* SMSC91C111 Ethernet Configuration */
>  #define CONFIG_SMC91
> @@ -121,7 +108,6 @@
>  #define CONFIG_CONS_INDEX  0
>
>  #define CONFIG_BAUDRATE115200
> -#define CONFIG_SYS_BAUDRATE_TABLE  { 9600, 19200, 38400, 57600, 115200 }
>  #define CONFIG_SYS_SERIAL0 V2M_UART0
>  #define CONFIG_SYS_SERIAL1 V2M_UART1
>
> @@ -166,10 +152,9 @@
>
>  /* Initial environment variables */
>  #define CONFIG_EXTRA_ENV_SETTINGS  \
> -   "kernel_addr=0x20\0"\
> -   "initrd_addr=0xa0\0"\
> -   "initrd_size=0x200\0"   \
> -   "fdt_addr=0x10\0"   \
> +   "kernel_addr=0x8008\0"  \
> +   "initrd_addr=0x9000\0"  \
> +   "fdt_addr=0x8800\0" \

Don't we want the _r variants here?

Rob
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] Please pull u-boot-ti/master

2014-05-13 Thread Tom Rini
Hey,

The following changes since commit d2a3e911390f9fc4d8c0ee4b3c7fc75f4fd3fd19:

  Merge branch 'u-boot/master' (2014-05-09 11:50:14 +0200)

are available in the git repository at:


  git://git.denx.de/u-boot-ti.git master

for you to fetch changes up to 97eeae1a0735fc17351f10b9978e966392badedc:

  mtd: nand: omap_gpmc: Fix update of read_ecc in oob (2014-05-13 19:48:17 
-0400)


Ash Charles (1):
  am335x: pepper: Add Gumstix Pepper AM335x-based machine

Belisko Marek (1):
  mtd: nand: omap_gpmc: Fix update of read_ecc in oob

Christian Riesch (1):
  arm, davinci: Use CONFIG_SPL_PAD_TO for padding the SPL in an ais image

Dmitry Lifshitz (3):
  ARM: OMAP5: add UART4 support
  ARM: OMAP5: Power: add LDO2 support for Palmas driver
  ARM: OMAP5: add CKO buffer control mask

Egli, Samuel (7):
  siemens: cosmetic: remove unused and rename defines
  siemens: update DDR3 parameters for dxr2
  siemens: add led cmd for flexible LED control
  siemens: change LED indication in DFU mode
  siemens: cosmetic: rename project_dir
  siemens:cosmetic, dxr2: rename dxr2 to draco
  siemens, draco: add new target

Khoronzhuk, Ivan (1):
  config: k2hk_evm: Add generic board support

Robert Nelson (2):
  omap3_beagle: use omap3-beagle-xm-ab.dtb for the xM AB revision
  omap3_beagle: xM A/B validate new dtb exits in file system

Yegor Yefremov (1):
  am33xx: add SSC enable macro

 Makefile |2 +-
 arch/arm/include/asm/arch-am33xx/clock.h |2 +
 arch/arm/include/asm/arch-omap5/clock.h  |3 +
 arch/arm/include/asm/arch-omap5/omap.h   |1 +
 board/gumstix/pepper/Makefile|   13 ++
 board/gumstix/pepper/board.c |  226 ++
 board/gumstix/pepper/board.h |   19 +++
 board/gumstix/pepper/mux.c   |   78 +++
 board/siemens/common/board.c |   46 +-
 board/siemens/{dxr2 => draco}/Makefile   |0
 board/siemens/{dxr2 => draco}/board.c|  112 ---
 board/siemens/{dxr2 => draco}/board.h|   26 ++--
 board/siemens/{dxr2 => draco}/mux.c  |2 +-
 board/siemens/pxm2/board.c   |   10 +-
 board/siemens/rut/board.c|2 +-
 boards.cfg   |4 +-
 drivers/mtd/nand/omap_gpmc.c |2 +-
 include/configs/da850evm.h   |1 +
 include/configs/draco.h  |   92 
 include/configs/dxr2.h   |4 +-
 include/configs/k2hk_evm.h   |1 +
 include/configs/omap3_beagle.h   |   10 +-
 include/configs/pepper.h |   99 +
 include/configs/pxm2.h   |2 +-
 include/configs/siemens-am33x-common.h   |8 +-
 include/palmas.h |4 +
 26 files changed, 686 insertions(+), 83 deletions(-)
 create mode 100644 board/gumstix/pepper/Makefile
 create mode 100644 board/gumstix/pepper/board.c
 create mode 100644 board/gumstix/pepper/board.h
 create mode 100644 board/gumstix/pepper/mux.c
 rename board/siemens/{dxr2 => draco}/Makefile (100%)
 rename board/siemens/{dxr2 => draco}/board.c (63%)
 rename board/siemens/{dxr2 => draco}/board.h (71%)
 rename board/siemens/{dxr2 => draco}/mux.c (99%)
 create mode 100644 include/configs/draco.h
 create mode 100644 include/configs/pepper.h

Thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [U-Boot, 2/2] omap3_beagle: xM A/B validate new dtb exits in file system

2014-05-13 Thread Tom Rini
On Fri, May 02, 2014 at 03:15:03PM -0500, robertcnel...@gmail.com wrote:

> From: Robert Nelson 
> 
> Fall back to previous dtb used when omap3-beagle-xm-ab.dtb doesn't exist in 
> file system
> 
> Signed-off-by: Robert Nelson 
> CC: Tom Rini 
> CC: Nishanth Menon 
> Acked-by: Tom Rini 

Applied to u-boot-ti/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [U-Boot,2/2] siemens, draco: add new target

2014-05-13 Thread Tom Rini
On Mon, May 05, 2014 at 04:50:44PM +0200, Egli, Samuel wrote:


> Signed-off-by: Samuel Egli 
> Reviewed-by: Roger Meier 
> Cc: Heiko Schocher 
> Cc: Wolfgang Denk 

Applied to u-boot-ti/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [U-Boot,3/4] ARM: OMAP5: add CKO buffer control mask

2014-05-13 Thread Tom Rini
On Sun, Apr 27, 2014 at 01:17:27PM +0300, Dmitry Lifshitz wrote:

> Add CKOBUFFER_CLK_EN bit mask enabling FREF_XTAL_CLK clock.
> 
> Signed-off-by: Dmitry Lifshitz 

Applied to u-boot-ti/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [U-Boot, 1/2] siemens:cosmetic, dxr2: rename dxr2 to draco

2014-05-13 Thread Tom Rini
On Mon, May 05, 2014 at 04:50:43PM +0200, Egli, Samuel wrote:

> The actual board name is draco and dxr2 is the target name.
> In the future we'll have different targets based on draco board.
> All changes are purely non-functional and basically rename dxr2
> to draco.
> 
> One style fix in board.c that existed already before.
> 
> Signed-off-by: Samuel Egli 
> Reviewed-by: Roger Meier 
> Cc: Heiko Schocher 
> Cc: Wolfgang Denk 

Applied to u-boot-ti/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] arm, davinci: Use CONFIG_SPL_PAD_TO for padding the SPL in an ais image

2014-05-13 Thread Tom Rini
On Wed, May 07, 2014 at 10:16:28AM +0200, Christian Riesch wrote:

> The commits
> 
> commit b7b5f1a16ca66dfdd817e7339f0e263a5b9f2758
> Author: Albert ARIBAUD 
> da850evm, da850_am18xxevm: convert to CONFIG_SPL_MAX_FOOTPRINT
> 
> and
> 
> commit e7497891e34efe5cb2b3a3dc7c6c096c012ede28
> Author: Albert ARIBAUD 
> cam_enc_4xx: convert to CONFIG_SPL_MAX_FOOTPRINT
> 
> replaced CONFIG_SPL_MAX_SIZE by CONFIG_SPL_MAX_FOOTPRINT. However,
> CONFIG_SPL_MAX_SIZE is used in the Makefile for padding the SPL
> when preparing an u-boot.ais image. By removing CONFIG_SPL_MAX_SIZE
> said commits broke the ais image of the da850evm and cam_enc_4xx
> configurations.
> 
> This patch converts the u-boot.ais target to use CONFIG_SPL_PAD_TO
> instead of CONFIG_SPL_MAX_SIZE for padding the SPL and adds
> a #define CONFIG_SPL_PAD_TO where it is required.
> 
> Signed-off-by: Christian Riesch 
> Reported-by: Tom Taylor 
> Cc: Sudhakar Rajashekhara 
> Cc: Heiko Schocher 
> Cc: Albert ARIBAUD 

Applied to u-boot-ti/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [U-Boot, 2/4] ARM: OMAP5: Power: add LDO2 support for Palmas driver

2014-05-13 Thread Tom Rini
On Sun, Apr 27, 2014 at 01:17:26PM +0300, Dmitry Lifshitz wrote:


> Add defines required to turn on LDO2 regulator.
> 
> Signed-off-by: Dmitry Lifshitz 

Applied to u-boot-ti/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] mtd: nand: omap_gpmc: Fix update of read_ecc in oob

2014-05-13 Thread Tom Rini
On Fri, Apr 25, 2014 at 12:00:07PM +0200, Belisko Marek wrote:

> We need to flip only one bit not assign.
> 
> Signed-off-by: Marek Belisko 
> Acked-by: Pekon Gupta 

Applied to u-boot-ti/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [U-Boot, 1/2] omap3_beagle: use omap3-beagle-xm-ab.dtb for the xM AB revision

2014-05-13 Thread Tom Rini
On Fri, May 02, 2014 at 03:14:39PM -0500, robertcnel...@gmail.com wrote:

> From: Robert Nelson 
> 
> As of v3.15-rc3, omap3-beagle-xm-ab.dtb now exists with the usb hub (ehci) 
> enabled.
> 
> For older kernels versions, cherry pick from mainline:
> ef78f3869c37c480f1d58462a760a40dabc823f4
> 
> Signed-off-by: Robert Nelson 
> CC: Tom Rini 
> CC: Nishanth Menon 

Applied to u-boot-ti/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [U-Boot,1/4] ARM: OMAP5: add UART4 support

2014-05-13 Thread Tom Rini
On Sun, Apr 27, 2014 at 01:17:25PM +0300, Dmitry Lifshitz wrote:

> Add UART4 base address.
> 
> Signed-off-by: Dmitry Lifshitz 

Applied to u-boot-ti/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [U-Boot,5/5] siemens: cosmetic: rename project_dir

2014-05-13 Thread Tom Rini
On Thu, Apr 24, 2014 at 05:57:56PM +0200, Egli, Samuel wrote:

> Signed-off-by: Samuel Egli 
> Cc: Roger Meier 
> Cc: Heiko Schocher 
> Cc: Wolfgang Denk 

Applied to u-boot-ti/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [U-Boot, 3/5] siemens: add led cmd for flexible LED control

2014-05-13 Thread Tom Rini
On Thu, Apr 24, 2014 at 05:57:54PM +0200, Egli, Samuel wrote:

> * remove setting LED in user button function.
>We want to decouple reading user button and setting LED. This
>two things need to be done independently.
> 
>  * led cmd can be used to control LEDs that are defined in board file
>having a led cmd, one can easily set LEDs in u-boot shell. For
>example bootcmd can be extended to disable status LED before
>loading kernel.
> 
> Signed-off-by: Samuel Egli 
> Cc: Roger Meier 
> Cc: Heiko Schocher 
> Cc: Wolfgang Denk 

Applied to u-boot-ti/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [U-Boot, 4/5] siemens: change LED indication in DFU mode

2014-05-13 Thread Tom Rini
On Thu, Apr 24, 2014 at 05:57:55PM +0200, Egli, Samuel wrote:

> In order to have the same LED indication like in another product
> when ready for updating, enable only red led and disable status
> LED when entering DFU mode.
> 
> The status LED is only switched off when defined in board file.
> 
> Signed-off-by: Samuel Egli 
> Cc: Roger Meier 
> Cc: Heiko Schocher 
> Cc: Wolfgang Denk 

Applied to u-boot-ti/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [U-Boot,v2] am33xx: add SSC enable macro

2014-05-13 Thread Tom Rini
On Sat, Apr 19, 2014 at 10:12:18PM +0200, Yegor Yefremov wrote:

> Signed-off-by: Yegor Yefremov 

Applied to u-boot-ti/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [U-Boot, 1/5] siemens: cosmetic: remove unused and rename defines

2014-05-13 Thread Tom Rini
On Thu, Apr 24, 2014 at 05:57:52PM +0200, Egli, Samuel wrote:

> For dxr2 board DXR2_IOCTRL_VAL is set by data in EEPROM. In pxm2
> board it does not make sense to have dxr2 as prefix. Replace it with
> more meaningful DDR prefix.
> 
> Signed-off-by: Samuel Egli 
> Cc: Pascal Bach 
> Cc: Roger Meier 
> Cc: Heiko Schocher 
> Cc: Wolfgang Denk 

Applied to u-boot-ti/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [U-Boot,2/5] siemens: update DDR3 parameters for dxr2

2014-05-13 Thread Tom Rini
On Thu, Apr 24, 2014 at 05:57:53PM +0200, Egli, Samuel wrote:

> * add parameters for factory and print them at start up to
>facilitate control of right DDR3 settings in EEPROM.
> 
>  * cosmetic changes in a couple of printfs
> 
> Signed-off-by: Samuel Egli 
> Cc: Roger Meier 
> Cc: Heiko Schocher 
> Cc: Wolfgang Denk 

Applied to u-boot-ti/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [U-Boot, v2] am335x: pepper: Add Gumstix Pepper AM335x-based machine

2014-05-13 Thread Tom Rini
On Wed, May 07, 2014 at 08:24:11AM -0700, Ash Charles wrote:

> This adds the Gumstix Pepper[1] single-board computer based on the
> TI AM335x processor. Schematics are available [2].
> 
> [1] https://store.gumstix.com/index.php/products/344/
> [2] https://pubs.gumstix.com/boards/PEPPER/
> 
> Signed-off-by: Ash Charles 

Applied to u-boot-ti/master (with a minor change to board.c to drop a
warning), thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2] omap4: duovero: Add Gumstix DuoVero machine.

2014-05-13 Thread Tom Rini
On Wed, May 07, 2014 at 09:02:25AM -0700, Ash Charles wrote:

> This adds the Gumstix DuoVero machine [1].  This is a OMAP4430-based
> computer-on-module (COM aka SOM) that can be mounted on various
> expansion boards with different peripherals.
[snip]
> + sr32((void *)CM_L3INIT_HSUSBHOST_CLKCTRL, 0, 32, utmi_clk);

We've dropped sr32 in a88e55c, please update and resent a v3, thanks!
(everything else seems fine).

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] armv8 relocation questions

2014-05-13 Thread Darwin Rambo

I'm hoping someone can help answer these questions about armv8 relocation.

The CONFIG_SYS_TEXT_BASE seems to be be usually setup to a decent amount 
of alignment. For the purposes of this discussion, let's say it would 
normally be 0x8800 and all is well. The relocation address moves to 
near the end of memory, to say, 0xfffa8000. So far so good.


Now let's say I want to shift the image a bit so that I can add a small 
32-byte header required by a previous bootloader. So I set 
CONFIG_SYS_TEXT_BASE to 0x8820, and the relocated address is still 
0xfffa8000 and the relocated vectors should be at 0xfffa9000. The image 
crashes so after some debugging, I find that the code appears to be 
relocated fine, but some sections have symbols that are not relocated 
properly. The vectors try to relocate to 0xfffa8fe0 and rodata.str1.1 
printf format strings are also 0x20 off. There are likely other offset 
sections with issues as well.


The relocation offset is 0x77fa7fe0 due to the calculations in 
arch/arm/lib/board.c. Simplifying, they look like this:


addr = CONFIG_SYS_SDRAM_BASE + gd->ram_size;

/* round down to next 4 kB limit */
addr &= ~(4096 - 1);
debug("Top of RAM usable for U-Boot at: %08lx\n", addr);

/*
 * reserve memory for U-Boot code, data & bss
 * round down to next 4 kB limit
 */
addr -= gd->mon_len;
addr &= ~(4096 - 1);

addr += 0x20; // hack to adjust relocaddr to aligned address...



gd->relocaddr = addr;
gd->start_addr_sp = addr_sp;
gd->reloc_off = addr - _TEXT_BASE;
debug("relocation Offset is: %08lx\n", gd->reloc_off);


Since _TEXT_BASE is 0x8820 and addr is 0xfffa8000, the reloc_off is 
a number like 0x77fa7fe0.


Now if I add 0x20 to 'addr' above just before the , relocaddr 
becomes 0x77fa8000 and the relocation works perfectly and no more 
crashes happen.


So my question - is the CONFIG_SYS_TEXT_BASE alignment requirement 
related to to any assumptions in the linker itself about image base 
alignment, specifically referring to creation of the rela.dyn sections 
and their use for image relocation?


A related question is if CONFIG_SYS_TEXT_BASE needs to be at a specific 
alignment. The maximum alignment in the armv8 code base is ".align 11" 
which I believe means 0x800 or 2048.


Note that an armv7 target appears to relocate properly with smaller 
offsets such as 0x20.


Thanks.

Best Regards,
Darwin Rambo


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Please pull mpc85xx

2014-05-13 Thread Tom Rini
On Tue, May 13, 2014 at 11:12:53AM -0500, York Sun wrote:

> Tom,
> 
> The following changes since commit 69c0d323e325f48f3124fb0696de3d68fcbcae8e:
> 
>   kbuild: Fix trailing whitespaces (2014-05-12 16:31:51 -0400)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-mpc85xx.git master
> 
> for you to fetch changes up to 8ad5d45e0044f5e64dbe7c6bd4e6aa0121a26b23:
> 
>   boards.cfg: fix a configuration error of ep8248 board (2014-05-13 11:00:01 
> -0500)
> 
> 
> Alexander Graf (4):
>   PPC 85xx QEMU: Always assume 1 core
>   PPC 85xx QEMU: Don't use HID1
>   powerpc/mpc85xx: Update TLB CAMs in relocated mode
>   PPC 85xx QEMU: Make a generic board file
> 
> Aneesh Bansal (2):
>   powerpc/mpc85xx: SECURE BOOT- secure boot target for t1040rdb
>   powerpc/mpc85xx: SECURE BOOT- Add secure boot target for T2080RDB
> 
> Chunhe Lan (1):
>   powerpc/85xx: Add T4240RDB board support
> 
> Masahiro Yamada (1):
>   boards.cfg: fix a configuration error of ep8248 board
> 
> Shaohui Xie (1):
>   powerpc/fman/memac: use default MDIO_HOLD value
> 
> Shaveta Leekha (2):
>   board/freescale/common: ZM7300 driver
>   board/b4qds: VID support
> 
> Shengzhou Liu (2):
>   powerpc/t208x: enable errata A006261, A006593, A006379
>   powerpc/85xx: add T4080 SoC support
> 
> Stefan Bigler (2):
>   kmp204x: Add support for the unit LEDs
>   kmp204x: handle dip-switch for factory settings
> 
> Valentin Longchamp (6):
>   kmp204x: selftest/factory test pin support
>   kmp204x: update the CONFIG_PRAM and CONFIG_KM_RESERVED_PRAM defines
>   kmp204x: complete the reset sequence and PRST configuration
>   kmp204x: update the RCW
>   kmp204x: add workaround for A-004849
>   kmp204x: enable the errata command
> 
> York Sun (6):
>   powerpc/mpc8572ds: Increase u-boot size to 768KB
>   powerpc/mpc85xx: Ignore FDT pointer for non-QEMU in cpu_init_early_f()
>   powerpc/freescale: Change the return value of mac_read_from_eeprom()
>   powerpc/mpc85xx: Fix boot_flag for calling board_init_f()
>   powerpc/mpc86xx: Fix boot_flag for calling board_init_f()
>   powerpc/freescale: Convert selected boards to generic board architecture
> 
> Zhao Qiang (1):
>   qe: disable qe when qe-ucode fails to be uploaded for "deep sleep"
> 
>  arch/powerpc/cpu/mpc85xx/Makefile   |2 +
>  arch/powerpc/cpu/mpc85xx/cpu.c  |   24 +
>  arch/powerpc/cpu/mpc85xx/cpu_init.c |   17 +-
>  arch/powerpc/cpu/mpc85xx/cpu_init_early.c   |2 +
>  arch/powerpc/cpu/mpc85xx/speed.c|3 +-
>  arch/powerpc/cpu/mpc85xx/spl_minimal.c  |4 +-
>  arch/powerpc/cpu/mpc85xx/start.S|4 +-
>  arch/powerpc/cpu/mpc85xx/t4240_ids.c|2 +
>  arch/powerpc/cpu/mpc85xx/t4240_serdes.c |2 +-
>  arch/powerpc/cpu/mpc86xx/start.S|1 +
>  arch/powerpc/cpu/mpc8xxx/cpu.c  |5 +-
>  arch/powerpc/include/asm/config_mpc85xx.h   |   19 +-
>  arch/powerpc/include/asm/fsl_errata.h   |5 +
>  arch/powerpc/include/asm/fsl_secure_boot.h  |3 +-
>  arch/powerpc/include/asm/immap_85xx.h   |6 +-
>  arch/powerpc/include/asm/processor.h|1 +
>  board/freescale/b4860qds/b4860qds.c |  240 +
>  board/freescale/common/Makefile |1 +
>  board/freescale/common/sys_eeprom.c |6 +-
>  board/freescale/common/zm7300.c |  235 +
>  board/freescale/common/zm7300.h |   22 +
>  board/freescale/qemu-ppce500/qemu-ppce500.c |   20 +
>  board/freescale/t4rdb/Makefile  |   12 +
>  board/freescale/t4rdb/ddr.c |  118 +
>  board/freescale/t4rdb/ddr.h |   78 +++
>  board/freescale/t4rdb/eth.c |  146 ++
>  board/freescale/t4rdb/law.c |   28 +
>  board/freescale/t4rdb/pci.c |   23 +
>  board/freescale/t4rdb/t4240rdb.c|  125 +
>  board/freescale/t4rdb/t4_pbi.cfg|   31 ++
>  board/freescale/t4rdb/t4_rcw.cfg|7 +
>  board/freescale/t4rdb/t4rdb.h   |   18 +
>  board/freescale/t4rdb/tlb.c |  111 
>  board/keymile/kmp204x/kmp204x.c |   75 ++-
>  board/keymile/kmp204x/kmp204x.h |3 +
>  board/keymile/kmp204x/pbi.cfg   |   43 +-
>  board/keymile/kmp204x/pci.c |   17 +-
>  board/keymile/kmp204x/qrio.c|   31 +-
>  board/keymile/kmp204x/rcw_kmp204x.cfg   |2 +-
>  boards.cfg  |7 +-
>  drivers/mmc/fsl_esdhc.c |2 +-
>  drivers/net/fm/Makefile |1 +
>  drivers/net/fm/memac_phy.c  |8 +-
>  drivers/qe/qe.c |9 +-
>  include/common.h|5 +-
>  in

[U-Boot] pull request for u-boot-tegra/master into ARM/master

2014-05-13 Thread Tom Warren
Albert,

Please pull u-boot-tegra/master into ARM/master. Thanks!

./MAKEALL -s tegra AOK, checkpatch.pl is OK, and ./MAKEALL -a arm only
shows failures that were already present in ARM/master.

The following changes since commit d2a3e911390f9fc4d8c0ee4b3c7fc75f4fd3fd19:

  Merge branch 'u-boot/master' (2014-05-09 11:50:14 +0200)

are available in the git repository at:

  git://git.denx.de/u-boot-tegra.git master

for you to fetch changes up to 2364e151e432b4ccf32dc9e6147121253d4ff86d:

  ARM: tegra: use a CPU freq that all SKUs can support (2014-05-13 10:41:32
-0700)


Stephen Warren (10):
  ARM: tegra: set CONFIG_SYS_MMC_MAX_DEVICE
  ARM: tegra: fix CPU VDD comment in Tegra30 CPU init code
  ARM: tegra: allow pinmux mux option not to be set by init tables
  ARM: tegra: add GPIO initialization table function
  ARM: tegra: add function to enable input clamping on tristate
  ARM: tegra: make use of GPIO init table on Jetson TK1
  ARM: tegra: clamp inputs on Jetson TK1
  ARM: tegra: update Venice2 pinmux
  ARM: tegra: Venice2 pinmux spreadsheet updates
  ARM: tegra: use a CPU freq that all SKUs can support

 arch/arm/cpu/arm720t/tegra-common/cpu.c|  10 +-
 arch/arm/cpu/arm720t/tegra30/cpu.c |  23 +-
 arch/arm/cpu/tegra-common/pinmux-common.c  |  19 +
 arch/arm/include/asm/arch-tegra/gpio.h |  20 +
 arch/arm/include/asm/arch-tegra/pinmux.h   |   5 +
 arch/arm/include/asm/arch-tegra/tegra_mmc.h|   2 -
 arch/arm/include/asm/arch-tegra114/pinmux.h|   1 +
 arch/arm/include/asm/arch-tegra124/pinmux.h|   1 +
 arch/arm/include/asm/arch-tegra20/pinmux.h |   1 +
 arch/arm/include/asm/arch-tegra30/pinmux.h |   1 +
 board/nvidia/jetson-tk1/jetson-tk1.c   |   6 +
 board/nvidia/jetson-tk1/pinmux-config-jetson-tk1.h | 256 +++---
 board/nvidia/venice2/pinmux-config-venice2.h   | 567
++---
 board/nvidia/venice2/venice2.c |  19 +-
 drivers/gpio/tegra_gpio.c  |  20 +
 drivers/mmc/tegra_mmc.c|  13 +-
 include/configs/beaver.h   |   3 +
 include/configs/cardhu.h   |   3 +
 include/configs/tegra-common.h |   7 +
 19 files changed, 565 insertions(+), 412 deletions(-)
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [U-Boot, 2/2] mkimage: Refactor mxsimage to use common crc32 code

2014-05-13 Thread Tom Rini
On Tue, May 13, 2014 at 07:27:18PM +1200, Charles Manning wrote:
> On Tuesday 13 May 2014 08:26:44 Tom Rini wrote:
> > On Tue, May 06, 2014 at 10:46:46AM +1200, Charles Manning wrote:
> > > mxsimage uses the same crc32 function as pblimage.
> > >
> > > Signed-off-by: Charles Manning 
> > > Acked-by: Stefano Babic 
> >
> > This introduces warnings:
> > tools/mxsimage.c: In function ‘sb_build_command_load’:
> > tools/mxsimage.c:979:2: warning: pointer targets in passing argument 2 of
> > ‘pbl_crc32’ differ in signedness [-Wpointer-sign] tools/pbl_crc32.h:11:10:
> > note: expected ‘const char *’ but argument is of type ‘uint8_t *’
> > u-boot/tools/mxsimage.c: In function ‘sb_verify_command’:
> > tools/mxsimage.c:1815:3: warning: pointer targets in passing argument 2 of
> > ‘pbl_crc32’ differ in signedness [-Wpointer-sign] tools/pbl_crc32.h:11:10:
> > note: expected ‘const char *’ but argument is of type ‘uint8_t *’
> 
> Thanks for the feedback.
> 
> Will it be acceptable to do the following:
> 
> *Change the called function from uint8_t * to const uint8_t
> *Cast the const char * pointer to const uint8_t *.

Sounds right.

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [RFC PATCH v2 13/13] tegra: Convert tegra GPIO driver to use driver model

2014-05-13 Thread Stephen Warren
On 05/09/2014 11:28 AM, Simon Glass wrote:
> This is an implementation of GPIOs for Tegra that uses driver model. It has
> been tested on trimslice and also using the new iotrace feature.
> 
> The implementation uses a top-level GPIO device (which has no actual GPIOS).
> Under this all the banks are created as separate GPIO devices.

I still believe that there is a single Tegra GPIO controller, and this
should be represented by a single device in the U-Boot device model.

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] mx25pdk: Add generic board support

2014-05-13 Thread Fabio Estevam
Signed-off-by: Fabio Estevam 
---
 include/configs/mx25pdk.h | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/include/configs/mx25pdk.h b/include/configs/mx25pdk.h
index aff2419..cfaf6ca 100644
--- a/include/configs/mx25pdk.h
+++ b/include/configs/mx25pdk.h
@@ -26,6 +26,8 @@
 #define CONFIG_SETUP_MEMORY_TAGS
 #define CONFIG_INITRD_TAG
 
+#define CONFIG_SYS_GENERIC_BOARD
+
 #define CONFIG_MACH_TYPE   MACH_TYPE_MX25_3DS
 
 /* Size of malloc() pool */
-- 
1.8.3.2

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [RFC PATCH v2 12/13] tegra: Enable driver model

2014-05-13 Thread Stephen Warren
On 05/09/2014 11:28 AM, Simon Glass wrote:
> Enable driver model for Tegra boards.

Seems fine to me,
Acked-by: Stephen Warren 
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [RFC PATCH v2 05/13] tegra: dts: Bring in GPIO bindings from linux

2014-05-13 Thread Stephen Warren
On 05/09/2014 11:28 AM, Simon Glass wrote:
> These files are taken from Linux 3.14.

> diff --git a/arch/arm/dts/tegra124.dtsi b/arch/arm/dts/tegra124.dtsi

> - gpio: gpio@6000d000 {
> + gpio: gpio@0,6000d000 {
>   compatible = "nvidia,tegra124-gpio", "nvidia,tegra30-gpio";
> - reg = <0x6000d000 0x1000>;
...
> + reg = <0x0 0x6000d000 0x0 0x1000>;

That reg value and node name are for #address-cells=<2>,
#size-cells=<2>. However, those properties are both =<1> in the U-Boot
tegra124.dtsi, and I'm not sure whether U-Boot even supports 64-bit DT
address values yet?

It'd probably be simplest to just use 32-bit address/size values in
tegra124.dtsi now, and import the conversion to 64-bit later?

Aside from that, this patch looks, so assuming the change to those two
lines above is reverted:

Acked-by: Stephen Warren 
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [RFC PATCH v2 04/13] Makefile: Support include files for .dts files

2014-05-13 Thread Stephen Warren
On 05/09/2014 11:28 AM, Simon Glass wrote:
> Linux supports this, and if we are to have compatible device tree files,
> U-Boot should also.
> 
> Avoid giving the device tree files access to U-Boot's include/ directory.
> Only include/dt-bindings is accessible.

Acked-by: Stephen Warren 

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Support for Tegra Jetson TK1

2014-05-13 Thread Stephen Warren
On 05/13/2014 06:06 AM, Hamza Farooq wrote:
> Hello,
> 
> does the current u-boot for Tegra branch fully support Jetson TK1 board?

Yes, I believe that what's in u-boot.git branch master should be enough
to boot the board. I've sent various patches since then to clean up the
pinmux configuration etc., but I don't think they're required for U-Boot
to work.

> The recent commits somehow look incomplete (I was not able to find
> RAM/Flash configuration).

I'm not sure what you mean by "RAM/Flash configuration". Anyway, it's
likely that the following links will be useful:

Background information on the Tegra boot process, flashing tools, etc.:
http://http.download.nvidia.com/tegra-public-appnotes/

The simplest way to flash a Tegra device for mainline usage:
https://github.com/NVIDIA/tegra-uboot-flasher-scripts
(See README-developer.txt to get the source and build it, then
README-user.txt to run it)

I hope that answers all your questions. Let me know if you have more.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] patman: Don't request full names from get_maintainer

2014-05-13 Thread Simon Glass
Hi Doug,

On 18 April 2014 15:32, Doug Anderson  wrote:
> Simon,
>
> On Fri, Apr 18, 2014 at 1:43 PM, Simon Glass  wrote:
>> Hi Doug,
>>
>> On 17 April 2014 12:47, Doug Anderson  wrote:
>>>
>>> The Linux get_maintainer.pl can often produce a whole lot of results.
>>> As a result you'll sometimes blow your CC field over 1024 characters
>>> and that can cause listservs to reject your message.
>>>
>>> As a stopgap, call get_maintainer.pl with "--non" so it doesn't
>>> include real names.  This will dramatically reduce the number of
>>> characters.
>>>
>>> Signed-off-by: Doug Anderson 
>>> ---
>>>
>>>  tools/patman/get_maintainer.py | 2 +-
>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/tools/patman/get_maintainer.py b/tools/patman/get_maintainer.py
>>> index 00b4939..a5160bc 100644
>>> --- a/tools/patman/get_maintainer.py
>>> +++ b/tools/patman/get_maintainer.py
>>> @@ -43,5 +43,5 @@ def GetMaintainer(fname, verbose=False):
>>>  print "WARNING: Couldn't find get_maintainer.pl"
>>>  return []
>>>
>>> -stdout = command.Output(get_maintainer, '--norolestats', fname)
>>> +stdout = command.Output(get_maintainer, '--norolestats', '--non', 
>>> fname)
>>>  return stdout.splitlines()
>>> --
>>> 1.9.1.423.g4596e3a
>>>
>>
>> Good to avoid this problem, but I wonder if we should check the size
>> and re-run the command if too long? That way we can keep names on the
>> thread when it is possible.
>>
>> Also, why is there a limit on CC - is that a limitation described in
>> the RFC, or just certain mailers? Can we get around it by specifying
>> multiple Cc tags?
>
> I believe that we're actually fighting a heuristic that the mailing
> list servers have to avoid spammers.  In the past I've seen my mail
> get through to list servers that weren't run by kernel.org but _not_
> the the ones at kernel.org.  Kind of like how if you want to write
> code that's for generic exynos 5 hardware you need to use exynos_5yyy
> and _not_ replace the y with x (because that would be adult content!)
>
> We could possibly only run the heuristic if the CC list was > 800
> characters?  I'd hesitate to getting too close to 1024 since I think
> there are instances where git will itself add a CC and that might blow
> us out.

Yes that seems reasonable - that way we get the names most of the time.

>
> ...another option is to just forget this patch and force people to add
> this to ".get_maintainer.conf"

I think the previous option is better.

Regards,
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Signed-off-by rules for U-Boot

2014-05-13 Thread Stefano Babic
Hi Stephen,

On 13/05/2014 20:03, Stephen Warren wrote:
> Does U-Boot follow the same Signed-off-by procedures as the Linux
> kernel, or is s-o-b only relevant in U-Boot for the initial patch
> submitter, and not maintainers?
> 

Yes, we discussed this in a previous thread, here Wolfgang's answer:

http://lists.denx.de/pipermail/u-boot/2012-October/137248.html


> I ask because there are a bunch of patches of mine that got applied
> without the maintainer adding their S-o-b when applying them, which
> would be very unusual for Linux, but perhaps it's normal for U-Boot?
> 

It should be normal for U-Boot. I never add my s-o-b to patches that I
only merge. Only if I change something I add my s-o-b. If this is not
the right rule, I am doing wrong since a lot of time ;-).

> Some likely non-exhaustive examples are:
> 
> 2813006fecda usb: ci_udc: allow multiple buffer allocs per ep
> 66b88b07cdab usb: ums: add error handling for failed registration
> 264e0e591bbd config: enable CMD_BMP when API+LCD is enabled
> 53eb768dfb97 aes: make apply_cbc_chain_data non-static
> 
> Yet others did get a maintainer s-o-b:
> 
> d381294aef4a ARM: tegra: pack pinmux data tables tighter
> def816a2ba87 mmc: set rca to 1 for MMC cards
> 91171091c673 ARM: tegra: make use of negative ENV_OFFSET on NVIDIA...
> 6ab6a650a418 disk: initialize name/part fields when returning a whole...
> c75942c7b792 sf: winbond: fix page_size
> 
> It seems like it mostly varies by maintainer.

Agree we should be consistent. I am not aware that the original rule was
changed, even if the behavior is different as the Linux kernel.

Regards,
Stefano


-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] patman: Suppress duplicate signoffs only for real patches

2014-05-13 Thread Simon Glass
There is an unfortunate bug in the signoff suppression logic. The first
pass is performed with 'git log', and all signoffs are added to the
supression set, such that the second time (when processing the real
patches) we always suppress the signoffs.

Correct this by only suppressing signoffs in the second pass.

Signed-off-by: Simon Glass 
---

 tools/patman/patchstream.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/tools/patman/patchstream.py b/tools/patman/patchstream.py
index 9f5682c..3228719 100644
--- a/tools/patman/patchstream.py
+++ b/tools/patman/patchstream.py
@@ -275,7 +275,8 @@ class PatchStream:
 
 # Suppress duplicate signoffs
 elif signoff_match:
-if self.commit.CheckDuplicateSignoff(signoff_match.group(1)):
+if (self.is_log or
+self.commit.CheckDuplicateSignoff(signoff_match.group(1))):
 out = [line]
 
 # Well that means this is an ordinary line
-- 
1.9.1.423.g4596e3a

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH v2] Because sandbox is not a real hardware, setting vendor=sandbox is almost meaningless.

2014-05-13 Thread Simon Glass
From: Masahiro Yamada 

This commit sets sandbox's vendor field to '-'.

It is a good thing that it reduces one level directory hierarchy.
The files board/sandbox/sandbox/* have been moved to board/sandbox/*.

Signed-off-by: Masahiro Yamada 
Signed-off-by: Simon Glass 
(modified to apply to latest upstream)

---

Changes in v2:
- Update to apply on current master

 arch/sandbox/lib/Makefile | 2 +-
 board/sandbox/Makefile| 7 +++
 {doc => board/sandbox}/README.sandbox | 0
 {arch/sandbox/lib => board/sandbox}/sandbox.c | 0
 boards.cfg| 2 +-
 5 files changed, 9 insertions(+), 2 deletions(-)
 create mode 100644 board/sandbox/Makefile
 rename {doc => board/sandbox}/README.sandbox (100%)
 rename {arch/sandbox/lib => board/sandbox}/sandbox.c (100%)

diff --git a/arch/sandbox/lib/Makefile b/arch/sandbox/lib/Makefile
index 6480ebf..4c1a38d 100644
--- a/arch/sandbox/lib/Makefile
+++ b/arch/sandbox/lib/Makefile
@@ -8,4 +8,4 @@
 #
 
 
-obj-y  += interrupts.o sandbox.o
+obj-y  += interrupts.o
diff --git a/board/sandbox/Makefile b/board/sandbox/Makefile
new file mode 100644
index 000..a0b9880
--- /dev/null
+++ b/board/sandbox/Makefile
@@ -0,0 +1,7 @@
+#
+# Copyright (c) 2011 The Chromium OS Authors.
+#
+# SPDX-License-Identifier: GPL-2.0+
+#
+
+obj-y  := sandbox.o
diff --git a/doc/README.sandbox b/board/sandbox/README.sandbox
similarity index 100%
rename from doc/README.sandbox
rename to board/sandbox/README.sandbox
diff --git a/arch/sandbox/lib/sandbox.c b/board/sandbox/sandbox.c
similarity index 100%
rename from arch/sandbox/lib/sandbox.c
rename to board/sandbox/sandbox.c
diff --git a/boards.cfg b/boards.cfg
index 9d90550..59b5c4c 100644
--- a/boards.cfg
+++ b/boards.cfg
@@ -1182,7 +1182,7 @@ Active  powerpc ppc4xx -   xilinx 
 ppc405-generic
 Active  powerpc ppc4xx -   xilinx  ppc405-generic  
xilinx-ppc405-generic_flash   
xilinx-ppc405-generic:SYS_TEXT_BASE=0xF7F6,RESET_VECTOR_ADDRESS=0xF7FC  
  Ricardo Ribalda 

 Active  powerpc ppc4xx -   xilinx  ppc440-generic  
xilinx-ppc440-generic 
xilinx-ppc440-generic:SYS_TEXT_BASE=0x0400,RESET_VECTOR_ADDRESS=0x0410,BOOT_FROM_XMD=1
Ricardo Ribalda 
 Active  powerpc ppc4xx -   xilinx  ppc440-generic  
xilinx-ppc440-generic_flash   
xilinx-ppc440-generic:SYS_TEXT_BASE=0xF7F6,RESET_VECTOR_ADDRESS=0xF7FC  
  Ricardo Ribalda 

-Active  sandbox sandbox-   - 
sandbox   - 

Simon Glass 
+Active  sandbox sandbox-   -   sandbox 
sandbox   - 

Simon Glass 
 Active  sh  sh2-   renesas rsk7203 
rsk7203   - 

Nobuhiro Iwamatsu :Nobuhiro Iwamatsu 

 Active  sh  sh2-   renesas rsk7264 
rsk7264   - 

Phil Edworthy 
 Active  sh  sh2-   renesas rsk7269 
rsk7269   - 

-
-- 
1.9.1.423.g4596e3a

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] gcc 4.3.5 and uboot 1.1.4.54 not working, uboot crashed

2014-05-13 Thread George Wang
Hello.

I just compiled uboot from gcc 4.3.5. When the uboot is running, it crashed
at ath_reg_wr_nf(), which is the SPI register write function. If I comment
it out, the uboot will keep going until it crashed again at
ath_pci_write_config(), which is PCI register write. Both of two call
#define ath_reg_wr(_phys, _val) do {\
ath_reg_wr_nf(_phys, _val); \
ath_reg_rd(_phys);  \


 #define ath_reg_wr_nf(_phys, _val) \
((*(volatile ath_reg_t *)KSEG1ADDR(_phys)) = (_val))


However When I use gcc 4.2.4 to compile the same uboot code, it works fun.
Any idea about what's going on here? I assume it must be related to BUS
configuration issue...

Thanks,

-George
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] Support for Tegra Jetson TK1

2014-05-13 Thread Hamza Farooq
Hello,

does the current u-boot for Tegra branch fully support Jetson TK1 board?
The recent commits somehow look incomplete (I was not able to find
RAM/Flash configuration). If it is the case, I'll be thankful if I could be
pointed in the right direction. It is obvious that I am not very
experienced with uboot but I am planning to learn/develop it for my Master
thesis.

PS: This is my first post ever in a mailing list. In case I am doing
something wrong, please point me to the right direction.

Thank you all for your patience

Regards,
Hamza Farooq
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] Signed-off-by rules for U-Boot

2014-05-13 Thread Stephen Warren
Does U-Boot follow the same Signed-off-by procedures as the Linux
kernel, or is s-o-b only relevant in U-Boot for the initial patch
submitter, and not maintainers?

I ask because there are a bunch of patches of mine that got applied
without the maintainer adding their S-o-b when applying them, which
would be very unusual for Linux, but perhaps it's normal for U-Boot?

Some likely non-exhaustive examples are:

2813006fecda usb: ci_udc: allow multiple buffer allocs per ep
66b88b07cdab usb: ums: add error handling for failed registration
264e0e591bbd config: enable CMD_BMP when API+LCD is enabled
53eb768dfb97 aes: make apply_cbc_chain_data non-static

Yet others did get a maintainer s-o-b:

d381294aef4a ARM: tegra: pack pinmux data tables tighter
def816a2ba87 mmc: set rca to 1 for MMC cards
91171091c673 ARM: tegra: make use of negative ENV_OFFSET on NVIDIA...
6ab6a650a418 disk: initialize name/part fields when returning a whole...
c75942c7b792 sf: winbond: fix page_size

It seems like it mostly varies by maintainer.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] boards.cfg: fix a configuration error of ep8248 board

2014-05-13 Thread York Sun
On 04/29/2014 10:55 PM, Masahiro Yamada wrote:
> "make ep8248_config" fails with an error like this:
> 
> $ make ep8248_config
> make: *** [ep8248_config] Error 1
> 
> Its cause is that there are two entries for "ep8248".
> 
> The first is around line 652 of boards.cfg. (as Active)
> 
> The second appears around line 1230. (as Orphan)
> 
> This bug was accidentally introduced by commit e7e90901.
> But it is not the author's fault. He just intended to change
> IDS8247 board.
> 
> The commiter added ep8248 entry by mistake when he resolved a conflict.
> 
> Signed-off-by: Masahiro Yamada 
> Cc: Heiko Schocher 
> Cc: Kim Phillips 
> ---

Applied to u-boot-mpc85xx/master.

Thanks,

York


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH V2] usb: ci_udc: parse QTD before over-writing it

2014-05-13 Thread Stephen Warren
From: Stephen Warren 

ci_udc only allocates a single QTD structure per EP. All data needs to be
extracted from the DTD prior to calling ci_ep_submit_next_request(), since
that fills the QTD with next transaction's parameters. Fix
handle_ep_complete() to extract the transaction (remaining) length before
kicking off the next transaction.

In practice, this only causes writes to UMS devices to fail for me. I may
have tested the final versions of my previous ci_udc patch only with
reads. More recently, I had patches applied locally that allocated a QTD
per USB request rather than per USB EP, although since that doesn't give
any performance benefit, I'm dropping those.

Fixes: 2813006fecda ("usb: ci_udc: allow multiple buffer allocs per ep")
Signed-off-by: Stephen Warren 
---
v2: Fix typo in commit description.
---
 drivers/usb/gadget/ci_udc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/usb/gadget/ci_udc.c b/drivers/usb/gadget/ci_udc.c
index 290863d61fc9..9cd003636a44 100644
--- a/drivers/usb/gadget/ci_udc.c
+++ b/drivers/usb/gadget/ci_udc.c
@@ -424,6 +424,7 @@ static void handle_ep_complete(struct ci_ep *ep)
item = ci_get_qtd(num, in);
ci_invalidate_qtd(num);
 
+   len = (item->info >> 16) & 0x7fff;
if (item->info & 0xff)
printf("EP%d/%s FAIL info=%x pg0=%x\n",
   num, in ? "in" : "out", item->info, item->page0);
@@ -435,7 +436,6 @@ static void handle_ep_complete(struct ci_ep *ep)
if (!list_empty(&ep->queue))
ci_ep_submit_next_request(ep);
 
-   len = (item->info >> 16) & 0x7fff;
ci_req->req.actual = ci_req->req.length - len;
ci_debounce(ci_req, in);
 
-- 
1.8.1.5

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2] powerpc/t208x: enable errata A006261, A006593, A006379

2014-05-13 Thread York Sun
On 04/23/2014 10:10 PM, Shengzhou Liu wrote:
> Enable errata A006261, A006593, A006379 for T208x.
> Additionally enable CONFIG_CMD_ERRATA for T2080RDB.
> 
> Signed-off-by: Shengzhou Liu 
> ---
> v2: rebase to latest 'master', and update has_erratum_a006261.
> 


Applied to u-boot-mpc85xx/master.

Thanks,

York


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 2/2] powerpc/mpc85xx: SECURE BOOT- Add secure boot target for T2080RDB

2014-05-13 Thread York Sun
On 04/22/2014 04:47 AM, Aneesh Bansal wrote:
> Secure Boot Target is added for T2080RDB
> 
> Changes:
> For Secure boot, CPC is configured as SRAM and used as house
> keeping area which needs to be disabled.
> So CONFIG_SYS_CPC_REINIT_F is defined for CONFIG_T2080RDB.
> 
> Signed-off-by: Aneesh Bansal 
> ---


Applied to u-boot-mpc85xx/master.

Thanks,

York


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v4] qe: disable qe when qe-ucode fails to be uploaded for "deep sleep"

2014-05-13 Thread York Sun
On 04/30/2014 03:45 AM, Zhao Qiang wrote:
> when qe-ucode fails to be uploaded, "deep sleep" will hang.
> if there is no qe-ucode, disable qe module for platforms
> which support "deep sleep"
> 
> Signed-off-by: Zhao Qiang 
> ---
> Changes for v2:
>   - add "#ifdef CONFIG_MPC85xx"
> Changes for v3:
>   - modify commit msg
> Changes for v4:
>   - modify "#ifdef CONFIG_MPC85xx" to "#ifdef CONFIG_DEEP_SLEEP"
> 


Applied to u-boot-mpc85xx/master.

Thanks,

York


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 2/2] board/b4qds: VID support

2014-05-13 Thread York Sun
On 04/11/2014 03:42 AM, Shaveta Leekha wrote:
> The fuse status register provides the values from on-chip
> voltage ID efuses programmed at the factory.
> These values define the voltage requirements for
> the chip. u-boot reads FUSESR and translates the values
> into the appropriate commands to set the voltage output
> value of an external voltage regulator.
> 
> B4860QDS has a PowerOne ZM7300 programmable digital Power
> Manager which is programmed as per the value read from
> the fuses.
> 
> Reference for this code is taken from t4qds VID implementation.
> 
> Signed-off-by: Shaveta Leekha 
> Signed-off-by: Poonam Aggrwal 
> ---
>

Applied to u-boot-mpc85xx/master.

Thanks,

York


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 0/5] PPC 85xx QEMU: Additional fixes

2014-05-13 Thread York Sun
On 04/30/2014 12:21 PM, Alexander Graf wrote:
> After the initial round of the e500 QEMU machine patches got merged
> I realized that we were missing 2 pieces of the puzzle to make everything
> great:
> 
>   - KVM support
>   - generic board file
> 
> With this patch set I can successfully run u-boot inside of KVM on an e500v2
> as well as an e500mc machine. I also no longer get warnings that I should
> convert the board to the generic board mechanism.
> 
> 
> Alex
> 
> Alexander Graf (5):
>   PPC 85xx QEMU: Always assume 1 core
>   PPC 85xx QEMU: Don't use HID1
>   powerpc/mpc85xx: Update TLB CAMs in relocated mode
>   PPC 85xx QEMU: Make a generic board file


Applied to u-boot-mpc85xx/master.

Thanks,

York

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 1/2] board/freescale/common: ZM7300 driver

2014-05-13 Thread York Sun
On 04/11/2014 03:42 AM, Shaveta Leekha wrote:
> Adds Support for PowerOne ZM7300 voltage regulator.
> This device is available on some Freescale Boards like B4860QDS
> and has to be programmed to adjust the voltage on the board.
> 
> The device is accessible via I2C interface.
> 
> Signed-off-by: Shaveta Leekha 
> Signed-off-by: Poonam Aggrwal 
> ---

Applied to u-boot-mpc85xx/master.

Thanks,

York


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 0/8] kmp204x patches for 2014.07

2014-05-13 Thread York Sun
On 04/30/2014 08:01 AM, Valentin Longchamp wrote:
> Things have stabilized quite a lot on the u-boot front for the Keymile
> kmp204x/kmcoge4 board. There are a few remaining patches that should get
> merged to have a complete support of the architecture/board in u-boot.
> These include:
>  - support for some additional functions of the QRIO board reset/control
>CPLD (LEDs, Factory Test signal)
>  - better reset sequence according to the tests/problems noticed during
>the development
>  - an update to the RCW (some external interrupts are unused and set as
>GPIOs now)
>  - an additionnal workaround in the PBL file (with an implementation
>enhancement for a previous one).
> 
> 
> Stefan Bigler (2):
>   kmp204x: Add support for the unit LEDs
>   kmp204x: handle dip-switch for factory settings
> 
> Valentin Longchamp (6):
>   kmp204x: selftest/factory test pin support
>   kmp204x: update the CONFIG_PRAM and CONFIG_KM_RESERVED_PRAM defines
>   kmp204x: complete the reset sequence and PRST configuration
>   kmp204x: update the RCW
>   kmp204x: add workaround for A-004849
>   kmp204x: enable the errata command
> 
>  board/keymile/kmp204x/kmp204x.c   | 75 
> ++-
>  board/keymile/kmp204x/kmp204x.h   |  3 ++
>  board/keymile/kmp204x/pbi.cfg | 43 +---
>  board/keymile/kmp204x/pci.c   | 17 
>  board/keymile/kmp204x/qrio.c  | 29 +-
>  board/keymile/kmp204x/rcw_kmp204x.cfg |  2 +-
>  include/configs/km/kmp204x-common.h   | 17 
>  7 files changed, 151 insertions(+), 35 deletions(-)
> 

Applied to u-boot-mpc85xx/master.

Thanks,

York

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2] powerpc/85xx: add T4080 SoC support

2014-05-13 Thread York Sun
On 04/25/2014 03:31 AM, Shengzhou Liu wrote:
> The T4080 SoC is a low-power version of the T4160.
> T4080 combines 4 dual-threaded Power Architecture e6500
> cores with single cluster and two memory complexes.
> 
> Signed-off-by: Shengzhou Liu 
> ---
> v2: add more comments.
> 

Applied to u-boot-mpc85xx/master.

Thanks,

York


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] powerpc/fman/memac: use default MDIO_HOLD value

2014-05-13 Thread York Sun
On 04/22/2014 05:21 AM, shh@gmail.com wrote:
> From: Shaohui Xie 
> 
> Current driver uses a Maximum value for MDIO_HOLD when doing 10G MDIO
> access, this is due to an errata A-006260 on T4 rev1.0 which is fixed
> on rev2.0, so remove the maximum value to use the default value for rev2.0.
> 
> Signed-off-by: Shaohui Xie 
> ---


Applied to u-boot-mpc85xx/master.

Thanks,

York

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 1/2] powerpc/mpc85xx: SECURE BOOT- secure boot target for t1040rdb

2014-05-13 Thread York Sun
On 04/22/2014 04:46 AM, Aneesh Bansal wrote:
> T1040RDB.h file is removed and a unified file T104xRDB.h is created.
> Hence macro CONFIG_T1040 is renamed to CONFIG_T104x.
> 
> Signed-off-by: Gaurav Kumar Rana 
> Signed-off-by: Aneesh Bansal 
> ---

Applied to u-boot-mpc85xx/master.

Thanks,

York


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] powerpc/85xx: Add T4240RDB board support

2014-05-13 Thread York Sun
On 04/14/2014 05:42 AM, Chunhe Lan wrote:
> T4240RDB board Specification
> 
> Memory subsystem:
>6GB DDR3
>128MB NOR flash
>2GB NAND flash
> 
> Ethernet:
>Eight 1G SGMII ports
>Four 10Gbps SFP+ ports
> 
> PCIe:
>Two PCIe slots
> 
> USB:
>Two USB2.0 Type A ports
> 
> SDHC:
>One SD-card port
> 
> SATA:
>One SATA port
> 
> UART:
>Dual RJ45 ports
> 
> Signed-off-by: Chunhe Lan 
> 


Applied to u-boot-mpc85xx/master.

Thanks,

York


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] Please pull mpc85xx

2014-05-13 Thread York Sun
Tom,

I forgot to add my "Reviewed-by" for this set. Let me know if I have to fix 
them.

York


On 05/13/2014 11:12 AM, York Sun wrote:
> Tom,
> 
> The following changes since commit 69c0d323e325f48f3124fb0696de3d68fcbcae8e:
> 
>   kbuild: Fix trailing whitespaces (2014-05-12 16:31:51 -0400)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-mpc85xx.git master
> 
> for you to fetch changes up to 8ad5d45e0044f5e64dbe7c6bd4e6aa0121a26b23:
> 
>   boards.cfg: fix a configuration error of ep8248 board (2014-05-13 11:00:01 
> -0500)
> 
> 
> Alexander Graf (4):
>   PPC 85xx QEMU: Always assume 1 core
>   PPC 85xx QEMU: Don't use HID1
>   powerpc/mpc85xx: Update TLB CAMs in relocated mode
>   PPC 85xx QEMU: Make a generic board file
> 
> Aneesh Bansal (2):
>   powerpc/mpc85xx: SECURE BOOT- secure boot target for t1040rdb
>   powerpc/mpc85xx: SECURE BOOT- Add secure boot target for T2080RDB
> 
> Chunhe Lan (1):
>   powerpc/85xx: Add T4240RDB board support
> 
> Masahiro Yamada (1):
>   boards.cfg: fix a configuration error of ep8248 board
> 
> Shaohui Xie (1):
>   powerpc/fman/memac: use default MDIO_HOLD value
> 
> Shaveta Leekha (2):
>   board/freescale/common: ZM7300 driver
>   board/b4qds: VID support
> 
> Shengzhou Liu (2):
>   powerpc/t208x: enable errata A006261, A006593, A006379
>   powerpc/85xx: add T4080 SoC support
> 
> Stefan Bigler (2):
>   kmp204x: Add support for the unit LEDs
>   kmp204x: handle dip-switch for factory settings
> 
> Valentin Longchamp (6):
>   kmp204x: selftest/factory test pin support
>   kmp204x: update the CONFIG_PRAM and CONFIG_KM_RESERVED_PRAM defines
>   kmp204x: complete the reset sequence and PRST configuration
>   kmp204x: update the RCW
>   kmp204x: add workaround for A-004849
>   kmp204x: enable the errata command
> 
> York Sun (6):
>   powerpc/mpc8572ds: Increase u-boot size to 768KB
>   powerpc/mpc85xx: Ignore FDT pointer for non-QEMU in cpu_init_early_f()
>   powerpc/freescale: Change the return value of mac_read_from_eeprom()
>   powerpc/mpc85xx: Fix boot_flag for calling board_init_f()
>   powerpc/mpc86xx: Fix boot_flag for calling board_init_f()
>   powerpc/freescale: Convert selected boards to generic board architecture
> 
> Zhao Qiang (1):
>   qe: disable qe when qe-ucode fails to be uploaded for "deep sleep"
> 
>  arch/powerpc/cpu/mpc85xx/Makefile   |2 +
>  arch/powerpc/cpu/mpc85xx/cpu.c  |   24 +
>  arch/powerpc/cpu/mpc85xx/cpu_init.c |   17 +-
>  arch/powerpc/cpu/mpc85xx/cpu_init_early.c   |2 +
>  arch/powerpc/cpu/mpc85xx/speed.c|3 +-
>  arch/powerpc/cpu/mpc85xx/spl_minimal.c  |4 +-
>  arch/powerpc/cpu/mpc85xx/start.S|4 +-
>  arch/powerpc/cpu/mpc85xx/t4240_ids.c|2 +
>  arch/powerpc/cpu/mpc85xx/t4240_serdes.c |2 +-
>  arch/powerpc/cpu/mpc86xx/start.S|1 +
>  arch/powerpc/cpu/mpc8xxx/cpu.c  |5 +-
>  arch/powerpc/include/asm/config_mpc85xx.h   |   19 +-
>  arch/powerpc/include/asm/fsl_errata.h   |5 +
>  arch/powerpc/include/asm/fsl_secure_boot.h  |3 +-
>  arch/powerpc/include/asm/immap_85xx.h   |6 +-
>  arch/powerpc/include/asm/processor.h|1 +
>  board/freescale/b4860qds/b4860qds.c |  240 +
>  board/freescale/common/Makefile |1 +
>  board/freescale/common/sys_eeprom.c |6 +-
>  board/freescale/common/zm7300.c |  235 +
>  board/freescale/common/zm7300.h |   22 +
>  board/freescale/qemu-ppce500/qemu-ppce500.c |   20 +
>  board/freescale/t4rdb/Makefile  |   12 +
>  board/freescale/t4rdb/ddr.c |  118 +
>  board/freescale/t4rdb/ddr.h |   78 +++
>  board/freescale/t4rdb/eth.c |  146 ++
>  board/freescale/t4rdb/law.c |   28 +
>  board/freescale/t4rdb/pci.c |   23 +
>  board/freescale/t4rdb/t4240rdb.c|  125 +
>  board/freescale/t4rdb/t4_pbi.cfg|   31 ++
>  board/freescale/t4rdb/t4_rcw.cfg|7 +
>  board/freescale/t4rdb/t4rdb.h   |   18 +
>  board/freescale/t4rdb/tlb.c |  111 
>  board/keymile/kmp204x/kmp204x.c |   75 ++-
>  board/keymile/kmp204x/kmp204x.h |3 +
>  board/keymile/kmp204x/pbi.cfg   |   43 +-
>  board/keymile/kmp204x/pci.c |   17 +-
>  board/keymile/kmp204x/qrio.c|   31 +-
>  board/keymile/kmp204x/rcw_kmp204x.cfg   |2 +-
>  boards.cfg  |7 +-
>  drivers/mmc/fsl_esdhc.c |2 +-
>  drivers/net/fm/Makefile |1 +
>  drivers/net/fm/memac_phy.c  |8 +-
>  drivers/qe/qe.c   

[U-Boot] Please pull mpc85xx

2014-05-13 Thread York Sun
Tom,

The following changes since commit 69c0d323e325f48f3124fb0696de3d68fcbcae8e:

  kbuild: Fix trailing whitespaces (2014-05-12 16:31:51 -0400)

are available in the git repository at:

  git://git.denx.de/u-boot-mpc85xx.git master

for you to fetch changes up to 8ad5d45e0044f5e64dbe7c6bd4e6aa0121a26b23:

  boards.cfg: fix a configuration error of ep8248 board (2014-05-13 11:00:01 
-0500)


Alexander Graf (4):
  PPC 85xx QEMU: Always assume 1 core
  PPC 85xx QEMU: Don't use HID1
  powerpc/mpc85xx: Update TLB CAMs in relocated mode
  PPC 85xx QEMU: Make a generic board file

Aneesh Bansal (2):
  powerpc/mpc85xx: SECURE BOOT- secure boot target for t1040rdb
  powerpc/mpc85xx: SECURE BOOT- Add secure boot target for T2080RDB

Chunhe Lan (1):
  powerpc/85xx: Add T4240RDB board support

Masahiro Yamada (1):
  boards.cfg: fix a configuration error of ep8248 board

Shaohui Xie (1):
  powerpc/fman/memac: use default MDIO_HOLD value

Shaveta Leekha (2):
  board/freescale/common: ZM7300 driver
  board/b4qds: VID support

Shengzhou Liu (2):
  powerpc/t208x: enable errata A006261, A006593, A006379
  powerpc/85xx: add T4080 SoC support

Stefan Bigler (2):
  kmp204x: Add support for the unit LEDs
  kmp204x: handle dip-switch for factory settings

Valentin Longchamp (6):
  kmp204x: selftest/factory test pin support
  kmp204x: update the CONFIG_PRAM and CONFIG_KM_RESERVED_PRAM defines
  kmp204x: complete the reset sequence and PRST configuration
  kmp204x: update the RCW
  kmp204x: add workaround for A-004849
  kmp204x: enable the errata command

York Sun (6):
  powerpc/mpc8572ds: Increase u-boot size to 768KB
  powerpc/mpc85xx: Ignore FDT pointer for non-QEMU in cpu_init_early_f()
  powerpc/freescale: Change the return value of mac_read_from_eeprom()
  powerpc/mpc85xx: Fix boot_flag for calling board_init_f()
  powerpc/mpc86xx: Fix boot_flag for calling board_init_f()
  powerpc/freescale: Convert selected boards to generic board architecture

Zhao Qiang (1):
  qe: disable qe when qe-ucode fails to be uploaded for "deep sleep"

 arch/powerpc/cpu/mpc85xx/Makefile   |2 +
 arch/powerpc/cpu/mpc85xx/cpu.c  |   24 +
 arch/powerpc/cpu/mpc85xx/cpu_init.c |   17 +-
 arch/powerpc/cpu/mpc85xx/cpu_init_early.c   |2 +
 arch/powerpc/cpu/mpc85xx/speed.c|3 +-
 arch/powerpc/cpu/mpc85xx/spl_minimal.c  |4 +-
 arch/powerpc/cpu/mpc85xx/start.S|4 +-
 arch/powerpc/cpu/mpc85xx/t4240_ids.c|2 +
 arch/powerpc/cpu/mpc85xx/t4240_serdes.c |2 +-
 arch/powerpc/cpu/mpc86xx/start.S|1 +
 arch/powerpc/cpu/mpc8xxx/cpu.c  |5 +-
 arch/powerpc/include/asm/config_mpc85xx.h   |   19 +-
 arch/powerpc/include/asm/fsl_errata.h   |5 +
 arch/powerpc/include/asm/fsl_secure_boot.h  |3 +-
 arch/powerpc/include/asm/immap_85xx.h   |6 +-
 arch/powerpc/include/asm/processor.h|1 +
 board/freescale/b4860qds/b4860qds.c |  240 +
 board/freescale/common/Makefile |1 +
 board/freescale/common/sys_eeprom.c |6 +-
 board/freescale/common/zm7300.c |  235 +
 board/freescale/common/zm7300.h |   22 +
 board/freescale/qemu-ppce500/qemu-ppce500.c |   20 +
 board/freescale/t4rdb/Makefile  |   12 +
 board/freescale/t4rdb/ddr.c |  118 +
 board/freescale/t4rdb/ddr.h |   78 +++
 board/freescale/t4rdb/eth.c |  146 ++
 board/freescale/t4rdb/law.c |   28 +
 board/freescale/t4rdb/pci.c |   23 +
 board/freescale/t4rdb/t4240rdb.c|  125 +
 board/freescale/t4rdb/t4_pbi.cfg|   31 ++
 board/freescale/t4rdb/t4_rcw.cfg|7 +
 board/freescale/t4rdb/t4rdb.h   |   18 +
 board/freescale/t4rdb/tlb.c |  111 
 board/keymile/kmp204x/kmp204x.c |   75 ++-
 board/keymile/kmp204x/kmp204x.h |3 +
 board/keymile/kmp204x/pbi.cfg   |   43 +-
 board/keymile/kmp204x/pci.c |   17 +-
 board/keymile/kmp204x/qrio.c|   31 +-
 board/keymile/kmp204x/rcw_kmp204x.cfg   |2 +-
 boards.cfg  |7 +-
 drivers/mmc/fsl_esdhc.c |2 +-
 drivers/net/fm/Makefile |1 +
 drivers/net/fm/memac_phy.c  |8 +-
 drivers/qe/qe.c |9 +-
 include/common.h|5 +-
 include/configs/B4860QDS.h  |   14 +
 include/configs/MPC8536DS.h |2 +
 include/configs/MPC8572DS.h |5 +-
 include/configs/MPC8641HPCN.h   |3 +
 include/configs/T208xRDB.h  |1 +
 include/config

Re: [U-Boot] [PATCH] sandbox: move source files from board/ to arch/sandbox/

2014-05-13 Thread Simon Glass
Hi Masahiro,

On 29 April 2014 22:53, Masahiro Yamada  wrote:
>
> Hi Simon,
>
> > > With this change, you can decrease one level hierarchy.
> > >
> > > board/sandbox/sandbox/sandbox.c
> > > ->  board/sandbox/sandbox.c
> >
> > That seems better to me. Will you update your patch?
> >
>
> Sure.
> Could you check this?
> http://patchwork.ozlabs.org/patch/344051/


Sorry I applied the wrong patch - the other one is not showing up in
patchwork for me. I'll see if I can figure it out.

Regards,
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] powerpc/mpc85xx: 32bit DDR changes for P1020/P1011

2014-05-13 Thread York Sun
On 05/12/2014 10:50 PM, Prabhakar Kushwaha wrote:
> The P1020/P1011 SOCs support max 32bit DDR width as opposed to P2020/P2010
> where max DDR data width supported is 64bit.
> 
> Add dynamic DDR size adjustment in second stage boot loader execution.
> 
> Signed-off-by: Prabhakar Kushwaha 
> ---
>  arch/powerpc/cpu/mpc85xx/cpu.c |7 ++-
>  1 file changed, 6 insertions(+), 1 deletion(-)
> 
> diff --git a/arch/powerpc/cpu/mpc85xx/cpu.c b/arch/powerpc/cpu/mpc85xx/cpu.c
> index 12e8e10..9c082b4 100644
> --- a/arch/powerpc/cpu/mpc85xx/cpu.c
> +++ b/arch/powerpc/cpu/mpc85xx/cpu.c
> @@ -347,7 +347,12 @@ phys_size_t initdram(int board_type)
>   defined(CONFIG_QEMU_E500)
>   return fsl_ddr_sdram_size();
>  #else
> - return (phys_size_t)CONFIG_SYS_SDRAM_SIZE * 1024 * 1024;
> + struct cpu_type *cpu = gd->arch.cpu;
> + /* P1020 and it's derivatives support max 32bit DDR width */
> + if (cpu->soc_ver == SVR_P1020 || cpu->soc_ver == SVR_P1011)
> + return CONFIG_SYS_SDRAM_SIZE * 1024 * 1024 / 2;
> + else
> + return CONFIG_SYS_SDRAM_SIZE * 1024 * 1024;
>  #endif
>  }
>  #else /* CONFIG_SYS_RAMBOOT */
> 

Prabhakar,

The second stage boot loader doesn't re-initialize DDR, right? The size should
come from either fsl_ddr_sdram_size() or the predefined macro
CONFIG_SYS_SDRAM_SIZE. It looks like you are modifying the predefiend one. Is
this what you want? If so, why don't you defined the macro with correct value?

York


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v2] zynq: treat ps7_init.c/h as external files to ignore them

2014-05-13 Thread Michal Simek
On 05/12/2014 05:18 AM, Masahiro Yamada wrote:
> ps7_init.c and ps7_init.h are supposed to be exported by hw project
> and copied to board/xilinx/zynq/ directory.
> 
> We want them to be ignored by git.
> So what we should do is to always treat them as external files
> rather than replacing ps7_init.c
> 
> This commit does:
> 
>  - Move a weak function ps7_init() to arch/arm/cpu/armv7/zynq/spl.c
>and delete board/xilinx/zynq/ps7_init.c
> 
>  - Compile board/xilinx/zynq/ps7_init.c only when it exists
> 
>  - Add .gitignore to ignore ps7_init.c/h
> 
> Signed-off-by: Masahiro Yamada 
> Signed-off-by: Michal Simek 
> ---
> 
> Changes in v2:
>   (only cosmetic changes in v2)
>   - Improve the comment in arch/arm/cpu/armv7/zynq/spl.c
>   s/replaced/overridden/
>   - Add Signed-off-by credit of Michal Simek
> 
>  arch/arm/cpu/armv7/zynq/spl.c |  8 
>  board/xilinx/zynq/.gitignore  |  1 +
>  board/xilinx/zynq/Makefile|  5 -
>  board/xilinx/zynq/ps7_init.c  | 12 
>  4 files changed, 13 insertions(+), 13 deletions(-)
>  create mode 100644 board/xilinx/zynq/.gitignore
>  delete mode 100644 board/xilinx/zynq/ps7_init.c

Applied to my zynq branch.

Thanks,
Michal

-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform




signature.asc
Description: OpenPGP digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] env_eeprom: Assign default environment during board_init_f

2014-05-13 Thread Michal Simek
Hi Tom,

On 05/09/2014 08:08 PM, Tom Rini wrote:
> On Fri, Apr 25, 2014 at 03:49:04PM +0200, Michal Simek wrote:
> 
>> From: Siva Durga Prasad Paladugu 
>>
>> Assign default environment and set env valid during board_init_f
>> before relocation as the actual environment will be read from
>> eeprom later.
>>
>> Signed-off-by: Siva Durga Prasad Paladugu 
>> Signed-off-by: Michal Simek 
>> ---
>>
>>  common/env_eeprom.c | 5 +
>>  1 file changed, 5 insertions(+)
>>
>> diff --git a/common/env_eeprom.c b/common/env_eeprom.c
>> index 0db2bb6..a4020bc 100644
>> --- a/common/env_eeprom.c
>> +++ b/common/env_eeprom.c
>> @@ -224,6 +224,7 @@ int env_init(void)
>>  #else
>>  int env_init(void)
>>  {
>> +#ifdef ENV_IS_EMBEDDED
>>  ulong crc, len, new;
>>  unsigned off;
>>  uchar buf[64];
>> @@ -256,6 +257,10 @@ int env_init(void)
>>  gd->env_addr= 0;
>>  gd->env_valid   = 0;
>>  }
>> +#else
>> +gd->env_addr = (ulong)&default_environment[0];
>> +gd->env_valid = 1;
>> +#endif
>>
>>  return 0;
>>  }
> 
> Sorry for the delay.  This looks fine but doesn't correct the redundant
> env case which has the same problem.  It would be nice if you had time
> to first consolidate env_init for both cases (see how env_nand.c does
> it) as well.  Thanks!

I have asked Siva to look at it but anyway if this is the right fix
it should be applied because it is fixing problem at least for one case.

Thanks,
Michal

-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Microblaze cpu - http://www.monstr.eu/fdt/
Maintainer of Linux kernel - Xilinx Zynq ARM architecture
Microblaze U-BOOT custodian and responsible for u-boot arm zynq platform




signature.asc
Description: OpenPGP digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v13 0/2] S5P: Exynos: Add GPIO numbering feature

2014-05-13 Thread Jaehoon Chung
Well..

In my case, exynos4 (trats2) is working fine until now.

On 05/13/2014 05:38 PM, Minkyu Kang wrote:
> Dear Akshay Saraswat,
> 
> On 13/05/14 14:00, Akshay Saraswat wrote:
>> Used a script to recheck/verify pin mapping and existing mapping
>> appears to be fine, returning correct bank and pin values.
>> Did ./MAKEALL -a arm and found all Exynos/S5P related boards
>> compiled successfully.
>> Couldn't test booting over all of them.
>> Tested U-Boot bootup over SMDK5420, SMDK5250, Snow.
>> Requesting maintaners to please test over affected SoCs/boards.
>>
>> Changes in V2:
>>  - Enabled CMD_GPIO as suggested by Simon Glass
>>and supported same for EXYNOS5.
>> Changes in V3:
>>  - New patch added to rename S5P GPIO definitions
>>to S5P_GPIO.
>>  - GPIO Table added to calculate the base address
>>of input gpio bank.
>> Changes in V4:
>>  - To have consistent 0..n-1 GPIO numbering the banks
>>are divided into different parts where ever they
>>have holes in them.
>>  - Function and table to support gpio command moved
>>to s5p-gpio driver.
>>  - Rebased on latest u-boot-samsung tree.
>> Changes in V5:
>>  - Rebased on latest u-boot-samsung tree.
>>  - Removed Exynos5 specific code in gpio driver api to
>>get bank.
>>  - Added #define HAVE_GENERIC_GPIO in config file
>>to remove conditinal CPU check in gpio driver.
>> Changes in V6:
>>  - Isolated config changes in a new patch.
>>  - Updated patches with corresponding changes for Exynos 5420.
>> Changes in V7:
>>  - Added changes for other SoCs like Exynos 4412, 4210 etc.
>> Changes in V8:
>>  - Changed Subject of patch 2/2 to reflect affected SoCs/boards.
>>  - Fixed arndale board compile time errors introduced due to
>>patch-set v7.
>> Changes in V9:
>>  - Fixed checkpatch errors.
>>  - Fixed naming error in exynos_gpio_data arrays which could
>>be the possible reason behind data abort witnessed over
>>Exynos4 boards.
>> Changes in v10:
>>  - Made arch-exynos/gpio.h -> arch/gpio.h.
>>  - Removed dead code.
>>  - Replaced exynos_gpio_get calls with corresponding pin number.
>>  - Edited pin numbmers in dts files.
>> Changes in v11:
>>  - Created and fixed pinmux_config for Exynos 4412.
>>  - Removed dead code.
>> Changes in v12:
>>  - Added change logs in individual patches.
>>  - Fixed few bugs and typos.
>>  - Added "Acked-by: Przemyslaw Marczak".
>> Changes in v13:
>>  - Fixed typo by replacing EXYNOS5420_GPIO_B10 with
>>EXYNOS5420_GPIO_B20 at line # 4149 of patch 1/2.
>>
>> Akshay Saraswat (2):
>>   S5P: Exynos: Add GPIO pin numbering and rename definitions
>>   S5P: Exynos: Config: Enable GPIO CMD config
>>
>>  arch/arm/cpu/armv7/exynos/pinmux.c |  561 +
>>  arch/arm/dts/exynos4210-origen.dts |4 +-
>>  arch/arm/dts/exynos4210-trats.dts  |6 +-
>>  arch/arm/dts/exynos4210-universal_c210.dts |4 +-
>>  arch/arm/dts/exynos4412-trats2.dts |4 +-
>>  arch/arm/include/asm/arch-exynos/cpu.h |   17 +-
>>  arch/arm/include/asm/arch-exynos/gpio.h| 1761 
>> +++-
>>  arch/arm/include/asm/arch-s5pc1xx/gpio.h   |  948 ---
>>  board/samsung/arndale/arndale.c|   11 +-
>>  board/samsung/goni/goni.c  |   32 +-
>>  board/samsung/smdk5250/exynos5-dt.c|   20 +-
>>  board/samsung/smdk5250/smdk5250.c  |   19 +-
>>  board/samsung/smdk5420/smdk5420.c  |   15 +-
>>  board/samsung/smdkc100/smdkc100.c  |5 +-
>>  board/samsung/smdkv310/smdkv310.c  |   19 +-
>>  board/samsung/trats/trats.c|   39 +-
>>  board/samsung/trats2/trats2.c  |   74 +-
>>  board/samsung/universal_c210/universal.c   |   51 +-
>>  drivers/gpio/s5p_gpio.c|  204 +++-
>>  include/configs/exynos5-dt.h   |2 +
>>  include/configs/s5p_goni.h |4 +-
>>  include/configs/s5pc210_universal.h|   16 +-
>>  include/configs/smdkv310.h |1 +
>>  include/configs/trats.h|8 +-
>>  include/configs/trats2.h   |4 +-
>>  25 files changed, 2918 insertions(+), 911 deletions(-)
>>
> 
> applied to u-boot-samsung.
> 
> 
> Dear SAMSUNG developers,
> 
> If you have any problem, please tell us immediately.
> 
> Thanks,
> Minkyu Kang.
> 

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [GIT PULL v2 ] fpga changes

2014-05-13 Thread Tom Rini
On Tue, May 13, 2014 at 09:48:07AM +0200, Michal Simek wrote:

> Hi Tom, [cc: Albert]
> 
> please pull these fpga changes to your tree.
> I have run buildman to ensure that there are no new compilation warnings.
> As you see from log below there are some warnings for m68k and astro board
> but they are there before my first commit.
> 
> When this is in the tree I will send pull request to Albert to get our
> Zynq changes to ARM tree.
> 
> Thanks,
> Michal
> 
> [u-boot]$ ./tools/buildman/buildman -b fpga apf27 zynq spear astro balloon3 
> pmc440 gen860t mvsmr mt_ventoux -sSed
> Summary of 10 commits for 35 boards (4 threads, 1 job per thread)
> 01: kbuild: Fix trailing whitespaces
>   m68k: +   astro_mcf5373l
> +/mnt/disk/fpga/.bm-work/01/arch/m68k/cpu/mcf532x/cpu_init.c: In function 
> 'cpu_init_f':
> +/mnt/disk/fpga/.bm-work/01/arch/m68k/cpu/mcf532x/cpu_init.c:211:10: warning: 
> unused variable 'wdog' [-Wunused-variable]
> +/mnt/disk/fpga/.bm-work/01/arch/m68k/lib/bootm.c: In function 
> 'do_bootm_linux':
> +/mnt/disk/fpga/.bm-work/01/arch/m68k/lib/bootm.c:53:8: warning: unused 
> variable 'rd_len' [-Wunused-variable]
> +/mnt/disk/fpga/.bm-work/01/arch/m68k/lib/bootm.c:99:12: warning: 
> 'initrd_start' may be used uninitialized in this function [-Wuninitialized]
> +/mnt/disk/fpga/.bm-work/01/arch/m68k/lib/bootm.c:99:12: warning: 
> 'initrd_end' may be used uninitialized in this function [-Wuninitialized]
> +/mnt/disk/fpga/.bm-work/01/arch/m68k/lib/bootm.c:99:12: warning: 'cmd_start' 
> may be used uninitialized in this function [-Wuninitialized]
> +/mnt/disk/fpga/.bm-work/01/arch/m68k/lib/bootm.c:99:12: warning: 'cmd_end' 
> may be used uninitialized in this function [-Wuninitialized]
> +/mnt/disk/fpga/.bm-work/01/board/astro/mcf5373l/mcf5373l.c: In function 
> 'initdram':
> +/mnt/disk/fpga/.bm-work/01/board/astro/mcf5373l/mcf5373l.c:83:5: warning: 
> pointer targets in passing argument 1 of 'get_ram_size' differ in signedness 
> [-Wpointer-sign]
> +/mnt/disk/fpga/.bm-work/01/include/common.h:470:6: note: expected 'long int 
> *' but argument is of type 'long unsigned int *'
> +/mnt/disk/fpga/.bm-work/01/board/astro/mcf5373l/fpga.c:168:2: warning: 
> initialization from incompatible pointer type [enabled by default]
> +/mnt/disk/fpga/.bm-work/01/board/astro/mcf5373l/fpga.c:168:2: warning: (near 
> initialization for 'altera_fns.write') [enabled by default]
> +/mnt/disk/fpga/.bm-work/01/board/astro/mcf5373l/fpga.c: In function 
> 'astro5373l_altera_load':
> +/mnt/disk/fpga/.bm-work/01/board/astro/mcf5373l/fpga.c:196:20: warning: 
> assignment from incompatible pointer type [enabled by default]
> 02: fpga: spartan2: Avoid CamelCase
> 03: fpga: spartan3: Avoid CamelCase
> 04: fpga: virtex2: Avoid CamelCase
> 05: fpga: xilinx: Avoid CamelCase for in Xilinx_desc
> 06: fpga: xilinx: Fix the rest of CamelCases
> 07: fpga: xilinx: Simplify load/dump/info function handling
>arm: (for 30/30 boards)  all -191.0  bss -1.7  data +9.3  rodata -54.9 
>  text -143.7
> zynq_zed   :  all -489  bss +24  data +36  rodata -153  text 
> -396
> zynq_zc770_xm013:  all -509  bss +4  data +36  rodata -153  text 
> -396
> zynq_zc770_xm012:  all -509  bss +4  data +36  rodata -153  text 
> -396
> zynq_zc70x :  all -517  bss -4  data +36  rodata -153  text 
> -396
> zynq_zc770_xm010:  all -521  bss -8  data +36  rodata -153  text 
> -396
> zynq_microzed  :  all -521  bss -8  data +36  rodata -153  text 
> -396
> x600   :  all -646  bss +4  data +16  rodata -182  text 
> -484
> apf27  :  all -650  data +16  rodata -182  text -484
> mt_ventoux :  all -654  bss -4  data +16  rodata -182  text 
> -484
> balloon3   :  all -714  bss -64  data +16  rodata -182  text 
> -484
>   m68k: (for 1/1 boards)  all -389.0  data -24.0  rodata -149.0  text 
> -216.0
> astro_mcf5373l :  all -389  data -24  rodata -149  text -216
>powerpc: (for 4/4 boards)  all -431.0  data +10.0  rodata -133.0  text 
> -308.0
> PMC440 :  all -364  data +40  rodata -148  text -256
> MVSMR  :  all -448  text -448
> GEN860T:  all -456  rodata -192  text -264
> GEN860T_SC :  all -456  rodata -192  text -264
> 08: fpga: zynq: Remove sparse warnings
> 09: fpga: zynq: Use helper functions for zynq dma
>arm: (for 30/30 boards)  all +22.2  rodata +11.8  text +10.4
> zynq_zc770_xm010:  all +119  bss +8  rodata +59  text +52
> zynq_microzed  :  all +119  bss +8  rodata +59  text +52
> zynq_zc70x :  all +119  bss +8  rodata +59  text +52
> zynq_zc770_xm013:  all +111  rodata +59  text +52
> zynq_zc770_xm012:  all +111  rodata +59  text +52
> zynq_zed   :  all +87  bss -24  rodata +59  text +52
> 10: fpga: zynq: Use helper function zynq_val

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

2014-05-13 Thread Tom Rini
On Mon, May 12, 2014 at 09:56:41PM +0200, Anatolij Gustschin wrote:

> Hey Tom,
> 
> The following changes since commit a405764c1ec835a41ccda943b9156aee25e15d5e:
> 
>   drivers/i2c/fsl_i2c: modify i2c_read to handle multi-byte write (2014-04-29 
> 07:10:58 +0200)
> 
> are available in the git repository at:
> 
>   git://git.denx.de/u-boot-video.git master
> 
> for you to fetch changes up to 1161f98db687a1cb263cbacdc7eb548a0354218d:
> 
>   at91: video: atmel_hlcdfb.c: fix bad timing configuration (2014-05-05 
> 11:50:16 +0200)
> 
> 
> Eric Nelson (1):
>   video: mxc_ipuv3_fb: stash frame buffer pointer in global data.
> 
> Stephen Warren (1):
>   config: enable CMD_BMP when API+LCD is enabled
> 
> Thomas Diener (1):
>   logos: Update of the syteco company logo
> 
> Wu, Josh (1):
>   at91: video: atmel_hlcdfb.c: fix bad timing configuration
> 
>  drivers/video/atmel_hlcdfb.c |8 
>  drivers/video/mxc_ipuv3_fb.c |5 +
>  include/config_fallbacks.h   |4 
>  tools/logos/syteco.bmp   |  Bin 11414 -> 11414 bytes
>  4 files changed, 13 insertions(+), 4 deletions(-)

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] net/designware: call phy_connect_dev() to properly setup phylib device

2014-05-13 Thread Tom Rini
On Tue, May 13, 2014 at 11:19:13AM +0100, Ian Campbell wrote:
> On Mon, 2014-05-12 at 21:53 -0400, Tom Rini wrote:
> > On Mon, Apr 28, 2014 at 08:14:05PM +0100, Ian Campbell wrote:
> > 
> > > This sets up the linkage from the phydev back to the ethernet device. This
> > > symptom of not doing this which I noticed was:
> > >  Waiting for PHY auto negotiation to complete
> > > rather than:
> > > dwmac.1c5 Waiting for PHY auto negotiation to complete
> > > 
> > > Signed-off-by: Ian Campbell 
> > > Cc: Alexey Brodkin 
> > 
> > Applied to u-boot/master, thanks!
> 
> Thanks, have you also seen "[PATCH v3 0/5] net/designware: fixes for
> data cache, phylib and burst size" which included this and some other
> (far more critical, since designware is currently unusable on ARM)
> fixes?

For the moment I've tossed all of the rest of them over to Albert to
grab along with the sunxi stuff.  Albert, if you're uncomfortable there
I can pick them up instead, thanks!

-- 
Tom


signature.asc
Description: Digital signature
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] please pull u-boot-samsung master

2014-05-13 Thread Minkyu Kang
Dear Albert,

The following changes since commit 7904b70885f3c589c239f6ac978f299a6744557f:

  ARM: highbank: use default prompt (2014-05-02 11:43:25 +0200)

are available in the git repository at:

  http://git.denx.de/u-boot-samsung 

for you to fetch changes up to 9b97b727dcfbdc02a0a78e4c1d81670742f28784:

  S5P: Exynos: Config: Enable GPIO CMD config (2014-05-13 15:20:38 +0900)


Akshay Saraswat (3):
  Exynos5: config: Enable FIT
  S5P: Exynos: Add GPIO pin numbering and rename definitions
  S5P: Exynos: Config: Enable GPIO CMD config

Inha Song (1):
  samsung: misc: add env default option to lcd menu

Jaehoon Chung (1):
  ARM: exynos: remove the unused code

Mateusz Zalega (1):
  ARM: Samsung: s5p_goni: maintainer update

Przemyslaw Marczak (4):
  samsung: misc: allows using environmental macros as args in menu commands
  samsung: misc: add gpt restore option to lcd menu
  samsung: misc: menu: increase delay in menu main loop
  samsung: misc: remove download mode info screen

Łukasz Majewski (2):
  trats2: config: fix: Set default console to ttySAC2
  trats: config: fix: Set default console to ttySAC2

 arch/arm/cpu/armv7/exynos/pinmux.c |  561 +
 arch/arm/dts/exynos4210-origen.dts |4 +-
 arch/arm/dts/exynos4210-trats.dts  |6 +-
 arch/arm/dts/exynos4210-universal_c210.dts |4 +-
 arch/arm/dts/exynos4412-trats2.dts |4 +-
 arch/arm/include/asm/arch-exynos/cpu.h |   17 +-
 arch/arm/include/asm/arch-exynos/gpio.h| 1761 +++-
 arch/arm/include/asm/arch-s5pc1xx/gpio.h   |  948 ---
 board/samsung/arndale/arndale.c|   11 +-
 board/samsung/common/misc.c|  130 +-
 board/samsung/goni/goni.c  |   32 +-
 board/samsung/smdk5250/exynos5-dt.c|   20 +-
 board/samsung/smdk5250/smdk5250.c  |   19 +-
 board/samsung/smdk5420/smdk5420.c  |   15 +-
 board/samsung/smdkc100/smdkc100.c  |5 +-
 board/samsung/smdkv310/smdkv310.c  |   19 +-
 board/samsung/trats/trats.c|   39 +-
 board/samsung/trats2/trats2.c  |   74 +-
 board/samsung/universal_c210/universal.c   |   51 +-
 boards.cfg |2 +-
 drivers/gpio/s5p_gpio.c|  204 +++-
 include/configs/exynos5-dt.h   |2 +
 include/configs/s5p_goni.h |4 +-
 include/configs/s5pc210_universal.h|   23 +-
 include/configs/smdk5250.h |4 +
 include/configs/smdk5420.h |4 +
 include/configs/smdkv310.h |1 +
 include/configs/snow.h |4 +
 include/configs/trats.h|   17 +-
 include/configs/trats2.h   |   17 +-
 include/samsung/misc.h |2 +
 31 files changed, 3002 insertions(+), 1002 deletions(-)

-- 
Thanks,
Minkyu Kang.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] net/designware: call phy_connect_dev() to properly setup phylib device

2014-05-13 Thread Ian Campbell
On Mon, 2014-05-12 at 21:53 -0400, Tom Rini wrote:
> On Mon, Apr 28, 2014 at 08:14:05PM +0100, Ian Campbell wrote:
> 
> > This sets up the linkage from the phydev back to the ethernet device. This
> > symptom of not doing this which I noticed was:
> >  Waiting for PHY auto negotiation to complete
> > rather than:
> > dwmac.1c5 Waiting for PHY auto negotiation to complete
> > 
> > Signed-off-by: Ian Campbell 
> > Cc: Alexey Brodkin 
> 
> Applied to u-boot/master, thanks!

Thanks, have you also seen "[PATCH v3 0/5] net/designware: fixes for
data cache, phylib and burst size" which included this and some other
(far more critical, since designware is currently unusable on ARM)
fixes?

Ian.

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v13 0/2] S5P: Exynos: Add GPIO numbering feature

2014-05-13 Thread Minkyu Kang
Dear Akshay Saraswat,

On 13/05/14 14:00, Akshay Saraswat wrote:
> Used a script to recheck/verify pin mapping and existing mapping
> appears to be fine, returning correct bank and pin values.
> Did ./MAKEALL -a arm and found all Exynos/S5P related boards
> compiled successfully.
> Couldn't test booting over all of them.
> Tested U-Boot bootup over SMDK5420, SMDK5250, Snow.
> Requesting maintaners to please test over affected SoCs/boards.
> 
> Changes in V2:
>   - Enabled CMD_GPIO as suggested by Simon Glass
> and supported same for EXYNOS5.
> Changes in V3:
>   - New patch added to rename S5P GPIO definitions
> to S5P_GPIO.
>   - GPIO Table added to calculate the base address
> of input gpio bank.
> Changes in V4:
>   - To have consistent 0..n-1 GPIO numbering the banks
> are divided into different parts where ever they
> have holes in them.
>   - Function and table to support gpio command moved
> to s5p-gpio driver.
>   - Rebased on latest u-boot-samsung tree.
> Changes in V5:
>   - Rebased on latest u-boot-samsung tree.
>   - Removed Exynos5 specific code in gpio driver api to
> get bank.
>   - Added #define HAVE_GENERIC_GPIO in config file
> to remove conditinal CPU check in gpio driver.
> Changes in V6:
>   - Isolated config changes in a new patch.
>   - Updated patches with corresponding changes for Exynos 5420.
> Changes in V7:
>   - Added changes for other SoCs like Exynos 4412, 4210 etc.
> Changes in V8:
>   - Changed Subject of patch 2/2 to reflect affected SoCs/boards.
>   - Fixed arndale board compile time errors introduced due to
> patch-set v7.
> Changes in V9:
>   - Fixed checkpatch errors.
>   - Fixed naming error in exynos_gpio_data arrays which could
> be the possible reason behind data abort witnessed over
> Exynos4 boards.
> Changes in v10:
>   - Made arch-exynos/gpio.h -> arch/gpio.h.
>   - Removed dead code.
>   - Replaced exynos_gpio_get calls with corresponding pin number.
>   - Edited pin numbmers in dts files.
> Changes in v11:
>   - Created and fixed pinmux_config for Exynos 4412.
>   - Removed dead code.
> Changes in v12:
>   - Added change logs in individual patches.
>   - Fixed few bugs and typos.
>   - Added "Acked-by: Przemyslaw Marczak".
> Changes in v13:
>   - Fixed typo by replacing EXYNOS5420_GPIO_B10 with
> EXYNOS5420_GPIO_B20 at line # 4149 of patch 1/2.
> 
> Akshay Saraswat (2):
>   S5P: Exynos: Add GPIO pin numbering and rename definitions
>   S5P: Exynos: Config: Enable GPIO CMD config
> 
>  arch/arm/cpu/armv7/exynos/pinmux.c |  561 +
>  arch/arm/dts/exynos4210-origen.dts |4 +-
>  arch/arm/dts/exynos4210-trats.dts  |6 +-
>  arch/arm/dts/exynos4210-universal_c210.dts |4 +-
>  arch/arm/dts/exynos4412-trats2.dts |4 +-
>  arch/arm/include/asm/arch-exynos/cpu.h |   17 +-
>  arch/arm/include/asm/arch-exynos/gpio.h| 1761 
> +++-
>  arch/arm/include/asm/arch-s5pc1xx/gpio.h   |  948 ---
>  board/samsung/arndale/arndale.c|   11 +-
>  board/samsung/goni/goni.c  |   32 +-
>  board/samsung/smdk5250/exynos5-dt.c|   20 +-
>  board/samsung/smdk5250/smdk5250.c  |   19 +-
>  board/samsung/smdk5420/smdk5420.c  |   15 +-
>  board/samsung/smdkc100/smdkc100.c  |5 +-
>  board/samsung/smdkv310/smdkv310.c  |   19 +-
>  board/samsung/trats/trats.c|   39 +-
>  board/samsung/trats2/trats2.c  |   74 +-
>  board/samsung/universal_c210/universal.c   |   51 +-
>  drivers/gpio/s5p_gpio.c|  204 +++-
>  include/configs/exynos5-dt.h   |2 +
>  include/configs/s5p_goni.h |4 +-
>  include/configs/s5pc210_universal.h|   16 +-
>  include/configs/smdkv310.h |1 +
>  include/configs/trats.h|8 +-
>  include/configs/trats2.h   |4 +-
>  25 files changed, 2918 insertions(+), 911 deletions(-)
> 

applied to u-boot-samsung.


Dear SAMSUNG developers,

If you have any problem, please tell us immediately.

Thanks,
Minkyu Kang.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] powerpc/mpc85xx: SECURE BOOT- corrected CSPR settings for BSC9132QDS NAND

2014-05-13 Thread Aneesh Bansal
In case of NAND boot, CSPR0 is configured as NAND and CSPR1
is configured as NOR.

Signed-off-by: Aneesh Bansal 
---
 include/configs/BSC9132QDS.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/configs/BSC9132QDS.h b/include/configs/BSC9132QDS.h
index e76a04b..7bb5d33 100644
--- a/include/configs/BSC9132QDS.h
+++ b/include/configs/BSC9132QDS.h
@@ -360,7 +360,7 @@ combinations. this should be removed later
 #endif
 
 /* Set up IFC registers for boot location NOR/NAND */
-#if defined(CONFIG_NAND)
+#if defined(CONFIG_NAND) || defined(CONFIG_NAND_SECBOOT)
 #define CONFIG_SYS_CSPR0   CONFIG_SYS_NAND_CSPR
 #define CONFIG_SYS_AMASK0  CONFIG_SYS_NAND_AMASK
 #define CONFIG_SYS_CSOR0   CONFIG_SYS_NAND_CSOR
-- 
1.8.1.4

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [GIT PULL v2 ] fpga changes

2014-05-13 Thread Michal Simek
Hi Tom, [cc: Albert]

please pull these fpga changes to your tree.
I have run buildman to ensure that there are no new compilation warnings.
As you see from log below there are some warnings for m68k and astro board
but they are there before my first commit.

When this is in the tree I will send pull request to Albert to get our
Zynq changes to ARM tree.

Thanks,
Michal

[u-boot]$ ./tools/buildman/buildman -b fpga apf27 zynq spear astro balloon3 
pmc440 gen860t mvsmr mt_ventoux -sSed
Summary of 10 commits for 35 boards (4 threads, 1 job per thread)
01: kbuild: Fix trailing whitespaces
  m68k: +   astro_mcf5373l
+/mnt/disk/fpga/.bm-work/01/arch/m68k/cpu/mcf532x/cpu_init.c: In function 
'cpu_init_f':
+/mnt/disk/fpga/.bm-work/01/arch/m68k/cpu/mcf532x/cpu_init.c:211:10: warning: 
unused variable 'wdog' [-Wunused-variable]
+/mnt/disk/fpga/.bm-work/01/arch/m68k/lib/bootm.c: In function 'do_bootm_linux':
+/mnt/disk/fpga/.bm-work/01/arch/m68k/lib/bootm.c:53:8: warning: unused 
variable 'rd_len' [-Wunused-variable]
+/mnt/disk/fpga/.bm-work/01/arch/m68k/lib/bootm.c:99:12: warning: 
'initrd_start' may be used uninitialized in this function [-Wuninitialized]
+/mnt/disk/fpga/.bm-work/01/arch/m68k/lib/bootm.c:99:12: warning: 'initrd_end' 
may be used uninitialized in this function [-Wuninitialized]
+/mnt/disk/fpga/.bm-work/01/arch/m68k/lib/bootm.c:99:12: warning: 'cmd_start' 
may be used uninitialized in this function [-Wuninitialized]
+/mnt/disk/fpga/.bm-work/01/arch/m68k/lib/bootm.c:99:12: warning: 'cmd_end' may 
be used uninitialized in this function [-Wuninitialized]
+/mnt/disk/fpga/.bm-work/01/board/astro/mcf5373l/mcf5373l.c: In function 
'initdram':
+/mnt/disk/fpga/.bm-work/01/board/astro/mcf5373l/mcf5373l.c:83:5: warning: 
pointer targets in passing argument 1 of 'get_ram_size' differ in signedness 
[-Wpointer-sign]
+/mnt/disk/fpga/.bm-work/01/include/common.h:470:6: note: expected 'long int *' 
but argument is of type 'long unsigned int *'
+/mnt/disk/fpga/.bm-work/01/board/astro/mcf5373l/fpga.c:168:2: warning: 
initialization from incompatible pointer type [enabled by default]
+/mnt/disk/fpga/.bm-work/01/board/astro/mcf5373l/fpga.c:168:2: warning: (near 
initialization for 'altera_fns.write') [enabled by default]
+/mnt/disk/fpga/.bm-work/01/board/astro/mcf5373l/fpga.c: In function 
'astro5373l_altera_load':
+/mnt/disk/fpga/.bm-work/01/board/astro/mcf5373l/fpga.c:196:20: warning: 
assignment from incompatible pointer type [enabled by default]
02: fpga: spartan2: Avoid CamelCase
03: fpga: spartan3: Avoid CamelCase
04: fpga: virtex2: Avoid CamelCase
05: fpga: xilinx: Avoid CamelCase for in Xilinx_desc
06: fpga: xilinx: Fix the rest of CamelCases
07: fpga: xilinx: Simplify load/dump/info function handling
   arm: (for 30/30 boards)  all -191.0  bss -1.7  data +9.3  rodata -54.9  
text -143.7
zynq_zed   :  all -489  bss +24  data +36  rodata -153  text 
-396
zynq_zc770_xm013:  all -509  bss +4  data +36  rodata -153  text 
-396
zynq_zc770_xm012:  all -509  bss +4  data +36  rodata -153  text 
-396
zynq_zc70x :  all -517  bss -4  data +36  rodata -153  text -396
zynq_zc770_xm010:  all -521  bss -8  data +36  rodata -153  text 
-396
zynq_microzed  :  all -521  bss -8  data +36  rodata -153  text -396
x600   :  all -646  bss +4  data +16  rodata -182  text -484
apf27  :  all -650  data +16  rodata -182  text -484
mt_ventoux :  all -654  bss -4  data +16  rodata -182  text -484
balloon3   :  all -714  bss -64  data +16  rodata -182  text 
-484
  m68k: (for 1/1 boards)  all -389.0  data -24.0  rodata -149.0  text -216.0
astro_mcf5373l :  all -389  data -24  rodata -149  text -216
   powerpc: (for 4/4 boards)  all -431.0  data +10.0  rodata -133.0  text -308.0
PMC440 :  all -364  data +40  rodata -148  text -256
MVSMR  :  all -448  text -448
GEN860T:  all -456  rodata -192  text -264
GEN860T_SC :  all -456  rodata -192  text -264
08: fpga: zynq: Remove sparse warnings
09: fpga: zynq: Use helper functions for zynq dma
   arm: (for 30/30 boards)  all +22.2  rodata +11.8  text +10.4
zynq_zc770_xm010:  all +119  bss +8  rodata +59  text +52
zynq_microzed  :  all +119  bss +8  rodata +59  text +52
zynq_zc70x :  all +119  bss +8  rodata +59  text +52
zynq_zc770_xm013:  all +111  rodata +59  text +52
zynq_zc770_xm012:  all +111  rodata +59  text +52
zynq_zed   :  all +87  bss -24  rodata +59  text +52
10: fpga: zynq: Use helper function zynq_validate_bitstream
   arm: (for 30/30 boards)  all +6.4  rodata +4.8  text +1.6
zynq_zc770_xm010:  all +40  bss +8  rodata +24  text +8
zynq_microzed  :  all +40  bss +8  rodata +24  text +8
zynq_zed   :  all +40  bss +

Re: [U-Boot] [U-Boot, 2/2] mkimage: Refactor mxsimage to use common crc32 code

2014-05-13 Thread Charles Manning
On Tuesday 13 May 2014 08:26:44 Tom Rini wrote:
> On Tue, May 06, 2014 at 10:46:46AM +1200, Charles Manning wrote:
> > mxsimage uses the same crc32 function as pblimage.
> >
> > Signed-off-by: Charles Manning 
> > Acked-by: Stefano Babic 
>
> This introduces warnings:
> tools/mxsimage.c: In function ‘sb_build_command_load’:
> tools/mxsimage.c:979:2: warning: pointer targets in passing argument 2 of
> ‘pbl_crc32’ differ in signedness [-Wpointer-sign] tools/pbl_crc32.h:11:10:
> note: expected ‘const char *’ but argument is of type ‘uint8_t *’
> u-boot/tools/mxsimage.c: In function ‘sb_verify_command’:
> tools/mxsimage.c:1815:3: warning: pointer targets in passing argument 2 of
> ‘pbl_crc32’ differ in signedness [-Wpointer-sign] tools/pbl_crc32.h:11:10:
> note: expected ‘const char *’ but argument is of type ‘uint8_t *’

Thanks for the feedback.

Will it be acceptable to do the following:

*Change the called function from uint8_t * to const uint8_t
*Cast the const char * pointer to const uint8_t *.

Thanks

Charles

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot