Re: [U-Boot] [PATCH 6/9] board/matrix_vision/mvblx/sys_eeprom.c: fix buffer overflow

2014-11-06 Thread Michael Jones
On 11/06/2014 02:03 PM, Wolfgang Denk wrote: Fix error detected by cppcheck: [board/matrix_vision/mvblx/sys_eeprom.c:353]: (error) Buffer is accessed out of bounds. Signed-off-by: Wolfgang Denk w...@denx.de cc: Michael Jones michael.jo...@matrix-vision.de --- board/matrix_vision/mvblx

[U-Boot] [PATCH 0/3] omap3: mvblx: small changes

2013-02-08 Thread Michael Jones
A few minor changes to mvblx Howard Gray (1): omap3: mvblx: change console to ttyO0 and make silent by default. Michael Jones (2): omap3: mvblx: select fpgafilename according to orientation omap3: mvblx: pass FPGA version to the kernel board/matrix_vision/mvblx/fpga.c | 14

[U-Boot] [PATCH 3/3] omap3: mvblx: pass FPGA version to the kernel

2013-02-08 Thread Michael Jones
Extract FPGA version from the .rbf and pass this info to the kernel. Signed-off-by: Michael Jones michael.jo...@matrix-vision.de --- board/matrix_vision/mvblx/fpga.c | 14 +- include/configs/omap3_mvblx.h|1 + 2 files changed, 14 insertions(+), 1 deletions(-) diff --git

[U-Boot] [PATCH 2/3] omap3: mvblx: select fpgafilename according to orientation

2013-02-08 Thread Michael Jones
Rather than load the FPGA file from the FAT partition, look at entry in system EEPROM to decide which file to retrieve directly from the EXT3 partition. Signed-off-by: Michael Jones michael.jo...@matrix-vision.de --- board/matrix_vision/mvblx/sys_eeprom.c | 24 include

[U-Boot] [PATCH 1/3] omap3: mvblx: change console to ttyO0 and make silent by default.

2013-02-08 Thread Michael Jones
From: Howard Gray howard.g...@matrix-vision.de Also, change bootdelay to 0 but allow pressing 'S' to stop at U-Boot prompt. Signed-off-by: Michael Jones michael.jo...@matrix-vision.de --- include/configs/omap3_mvblx.h | 18 +- 1 files changed, 13 insertions(+), 5 deletions

[U-Boot] [PATCH] Remove unused CONFIG_SYS_I2C_BUS[_SELECT]

2013-01-16 Thread Michael Jones
CONFIG_SYS_I2C_BUS and CONFIG_SYS_I2C_BUS_SELECT don't appear anywhere outside of config files. Signed-off-by: Michael Jones michael.jo...@matrix-vision.de --- As far as I can tell, these CONFIG options never have existed outside of these config files - looking at omap3_beagle.h as an example

[U-Boot] [PATCH] README: supported boards are in boards.cfg, not Makefile

2012-03-16 Thread Michael Jones
Signed-off-by: Michael Jones michael.jo...@matrix-vision.de --- README |8 +++- 1 files changed, 3 insertions(+), 5 deletions(-) diff --git a/README b/README index 8964672..a6a91be 100644 --- a/README +++ b/README @@ -3421,7 +3421,7 @@ is done by typing: make NAME_config

Re: [U-Boot] OMAP4 u-boot broken (was [PATCH] usb: align usb_endpoint_descriptor to 16-bit boundary)

2011-12-14 Thread Michael Jones
On 12/13/2011 02:29 PM, Aneesh V wrote: Hi Kristiansson, On Friday 21 October 2011 12:55 AM, Stefan Kristiansson wrote: The usb_endpoint_descriptor struct is 7 bytes large and is defined as an array (ep_desc[USB_MAXENDPOINTS]) in the usb_interface struct in include/usb.h This fact will result

[U-Boot] [PATCH] mvblx: punt unused clean/distclean targets

