Re: [coreboot] small patch for prototypes and unused variables

2009-10-28 Thread Stefan Reinauer
Myles Watson wrote: Maciej, Thanks for the patch. I think most of it is ready to be committed. Index: src/lib/clog2.c === --- src/lib/clog2.c (revision 4869) +++ src/lib/clog2.c (working copy) @@ -7,6 +7,8 @@ /* Assume

Re: [coreboot] small patch for prototypes and unused variables

2009-10-28 Thread Myles Watson
-Original Message- From: Stefan Reinauer [mailto:ste...@coresystems.de] Sent: Wednesday, October 28, 2009 4:25 AM To: Myles Watson Cc: 'Maciej Pijanka'; coreboot@coreboot.org Subject: Re: [coreboot] small patch for prototypes and unused variables Myles Watson wrote: Maciej

Re: [coreboot] small patch for prototypes and unused variables

2009-10-28 Thread Stefan Reinauer
Myles Watson wrote: -Original Message- From: Stefan Reinauer [mailto:ste...@coresystems.de] Sent: Wednesday, October 28, 2009 4:25 AM To: Myles Watson Cc: 'Maciej Pijanka'; coreboot@coreboot.org Subject: Re: [coreboot] small patch for prototypes and unused variables Myles

Re: [coreboot] small patch for prototypes and unused variables

2009-10-27 Thread Maciej Pijanka
Hello as Uwe sugested, i checked abuild -a, and found one romcc prototype problem, fixed patch in attachment. best regards Maciej -- Maciej Pijanka, PLD-Linux Developer, Reg Linux user #133161 POE/Perl user coreboot-4869-prototypes-updated.patch Description: Binary data -- coreboot mailing

Re: [coreboot] small patch for prototypes and unused variables

2009-10-27 Thread Myles Watson
Maciej, Thanks for the patch. I think most of it is ready to be committed. Index: src/lib/clog2.c === --- src/lib/clog2.c (revision 4869) +++ src/lib/clog2.c (working copy) @@ -7,6 +7,8 @@ /* Assume 8 bits per byte */

Re: [coreboot] small patch for prototypes and unused variables

2009-10-27 Thread Maciej Pijanka
On 27/10/2009, Myles Watson myle...@gmail.com wrote: Maciej, Thanks for the patch. I think most of it is ready to be committed. Index: src/lib/clog2.c === --- src/lib/clog2.c (revision 4869) +++ src/lib/clog2.c (working

Re: [coreboot] small patch for prototypes and unused variables

2009-10-27 Thread Myles Watson
On Tue, Oct 27, 2009 at 7:32 AM, Maciej Pijanka maciej.pija...@gmail.comwrote: On 27/10/2009, Myles Watson myle...@gmail.com wrote: Maciej, Thanks for the patch. I think most of it is ready to be committed. Index: src/lib/clog2.c

Re: [coreboot] small patch for prototypes and unused variables

2009-10-27 Thread Maciej Pijanka
On 27/10/2009, Myles Watson myle...@gmail.com wrote: On Tue, Oct 27, 2009 at 7:32 AM, Maciej Pijanka maciej.pija...@gmail.comwrote: On 27/10/2009, Myles Watson myle...@gmail.com wrote: Maciej, Thanks for the patch. I think most of it is ready to be committed. Index:

Re: [coreboot] small patch for prototypes and unused variables

2009-10-27 Thread Myles Watson
Acked-by: Myles Watson myle...@gmail.com Thanks, Myles -- coreboot mailing list: coreboot@coreboot.org http://www.coreboot.org/mailman/listinfo/coreboot

Re: [coreboot] small patch for prototypes and unused variables

2009-10-27 Thread Myles Watson
On Tue, Oct 27, 2009 at 8:20 AM, Myles Watson myle...@gmail.com wrote: Acked-by: Myles Watson myle...@gmail.com Rev 4871. Thanks, Myles -- coreboot mailing list: coreboot@coreboot.org http://www.coreboot.org/mailman/listinfo/coreboot

Re: [coreboot] small patch for prototypes and unused variables

2009-10-27 Thread ron minnich
In v3 IIRC we decided on an include file, lib.h, which had these sorts of nuisance prototypes. ron -- coreboot mailing list: coreboot@coreboot.org http://www.coreboot.org/mailman/listinfo/coreboot

Re: [coreboot] small patch for prototypes and unused variables

2009-10-27 Thread Myles Watson
On Tue, Oct 27, 2009 at 8:58 AM, ron minnich rminn...@gmail.com wrote: In v3 IIRC we decided on an include file, lib.h, which had these sorts of nuisance prototypes. Something like that would be fine with me, but since we'd do it just to do it, not to make the code cleaner, maybe it's just as

Re: [coreboot] small patch for prototypes and unused variables

2009-10-27 Thread ron minnich
On Tue, Oct 27, 2009 at 8:17 AM, Myles Watson myle...@gmail.com wrote: Something like that would be fine with me, but since we'd do it just to do it, not to make the code cleaner, maybe it's just as good to put the prototypes by the functions. In the case of that function you need a proto

Re: [coreboot] small patch for prototypes and unused variables

2009-10-27 Thread Myles Watson
On Tue, Oct 27, 2009 at 10:10 AM, ron minnich rminn...@gmail.com wrote: On Tue, Oct 27, 2009 at 8:17 AM, Myles Watson myle...@gmail.com wrote: Something like that would be fine with me, but since we'd do it just to do it, not to make the code cleaner, maybe it's just as good to put the

[coreboot] small patch for prototypes and unused variables

2009-10-26 Thread Maciej Pijanka
Hello see patch in attachment, best regards Maciej -- Maciej Pijanka, PLD-Linux Developer, Reg Linux user #133161 POE/Perl user coreboot-4869-prototypes.patch Description: Binary data -- coreboot mailing list: coreboot@coreboot.org http://www.coreboot.org/mailman/listinfo/coreboot