[U-Boot] [PATCH] cosmetic: board: pm9263 rewrite old style stuct init

this prevent some warnings when compiling with clang cc: Stelian Pop Signed-off-by: Jeroen Hofstee --- board/ronetix/pm9263/pm9263.c | 28 ++-- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/board/ronetix/pm9263/pm9263.c b/board/ronetix/pm9263/pm9263.c i

[U-Boot] [PATCH] imximage_hynix.cfg: fix unterminated comment

cc: Troy Kisky Signed-off-by: Jeroen Hofstee --- board/ttcontrol/vision2/imximage_hynix.cfg | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/board/ttcontrol/vision2/imximage_hynix.cfg b/board/ttcontrol/vision2/imximage_hynix.cfg index 4e6583a..c74973e 100644 --- a/board/ttcon

Re: [U-Boot] [PATCH] arm: Allow u-boot to run from offset base address

Hi Steve, On Tue, 10 Jun 2014 12:38:42 -0700, Steve Rae wrote: > > > On 14-06-10 11:13 AM, Wolfgang Denk wrote: > > Dear Steve, > > > > In message <539746c4.9040...@broadcom.com> you wrote: > >> > >> There could be "many" reasons why the CONFIG_SYS_TEXT_BASE is not > >> aligned on a 0x1000 byt

[U-Boot] [PATCH] dfu: fix some issues with reads/uploads

From: Stephen Warren FIXME: reword after updates to use "*size" command for fs... DFU read support appears to rely upon dfu->read_medium() updating the passed-by-reference len parameter to indicate the remaining size available for reading. dfu_read_medium_mmc() never does this, and the implemen

[U-Boot] [PATCH] usb: ci_udc: terminate ep0 INs with a zlp when required

From: Stephen Warren Sometimes, a zero-length packet is required at the end of an IN transaction so that the host knows the device is done sending data. Enhance ci_udc to send a zlp when necessary. See the comments for more details. Signed-off-by: Stephen Warren --- This depends on all the othe

Re: [U-Boot] [PATCH] dfu: fix some issues with reads/uploads

On 06/10/2014 03:25 PM, Stephen Warren wrote: > From: Stephen Warren > > FIXME: reword after updates to use "*size" command for fs... > > DFU read support appears to rely upon dfu->read_medium() updating the > passed-by-reference len parameter to indicate the remaining size > available for readi

[U-Boot] [PATCH] ubifs: don't self assign values

It seems the code tries to trick the compiler the argument is actually used. However compilers became too smart too fool them so easily an now warn. Checking gcc and clang does not seem to emit a warning. If so it should be decorated with unused / (void). cc: Stefan Roese Signed-off-by: Jeroen Ho

[U-Boot] [PATCH] LzmaTools: don't self assign values

It seems the code tries to trick the compiler the argument is actually used. However compilers became too smart to fool them so easily an now warn. Gcc and clang don't seem to emit a warning when the argument is unused. If so it should be decorated with unused / (void). Signed-off-by: Jeroen Hofst

[U-Boot] [PATCH] include/dm.h: fix inclusion guard

cc: Simon Glass Signed-off-by: Jeroen Hofstee --- include/dm.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/dm.h b/include/dm.h index 8bbb21b..a179c8a 100644 --- a/include/dm.h +++ b/include/dm.h @@ -5,7 +5,7 @@ */ #ifndef _DM_H_ -#define _DM_H +#define _DM_H_

[U-Boot] [PATCH] test: dfu: script enhancements

From: Stephen Warren Various misc enhancements to dfu_gadget_test.sh: * After every write (download), perform a write to a different file with different data. This ensures that the DFU buffer's content is replaced, so that if the read (upload) succeeds, we know that the correct data was ac

[U-Boot] [PATCH] common/cli_hush.c: remove unnecessary double braces