2011-11-07 Thread Michael Jones
Signed-off-by: Michael Jones michael.jo...@matrix-vision.de --- This would have been cleaned up by commit 464c7920 (punt unused clean/distclean targets), but these files were in the pipeline to go upstream when that patch was submitted. board/matrix_vision/mvblx/Makefile |6 -- 1 files

[U-Boot] [PATCH] mv_common.c: get rid of 'defined but not used' warning

2011-10-20 Thread Michael Jones
When CONFIG_ENV_IS_NOWHERE, got following warning: mv_common.c:32:14: warning: 'entries_to_keep' defined but not used. Get rid of this warning. Signed-off-by: Michael Jones michael.jo...@matrix-vision.de --- board/matrix_vision/common/mv_common.c |2 +- 1 files changed, 1 insertions(+), 1

Re: [U-Boot] [PATCH v5] OMAP3: mvblx: Initial support for mvBlueLYNX-X

2011-10-19 Thread Michael Jones
Hi Tom, On 10/18/2011 07:31 PM, Tom Rini wrote: On Wed, Oct 5, 2011 at 8:08 AM, Michael Jones michael.jo...@matrix-vision.de wrote: Add support for the MATRIX VISION mvBlueLYNX-X, an OMAP3-based intelligent camera. Signed-off-by: Michael Jonesmichael.jo...@matrix-vision.de Configuring

Re: [U-Boot] [PATCH v4] OMAP3: mvblx: Initial support for mvBlueLYNX-X

2011-10-07 Thread Michael Jones
Hi Wolfgang, On 10/06/2011 11:51 PM, Wolfgang Denk wrote: Dear Michael Jones, In message1313571318-17795-1-git-send-email-michael.jo...@matrix-vision.de you wrote: Add support for the MATRIX VISION mvBlueLYNX-X, an OMAP3-based intelligent camera. Signed-off-by: Michael Jonesmichael.jo

Re: [U-Boot] [PATCH] README: fix documentation of CONFIG_SHOW_BOOT_PROGRESS

2011-10-07 Thread Michael Jones
Hi Wolfgang, On 10/07/2011 10:04 AM, Wolfgang Denk wrote: SOme previous changes added code right in the middle of the typo- double capitalization description of CONFIG_SHOW_BOOT_PROGRESS. Move this text down. Fix formatting while we are at it. Signed-off-by: Wolfgang Denkw...@denx.de

Re: [U-Boot] [STATUS] ARM

2011-08-23 Thread Michael Jones
Hi Albert, On 08/22/2011 10:15 AM, Simon Schwarz wrote: Hi Albert, On 08/20/2011 01:12 PM, Albert ARIBAUD wrote: snip Sandeep has requested that I pick these two in u-boot-arm. snip So is right that everything else in the responsibility of Sandeep won't make it into the next

Re: [U-Boot] [PATCH v2 4/7] mmc: rescan fails on empty slot

2011-08-17 Thread Michael Jones
at 9:40 AM, Michael Jones michael.jo...@matrix-vision.de wrote: Fail in 'mmc rescan' if mmc_init() returns error I think, if we're going to do this, we should change them all. I agree it would be logical to return failed from the other mmc commands if mmc_init() fails. I can submit

Re: [U-Boot] [PATCH v2 6/7] ARMV7: OMAP: I2C driver: Write more than 1 byte at a time in i2c_write

2011-07-27 Thread Michael Jones
Hi Heiko, Thanks for the review. On 07/27/2011 08:07 AM, Heiko Schocher wrote: Hello Michael, Sorry for the long delay... Michael Jones wrote: This allows the EEPROM layer to send a single i2c write command per page, and wait CONFIG_SYS_EEPROM_PAGE_WRITE_DELAY_MS between i2c write

[U-Boot] [PATCH v3] ARMV7: OMAP: I2C driver: Write more than 1 byte at a time in i2c_write

