Re: [U-Boot] [PATCH V2 1/2] test/py: fix off-by-one error in spawn matching code

On 5 February 2016 at 18:04, Stephen Warren wrote: > From: Stephen Warren > > A regex match object's .end() value is already the index after the match, > not the index of the last character in the match, so there's no need to > add 1 to point past the

Re: [U-Boot] [PATCH] test/py: support running sandbox under gdbserver

Hi Stephen, On 6 February 2016 at 13:34, Stephen Warren wrote: > On 02/06/2016 01:30 PM, Simon Glass wrote: >> On 4 February 2016 at 16:11, Stephen Warren wrote: >>> Implement command--line option --gdbserver COMM, which does two things: >>> >>> a)

[U-Boot] [PATCH 2/2] video: tegra: Enable the 'lcd' env variable work-around

Enable this option on all tegra boards. Signed-off-by: Simon Glass --- arch/arm/mach-tegra/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/mach-tegra/Kconfig b/arch/arm/mach-tegra/Kconfig index 66298d5..3540eda 100644 --- a/arch/arm/mach-tegra/Kconfig

[U-Boot] [PATCH 1/2] dm: video: Add a temporary work-around for old stdout var

Boards with a saved environment may use 'lcd' in their stdout environment variable, expecting that this will enable output to the LCD. When the board moves to use driver model for video, this will no-longer work. Add a work-around to fix this. A warning messages is printed, and we will remove the

Re: [U-Boot] [PATCH 2/2] video: tegra: Enable the 'lcd' env variable work-around

On Sat, 6 Feb 2016 14:31:38 -0700 Simon Glass s...@chromium.org wrote: > Enable this option on all tegra boards. > > Signed-off-by: Simon Glass > --- > > arch/arm/mach-tegra/Kconfig | 1 + > 1 file changed, 1 insertion(+) Acked-by: Anatolij Gustschin

Re: [U-Boot] [PATCH 1/2] dm: video: Add a temporary work-around for old stdout var

Hi Simon, On Sat, 6 Feb 2016 14:31:37 -0700 Simon Glass s...@chromium.org wrote: > Boards with a saved environment may use 'lcd' in their stdout environment > variable, expecting that this will enable output to the LCD. When the board > moves to use driver model for video, this will no-longer

Re: [U-Boot] [PATCH V2 1/2] test/py: fix off-by-one error in spawn matching code

On 6 February 2016 at 13:30, Simon Glass wrote: > On 5 February 2016 at 18:04, Stephen Warren wrote: >> From: Stephen Warren >> >> A regex match object's .end() value is already the index after the match, >> not the index of the last

Re: [U-Boot] [PATCH] test/py: HTML awesome!

On 6 February 2016 at 13:29, Simon Glass wrote: > On 3 February 2016 at 16:46, Stephen Warren wrote: >> From: Stephen Warren >> >> Implement three improvements to the HTML log file: >> - Ability to expand/contract sections. All

Re: [U-Boot] [PATCH V2 2/2] test/py: capture the entire U-Boot version at boot

On 6 February 2016 at 13:30, Simon Glass wrote: > On 5 February 2016 at 18:04, Stephen Warren wrote: >> From: Stephen Warren >> >> The existing regex simply ensures that the captured version string doesn't >> go past the end of a

Re: [U-Boot] [PATCH] test/py: support running sandbox under gdbserver

On 02/06/2016 01:30 PM, Simon Glass wrote: > On 4 February 2016 at 16:11, Stephen Warren wrote: >> Implement command--line option --gdbserver COMM, which does two things: >> >> a) Run the sandbox process under gdbserver, using COMM as gdbserver's >>communication

Re: [U-Boot] [PATCH 1/2] ARM: rpi: update memory layout env. var. documentation

Hi Stephen, Looks good to me then. I wasn't sure how U-Boot was typically used on the RPi. Regards, Jonathan On Sunday, 7 February 2016, Stephen Warren wrote: > On 02/06/2016 12:30 AM, Jonathan Liu wrote: > > Hi Stephen, > > > > I actually read the DT loaded by RPi's