Clang interpretes an if condition like "if ((a = b) == NULL) as it tries to assign a value in a statement. Hence if you do "if ((something)) it warns you that you might be confused. Hence drop the double braces for plane if statements. Simon Glass Signed-off-by: Jeroen Hofstee --- common/cli_h

[U-Boot] [PATCH] board:keymile: remove unnecessary double braces

Clang interpretes an if condition like "if ((a = b) == NULL) as it tries to assign a value in a statement. Hence if you do "if ((something)) it warns you that you might be confused. Hence drop the double braces for plane if statements. cc: Holger Brunck Signed-off-by: Jeroen Hofstee --- board/

[U-Boot] [PATCH] jffs2:jffs2_1pass.c: remove double braces

Clang interpretes an if condition like  "if ((a = b) == NULL) as it tries to assign a value in a statement. Hence if you do "if ((something)) it warns you that you might be confused. Hence drop the double braces for plane if statements. Signed-off-by: Jeroen Hofstee --- fs/jffs2/jffs2_1pass.c |

[U-Boot] [PATCH] sandbox: restore ability to access host fs through standard commands

From: Stephen Warren Commit 95fac6ab4589 "sandbox: Use os functions to read host device tree" removed the ability for get_device_and_partition() to handle the "host" device type, and redirect accesses to it to the host filesystem. This broke some unit tests that use this feature. So, revert that

[U-Boot] [PATCH] common/xyzModem.c: move empty statements to newline

To prevent a warning for clang the loop without a body is made more clear by moving it to a line of its own. This prevents a clang warning. cc: sba...@denx.de Signed-off-by: Jeroen Hofstee --- This ignores checkpatch warnings since somone forgot to burn the GNU code style manual. jeroen@yellow:

Re: [U-Boot] [PATCH] arm: Allow u-boot to run from offset base address

On 14-06-10 01:35 PM, Wolfgang Denk wrote: Dear Steve, In message <53975ec2.1080...@broadcom.com> you wrote: I still cannot understand why _start and CONFIG_SYS_TEXT_BASE would have to be the same. There is no such requirement. What exactly prevents you from assigning _start a location at

[U-Boot] [PATCH v5 09/15] dm: Cast away the const-ness of the global_data pointer

In a very few cases we need to adjust the driver model root device, such as when setting it up at initialisation. Add a macro to make this easier. Signed-off-by: Simon Glass --- Changes in v5: None Changes in v4: None Changes in v3: - Fix typo in commit subject Changes in v2: - Add new patch to

[U-Boot] [PATCH v5 05/15] dm: Rename struct device_id to udevice_id

It is best to avoid having any occurence of 'struct device' in driver model, so rename to achieve this. Signed-off-by: Simon Glass --- Changes in v5: None Changes in v4: - Add new patch to rename struct device_id to udevice_id Changes in v3: None Changes in v2: None doc/driver-model/README.tx

[U-Boot] [PATCH v5 0/15] Collected driver model bug-fixes and docs

This series collects some of the patches from the Tegra GPIO conversion to driver model. That work is still in progress, but the bug fixes and iotracing feature should go into this release I think. Also the documentation improvements may as well follow since the existings docs are proven inadequate

[U-Boot] [PATCH v5 08/15] dm: Add missing header files in lists and root

These files don't compile in some architectures. Fix it by adding the missing headers. Signed-off-by: Simon Glass --- Changes in v5: None Changes in v4: None Changes in v3: None Changes in v2: - Add new patch to add missing header files in lists and root drivers/core/lists.c | 1 + drivers/cor

[U-Boot] [PATCH v5 07/15] dm: Use case-insensitive comparison for GPIO banks

We want 'N0' and 'n0' to mean the same thing, so ensure that case is not considered when naming GPIO banks. Signed-off-by: Simon Glass --- Changes in v5: None Changes in v4: None Changes in v3: None Changes in v2: - Add new patch to use case-insensitive comparison for GPIO banks drivers/gpio/g

