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@xxxxxxxxxxx P: 512.691.8044
