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 cc: Michael Jones --- board/matrix_vision/mvblx/sys_eeprom.c | 2 +- 1 file changed, 1

[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 Also, change bootdelay to 0 but allow pressing 'S' to stop at U-Boot prompt. Signed-off-by: Michael Jones --- include/configs/omap3_mvblx.h | 18 +- 1 files changed, 13 insertions(+), 5 deletions(-) diff --git a/include/configs/omap3_mvblx.h

[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 --- board/matrix_vision/mvblx/sys_eeprom.c | 24 include/configs/omap3_mvblx.h

[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 --- board/matrix_vision/mvblx/fpga.c | 14 +- include/configs/omap3_mvblx.h|1 + 2 files changed, 14 insertions(+), 1 deletions(-) diff --git a/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] 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 --- 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 --- 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 where "NAME_config" is t

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 --- 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 changed, 0 insert

[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 --- board/matrix_vision/common/mv_common.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --g

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 > wrote: >> Add support for the MATRIX VISION mvBlueLYNX-X, an OMAP3-based >> intelligent camera. >> >> Signed-off-by: Michael Jones > > Configuring for omap

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 Denk > --- >

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 message<1313571318-17795-1-git-send-email-michael.jo...@matrix-vision.de> > you wrote: >> Add support for the MATRIX VISION mvBlueLYNX-X, an OMAP3-based >> intelligent cam

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. >> > > > So is right that everything else in the responsibility of Sandeep won't > make it into t

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

2011-08-17 Thread Michael Jones
On Mon, Jul 18, 2011 at 9:40 AM, Michael Jones >> 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

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

2011-08-01 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 --- Changes for v4: - Added missing args for printf on out-of-range Changes for v3: - Rewrote error messages to

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 writes: >>> >>>> In the only implementation

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 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-re

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

2011-07-27 Thread Michael Jones
Signed-off-by: Michael Jones --- 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 --- a/drivers/i2c

[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 --- Changes for v3: - Rewrote error messages to be more descriptive. Changes for v2: - None. Resubmitting to

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_EEPR

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

2011-07-20 Thread Michael Jones
;mmcinfo': Currently this basically prints '0' for all the fields when the slot is empty. 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() re

[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 --- Changes for v2: - None. Resubmitting to include custodian in cc: drivers/i2c/omap24xx_i2c.c | 134

[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 --- Changes for v2: - None. Resubmitting to include custodian in c

[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 --- 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/common/cmd_mmc.c ind

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

2011-07-18 Thread Michael Jones
From: "Howard D. Gray" Signed-off-by: Michael Jones --- 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 ++ 2 files changed, 22 insertions(+), 1

[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 --- drivers/i2c/omap24xx_i2c.c | 134 ++- 1 files changed, 56 insertions

[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 --- 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 +++ b/common

[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 --- board/matrix_vision/common/mv_common.c |2 ++ 1 files changed, 2

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

2011-07-15 Thread Michael Jones
From: "Howard D. Gray" Signed-off-by: Michael Jones --- 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 --git a/arch/arm/cpu/armv7/omap3/sys_info.c b/ar

[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 --- 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..567099e 100644 --- a/drivers/fpga

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

2011-07-15 Thread Michael Jones
Signed-off-by: Michael Jones --- 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 struct { #define

[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 MAINTA

[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'

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

2011-07-15 Thread Michael Jones
Signed-off-by: Michael Jones --- 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( mac, 3, 1, do_mac

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

2011-07-15 Thread Michael Jones
Signed-off-by: Michael Jones --- 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 @@ -60,7 +60,7 @@ int altera_load

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

2011-07-15 Thread Michael Jones
Signed-off-by: Michael Jones --- 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 +- include/altera.h

[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

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, "I2

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

[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 eve

[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 re

[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 --- include/configs/CPC45.h|3 --- include/configs/CU824.h|3 --- include/configs/MigoR.h|1 - include