[U-Boot] [PATCH v5 04/15] Makefile: Support include files for .dts files

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. Signed-off-by: Simon Glass Acked-by: Stephen Warren Reviewed-by: Masahiro Yamada ---

[U-Boot] [PATCH v5 01/15] Add an I/O tracing feature

When debugging drivers it is useful to see what I/O accesses were done and in what order. Even if the individual accesses are of little interest it can be useful to verify that the access pattern is consistent each time an operation is performed. In this case a checksum can be used to characterise

[U-Boot] [PATCH v5 02/15] arm: Support iotrace feature

Support the iotrace feature for ARM, when enabled. Signed-off-by: Simon Glass --- Changes in v5: None Changes in v4: None Changes in v3: None Changes in v2: - Add a new patch to enable iotrace for arm arch/arm/include/asm/io.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/arch/arm/inc

[U-Boot] [PATCH v5 14/15] dm: Tidy up four minor code nits

There is a spelling mistake and two functions are missing comments altogether. Also the flags declaration is correct, but doesn't follow style. Finally, the uclass_get_device() function has some errors in its documentation. Fix these problems. Signed-off-by: Simon Glass Acked-by: Marek Vasut -

[U-Boot] [PATCH v5 15/15] dm: Expand and improve the device lifecycle docs

The lifecycle of a device is an important part of driver model. Add to the existing documentation and clarify it. Reported-by: Jon Loeliger Signed-off-by: Simon Glass --- Thanks for Jon Loeliger for helping with the text and suggesting improvements. (Jon please comment/adjust to help clarify

[U-Boot] [PATCH v5 06/15] dm: Update README to encourage conversion to driver model

Add a note to encourage people to convert drivers to use driver model. Signed-off-by: Simon Glass --- Changes in v5: None Changes in v4: None Changes in v3: None Changes in v2: - Update README to encourage conversion to driver model README | 5 + 1 file changed, 5 insertions(+) diff --git

[U-Boot] [PATCH v5 03/15] sandbox: Support iotrace feature

Support the iotrace feature for sandbox, and enable it, using some dummy I/O access methods. Signed-off-by: Simon Glass --- Changes in v5: None Changes in v4: - Correct typo in CONFIG_CMD_IOTRACE Changes in v3: None Changes in v2: - Add a new patch to enable iotrace for sandbox arch/sandbox/i

[U-Boot] [PATCH v5 13/15] tegra: Enable driver model

Enable driver model for Tegra boards. Signed-off-by: Simon Glass Acked-by: Stephen Warren --- Changes in v5: None Changes in v4: None Changes in v3: - Enable dm command in this patch instead of the next Changes in v2: - Split out a separate patch to enable driver model for tegra include/con

[U-Boot] [PATCH v5 10/15] dm: Allow driver model tests only for sandbox

The GPIO tests require the sandbox GPIO driver, so cannot be run on other platforms. Similarly for the 'dm test' command. Signed-off-by: Simon Glass --- Changes in v5: None Changes in v4: None Changes in v3: None Changes in v2: - Add new patch to allow driver model tests only for sandbox test/

[U-Boot] [PATCH v5 12/15] tegra: dts: Bring in GPIO bindings from linux

These files are taken from Linux 3.14. Signed-off-by: Simon Glass Acked-by: Stephen Warren --- Changes in v5: None Changes in v4: - Remove 64-bit addresses which are not used in U-Boot Changes in v3: - Bring in GPIO bindings for tegra{30,114,124} also Changes in v2: - Add new patch to bring

[U-Boot] [PATCH v5 11/15] dm: Fix printf() strings in the 'dm' command

The values here are int, but the map_to_sysmem() call can return a long. Add a cast to deal with this. Signed-off-by: Simon Glass --- Changes in v5: None Changes in v4: None Changes in v3: None Changes in v2: - Add new patch to fix printf() strings in the 'dm' command test/dm/cmd_dm.c | 8

Re: [U-Boot] [PATCH] arm: Allow u-boot to run from offset base address

On 14-06-10 02:20 PM, Albert ARIBAUD wrote: Hi Steve, On Tue, 10 Jun 2014 12:38:42 -0700, Steve Rae wrote: On 14-06-10 11:13 AM, Wolfgang Denk wrote: Dear Steve, In message <539746c4.9040...@broadcom.com> you wrote: There could be "many" reasons why the CONFIG_SYS_TEXT_BASE is not ali

Re: [U-Boot] [PATCH] usb: ci_udc: terminate ep0 INs with a zlp when required

On Tuesday, June 10, 2014 at 11:27:39 PM, Stephen Warren wrote: > From: Stephen Warren > > Sometimes, a zero-length packet is required at the end of an IN > transaction so that the host knows the device is done sending data. > Enhance ci_udc to send a zlp when necessary. See the comments for > mo

Re: [U-Boot] [PATCH] dfu: Disable default calculation of CRC32

On Tuesday, June 10, 2014 at 12:25:59 PM, Lukasz Majewski wrote: > Patch (SHA1: bd694244db7bc969954) > dfu: Introduction of the "dfu_hash_algo" env variable for checksum method > setting > > already introduced more generic handling of the crc32 calculation. > Up till now the CRC32 of received data

Re: [U-Boot] [PULL] u-boot-usb/master

The previous PR was somehow lost I presume, so please find a bigger one: The following changes since commit 90b51c33f362926e17d4c07dcef1ce822abaa89f: Merge branch 'master' of git://git.denx.de/u-boot-arm (2014-05-30 11:34:39 -0400) are available in the git repository at: git://git.denx.de

Re: [U-Boot] [PATCH 1/4] usb: ci_udc: call udc_disconnect() from ci_pullup()

On Tuesday, June 10, 2014 at 07:02:35 PM, Stephen Warren wrote: > From: Stephen Warren > > ci_pullup()'s !is_on path contains a cut/paste copy of udc_disconnect(). > Remove the duplication by simply calling udc_disconnect() instead. > > Signed-off-by: Stephen Warren Applied all, thanks. Best

Re: [U-Boot] [PATCH 08/10] odroid: add board file for Odroid X2/U3 based on Samsung Exynos4412

Hi Przemyslaw, In U3 board, cooling pan is not work. I think, cooling pan setting is need in board_gpio_init(). (X2 board use cooling pan pwr form USB port) best regards, Inha Song. On Tue, 10 Jun 2014 13:33:02 +0200 Przemyslaw Marczak wrote: > This board file supports standard features of Odr

Re: [U-Boot] [PULL] u-boot-usb/master

On Wed, Jun 11, 2014 at 02:34:21AM +0200, Marek Vasut wrote: > The previous PR was somehow lost I presume, so please find a bigger one: > > The following changes since commit 90b51c33f362926e17d4c07dcef1ce822abaa89f: > > Merge branch 'master' of git://git.denx.de/u-boot-arm (2014-05-30 11:34:3

Re: [U-Boot] [PULL] u-boot-usb/master

On Wednesday, June 11, 2014 at 03:49:21 AM, Tom Rini wrote: > On Wed, Jun 11, 2014 at 02:34:21AM +0200, Marek Vasut wrote: > > The previous PR was somehow lost I presume, so please find a bigger one: > > > > The following changes since commit 90b51c33f362926e17d4c07dcef1ce822abaa89f: > > Merge b

Re: [U-Boot] [PATCH 04/10] samsung: misc: set the dfu bootloader setting at boot time.

Hi Przemyslaw, I have a question. "dfu_alt_bootloader" env is settings successfully > dfu_alt_bootloader=u-boot raw 0x3e 0x800 mmcpart 1 But, In order to replace bootloader binary in thor mode, How can I use the "dfu_alt_bootloader" env? DFU gadget use "dfu_alt_info" env for DFU entities config

Re: [U-Boot] [PATCH] include/dm.h: fix inclusion guard

On 10 June 2014 17:52, Jeroen Hofstee wrote: > cc: Simon Glass > Signed-off-by: Jeroen Hofstee Acked-by: Simon Glass ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] tps6586x.h: fix inclusion guard

