Re: [PATCH 1/1] Initialization of Alchemy boards

[Kevin Hickey <khickey@rmicorp.com>]

Dmitri,

On Mon, 2008-07-28 at 10:08 +0300, Dmitri Vorobiev wrote:
> Kevin Hickey wrote:
> > I found this when I updated to version 2.6.26.  None of my development
> > boards would boot.  It appears that a previous update changed some calls
> > to simple_strtol to strict_strtol but did not account for the different
> > call semantics.
> 
> Hi Kevin,
> 
> 1) you forgot to sign your patch off;
Indeed I did.  Is it sufficient to reply to the original post and add a
sign-off line?
> 2) please consider using git to generate and submit patches. At least according to my experience, it's much more convenient than any other approach.
It's starting to look that way to me.  I started my work here using SVN
because that's what I'm familiar with.  I did some reading about git
this weekend and I think I'm going to adopt it for future kernel
development.
> 3) As I can see, the board-specific code is almost identical for the pb1x00 boards. While at it, would you please try to merge it?
Yes, but in a future patch.  I'd like to get this one accepted ASAP
because it prevents 2.6.26+ from booting on any of our development
boards.  Next, I want to go through all of the db and pb board specific
code and see if I can condense any of it.
> 
> Regards,
> Dmitri
Thanks,
Kevin
> 
> > 
> > Index: arch/mips/au1000/pb1000/init.c
> > ===================================================================
> > --- arch/mips/au1000/pb1000/init.c
> > +++ arch/mips/au1000/pb1000/init.c
> > @@ -52,6 +52,6 @@
> >  	if (!memsize_str)
> >  		memsize = 0x04000000;
> >  	else
> > -		memsize = strict_strtol(memsize_str, 0, NULL);
> > +		strict_strtol(memsize_str, 0, &memsize);
> >  	add_memory_region(0, memsize, BOOT_MEM_RAM);
> >  }
> > Index: arch/mips/au1000/pb1100/init.c
> > ===================================================================
> > --- arch/mips/au1000/pb1100/init.c
> > +++ arch/mips/au1000/pb1100/init.c
> > @@ -54,7 +54,7 @@
> >  	if (!memsize_str)
> >  		memsize = 0x04000000;
> >  	else
> > -		memsize = strict_strtol(memsize_str, 0, NULL);
> > +		strict_strtol(memsize_str, 0, &memsize);
> >  
> >  	add_memory_region(0, memsize, BOOT_MEM_RAM);
> >  }
> > Index: arch/mips/au1000/pb1200/init.c
> > ===================================================================
> > --- arch/mips/au1000/pb1200/init.c
> > +++ arch/mips/au1000/pb1200/init.c
> > @@ -53,6 +53,6 @@
> >  	if (!memsize_str)
> >  		memsize = 0x08000000;
> >  	else
> > -		memsize = strict_strtol(memsize_str, 0, NULL);
> > +		strict_strtol(memsize_str, 0, &memsize);
> >  	add_memory_region(0, memsize, BOOT_MEM_RAM);
> >  }
> > Index: arch/mips/au1000/mtx-1/init.c
> > ===================================================================
> > --- arch/mips/au1000/mtx-1/init.c
> > +++ arch/mips/au1000/mtx-1/init.c
> > @@ -55,6 +55,6 @@
> >  	if (!memsize_str)
> >  		memsize = 0x04000000;
> >  	else
> > -		memsize = strict_strtol(memsize_str, 0, NULL);
> > +		strict_strtol(memsize_str, 0, &memsize);
> >  	add_memory_region(0, memsize, BOOT_MEM_RAM);
> >  }
> > Index: arch/mips/au1000/pb1500/init.c
> > ===================================================================
> > --- arch/mips/au1000/pb1500/init.c
> > +++ arch/mips/au1000/pb1500/init.c
> > @@ -53,6 +53,6 @@
> >  	if (!memsize_str)
> >  		memsize = 0x04000000;
> >  	else
> > -		memsize = strict_strtol(memsize_str, 0, NULL);
> > +		strict_strtol(memsize_str, 0, &memsize);
> >  	add_memory_region(0, memsize, BOOT_MEM_RAM);
> >  }
> > Index: arch/mips/au1000/xxs1500/init.c
> > ===================================================================
> > --- arch/mips/au1000/xxs1500/init.c
> > +++ arch/mips/au1000/xxs1500/init.c
> > @@ -53,6 +53,6 @@
> >  	if (!memsize_str)
> >  		memsize = 0x04000000;
> >  	else
> > -		memsize = strict_strtol(memsize_str, 0, NULL);
> > +		strict_strtol(memsize_str, 0, &memsize);
> >  	add_memory_region(0, memsize, BOOT_MEM_RAM);
> >  }
> > Index: arch/mips/au1000/pb1550/init.c
> > ===================================================================
> > --- arch/mips/au1000/pb1550/init.c
> > +++ arch/mips/au1000/pb1550/init.c
> > @@ -53,6 +53,6 @@
> >  	if (!memsize_str)
> >  		memsize = 0x08000000;
> >  	else
> > -		memsize = strict_strtol(memsize_str, 0, NULL);
> > +		strict_strtol(memsize_str, 0, &memsize);
> >  	add_memory_region(0, memsize, BOOT_MEM_RAM);
> >  }
> > Index: arch/mips/au1000/db1x00/init.c
> > ===================================================================
> > --- arch/mips/au1000/db1x00/init.c
> > +++ arch/mips/au1000/db1x00/init.c
> > @@ -57,6 +57,6 @@
> >  	if (!memsize_str)
> >  		memsize = 0x04000000;
> >  	else
> > -		memsize = strict_strtol(memsize_str, 0, NULL);
> > +		strict_strtol(memsize_str, 0, &memsize);
> >  	add_memory_region(0, memsize, BOOT_MEM_RAM);
> >  }
> > 
> 
-- 
Kevin Hickey
Alchemy Solutions
RMI Corporation
khickey@RMICorp.com
P: 512.691.8044