Re: [PATCH] binman: Avoid requiring a home directory on startup

2023-02-14 Thread Mike Frysinger
On Tue, Feb 14, 2023 at 3:08 PM Tom Rini wrote: > Downloading things from the internet and putting them in to the default > PATH always and forever is also kinda not great? you just described a standard distribution. this is like literally how all of them work. not to mention every other

Re: [PATCH] binman: Avoid requiring a home directory on startup

2023-02-14 Thread Mike Frysinger
considering, iiuc, the user has to execute the command to opt-in to installing the programs, putting it into a dir that's in $PATH is reasonable i would quibble that ~/bin is archaic and everyone should be using ~/.local/bin nowadays. but that can be a followup.

Re: [U-Boot] [PATCH 3/5] blackfin: replace bfin_gen_rand_mac() with eth_random_addr()

2014-04-17 Thread Mike Frysinger
On Thu 17 Apr 2014 17:00:30 Masahiro Yamada wrote: --- a/include/configs/tcm-bf537.h +++ b/include/configs/tcm-bf537.h @@ -73,7 +73,7 @@ #define CONFIG_HOSTNAME tcm-bf537 /* Uncomment next line to use fixed MAC address */ /* #define CONFIG_ETHADDR02:80:ad:20:31:e8 */ -

Re: [U-Boot] [PATCH 0/5] Do not use __DATE__ and __TIME__ anymore

2014-04-17 Thread Mike Frysinger
On Thu 17 Apr 2014 17:00:27 Masahiro Yamada wrote: The aim of this series is to prohibit using __DATE__ and __TIME__. as long as the version.h isn't impacted, this should fine. i don't think it is as it looks like the build will generate U_BOOT_DATE and U_BOOT_TIME into a const string by

Re: [U-Boot] [RFC] i2c_reloc_fixup fails on m68k

2013-10-14 Thread Mike Frysinger
i2c_adap_p is already relocated. Which toolchain? Can anybody confirm this? Added Mike Frysinger, Sonic Zhang (for blackfin) Jason Jin (for m68k) and Macpaul Lin (for nds32) to Cc ... I think also m68k, backfin and nds32 systems are affected Sorry, I have no such system ... maybe

Re: [U-Boot] Change of NDS32 Custodian

2013-08-24 Thread Mike Frysinger
On Wednesday 31 July 2013 06:40:37 Marek Vasut wrote: Dear Macpaul Lin, Here is a short announcement about a change in the U-boot NDS32 custodian. I have no longer working in Andes Technology Corporation and cannot provide the community the best support on NDS32 architecture. Kuan-Yu

Re: [U-Boot] [PATCH v1 1/3] blackfin: Enable early print via the generic serial API.

2013-05-12 Thread Mike Frysinger
On Monday 13 May 2013 00:20:15 Sonic Zhang wrote: Remove blackfin specific implementation of the generic serial API when early print macro is defined. In BFIN_BOOT_BYPASS mode, don't call generic serial_puts, because early print in bypass mode is running before code binary is relocated to

Re: [U-Boot] [PATCH v1] bfin: Move gpio support for bf54x and bf60x into the generic driver folder.

2013-05-02 Thread Mike Frysinger
On Thursday 02 May 2013 01:55:25 Sonic Zhang wrote: The gpio spec for bf54x and bf60x differ a lot from the old gpio driver for bf5xx. A lot of machine macros are used to accomodate both code in one gpio driver. This patch split the old gpio driver and move new gpio2 support to the generic

Re: [U-Boot] [PATCH v1] mmc: bfin: Ensure MMR writing is done before next command.

2013-04-27 Thread Mike Frysinger
On Saturday 27 April 2013 07:04:10 Sonic Zhang wrote: From: Sonic Zhang sonic.zh...@analog.com - Ensure MMR writing is done before next command. - Invalidate the buffer before starting to read. this patch was written by Cliff, not yourself. the description really doesn't match reality, and

Re: [U-Boot] [PATCH v1 1/2] bfin: Remove spi dma function in bf5xx.

2013-03-12 Thread Mike Frysinger
On Monday 04 March 2013 02:20:08 Sonic Zhang wrote: From: Scott Jiang scott.jiang.li...@gmail.com BF5xx rx dma causes spi flash random read error. Accually spi controller has problems both on tx and rx dma. So remove spi dma support in u-boot. this is wrong, and imo, unnecessary. it's

Re: [U-Boot] [PATCH 01/15] fdt: Use sed instead of cpp to pre-process the dtc

2012-12-28 Thread Mike Frysinger
On Friday 28 December 2012 09:55:52 Simon Glass wrote: On Thu, Dec 27, 2012 at 4:03 PM, Stephen Warren wrote: On 12/26/2012 03:28 PM, Simon Glass wrote: Include file support in dtc is still not available in common distributions so we need to keep our preprocessing arrangement around for a

[U-Boot] [PATCH 1/3] fw_env: fix type of len