On 10 June 2014 17:01, Jeroen Hofstee wrote: > cc: Simon Glass > Signed-off-by: Jeroen Hofstee Acked-by: Simon Glass ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] sandbox: restore ability to access host fs through standard commands

Hi, Stephen On 6/11/2014 6:43 AM, Stephen Warren wrote: From: Stephen Warren Commit 95fac6ab4589 "sandbox: Use os functions to read host device tree" removed the ability for get_device_and_partition() to handle the "host" device type, and redirect accesses to it to the host filesystem. This br

Re: [U-Boot] [U-boot] sandbox test script question

Hi, Tiger On 6/10/2014 7:01 PM, tiger...@via-alliance.com wrote: Hi, Simon: Is there any doc described how to use 'sb load host / sb ls host' etc cmds? so far seems no doc yet. If you want to access the host file system, you need apply a patch: http://patchwork.ozlabs.org/patch/358406/ (fix th

[U-Boot] WIP for SPI

Hi Jon, I thought I should mention that I spent time on a flight to look at SPI with driver model. I have put the WIP code in branch 'working' in u-boot-dm.git. Note it doesn't work, and is very early. Also note that many of the patches have not been posted - I just want to make it clear what I am

Re: [U-Boot] [U-boot] sandbox test script question

Hi Tiger, On 10 June 2014 23:36, Josh Wu wrote: > Hi, Tiger > > > On 6/10/2014 7:01 PM, tiger...@via-alliance.com wrote: >> >> Hi, Simon: >> Is there any doc described how to use 'sb load host / sb ls host' etc >> cmds? > > so far seems no doc yet. > If you want to access the host file system, yo