2011-07-27 Thread Michael Jones
This allows the EEPROM layer to send a single i2c write command per page, and wait CONFIG_SYS_EEPROM_PAGE_WRITE_DELAY_MS between i2c write commands. Signed-off-by: Michael Jones michael.jo...@matrix-vision.de --- Changes for v3: - Rewrote error messages to be more descriptive. Changes for v2

[U-Boot] [PATCH] ARMV7: OMAP: I2C driver: cosmetic: make checkpatch-compatible

2011-07-27 Thread Michael Jones
Signed-off-by: Michael Jones michael.jo...@matrix-vision.de --- drivers/i2c/omap24xx_i2c.c | 105 ++-- 1 files changed, 53 insertions(+), 52 deletions(-) diff --git a/drivers/i2c/omap24xx_i2c.c b/drivers/i2c/omap24xx_i2c.c index 19e024d..c812001 100644

Re: [U-Boot] [PATCH 4/4] cmd_mac: fix help for 'mac read'

2011-07-27 Thread Michael Jones
Hi Detlev, On 07/15/2011 05:17 PM, Detlev Zundel wrote: Michael Jones michael.jo...@matrix-vision.de writes: In the only implementation of 'mac read', it doesn't display the contents of the eeprom as the help indicated unless compiled with DEBUG. It only re-reads the contents

Re: [U-Boot] [PATCH 4/4] cmd_mac: fix help for 'mac read'

2011-07-27 Thread Michael Jones
Hi Detlev, Thanks for the reply. On 07/27/2011 03:54 PM, Detlev Zundel wrote: Hi Michael, Hi Detlev, On 07/15/2011 05:17 PM, Detlev Zundel wrote: Michael Jones michael.jo...@matrix-vision.de writes: In the only implementation of 'mac read', it doesn't display the contents

Re: [U-Boot] [PATCH 4/7] mmc: rescan fails on empty slot

2011-07-20 Thread Michael Jones
. It probably would be more logical to print device init failed or similar instead. -Michael Michael Jones wrote: Fail in 'mmc rescan' if mmc_init() returns error Signed-off-by: Michael Jones michael.jo...@matrix-vision.de --- common/cmd_mmc.c |6 -- 1 files changed, 4 insertions(+), 2

[U-Boot] [PATCH v2 3/7] ARMV7: OMAP3: Add 37xx ESx revision numbers.

2011-07-18 Thread Michael Jones
From: Howard D. Gray howard.g...@matrix-vision.de Signed-off-by: Michael Jones michael.jo...@matrix-vision.de --- Changes for v2: - None. Resubmitting to include custodian in cc: arch/arm/cpu/armv7/omap3/sys_info.c | 13 - arch/arm/include/asm/arch-omap3/omap3.h | 10

[U-Boot] [PATCH v2 4/7] mmc: rescan fails on empty slot

2011-07-18 Thread Michael Jones
Fail in 'mmc rescan' if mmc_init() returns error Signed-off-by: Michael Jones michael.jo...@matrix-vision.de --- Changes for v2: - None. Resubmitting to include custodian in cc: common/cmd_mmc.c |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/common/cmd_mmc.c b

[U-Boot] [PATCH v2 5/7] mv_common.c: don't compile reset_environment if ENV_IS_NOWHERE

2011-07-18 Thread Michael Jones
Doesn't make sense to provide this function to boards which defined CONFIG_ENV_IS_NOWHERE. Such a board gets a linking error because common/env_nowhere.c doesn't define saveenv(). Signed-off-by: Michael Jones michael.jo...@matrix-vision.de --- Changes for v2: - None. Resubmitting to include

[U-Boot] [PATCH v2 6/7] ARMV7: OMAP: I2C driver: Write more than 1 byte at a time in i2c_write

2011-07-18 Thread Michael Jones
This allows the EEPROM layer to send a single i2c write command per page, and wait CONFIG_SYS_EEPROM_PAGE_WRITE_DELAY_MS between i2c write commands. Signed-off-by: Michael Jones michael.jo...@matrix-vision.de --- Changes for v2: - None. Resubmitting to include custodian in cc: drivers/i2c