Re: [U-Boot] [PATCH] test/py: HTML awesome!

On 3 February 2016 at 16:46, Stephen Warren wrote: > From: Stephen Warren > > Implement three improvements to the HTML log file: > - Ability to expand/contract sections. All passing sections are contracted > at file load time so the user can

Re: [U-Boot] [PATCH] test/py: exit(1) if there are problems running py.test

On 3 February 2016 at 10:42, Stephen Warren wrote: > From: Stephen Warren > > The test/py/test.py wrapper script catches exceptions thrown when > exec()ing py.test in order to print a helpful error message. However, > the exception handling code

Re: [U-Boot] [PATCH V2 1/2] Makefile: generate symbol list from u-boot

On 2 February 2016 at 13:41, Stephen Warren wrote: > From: Stephen Warren > > This information may be useful for both debugging, and processes that want > to perform simple forms of introspection on the U-Boot binary, such as > determining the set of

Re: [U-Boot] [PATCH V2 2/2] test/py: run all "ut" subtests

On 2 February 2016 at 13:41, Stephen Warren wrote: > From: Stephen Warren > > Invoke each "ut"-based unit test as a separate pytest. > > Now that the DM unit test runs under test/py, remove the manual shell > script that invokes it. > > Signed-off-by:

Re: [U-Boot] [PATCH V2 2/2] test/py: capture the entire U-Boot version at boot

On 5 February 2016 at 18:04, Stephen Warren wrote: > From: Stephen Warren > > The existing regex simply ensures that the captured version string doesn't > go past the end of a line. We really want to grab as much as possible. Do > this by explicitly

Re: [U-Boot] [PATCH] test/py: support running sandbox under gdbserver

Hi Stephen, On 4 February 2016 at 16:11, Stephen Warren wrote: > From: Stephen Warren > > Implement command--line option --gdbserver COMM, which does two things: > > a) Run the sandbox process under gdbserver, using COMM as gdbserver's >

Re: [U-Boot] [PATCH] gunzip: remove avail_in recalculation

On 29 January 2016 at 13:26, Stephen Warren wrote: > From: Stephen Warren > > Current, the following passes: > ./u-boot -d arch/sandbox/dts/test.dtb -c 'ut_image_decomp' > but the following fails: > ./u-boot -d arch/sandbox/dts/test.dtb -c 'ut

Re: [U-Boot] [PATCH] test/py: exit(1) if there are problems running py.test

On 6 February 2016 at 13:29, Simon Glass wrote: > On 3 February 2016 at 10:42, Stephen Warren wrote: >> From: Stephen Warren >> >> The test/py/test.py wrapper script catches exceptions thrown when >> exec()ing py.test in order to

Re: [U-Boot] help on using patman

Hi Mugunthan, On 4 February 2016 at 23:24, Mugunthan V N wrote: > Hi Simon > > On Thursday 04 February 2016 09:21 PM, Simon Glass wrote: >> Hi Mugunthan, >> >> On 4 February 2016 at 08:09, Mugunthan V N wrote: >>> Hi Simon >>> >>> I tried to use patman

Re: [U-Boot] [RFC PATCH] itest: allow map_physmem to return 0 in limited cases

On 6 February 2016 at 13:29, Simon Glass wrote: > On 29 January 2016 at 16:10, Stephen Warren wrote: >> From: Stephen Warren >> >> On some systems, RAM starts at address 0. If the user executes itest >> against address 0 on such a

Re: [U-Boot] [PATCH] serial: pl01x: Add support for devices with the rate pre-configured.