Re: [U-Boot] [PATCH] ubifs: don't self assign values

Hello Jeroen, Am 10.06.2014 23:27, schrieb Jeroen Hofstee: It seems the code tries to trick the compiler the argument is actually used. However compilers became too smart too fool them so easily an now warn. Checking gcc and clang does not seem to emit a warning. If so it should be decorated wit

Re: [U-Boot] [PATCH] arm: Allow u-boot to run from offset base address

Dear Albert, In message you wrote: > > 1. A typical ARM binary image is linked to a base address, defined by >preprocessor macro (aka U-Boot config option) CONFIG_SYS_TEXT_BASE. We shoul specifically keep in mind here that the "base address" is defined as the start address of the text segme

Re: [U-Boot] [PATCH] arm: Allow u-boot to run from offset base address

Dear Steve, In message <53979199.5010...@broadcom.com> you wrote: > > OK - I think that one of the alternate proposals would be to > conditionally reserve a "32 byte block" prior to the _start symbol (in > "arch/arm/cpu/armv8/start.S") which would then be filled in by a > post-processing step.

Re: [U-Boot] [PATCH] arm: Allow u-boot to run from offset base address

Dear Steve, In message <53979f6a.90...@broadcom.com> you wrote: > > Yes - copied from MMC to RAM, but not really necessary to do this at > "link stage or earlier" (we do it with a post-processing tool to prepend > a header onto 'u-boot.bin') What exactly is the fuction of this post-processing

Re: [U-Boot] [U-boot] sandbox test script question

Hi, Josh: Thanks a lot! Follwing these steps, I could load itb to memory: 1. sb bind 0 /lionfs 2. ext4load host 0 10 testkernel.itb Best wishes, ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] arm: Allow u-boot to run from offset base address

Hi Wolfgang, On Wed, 11 Jun 2014 06:49:28 +0200, Wolfgang Denk wrote: > Dear Steve, > > In message <53979199.5010...@broadcom.com> you wrote: > > > > OK - I think that one of the alternate proposals would be to > > conditionally reserve a "32 byte block" prior to the _start symbol (in > > "a

<    1   2