[U-Boot] [PATCH 0/4] cosmetic

2011-07-15 Thread Michael Jones
Here are some minor changes I've been collecting. Most of it is purely cosmetic, but there are some printf and help output changes, too. Michael Jones (4): cosmetic altera: fix printf typo cmd_mac: cleanup help cmd_mac: fix help for 'mac read' README |6

[U-Boot] [PATCH 1/4] cosmetic

2011-07-15 Thread Michael Jones
Signed-off-by: Michael Jones michael.jo...@matrix-vision.de --- README |6 +++--- arch/arm/include/asm/global_data.h |2 +- common/cmd_eeprom.c|2 +- common/cmd_i2c.c |4 ++-- drivers/i2c/omap24xx_i2c.c |2

[U-Boot] [PATCH 2/4] altera: fix printf typo

2011-07-15 Thread Michael Jones
Signed-off-by: Michael Jones michael.jo...@matrix-vision.de --- drivers/fpga/altera.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/fpga/altera.c b/drivers/fpga/altera.c index 650f2ec..103d81c 100644 --- a/drivers/fpga/altera.c +++ b/drivers/fpga/altera.c

[U-Boot] [PATCH 3/4] cmd_mac: cleanup help

2011-07-15 Thread Michael Jones
Signed-off-by: Michael Jones michael.jo...@matrix-vision.de --- common/cmd_mac.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/common/cmd_mac.c b/common/cmd_mac.c index 1884c2a..0f924b7 100644 --- a/common/cmd_mac.c +++ b/common/cmd_mac.c @@ -30,7 +30,7 @@ U_BOOT_CMD

[U-Boot] [PATCH 4/4] cmd_mac: fix help for 'mac read'

2011-07-15 Thread Michael Jones
In the only implementation of 'mac read', it doesn't display the contents of the eeprom as the help indicated unless compiled with DEBUG. It only re-reads the contents of the EEPROM into memory. Displaying the contents of the EEPROM is done by passing no arguments to 'mac'. Signed-off-by: Michael

[U-Boot] [PATCH 0/7] OMAP3: Add support for mvBlueLYNX-X

2011-07-15 Thread Michael Jones
. Michael Jones (5): fpga: support FPP Cyclone configuration mmc: rescan fails on empty slot mv_common.c: don't compile reset_environment if ENV_IS_NOWHERE ARMV7: OMAP: I2C driver: Write more than 1 byte at a time in i2c_write mvblx: Initial support for mvBlueLYNX-X MAINTAINERS

[U-Boot] [PATCH 1/7] fpga: add #define for Altera Cyclone EP3C5

2011-07-15 Thread Michael Jones
Signed-off-by: Michael Jones michael.jo...@matrix-vision.de --- include/ACEX1K.h |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/include/ACEX1K.h b/include/ACEX1K.h index ae20164..2bb9226 100644 --- a/include/ACEX1K.h +++ b/include/ACEX1K.h @@ -76,6 +76,7 @@ typedef

[U-Boot] [PATCH 2/7] fpga: support FPP Cyclone configuration

2011-07-15 Thread Michael Jones
Support FPGAs which use Fast Passive Parallel configuration Signed-off-by: Michael Jones michael.jo...@matrix-vision.de --- drivers/fpga/cyclon2.c | 10 ++ 1 files changed, 10 insertions(+), 0 deletions(-) diff --git a/drivers/fpga/cyclon2.c b/drivers/fpga/cyclon2.c index 4622b4e

[U-Boot] [PATCH 3/7] ARMV7: OMAP3: Add 37xx ESx revision numbers.