2012-11-10 Thread Mike Frysinger
-off-by: Mike Frysinger vap...@gentoo.org --- tools/env/fw_env.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tools/env/fw_env.c b/tools/env/fw_env.c index 9b023e8..02f97c0 100644 --- a/tools/env/fw_env.c +++ b/tools/env/fw_env.c @@ -379,7 +379,8 @@ int fw_env_write(char

[U-Boot] [PATCH 3/3] fw_env: fix building w/out a config.h

2012-11-10 Thread Mike Frysinger
Signed-off-by: Mike Frysinger vap...@gentoo.org --- tools/env/Makefile | 11 ++- tools/env/fw_env.h | 25 - 2 files changed, 10 insertions(+), 26 deletions(-) diff --git a/tools/env/Makefile b/tools/env/Makefile index ab73c8c..62a113a 100644 --- a/tools/env

[U-Boot] [PATCH 2/3] fw_env: fix incorrect usage of open(O_CREAT)

2012-11-10 Thread Mike Frysinger
/include/bits/fcntl2.h:50:24: error: call to '__open_missing_mode' declared with attribute error: open with O_CREAT in second argument needs 3 arguments Signed-off-by: Mike Frysinger vap...@gentoo.org --- tools/env/fw_env_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

Re: [U-Boot] [PATCH 2/2] bfin: Fix warning in bfin_mac

2012-09-12 Thread Mike Frysinger
but not used [-Wunused-but-set-variable] Signed-off-by: Marek Vasut ma...@denx.de Cc: Mike Frysinger vap...@gentoo.org Guess I should have Cced Joe too ... Acked-by: Joe Hershberger joe.hershber...@ni.com i can pick this up, or you can. it'll be another week before i'm back home working

Re: [U-Boot] [PATCH 1/2] bfin: Disable -fstack-usage

2012-09-11 Thread Mike Frysinger
On Tue, Sep 11, 2012 at 3:08 PM, Marek Vasut wrote: The GCC does not support this on blackfin, disable it. err, no, you're probably using gcc-4.5.x which didn't support -fstack-usage. that is not specific to Blackfin as gcc didn't add it until 4.6.x. why doesn't the cc-option check work ?

Re: [U-Boot] [PATCH 2/2] bfin: Fix warning in bfin_mac

2012-09-11 Thread Mike Frysinger
Acked-by: Mike Frysinger vap...@gentoo.org -mike ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 1/3] MIPS: factor out endianess flag handling to arch config.mk

2012-08-25 Thread Mike Frysinger
On Thursday 23 August 2012 18:50:47 daniel.schwierz...@gmail.com wrote: From: Daniel Schwierzeck daniel.schwierz...@gmail.com This is CPU independent and should be configured architecture-wide. nice Acked-by: Mike Frysinger vap...@gentoo.org -mike signature.asc Description

Re: [U-Boot] Blackfin BF54x Silicon revision 0.4 and uClinux

2012-08-25 Thread Mike Frysinger
On Friday 24 August 2012 10:09:37 Dimitar Penev wrote: My question is not directly applicable with u-boot but I think here I may find the answer which I still don't get from the Analog Device forum. well, they'd be the people to pester Does anyone got working hardware based on Blackfin BF54x

Re: [U-Boot] [PATCH 04/12] tools/env: Reduce the impact on real-time processes

2012-08-23 Thread Mike Frysinger
On Thursday 23 August 2012 12:26:08 Joe Hershberger wrote: On Wed, Aug 22, 2012 at 10:30 PM, Mike Frysinger wrote: On Friday 17 August 2012 16:49:38 Joe Hershberger wrote: + * Break reads up into very small chunks so fw_printenv doesn't + * block the kernel long enough to starve other

Re: [U-Boot] [PATCH 3/5] MX28: SPI: Supercharge the SPI driver

2012-08-22 Thread Mike Frysinger
On Tuesday 21 August 2012 22:17:27 Marek Vasut wrote: This change implements DMA chaining into SPI driver. This allows the transfers to go much faster, while also fixing SF issues. might be nice to have the word DMA in the patch summary MX28: SPI: Supercharge the SPI driver w/DMA -mike

Re: [U-Boot] Blackfin BF547 board testing.

2012-08-22 Thread Mike Frysinger
On Wednesday 22 August 2012 11:13:54 Dimitar Penev wrote: - GCC assumes rev 0.2 of the CPU is this OK with the rev 0.4 which we have on board? err, gcc assumes nothing. this is controlled by your board config's CONFIG_BFIN_CPU setting. The first reason for this behavior seems to be memory

Re: [U-Boot] [PATCH 3/5] MX28: SPI: Supercharge the SPI driver

2012-08-22 Thread Mike Frysinger
On Wednesday 22 August 2012 13:43:19 Marek Vasut wrote: Dear Mike Frysinger, On Tuesday 21 August 2012 22:17:27 Marek Vasut wrote: This change implements DMA chaining into SPI driver. This allows the transfers to go much faster, while also fixing SF issues. might be nice to have

Re: [U-Boot] [PATCH 3/6] net: asix: split out basic reset function

2012-08-22 Thread Mike Frysinger
On Wednesday 22 August 2012 06:09:04 Lucas Stach wrote: The basic device reset ensures that the device is ready to service commands and does not need to get redone before each network operation. Split out the basic reset from asix_init() and instead call it from asix_eth_get_info(), so that

Re: [U-Boot] [Patch V3 0/4] add mips64 cpu support

2012-08-22 Thread Mike Frysinger
On Monday 20 August 2012 10:22:22 Zhizhou Zhang wrote: This patch add mips64 cpu support. Changes in V3: - merge related files into one patch, no longer one file one patch. - add detailed commit message. - remove standalone example. it's too complicate. do you keep sending these

