Re: [coreboot] [RFC] Error out on implicit declarations

[Carl-Daniel Hailfinger]

On 22.12.2008 17:32, Myles Watson wrote:
> > -----Original Message-----
> > From: Carl-Daniel Hailfinger [mailto:[e-mail]]
> > Sent: Monday, December 22, 2008 8:48 AM
> > To: [e-mail]
> > Cc: Myles Watson; Coreboot
> > Subject: Re: [coreboot] [RFC] Error out on implicit declarations
> >
> > On 18.12.2008 15:52, Jordan Crouse wrote:
> > > Myles Watson wrote:
> > > > On Thu, Dec 18, 2008 at 3:18 AM, Carl-Daniel Hailfinger <
> > > > [e-mail]> wrote:
> > > > > Bao, Zheng found a bug which killed SATA booting on my board.
> > > > >
> > > > > This happened because we do not error out on implicit function
> > > > > declarations. The linker has no way of checking whether the implicitly
> > > > > assumed function signature is identical to the real signature, so
> > > > > mismatches can occur and these mismatches are practically impossible
> >
> > to
> > > > > debug because the code looks completely correct.
> > > > >
> > > > > Adding -Werror-implicit-function-declaration to our CFLAGS would solve
> > > > > this problem nicely, but a lot of files in the tree need to be fixed.
> > > >
> > > > I think this is a great idea.  Isn't the correct order to fix all the
> > > > warnings, then make it an error?
> > >
> > > Yeah - the unfortunate thing about changes like this is that you end
> > > up being responsible for fixing the errors.. :)
> >
> > Fix implicit declarations of get_bus_conf.
> >
> > Signed-off-by: Carl-Daniel Hailfinger
>
> Acked-by: Myles Watson

Thanks, r3835.

Regards,
Carl-Daniel