2011-07-15 Thread Michael Jones
From: Howard D. Gray howard.g...@matrix-vision.de Signed-off-by: Michael Jones michael.jo...@matrix-vision.de --- arch/arm/cpu/armv7/omap3/sys_info.c | 13 - arch/arm/include/asm/arch-omap3/omap3.h | 10 ++ 2 files changed, 22 insertions(+), 1 deletions(-) diff

[U-Boot] [PATCH 5/7] mv_common.c: don't compile reset_environment if ENV_IS_NOWHERE

2011-07-15 Thread Michael Jones
Doesn't make sense to provide this function to boards which defined CONFIG_ENV_IS_NOWHERE. Such a board gets a linking error because common/env_nowhere.c doesn't define saveenv(). Signed-off-by: Michael Jones michael.jo...@matrix-vision.de --- board/matrix_vision/common/mv_common.c |2 ++ 1

[U-Boot] [PATCH 4/7] mmc: rescan fails on empty slot

2011-07-15 Thread Michael Jones
Fail in 'mmc rescan' if mmc_init() returns error Signed-off-by: Michael Jones michael.jo...@matrix-vision.de --- common/cmd_mmc.c |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/common/cmd_mmc.c b/common/cmd_mmc.c index 176646d..28918f6 100644 --- a/common/cmd_mmc.c

[U-Boot] [PATCH 6/7] ARMV7: OMAP: I2C driver: Write more than 1 byte at a time in i2c_write

2011-07-15 Thread Michael Jones
This allows the EEPROM layer to send a single i2c write command per page, and wait CONFIG_SYS_EEPROM_PAGE_WRITE_DELAY_MS between i2c write commands. Signed-off-by: Michael Jones michael.jo...@matrix-vision.de --- drivers/i2c/omap24xx_i2c.c | 134 ++- 1

Re: [U-Boot] I2C: OMAP: spurious i2c probe addresses

2011-05-26 Thread Michael Jones
On 05/25/2011 05:38 PM, Michael Jones wrote: While running v2011.06-rc1, I noticed some new behavior on my OMAP3 i2c bus. I tracked it to commit 0e57968a215d1b, I2C: OMAP: detect more devices when probing an i2c bus. It detects more devices indeed, such as some that don't even exist. Even

Re: [U-Boot] I2C: OMAP: spurious i2c probe addresses

2011-05-26 Thread Michael Jones
On 05/26/2011 11:23 AM, Nick Thompson wrote: On 26/05/11 08:03, Michael Jones wrote: On 05/25/2011 05:38 PM, Michael Jones wrote: While running v2011.06-rc1, I noticed some new behavior on my OMAP3 i2c bus. I tracked it to commit 0e57968a215d1b, I2C: OMAP: detect more devices when probing

[U-Boot] [RFC] testing mmc slot w/ generic mmc

2011-05-25 Thread Michael Jones
I have a custom OMAP3-based board with 2 MMC slots. During my boot sequence, I want to test whether a card is present in mmc slot 1, and boot from it if so. Otherwise it will continue to look at other options. Until now, w/o CONFIG_GENERIC_MMC, I have used 'mmc init 1' to do this, which would

[U-Boot] I2C: OMAP: spurious i2c probe addresses

2011-05-25 Thread Michael Jones
While running v2011.06-rc1, I noticed some new behavior on my OMAP3 i2c bus. I tracked it to commit 0e57968a215d1b, I2C: OMAP: detect more devices when probing an i2c bus. It detects more devices indeed, such as some that don't even exist. Even better than that, it detects different devices

[U-Boot] [PATCH] Remove remnants of obsolete CONFIG_SYS_GBL_DATA_SIZE comments

2011-04-19 Thread Michael Jones
commit 25ddd1fb0a2281b182529afbc8fda5de2dc16d96 left remnants of many comments about CONFIG_SYS_GBL_DATA_SIZE. Signed-off-by: Michael Jones michael.jo...@matrix-vision.de --- include/configs/CPC45.h|3 --- include/configs/CU824.h|3 --- include/configs/MigoR.h