Hi Eric, On 5 February 2016 at 14:43, Eric Anholt wrote: > For Raspberry Pi, we had the input clock rate to the pl011 fixed in > the rpi.c file, but it may be changed by firmware due to user changes > to config.txt. Since the firmware always sets up the uart (default > 115200

Re: [U-Boot] [PATCH] gunzip: remove avail_in recalculation

On 6 February 2016 at 13:29, Simon Glass wrote: > On 29 January 2016 at 13:26, Stephen Warren wrote: >> From: Stephen Warren >> >> Current, the following passes: >> ./u-boot -d arch/sandbox/dts/test.dtb -c 'ut_image_decomp' >>

Re: [U-Boot] [PATCH] serial: zynq: Change logic in putc

On 3 February 2016 at 07:34, Michal Simek wrote: > Sync logic with Linux kernel where TX empty flag is checked before char > is sent. > This logic is fixing problem with console on zynqmp platform. > > For example: > DRAM: 2 GiB > Enabling Caches... > EL Level: ��

Re: [U-Boot] [RFC PATCH] itest: allow map_physmem to return 0 in limited cases

On 29 January 2016 at 16:10, Stephen Warren wrote: > From: Stephen Warren > > On some systems, RAM starts at address 0. If the user executes itest > against address 0 on such a system, it will call map_physmem(0, ...) > which will return 0 back; mapping

Re: [U-Boot] [PATCH v2 3/7] drivers: block: disk-uclass: implement scsi_init()

Hi Bin, On 3 February 2016 at 04:59, Mugunthan V N wrote: > > Implement scsi_init() api to probe driver model based sata > devices. > > Signed-off-by: Mugunthan V N > --- > drivers/block/disk-uclass.c | 39 +++ > 1

Re: [U-Boot] [PATCH v2 00/23] dm: tegra: Convert tegra20 and tegra124 video drivers to driver model

Hi Stephen, On 1 February 2016 at 17:36, Stephen Warren wrote: > On 02/01/2016 05:28 PM, Simon Glass wrote: >> >> Hi Stephen, >> >> On 1 February 2016 at 17:17, Stephen Warren wrote: >>> >>> On 02/01/2016 05:05 PM, Simon Glass wrote:

Re: [U-Boot] [PATCH 3/4] spi: omap3: Convert to DM

Hi Simon, Tom, I assume the approach you are taking is also valuable for the i2c: omap24xx patch serie: http://lists.denx.de/pipermail/u-boot/2016-January/241676.html What are your recommendation about the pending patches ? Should i send back only the one not taking care of the DM conversion

Re: [U-Boot] [PATCH v2] Use correct spelling of "U-Boot"

Hi Bin, On Fri, 5 Feb 2016 19:30:11 -0800 Bin Meng bmeng...@gmail.com wrote: > Correct spelling of "U-Boot" shall be used in all written text > (documentation, comments in source files etc.). > > Signed-off-by: Bin Meng > Reviewed-by: Heiko Schocher >

Re: [U-Boot] [PATCH 5/6] bzlib: Try another way to fix an unused variable

On Sat, 30 Jan 2016 15:45:18 -0700 Simon Glass s...@chromium.org wrote: > Use __maybe_unused which should avoid the Coverity error. > > Reported-by: Coverity (CID: 134900) > > Signed-off-by: Simon Glass > --- > > lib/bzip2/bzlib_compress.c | 7 +++ > 1 file changed, 3

[U-Boot] [PATCH v3] net: davinci_emac: fix NULL check after pointer dereference

From: Vishwas Srivastava NULL check is made after the pointer dereference. This patch fixes this issue. Signed-off-by: Vishwas Srivastava CC: Joe Hershberger Reviewed-by: Tom Rini Acked-by: Joe

[U-Boot] [PATCH] video: bcm2835: use bus_to_phys() on FB address

The FB address is generated by the firmware running on the GPU/VideoCore and is a "bus" address. This address is the other side of an IOMMU from the ARM upon which U-Boot is running. Use bus_to_phys() to convert this to an ARM physical address. Without this, U-Boot seems to work just fine, but

[U-Boot] Pull request: u-boot-staging

Hi Tom, The following changes since commit 94985cc9d3d7ed991ca8b2627d5894df5ea68f49: Merge branch 'master' of git://git.denx.de/u-boot-usb (2016-02-04 06:56:23 -0500) are available in the git repository at: git://git.denx.de/u-boot-staging.git ag...@denx.de for you to fetch changes up

Re: [U-Boot] [PATCH v3] net: davinci_emac: fix NULL check after pointer dereference

On Sat, 6 Feb 2016 16:40:59 +0100 Anatolij Gustschin ag...@denx.de wrote: > From: Vishwas Srivastava > > NULL check is made after the pointer dereference. This patch > fixes this issue. > > Signed-off-by: Vishwas Srivastava > CC: Joe

Re: [U-Boot] [PATCH 1/2] ARM: rpi: update memory layout env. var. documentation

On 02/06/2016 12:30 AM, Jonathan Liu wrote: > Hi Stephen, > > I actually read the DT loaded by RPi's binary firmware on an RPi 2 in a > U-Boot script: > fdt addr ${fdt_addr_r} && fdt get value bootargs /chosen bootargs > fatload mmc 0:1 ${kernel_addr_r} uImage > bootm ${kernel_addr_r} -

Re: [U-Boot] [PATCH] video: bcm2835: use bus_to_phys() on FB address

Hi Stephen, On Sat, 6 Feb 2016 10:42:43 -0700 Stephen Warren swar...@wwwdotorg.org wrote: > The FB address is generated by the firmware running on the GPU/VideoCore > and is a "bus" address. This address is the other side of an IOMMU from > the ARM upon which U-Boot is running. Use

Re: [U-Boot] [PATCH V2] fdt: introduce fdtdec_get_child_count

On Mon, 1 Feb 2016 13:31:15 +0800 Peng Fan van.free...@gmail.com wrote: > Introduce fdtdec_get_child_count for get the number of subnodes > of one parent node. > > Signed-off-by: Peng Fan > Cc: Simon Glass > --- > > V2: > correct spell error. >

[U-Boot] [PATCH] net: fix wrong initialization in davinci-emac driver

From: Vishwas Srivastava emac module of the davinci platform supports only 8 tx and 8 rx channels (total 16). emac driver for davinci platform, however, while doing initialization of the dma descriptor head pointers, wrongly initializes the 16 head pointers (instead of

Re: [U-Boot] [PATCH 6/6] rockchip: Drop old CONFIG_VIDEO_ROTATION option

On Sat, 30 Jan 2016 15:45:19 -0700 Simon Glass s...@chromium.org wrote: > The option was renamed to CONFIG_CONSOLE_ROTATION and Rockchip boards > were not updated. However this option is is not needed by default for > Rockchip since we don't need a rotated console for current boards. So just >

Re: [U-Boot] 9c11135 breaks bootz

That fixed it! Thank you adam On Fri, Feb 5, 2016 at 6:51 AM, Masahiro Yamada wrote: > Hi Adam, > > > 2016-02-05 20:58 GMT+09:00 Adam Ford : >> I have the same problem on a TI DM3730 > > > Did you see this? > >

Re: [U-Boot] U-Boot 2016.01 on Hikey 96

Hi Stefano, On 26 January 2016 at 15:58, Stefano Babic wrote: > > I have a Hikey 96 and I wanted to check U-Boot on ARM-64. I followed > your README in hisilicon/hikey and I run all steps without any apparent > issue. U-Boot is current Tom's tree. Just this step: > sudo

Re: [U-Boot] [PATCH] x86: Clean up SPI flash drivers in defconfig

Hi Bin, On 6 February 2016 at 09:46, Bin Meng wrote: > Hi Jagan, Simon, > > On Thu, Jan 21, 2016 at 1:53 PM, Bin Meng wrote: >> Hi, >> >> On Tue, Dec 15, 2015 at 2:27 PM, Jagan Teki wrote: >>> >>> On 15 December 2015 at 11:48, Bin

Re: [U-Boot] [PATCH 4/6] video: test: Adjust order of file closure

On Sat, 30 Jan 2016 15:45:17 -0700 Simon Glass s...@chromium.org wrote: > Close the file earlier to hopefully fix a Coverity error. > > Reported-by: Coverity (CID: 134901) > > Signed-off-by: Simon Glass > --- > > test/dm/video.c | 2 +- > 1 file changed, 1 insertion(+), 1

[U-Boot] [PATCH] serial: pl01x: Add support for devices with the rate pre-configured.

For Raspberry Pi, we had the input clock rate to the pl011 fixed in the rpi.c file, but it may be changed by firmware due to user changes to config.txt. Since the firmware always sets up the uart (default 115200 output unless the user changes it), we can just skip our own uart init to simplify

[U-Boot] [PATCH 1/2] ARM: rpi: update memory layout env. var. documentation

Hi Stephen, I actually read the DT loaded by RPi's binary firmware on an RPi 2 in a U-Boot script: fdt addr ${fdt_addr_r} && fdt get value bootargs /chosen bootargs fatload mmc 0:1 ${kernel_addr_r} uImage bootm ${kernel_addr_r} - ${fdt_addr_r} Essentially this loads the kernel with the same

[U-Boot] Fwd: Patman use (was: [PATCHv2] common/memsize.c: Simplify RAM size detection)

(Sorry Albert for the dup, I accidentally sent the reply only to you.) -- Forwarded message -- From: Eddy Petrișor Date: 2016-02-05 9:06 GMT+02:00 Subject: Re: Patman use (was: [U-Boot] [PATCHv2] common/memsize.c: Simplify RAM size detection) To: Albert

Re: [U-Boot] [PATCH 2/6] cmd: Fix control bmp_display()

On Sat, 30 Jan 2016 15:45:15 -0700 Simon Glass s...@chromium.org wrote: > All paths should share the same return. > > Reported-by: Coverity (CID:134903) > > Signed-off-by: Simon Glass > --- > > cmd/bmp.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) applied to

Re: [U-Boot] [PATCH 1/6] microblaze: Correct build error in eth-uclass.c

Hi Simon, On Sat, 30 Jan 2016 15:45:14 -0700 Simon Glass s...@chromium.org wrote: > This fixes the following error when building microblaze-generic: > > net/eth-uclass.c: In function 'eth_post_probe': > net/eth-uclass.c:466:18: error: 'gd' undeclared (first use in this function) >

Re: [U-Boot] [PATCH 3/6] video: Use 'int' for loop variables instead of short

On Sat, 30 Jan 2016 15:45:16 -0700 Simon Glass s...@chromium.org wrote: > Using short doesn't save anything and is confusing when the width and height > variables are ulong. > > This may fix Coverity CID134902 but I doubt it. > > Signed-off-by: Simon Glass > --- > >

Re: [U-Boot] [PATCH] net: fix wrong initialization in davinci-emac driver

On Sat, 6 Feb 2016 15:28:26 +0100 Anatolij Gustschin ag...@denx.de wrote: > From: Vishwas Srivastava > > emac module of the davinci platform supports only 8 tx and 8 > rx channels (total 16). emac driver for davinci platform, > however, while doing initialization of the

[U-Boot] RealTek RTL8211E u-boot driver

Hi guys, I am having one issue with RealTek RTL8211E u-boot driver. I found you have added support for this phy in past. The problem which I have is with rtl8211x_parse_status() where you are working with page 11. What was wrong on genphy_parse_link function? The problem I do see is that

Re: [U-Boot] [PATCH] ppc: xilinx-ppc405-generic

Hi Ricardo, On Thu, 4 Feb 2016 14:45:00 +0100 Ricardo Ribalda Delgado ricardo.riba...@gmail.com wrote: > Fix device tree name: > > +(xilinx-ppc405-generic) Device Tree Source is not correctly specified. > +(xilinx-ppc405-generic) Please define 'CONFIG_DEFAULT_DEVICE_TREE' >