Re: [U-Boot] U-boot Saveenv question
How did you verified that your environment get saved? What is the result of running the 'flinfo'? Is CONFIG_ENV_OVERWRITE defined in your board file? Kind regards, Andrejs Cainikovs. Suresh Bhaskaran wrote: > Hello, > > I have a question relating to doing saveenv from the u-boot command line > (after hitting key to stop autoboot), and having u-boot come up during > the next reboot with the saved environment. > > First of all, I apologize in advance if this question is out of place in > this mailing list. > > This mailing list seems to be for u-boot patch upgrades, rather than for > general questions? > > Anyway, it's the best list on u-boot I was able to find: > > 1. I upgraded from u-boot 1.1.3 (2005) to u-boot 1.3.4 (2008) > 2. I've put in the board/target-specific files incorporated, and have > it working. > > 3. However, when I do a saveenv (e.g. after "set serverip 10.0.0.1"), > and I do a reboot, the new environment variable doesn't show up. > a. (I've verified that it actually gets written to flash...) > > 4. Is there something I else I need to do to "turn on" re-booting from > saved environment? > > Thanks in advance. > > Suresh Bhaskaran > ___ > U-Boot mailing list > U-Boot@lists.denx.de > http://lists.denx.de/mailman/listinfo/u-boot > ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] i2c: merge all i2c_reg_read() and i2c_reg_write() into inline functions
On Fri, Oct 3, 2008 at 6:39 PM, Wolfgang Denk <[EMAIL PROTECTED]> wrote: > This is your opinion. I think you cannot really speak for others. I > think you should ask the respective maintainers of the code first > before changing their code. I think the MAINTAINERS file might be out of date with respect to the blackfin and pxa boards. It lists Mike Frysinger and the blackfin maintainer and Cliff Brake as the pxa maintainer, but the mailing lists archives and the git repository show that these two individuals have not been involved in any patches for these platforms for the past six months. So who are the maintainers for blackfin and pxa? -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] Fat32 and VFAT support
Hi eric, Eric Nelson wrote: > Hello all, > > I've recently been looking into a problem loading files from a FAT32 > filesystem > and (re)discovered that there are issues with this code. > > Some of the symptoms are related to problems discussed in a thread back in > April titled "USB SUPPORT & get_vfatname". > > http://lists.denx.de/pipermail/u-boot/2008-April/033117.html > > I ran into the issue(s) on an SD(MMC) card when I noticed a failure to load > a certain file from the card. > > I've been able to narrow down a set of problems in fs/fat.c. I'm not sure > when I'll have the time to generate a proper patch, but wanted to at least > write some notes on the nature of the problem to save others some grief. > > The first problem I encountered was that the main loop in fs/fat.c/do_fat_read > was built for fixed-size root directories. The Microsoft document > "FAT: General Overview of On-Disk Format" > says this on page 22: > "For FAT32, the root directory can be of variable size and is a > cluster chain, just like any other ..." > > When running the command: > > fatls mmc 0 > > I try to investigate but I don't have spare time to fix. I send to the list an linux-user space program that use the u-boot routine to read on the image. You can use that to track down the problem. Regards Michael ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] i2c: merge all i2c_reg_read() and i2c_reg_write() into inline functions
On Sunday 05 October 2008, Timur Tabi wrote: > On Fri, Oct 3, 2008 at 6:39 PM, Wolfgang Denk <[EMAIL PROTECTED]> wrote: > > This is your opinion. I think you cannot really speak for others. I > > think you should ask the respective maintainers of the code first > > before changing their code. > > I think the MAINTAINERS file might be out of date with respect to the > blackfin and pxa boards. It lists Mike Frysinger and the blackfin > maintainer and Cliff Brake as the pxa maintainer, but the mailing > lists archives and the git repository show that these two individuals > have not been involved in any patches for these platforms for the past > six months. > > So who are the maintainers for blackfin and pxa? I'm pretty sure that Mike is still responsible for Blackfin. Not sure about Cliff. BTW: I second your patch. So from me: Acked-by: Stefan Roese <[EMAIL PROTECTED]> Thanks. Best regards, Stefan = DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: [EMAIL PROTECTED] = ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] MIMC200 board now uses CONFIG_DISABLE_CONSOLE
Mark Jackson <[EMAIL PROTECTED]> wrote: > Changed MIMC200 board setup and config to use CONFIG_DISABLE_CONSOLE. > Also fixed default uImage location. > > Signed-off-by: Mark Jackson <[EMAIL PROTECTED]> Applied to avr32 next, thanks. Haavard ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH] Red Black Tree support
RB-tree support on U-Boot Now it's used at UBI module. Of course other modules can use it. Signed-off-by: Kyungmin Park <[EMAIL PROTECTED]> --- diff --git a/include/linux/rbtree.h b/include/linux/rbtree.h new file mode 100644 index 000..a4956c4 --- /dev/null +++ b/include/linux/rbtree.h @@ -0,0 +1,160 @@ +/* + Red Black Trees + (C) 1999 Andrea Arcangeli <[EMAIL PROTECTED]> + + This program is free software; you can redistribute it and/or modify + it under the terms of the GNU General Public License as published by + the Free Software Foundation; either version 2 of the License, or + (at your option) any later version. + + This program is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + GNU General Public License for more details. + + You should have received a copy of the GNU General Public License + along with this program; if not, write to the Free Software + Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA + + linux/include/linux/rbtree.h + + To use rbtrees you'll have to implement your own insert and search cores. + This will avoid us to use callbacks and to drop drammatically performances. + I know it's not the cleaner way, but in C (not in C++) to get + performances and genericity... + + Some example of insert and search follows here. The search is a plain + normal search over an ordered tree. The insert instead must be implemented + int two steps: as first thing the code must insert the element in + order as a red leaf in the tree, then the support library function + rb_insert_color() must be called. Such function will do the + not trivial work to rebalance the rbtree if necessary. + +--- +static inline struct page * rb_search_page_cache(struct inode * inode, +unsigned long offset) +{ + struct rb_node * n = inode->i_rb_page_cache.rb_node; + struct page * page; + + while (n) + { + page = rb_entry(n, struct page, rb_page_cache); + + if (offset < page->offset) + n = n->rb_left; + else if (offset > page->offset) + n = n->rb_right; + else + return page; + } + return NULL; +} + +static inline struct page * __rb_insert_page_cache(struct inode * inode, + unsigned long offset, + struct rb_node * node) +{ + struct rb_node ** p = &inode->i_rb_page_cache.rb_node; + struct rb_node * parent = NULL; + struct page * page; + + while (*p) + { + parent = *p; + page = rb_entry(parent, struct page, rb_page_cache); + + if (offset < page->offset) + p = &(*p)->rb_left; + else if (offset > page->offset) + p = &(*p)->rb_right; + else + return page; + } + + rb_link_node(node, parent, p); + + return NULL; +} + +static inline struct page * rb_insert_page_cache(struct inode * inode, +unsigned long offset, +struct rb_node * node) +{ + struct page * ret; + if ((ret = __rb_insert_page_cache(inode, offset, node))) + goto out; + rb_insert_color(node, &inode->i_rb_page_cache); + out: + return ret; +} +--- +*/ + +#ifndef_LINUX_RBTREE_H +#define_LINUX_RBTREE_H + +#include + +struct rb_node +{ + unsigned long rb_parent_color; +#defineRB_RED 0 +#defineRB_BLACK1 + struct rb_node *rb_right; + struct rb_node *rb_left; +} __attribute__((aligned(sizeof(long; +/* The alignment might seem pointless, but allegedly CRIS needs it */ + +struct rb_root +{ + struct rb_node *rb_node; +}; + + +#define rb_parent(r) ((struct rb_node *)((r)->rb_parent_color & ~3)) +#define rb_color(r) ((r)->rb_parent_color & 1) +#define rb_is_red(r) (!rb_color(r)) +#define rb_is_black(r) rb_color(r) +#define rb_set_red(r) do { (r)->rb_parent_color &= ~1; } while (0) +#define rb_set_black(r) do { (r)->rb_parent_color |= 1; } while (0) + +static inline void rb_set_parent(struct rb_node *rb, struct rb_node *p) +{ + rb->rb_parent_color = (rb->rb_parent_color & 3) | (unsigned long)p; +} +static inline void rb_set_color(struct rb_node *rb, int color) +{ + rb->rb_parent_color = (rb->rb_parent_color & ~1) | color; +} + +#define RB_ROOT(struct rb_root) { NULL, } +#definerb_entry(ptr, type, member) container_of(ptr, type, member) + +#def
[U-Boot] [PATCH] Use correct kzalloc
I'm not sure calloc returns all zero values. Actually the calloc uses the __libc_alloc. Instead of ambiguity, it uses the correct kzalloc. Signed-off-by: Kyungmin Park <[EMAIL PROTECTED]> --- diff --git a/include/linux/mtd/compat.h b/include/linux/mtd/compat.h index 9036b74..840de4c 100644 --- a/include/linux/mtd/compat.h +++ b/include/linux/mtd/compat.h @@ -18,7 +18,12 @@ #define KERN_DEBUG #define kmalloc(size, flags) malloc(size) -#define kzalloc(size, flags) calloc(size, 1) +#define kzalloc(size, flags) ({ \ + void *__ret = malloc(size); \ + if (__ret) \ + memset(__ret, 0, size); \ + __ret; \ +}) #define vmalloc(size) malloc(size) #define kfree(ptr) free(ptr) #define vfree(ptr) free(ptr) ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] i2c: merge all i2c_reg_read() and i2c_reg_write() into inline functions
Mike and Cliff, This I2C patch converts some I2C functions t inline, but in the case of blackfin and pxa, it removes the PRINTD() call as well. If you're okay with that, please ACK this patch. You can find the original patch in the mailing list archive here: http://lists.denx.de/pipermail/u-boot/2008-October/041173.html On Thu, Oct 2, 2008 at 10:06 AM, Timur Tabi <[EMAIL PROTECTED]> wrote: > All implementations of the functions i2c_reg_read() and i2c_reg_write() are > identical. We can save space and simplify the code by converting these > functions into inlines and putting them in i2c.h. > > Signed-off-by: Timur Tabi <[EMAIL PROTECTED]> > --- > > I'm posting this patch because I'm enhancing the I2C routines to support > multiple I2C busses more easily, but I need to clean up the existing code > first. > > cpu/arm920t/at91rm9200/i2c.c | 14 -- > cpu/arm926ejs/davinci/i2c.c | 17 - > cpu/blackfin/i2c.c| 16 > cpu/mpc512x/i2c.c | 17 - > cpu/mpc5xxx/i2c.c | 16 > cpu/mpc8220/i2c.c | 16 > cpu/mpc824x/drivers/i2c/i2c.c | 14 -- > cpu/mpc8260/i2c.c | 16 > cpu/mpc8xx/i2c.c | 20 > cpu/ppc4xx/i2c.c | 20 > cpu/pxa/i2c.c | 15 --- > drivers/i2c/fsl_i2c.c | 16 > drivers/i2c/soft_i2c.c| 19 --- > include/i2c.h | 15 +-- > 14 files changed, 13 insertions(+), 218 deletions(-) -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] Change UEC PHY interface to RGMII on MPC8568MDS
Andy Fleming wrote: > From: Haiying Wang <[EMAIL PROTECTED]> > > Change UEC phy interface from GMII to RGMII on MPC8568MDS board > > Because on MPC8568MDS, GMII interface is only recommended for 1000Mbps speed, > but RGMII interface can work at 10/100/1000Mbps, and RGMII interface works > more stable. > > Now both UEC1 and UEC2 can work properly under u-boot. > > It is also in consistent with the kernel setting for 8568 UEC phy interface. > > Signed-off-by: Haiying Wang <[EMAIL PROTECTED]> > --- Applied to net/next. regards, Ben ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v3] [83xx] Adds two more ethernet interface to 83xx
richardretanubun wrote: > Added as a convenience for other platforms that uses MPC8360 (has 8 UCC). > Six eth interface is chosen because the platform I am using combines > UCC1&2 and UCC3&4 as 1000 Eth and the other four UCCs as 10/100 Eth. > > Signed-off-by: Richard Retanubun <[EMAIL PROTECTED]> > --- Applied to net/next regards, Ben ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 2/6] net: express the first argument to NetSetTimeout() in milliseconds
Bartlomiej Sieka wrote: > Enforce millisecond semantics of the first argument to NetSetTimeout() -- > the change is transparent for well-behaving boards (CFG_HZ == 1000 and > get_timer() countiing in milliseconds). > > Rationale for this patch is to enable millisecond granularity for > network-related timeouts, which is needed for the upcoming automatic > software update feature. > > Summary of changes: > - do not scale the first argument to NetSetTimeout() by CFG_HZ > - change timeout values used in the networking code to milliseconds > > Signed-off-by: Rafal Czubak <[EMAIL PROTECTED]> > Signed-off-by: Bartlomiej Sieka <[EMAIL PROTECTED]> > --- Applied to net/next regards, Ben ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH v2 3/6] net: Make TFTP server timeout configurable
Bartlomiej Sieka wrote: > There are two aspects of a TFTP transfer involving timeouts: > 1. timeout waiting for initial server reply after sending RRQ > 2. timeouts while transferring actual data from the server > > Since the upcoming auto-update feature attempts a TFTP download during each > boot, it is undesirable to have a long delay when the TFTP server is not > available. Thus, this commit makes the server timeout (1.) configurable by two > global variables: > > TftpRRQTimeoutMSecs > TftpRRQTimeoutCountMax > > TftpRRQTimeoutMSecs overrides default timeout when trying to connect to a TFTP > server, TftpRRQTimeoutCountMax overrides default number of connection retries. > The total delay when trying to download a file from a non-existing TFTP server > is TftpRRQTimeoutMSecs x TftpRRQTimeoutCountMax milliseconds. > > Timeouts during file transfers (2.) are unaffected. > > Signed-off-by: Rafal Czubak <[EMAIL PROTECTED]> > Signed-off-by: Bartlomiej Sieka <[EMAIL PROTECTED]> > --- Applied to net/next regards, Ben ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] u-boot randomly lost params.
I am using 1.3.2, and discovered that the params randomly got lost. This does not happen every time, but does happen from time to time. I can not repliably reproduce it. Any hints? I am using a TI6446 EVM. Someone over at the TI mailing list suggests that the size of params is bigger than the param sector size. How to check that? T.I.A ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] TI DM6446 EVM logo
Hi, Is there any version of u-boot that can display a logo for TI DM6446 EVM? I looked at the code, it seems that the video device hw_init() needs to be implemented. T.I.A ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] Custodian update
Wolfgang Denk schrieb: > Hi everybody, > > here some custodian news: > > 1) Kyungmin Park is now custodian for the (new) "u-boot-ubi" reposi- >tory where UBI related work will be added. Thanks to Kyungmin Park >for volunteering. > > 2) Anatolij Gustschin is now custodian for the "u-boot-video" reposi- >tory where graphics and video driver related work will be added. >Thanks to Anatolij Gustschin for volunteering, and thanks to >Rodolfo Giometti for his work done so far. > What's about the ARM custodian entry in U-Boot's Gitweb? Peter or Jean-Christophe? -- Kind regards, Jens ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot