[flashrom] [Patch] Boards: Fix up several issues with nvidia_mcp_gpio_set.
Carl-Daniel Hailfinger
c-d.hailfinger.devel.2006 at gmx.net
Thu Jan 7 21:44:19 CET 2010
On 21.12.2009 17:10, Michael Karcher wrote:
> Am Montag, den 21.12.2009, 16:52 +0100 schrieb Luc Verhaegen:
>
>> - if ((gpio < 0) || (gpio > 31)) {
>> + if ((gpio < 0) || (gpio > 0x40)) {
>>
> Sorry, I have to reject this one too. gpio == 0x40 is most probably
> invalid, as it would be gpio_base + 0x100. This should read "gpio >=
> 0x40".
>
> Nacked-by: Michael Karcher <flashrom at mkarcher.dialup.fu-berlin.de>
>
>
>> + default:
>> + fprintf(stderr,
>> + "\nERROR: no nVidia LPC/SMBus controller found.\n");
>> return -1;
>> + }
>> + break;
>>
> Maybe make that error message
> "no supported nVidia LPC/SMBus controller found.\n"
>
Luc, could you resubmit with the review comments addressed? I'd like to
get this in. Thanks.
Regards,
Carl-Daniel
--
Developer quote of the year:
"We are juggling too many chainsaws and flaming arrows and tigers."
More information about the flashrom
mailing list