Re: [flashrom] AMD - SP5100 - take SPI ownership (1/2)

[Stefan Tauner]

On Wed, 27 Mar 2013 16:28:22 +0000
"Racine, Michel"  wrote:
> With your patch, I had a small compile issue (imc.c, function mbox_wait_ack, commented call to msg_pwarn).

Hello,

thanks for testing it. What kind of compile issue? There should not be
any problem with calls to msg_pwarn with that patch.
> I attempted to use the exe, but it cannot turn off IMC. Not sure what you need from here. I can change the code, or gather any data you want.

That's interesting, I can't comment on that yet and there isn't a lot
of debug code in there yet, so running with additional -V parameters
won't help much I guess.

AMD - SP5100 - take SPI ownership (1/2)

[Racine, Michel]

Hello,

I attached imc.c. The compiler complained with:

imc.c: In function ‘mbox_wait_ack’:
imc.c:107:2: error: implicit declaration of function ‘msg_pwarn’ [-Werror=implicit-function-declaration]
cc1: all warnings being treated as errors
make: *** [imc.o] Error 1

Thanks,
Michel.

-----Original Message-----
From: Stefan Tauner [mailto:[e-mail]] 
Sent: Wednesday, March 27, 2013 10:44 AM
To: Racine, Michel
Cc: [e-mail]; Rudolf Marek; roxfan
Subject: Re: [flashrom] AMD - SP5100 - take SPI ownership (1/2)

On Wed, 27 Mar 2013 16:28:22 +0000
"Racine, Michel"  wrote:
> With your patch, I had a small compile issue (imc.c, function mbox_wait_ack, commented call to msg_pwarn).

Hello,

thanks for testing it. What kind of compile issue? There should not be any problem with calls to msg_pwarn with that patch.
> I attempted to use the exe, but it cannot turn off IMC. Not sure what you need from here. I can change the code, or gather any data you want.

That's interesting, I can't comment on that yet and there isn't a lot of debug code in there yet, so running with additional -V parameters won't help much I guess.