Re: [U-Boot] [Patch V3 1/4] [MIPS] Add support for MIPS64 cpus

2012-08-22 Thread Mike Frysinger
On Monday 20 August 2012 10:22:23 Zhizhou Zhang wrote: +void __weak _machine_restart(void) +{ +} this should be: void __noreturn __weak _machine_restart(void) { while (1); } there should also be a prototype for this in one of the mips64 headers +int do_reset(cmd_tbl_t *cmdtp, int

Re: [U-Boot] [[Patch V2] mips: 01/16] add mips64 standalone support

2012-08-22 Thread Mike Frysinger
On Saturday 18 August 2012 22:25:24 Zhi-zhou Zhang wrote: On 8/18/12, Mike Frysinger vap...@gentoo.org wrote: On Saturday 18 August 2012 08:22:51 Zhi-zhou Zhang wrote: On Sat, Aug 18, 2012 at 3:31 AM, Mike Frysinger wrote: On Friday 17 August 2012 11:30:44 Zhizhou Zhang wrote

Re: [U-Boot] [PATCH 01/12] tools/env: Use a board-specific default env

2012-08-22 Thread Mike Frysinger
On Friday 17 August 2012 16:49:35 Joe Hershberger wrote: */ #define CONFIG_FILE /etc/fw_env.config +#ifndef CONFIG_FILE this doesn't make any sense. CONFIG_FILE is defined literally right above this check. -mike signature.asc Description: This is a digitally signed message part.

Re: [U-Boot] [PATCH 02/12] tools/env: Remove unneeded complexity

2012-08-22 Thread Mike Frysinger
On Friday 17 August 2012 16:49:36 Joe Hershberger wrote: --- a/tools/env/fw_env.c +++ b/tools/env/fw_env.c - memset(value, 0, len - strlen(name)); + memset(value, 0, len); side note: this memset is mostly useles as the value buffer largely gets

Re: [U-Boot] [PATCH 04/12] tools/env: Reduce the impact on real-time processes

2012-08-22 Thread Mike Frysinger
On Friday 17 August 2012 16:49:38 Joe Hershberger wrote: + * Break reads up into very small chunks so fw_printenv doesn't + * block the kernel long enough to starve other kernel tasks. err, this sounds like a bug in your kernel driver. why should userspace be working

Re: [U-Boot] [PATCH 05/12] tools/env: Serialize calls to fw_*env

2012-08-22 Thread Mike Frysinger
On Friday 17 August 2012 16:49:39 Joe Hershberger wrote: Use a lock file at /var/lock/fw_printenv.lock. the lock should be per-MTD, not per-system. + if (-1 == flock(lockfd, LOCK_EX)) { i guess flock() on the fd returned from the mtd device itself doesn't work ? -mike signature.asc

Re: [U-Boot] [PATCH 06/12] env: Make the silent env var take effect immediately

2012-08-22 Thread Mike Frysinger
On Friday 17 August 2012 16:49:40 Joe Hershberger wrote: --- a/common/cmd_nvedit.c +++ b/common/cmd_nvedit.c +#if defined(CONFIG_SILENT_CONSOLE) \ + defined(CONFIG_SILENT_CONSOLE_UPDATE_ON_SET) i wonder if the CONFIG_SILENT_CONSOLE check is needed. setting the latter without the

Re: [U-Boot] [PATCH 10/12] env: acl: Add environment variable access control list

2012-08-22 Thread Mike Frysinger
On Friday 17 August 2012 16:49:44 Joe Hershberger wrote: --- a/common/cmd_nvedit.c +++ b/common/cmd_nvedit.c +#if defined(CONFIG_ENV_ACL) +#include env_acl.h +#endif the header should not need protection just to be included +#ifdef CONFIG_ENV_ACL + if

Re: [U-Boot] [PATCH 12/12] env: cosmetic: Consilidate the default env definition

2012-08-22 Thread Mike Frysinger
On Friday 17 August 2012 16:49:46 Joe Hershberger wrote: There used to be a huge structure duplicated 3 times in the source. if this actually works: Acked-by: Mike Frysinger vap...@gentoo.org -mike signature.asc Description: This is a digitally signed message part

Re: [U-Boot] [PATCH 03/16] Blackfin: Bf60x: support big cplb page

2012-08-22 Thread Mike Frysinger
On Monday 20 August 2012 04:00:21 Bob Liu wrote: On Wed, Aug 8, 2012 at 12:48 PM, Mike Frysinger wrote: On Tuesday 07 August 2012 04:07:42 Bob Liu wrote: --- a/arch/blackfin/lib/board.c +++ b/arch/blackfin/lib/board.c void init_cplbtables(void) { ... +#if defined(__ADSPBF60x__

Re: [U-Boot] [PATCH 06/16] Blackfin: Bf609-ezkit: implement soft switch

2012-08-22 Thread Mike Frysinger
On Monday 20 August 2012 04:11:17 Bob Liu wrote: On Wed, Aug 8, 2012 at 12:59 PM, Mike Frysinger vap...@gentoo.org wrote: start with the address that the newest boards are using, and then fallback to the older ones. this way there's no runtime penalty on newer boards, but older ones

Re: [U-Boot] [Patch v3 1/2] net: introduce transparent driver private in ueth_data

2012-08-21 Thread Mike Frysinger
On Tuesday 21 August 2012 09:23:12 Lucas Stach wrote: Avoid clutter in ueth_data. Individual drivers should not mess with structures belonging to the core like this. nice work -mike signature.asc Description: This is a digitally signed message part.

Re: [U-Boot] [Patch v3 2/2] net: asix: add support for AX88772B

2012-08-21 Thread Mike Frysinger
On Tuesday 21 August 2012 09:23:13 Lucas Stach wrote: + struct ueth_data *dev = (struct ueth_data *)eth-priv; + struct asix_private *priv = (struct asix_private *)dev-dev_priv; not that it's harmful, but priv is void*, so the casts aren't needed + ALLOC_CACHE_ALIGN_BUFFER(unsigned

Re: [U-Boot] [PATCH 1/4] spiflash: at25: using common spi flash operation

2012-08-21 Thread Mike Frysinger
On Tuesday 21 August 2012 07:26:27 Andreas Bießmann wrote: On 20.08.2012 08:32, Bo Shen wrote: Using common spi flash operation function to replace private operation funtion This patch is based on http://patchwork.ozlabs.org/patch/177896/ which has been merged by Mike frysinger Mike

Re: [U-Boot] [PATCH 2/4] spi: atmel: add WDRBT bit to avoid receive overrun

2012-08-21 Thread Mike Frysinger
On Tuesday 21 August 2012 07:11:18 Andreas Bießmann wrote: On 20.08.2012 08:32, Bo Shen wrote: The atmel at91sam9x5 series spi has feature to avoid receive overren Using the patch to enable it Signed-off-by: Bo Shen voice.s...@atmel.com Acked-by: Andreas Bießmann

Re: [U-Boot] [PATCH v2] net: asix: add support for AX88772B

2012-08-20 Thread Mike Frysinger
On Monday 20 August 2012 13:52:18 Lucas Stach wrote: + memset(buf, 0, ETH_ALEN); this memset is pointless + for (i = 0; i (ETH_ALEN 1); i++) { + memset(read_buf, 0, 2); this memset is pointless + memcpy((buf + i*2), read_buf, 2); use spaces: i * 2 +

Re: [U-Boot] [PATCH 6/6] fdt: Add a Linux tool for reading values from FDT files

2012-08-20 Thread Mike Frysinger
On Monday 20 August 2012 19:40:45 Joe Hershberger wrote: On Fri, Aug 17, 2012 at 6:35 PM, Mike Frysinger wrote: On Friday 17 August 2012 16:34:40 Joe Hershberger wrote: Designed to be able to access itb files on a filesystem or an mtd partition. Supports print and list (like the fdt

Re: [U-Boot] [PATCH 6/6] fdt: Add a Linux tool for reading values from FDT files

2012-08-20 Thread Mike Frysinger
On Monday 20 August 2012 20:51:32 Joe Hershberger wrote: On Mon, Aug 20, 2012 at 7:24 PM, Mike Frysinger wrote: On Monday 20 August 2012 19:40:45 Joe Hershberger wrote: On Fri, Aug 17, 2012 at 6:35 PM, Mike Frysinger wrote: On Friday 17 August 2012 16:34:40 Joe Hershberger wrote

Re: [U-Boot] [PATCH v5] arm: rmobile: Add support Renesas SH73A0

2012-08-19 Thread Mike Frysinger
On Sunday 19 August 2012 10:40:05 Nobuhiro Iwamatsu wrote: --- /dev/null +++ b/arch/arm/cpu/armv7/rmobile/board.c +#ifdef CONFIG_DISPLAY_BOARDINFO +int checkboard(void) +{ + printf(Board: %s\n, sysinfo.board_string); + return 0; +} +#endif /* CONFIG_DISPLAY_BOARDINFO */ seems

Re: [U-Boot] [[Patch V2] mips: 01/16] add mips64 standalone support

2012-08-18 Thread Mike Frysinger
On Saturday 18 August 2012 08:22:51 Zhi-zhou Zhang wrote: On Sat, Aug 18, 2012 at 3:31 AM, Mike Frysinger wrote: On Friday 17 August 2012 11:30:44 Zhizhou Zhang wrote: --- a/arch/mips/config.mk +++ b/arch/mips/config.mk +ifeq $(CPU) mips64 +CONFIG_STANDALONE_LOAD_ADDR

Re: [U-Boot] [PATCH 1/9] SOUND: Add I2S driver

2012-08-17 Thread Mike Frysinger
On Friday 17 August 2012 08:48:59 Andrew Dyer wrote: The code in the i2s.c file is still full of samsung soc specific stuff, so I think the filename is misleading. Something like samsung-i2s.c would be more appropriate. +1 -mike signature.asc Description: This is a digitally signed message

Re: [U-Boot] [PATCH 1/3] spi: add atmel at25df321 serial flash support

2012-08-17 Thread Mike Frysinger
On Friday 17 August 2012 01:08:01 Bo Shen wrote: On 8/17/2012 12:18, Mike Frysinger wrote: On Thursday 16 August 2012 00:44:25 Bo Shen wrote: Add atmel at25df321 serial flash support After reading the spi framework. I found there are common interface for serial flash. So, I plan to use

Re: [U-Boot] [[Patch V2] mips: 01/16] add mips64 standalone support

2012-08-17 Thread Mike Frysinger
On Friday 17 August 2012 11:30:44 Zhizhou Zhang wrote: --- a/arch/mips/config.mk +++ b/arch/mips/config.mk +ifeq $(CPU) mips64 +CONFIG_STANDALONE_LOAD_ADDR ?= 0xFfffFfff8020 -T mips64.lds +else CONFIG_STANDALONE_LOAD_ADDR ?= 0x8020 -T mips.lds +endif the cpu config.mk is sourced

Re: [U-Boot] [[Patch V2] mips: 03/16] add cpu/mips64/config.mk

2012-08-17 Thread Mike Frysinger
On Friday 17 August 2012 11:30:46 Zhizhou Zhang wrote: --- /dev/null +++ b/arch/mips/cpu/mips64/config.mk +ENDIANNESS = -EL + +MIPSFLAGS += $(ENDIANNESS) seems to me this logic should get moved out of mips32/config.mk and into the top level mips config.mk -mike signature.asc Description:

Re: [U-Boot] [[Patch V2] mips: 04/16] add cpu/mips64/cpu.c

2012-08-17 Thread Mike Frysinger
On Friday 17 August 2012 11:30:47 Zhizhou Zhang wrote: --- /dev/null +++ b/arch/mips/cpu/mips64/cpu.c +void __attribute__((weak)) _machine_restart(void) +{ +} change to __weak (include linux/compiler.h if you need to). +int do_reset(cmd_tbl_t *cmdtp, int flag, int argc, char * const

Re: [U-Boot] [[Patch V2] mips: 06/16] add cpu/mips64/start.S

2012-08-17 Thread Mike Frysinger
On Friday 17 August 2012 11:30:49 Zhizhou Zhang wrote: remove cache init this commit message doesn't make much sense. if it meant to be covering what changed since v1, put it below the --- marker below. --- /dev/null +++ b/arch/mips/cpu/mips64/start.S + .globl _start + .text

Re: [U-Boot] [PATCH 2/2] Add unlzo command

2012-08-17 Thread Mike Frysinger
On Friday 17 August 2012 16:59:44 Joe Hershberger wrote: --- a/common/Makefile +++ b/common/Makefile ifdef CONFIG_LZMA COBJS-$(CONFIG_CMD_UNLZMA) += cmd_unlzma.o endif +ifdef CONFIG_LZO +COBJS-$(CONFIG_CMD_UNLZO) += cmd_unlzo.o +endif imo, these ifdefs shouldn't exist. the commands

Re: [U-Boot] [PATCH 1/2] Add unlzma command

2012-08-17 Thread Mike Frysinger
On Friday 17 August 2012 16:59:43 Joe Hershberger wrote: Provide access to the lzma decompressor from the command line. same feedback for this file as the unlzo command feedback -mike signature.asc Description: This is a digitally signed message part.

Re: [U-Boot] [PATCH 1/3] hush: Add default value substitution support

2012-08-17 Thread Mike Frysinger
On Friday 17 August 2012 16:26:29 Joe Hershberger wrote: Use standard sh syntax: ${VAR:-default} Use default value: if VAR is set and non-null, expands to $VAR. Otherwise, expands to default. ${VAR:=default} Set default value: if VAR is set and non-null, expands to $VAR.

Re: [U-Boot] [PATCH 3/3] hush: Include file and line number when reporting syntax errors

2012-08-17 Thread Mike Frysinger
On Friday 17 August 2012 16:26:31 Joe Hershberger wrote: Make debugging script problems easier just like non-u-boot. err, but you're posting hush.c all the time, and the line number of the source code in hush.c, not the line of the shell script --- a/common/hush.c +++ b/common/hush.c

Re: [U-Boot] [PATCH 6/6] fdt: Add a Linux tool for reading values from FDT files

2012-08-17 Thread Mike Frysinger
On Friday 17 August 2012 16:34:40 Joe Hershberger wrote: Designed to be able to access itb files on a filesystem or an mtd partition. Supports print and list (like the fdt command) and also offset for finding the offset and size of a given property in an FDT file. This is especially

Re: [U-Boot] [PATCH] Cleanup cache command prints

2012-08-17 Thread Mike Frysinger
On Friday 17 August 2012 16:54:23 Joe Hershberger wrote: Only print when queried, not every time the setting is changed. considering there's a lot more style changes here than functional, it'd probably be good to split up into two as for the basic idea, makes sense to me -mike signature.asc

Re: [U-Boot] [PATCH] Output strings from echo with puts where easy

2012-08-17 Thread Mike Frysinger
On Friday 17 August 2012 16:55:18 Joe Hershberger wrote: --- a/common/cmd_echo.c +++ b/common/cmd_echo.c - while ((c = *p++) != '\0') { - if (c == '\\' *p == 'c') { - putnl = 0; - p++; -

Re: [U-Boot] [PATCH] Change dead code in test cmd to debug output

2012-08-17 Thread Mike Frysinger
On Friday 17 August 2012 16:56:12 Joe Hershberger wrote: --- a/common/cmd_test.c +++ b/common/cmd_test.c -#if 0 +#ifdef DEBUG { - printf(test:); + debug(test(%d):, argc); left = 1; while (argv[left]) -

Re: [U-Boot] [PATCH] Add a new ini command

2012-08-17 Thread Mike Frysinger
On Friday 17 August 2012 16:56:57 Joe Hershberger wrote: This allows you to read ini-formatted data from anywhere and then import one of the sections into the environment This is based on rev 16 at http://code.google.com/p/inih/ document it in top level README ? --- /dev/null +++

Re: [U-Boot] [PATCH] Add a command to access the system timer

2012-08-17 Thread Mike Frysinger
On Friday 17 August 2012 16:57:58 Joe Hershberger wrote: Two sub-commands... start and get. * start sets the reference. * get prints out the time since the last start (in sec.msec format). If get is called without start, returns time since boot. Simple way to benchmark an operation: timer

Re: [U-Boot] [PATCH] Add a memory get command

2012-08-17 Thread Mike Frysinger
On Friday 17 August 2012 16:58:41 Joe Hershberger wrote: --- a/common/cmd_mem.c +++ b/common/cmd_mem.c +#ifdef CONFIG_CMD_MEM_GET not the greatest name ... +int do_mem_mg(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) static + size = cmd_get_data_size(argv[0], 4); +

Re: [U-Boot] [PATCH 1/4] Implement verify option for sha1sum command

2012-08-17 Thread Mike Frysinger
On Friday 17 August 2012 17:00:45 Joe Hershberger wrote: Loosely based on CONFIG_CRC32_VERIFY. i've always found that code logic in desperate need of cleaning up. let's please not copy this wart. can't we unify these two code paths ? -mike signature.asc Description: This is a digitally

Re: [U-Boot] [PATCH 08/12] env: Check for NULL pointer in envmatch()

2012-08-17 Thread Mike Frysinger
On Friday 17 August 2012 16:49:42 Joe Hershberger wrote: If the pointer passed into envmatch() is NULL, return -1 instead of crashing. so don't pass in NULL ? -mike signature.asc Description: This is a digitally signed message part. ___ U-Boot

Re: [U-Boot] [PATCH 1/2] net: Allow netconsole settings to change after nc_start

2012-08-17 Thread Mike Frysinger
On Friday 17 August 2012 15:48:20 Joe Hershberger wrote: --- a/drivers/net/netconsole.c +++ b/drivers/net/netconsole.c +static int refresh_settings_from_env(void) this func always returns 0. why not change it to void to simplify the code (source and generated) ? + char *p; const

Re: [U-Boot] [PATCH 2/7 V4] SPI: Add W25Q32 to Winbond SPI flash table

2012-08-17 Thread Mike Frysinger
On Friday 17 August 2012 04:28:02 Joonyoung Shim wrote: 2012/8/2 Rajeshwari Shinde rajeshwar...@samsung.com: + { + .id = 0x5014, + .l2_page_size = 8, + .pages_per_sector = 16, +

Re: [U-Boot] [PATCH v5] Consolidate bootcount code into drivers/bootcount

2012-08-16 Thread Mike Frysinger
On Thursday 16 August 2012 23:55:41 Stefan Roese wrote: This patch moves all bootcount implementations into a common directory: drivers/bootcount. The generic bootcount driver is now usable not only by powerpc platforms, but others as well. Acked-by: Mike Frysinger vap...@gentoo.org -mike

Re: [U-Boot] [PATCH 1/3] spi: add atmel at25df321 serial flash support

2012-08-16 Thread Mike Frysinger
On Thursday 16 August 2012 00:44:25 Bo Shen wrote: Add atmel at25df321 serial flash support thanks, merged into my spi flash branch -mike signature.asc Description: This is a digitally signed message part. ___ U-Boot mailing list U-Boot@lists.denx.de

Re: [U-Boot] [PATCH] README: Update ver env var description

2012-08-16 Thread Mike Frysinger
On Monday 13 August 2012 09:01:14 Benoît Thébaudeau wrote: Commit 155cb01 replaced the read-only property of the ver env var with an auto-restoring behavior. Update the README file accordingly. Acked-by: Mike Frysinger vap...@gentoo.org -mike signature.asc Description: This is a digitally

Re: [U-Boot] [PATCH] sf: winbond: fix page_size

2012-08-14 Thread Mike Frysinger
On Monday 13 August 2012 18:46:21 Stephen Warren wrote: Commit a4ed3b6 sf: inline data constants modified winbond.c's page_size from 256 to 4096. This prevents either/both of sf write writing the correct data, or sf read from reading the correct data back. yeah, sorry about that. merged into

[U-Boot] Pull request u-boot-blackfin.git (sf branch)

2012-08-14 Thread Mike Frysinger
The following changes since commit 4d3c95f5ea7c737a21cd6b9c59435ee693b3f127: zfs: Add ZFS filesystem support (2012-08-09 23:42:20 +0200) are available in the git repository at: git://www.denx.de/git/u-boot-blackfin.git sf for you to fetch changes up to

Re: [U-Boot] [PATCH] sf: spansion: Add support for S25FL256S

2012-08-14 Thread Mike Frysinger
On Tuesday 14 August 2012 07:11:22 Michal Simek wrote: Add support for Spansion S25FL256S SPI flash. thanks, added to my sf branch -mike signature.asc Description: This is a digitally signed message part. ___ U-Boot mailing list U-Boot@lists.denx.de

Re: [U-Boot] [Patch 0/17] New 'qemu-mips64' test architecture for U-Boot

2012-08-14 Thread Mike Frysinger
On Tuesday 14 August 2012 11:06:51 Zhi-zhou Zhang wrote: As so far, U-Boot has already supports mips32. But there are some differences between mips64 and mips32. Such as different ABI, different output format. so this patch adds mips64 support. many of your patches are line wrapped. please

Re: [U-Boot] [Patch 0/17] New 'qemu-mips64' test architecture for U-Boot

2012-08-14 Thread Mike Frysinger
On Tuesday 14 August 2012 11:49:49 Mike Frysinger wrote: On Tuesday 14 August 2012 11:06:51 Zhi-zhou Zhang wrote: As so far, U-Boot has already supports mips32. But there are some differences between mips64 and mips32. Such as different ABI, different output format. so this patch adds

Re: [U-Boot] [PATCH 1/4] spi_nor: add m25p32 spi nor driver

2012-08-14 Thread Mike Frysinger
On Tuesday 14 August 2012 06:58:31 Terry Lv wrote: This driver is ported from kernel and can handle a series of spi nor flash including m25pxx, m45pxx, w25xxx, cat25xxx, sst25vf0xxx, sst25wfxxx, s25xxx, mx25xxx, at25xxx, at26xxx, en25xxx, xxxs33b. it doesn't make much sense to add a driver

Re: [U-Boot] smc911x and CONFIG_NET_MULTI

2012-08-14 Thread Mike Frysinger
On Tuesday 14 August 2012 12:25:34 Dimitar Penev wrote: We have developed a custom blackfin board having two LAN9218 chips on board. Is the available smc911x driver tested with multiple network chips on board? all net drivers have been converted to net multi a long time ago. The base address

Re: [U-Boot] [PATCH] dlmalloc: remove manual reloc alias warning

2012-08-13 Thread Mike Frysinger
On Monday 13 August 2012 05:02:03 Andreas Bießmann wrote: From: Andreas Bießmann biessm...@corscience.de The avr32 architecture (and some others) require manual relocation. Due to the previous error all avr32 boards gave warnings in MAKEALL wich makes it hard to find new warnings. This

Re: [U-Boot] [PATCH v4] Consolidate bootcount code into drivers/bootcount

2012-08-13 Thread Mike Frysinger
On Monday 13 August 2012 07:56:03 Stefan Roese wrote: rename arch/blackfin/cpu/bootcount.c = drivers/bootcount/bootcount_blackfin.c (100%) create mode 100644 hmm, file was moved, but forgot to update arch/blackfin/cpu/Makefile to no longer refer to bootcount.c, so it fails. once that's

Re: [U-Boot] [PATCH 4/5] README: The ver env var is not read-only

2012-08-12 Thread Mike Frysinger
On Sunday 12 August 2012 09:58:24 Benoît Thébaudeau wrote: Dear Wolfgang Denk, The correct fix for this would be the introduction of variable types, including flags like read-only (as for serial# or ethaddr) or volatile (i. e. not included in saveenv, as for filesize etc.) I have been

Re: [U-Boot] [PATCH 5/5] Add env var giving the board revision

2012-08-12 Thread Mike Frysinger
On Sunday 12 August 2012 10:02:48 Benoît Thébaudeau wrote: Dear Wolfgang Denk, I have searched such a usage in the tree, but did not find any, so this should not break anything. You cannot expect to see the real, production environments in the mainline source tree. Right, but

Re: [U-Boot] [PATCH 5/5] Add env var giving the board revision

2012-08-12 Thread Mike Frysinger
On Sunday 12 August 2012 13:11:03 Benoît Thébaudeau wrote: OK. Actually, the only reason for which I need this patch is to make a variable read-only, and the only reason for which you reject it is because you fear that it breaks something. and because it bloats the codebase for 0 gain for the

Re: [U-Boot] [PATCH v3] Consolidate bootcount code into drivers/bootcount

2012-08-11 Thread Mike Frysinger
On Tuesday 05 June 2012 02:37:55 Stefan Roese wrote: --- /dev/null +++ b/drivers/bootcount/Makefile +COBJS-$(CONFIG_BFIN_CPU) += bootcount_blackfin.o needs to be CONFIG_BLACKFIN +all: $(LIB) unused rule - delete --- /dev/null +++ b/include/bootcount.h +#ifdef

Re: [U-Boot] [PATCH 3/5] env import/export: Remove from help if disabled

2012-08-11 Thread Mike Frysinger
Acked-by: Mike Frysinger vap...@gentoo.org -mike signature.asc Description: This is a digitally signed message part. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 4/5] README: The ver env var is not read-only

2012-08-11 Thread Mike Frysinger
On Friday 10 August 2012 13:45:57 Benoît Thébaudeau wrote: --- u-boot-4d3c95f.orig/README +++ u-boot-4d3c95f/README If this variable is defined, an environment variable named ver is created by U-Boot showing the U-Boot version as printed by the

Re: [U-Boot] [PATCH] eth_write_hwaddr: Return error for invalid MACs

2012-08-11 Thread Mike Frysinger
Acked-by: Mike Frysinger vap...@gentoo.org -mike signature.asc Description: This is a digitally signed message part. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 5/5] Add env var giving the board revision

2012-08-11 Thread Mike Frysinger
On Friday 10 August 2012 13:46:08 Benoît Thébaudeau wrote: The board revision can be a useful env var, like its serial number. unless i missed something, there is no standard rev variable today, which means this change can easily break anyone who happens to already be using a variable named

Re: [U-Boot] [PATCH] mtest: Disable dcache during test

2012-08-11 Thread Mike Frysinger
On Saturday 11 August 2012 10:17:04 Benoît Thébaudeau wrote: On 08/11/2012 05:18 AM, Mike Frysinger wrote: On Friday 10 August 2012 15:16:15 Benoît Thébaudeau wrote: mtest is supposed to test many types of memory accesses in many different conditions. If dcache is enabled, memory

Re: [U-Boot] [PATCH] mtest: Disable dcache during test

2012-08-11 Thread Mike Frysinger
On Saturday 11 August 2012 16:05:36 Benoît Thébaudeau wrote: On 08/11/2012 21:47, Andrew Dyer wrote: I agree with Mike, use the current dcache settings. U-boot has always assumed the user knew what they were doing. If you want to print a small message with the dcache setting that makes

Re: [U-Boot] Is there file size limiation of tftp

2012-08-11 Thread Mike Frysinger
On Saturday 11 August 2012 21:21:03 J.Hwan Kim wrote: Is there any limit of file size for tftpboot? not really. how much unused RAM do you have ? I have a problem downloading the file of which file size is 65MB. did you overwrite u-boot or something ? -mike signature.asc Description: This

Re: [U-Boot] [PATCH] sf: stmicro: Add support for N25Q128A

2012-08-10 Thread Mike Frysinger
On Friday 10 August 2012 08:21:46 Michal Simek wrote: Add support for Numonyx N25Q128A SPI flash. thanks, merged into my sf branch -mike signature.asc Description: This is a digitally signed message part. ___ U-Boot mailing list U-Boot@lists.denx.de

Re: [U-Boot] [PATCH v3] arm: rmobile: Add support Renesas SH73A0

2012-08-10 Thread Mike Frysinger
On Friday 10 August 2012 03:41:17 Nobuhiro Iwamatsu wrote: --- a/arch/arm/cpu/armv7/rmobile/Makefile +++ b/arch/arm/cpu/armv7/rmobile/Makefile +clean: + rm -f $(SOBJS) $(OBJS) + dead code - delete -mike signature.asc Description: This is a digitally signed message part.

Re: [U-Boot] [PATCH] serial: CONSOLE macro is not used

2012-08-10 Thread Mike Frysinger
Acked-by: Mike Frysinger vap...@gentoo.org -mike signature.asc Description: This is a digitally signed message part. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] Why no cache flushing in do_go_exec()

2012-08-10 Thread Mike Frysinger
On Thursday 09 August 2012 22:38:09 Charles Manning wrote: I'm helping to work through an issue where some code is loaded into RAM this is generally where cache is supposed to be flushed. you didn't describe how exactly the code is being loaded into RAM though. and go xxx is issued to then

Re: [U-Boot] [PATCH] mtest: Disable dcache during test

2012-08-10 Thread Mike Frysinger
On Friday 10 August 2012 15:16:15 Benoît Thébaudeau wrote: mtest is supposed to test many types of memory accesses in many different conditions. If dcache is enabled, memory accesses are likely bursts, and some memory accesses are simply skipped. Hence, dcache should be disabled during mtest

Re: [U-Boot] [PATCH 1/2] doc/git-mailrc: Add 'rmobile' alias

2012-08-09 Thread Mike Frysinger
On Thursday 09 August 2012 10:30:23 Nobuhiro Iwamatsu wrote: --- a/doc/git-mailrc +++ b/doc/git-mailrc alias tegra uboot, Simon Glass s...@chromium.org, Tom Warren alias ti uboot, Tom Rini tr...@ti.com +alias rmobileuboot, iwamatsu please keep it sorted -mike

Re: [U-Boot] Sandbox: auto exiting on pipe closure

2012-08-09 Thread Mike Frysinger
On Thursday 09 August 2012 16:39:20 Wolfgang Denk wrote: Mike Frysinger wrote: On Monday 23 April 2012 02:41:08 Wolfgang Denk wrote: 2) Sandbox does no handle EOF on stadin; this makes it impossible to use it in test scripts. For example, something like this should

Re: [U-Boot] [PATCH 1/3] COMMON: Add __stringify() function

2012-08-08 Thread Mike Frysinger
Acked-by: Mike Frysinger vap...@gentoo.org -mike signature.asc Description: This is a digitally signed message part. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 3/3] COMMON: Use __stringify() instead of MK_STR()

2012-08-08 Thread Mike Frysinger
Acked-by: Mike Frysinger vap...@gentoo.org -mike signature.asc Description: This is a digitally signed message part. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 2/3] COMMON: Use __stringify() instead of xstr()

2012-08-08 Thread Mike Frysinger
Acked-by: Mike Frysinger vap...@gentoo.org -mike signature.asc Description: This is a digitally signed message part. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 1/3] COMMON: Add __stringify() function

2012-08-08 Thread Mike Frysinger
On Wednesday 08 August 2012 06:52:17 Marek Vasut wrote: This function converts static number to string in preprocessor. This is useful as it allows higher usage of puts() in favour of printf() a few more to convert: include/nios2.h:#define _str_(x) #x include/configs/astro_mcf5373l.h:#define

  1   2   3   4   5   6   7   8   9   10   >