[flashrom] [PATCH] Kill spurious cbtable related board enable warning

Carl-Daniel Hailfinger c-d.hailfinger.devel.2006 at gmx.net
Mon Jul 27 20:27:20 CEST 2009


On 27.07.2009 20:00, Stefan Reinauer wrote:
> On 7/27/09 7:39 PM, Carl-Daniel Hailfinger wrote:
>   
>> Index: flashrom-cbtable_boardenable_nowarn/board_enable.c
>> ===================================================================
>> --- flashrom-cbtable_boardenable_nowarn/board_enable.c	(Revision 667)
>> +++ flashrom-cbtable_boardenable_nowarn/board_enable.c	(Arbeitskopie)
>> @@ -1134,7 +1134,14 @@
>>  	if (partmatch)
>>  		return partmatch;
>>  
>> -	printf("\nUnknown vendor:board from coreboot table or -m option: %s:%s\n\n", vendor, part);
>> +	if (partvendor_from_cbtable) {
>> +		printf("vendor:board from coreboot table is %s:%s, apparently "
>> +		       "no board specific flash chip write enable is needed.\n",
>> +		       vendor, part);
>> +	} else {
>> +		printf("\nUnknown vendor:board from -m option: %s:%s\n\n",
>> +		       vendor, part);
>> +	}
>>  	return NULL;
>>  }
>>   
>>     
>
> Can we get a method to make the coreboot table case not output any odd
> technical message at all?
>   

Sure, I can remove the if part and transform the else part to
if (!partvendor_from_cbtable)

Would that be OK for you?

Regards,
Carl-Daniel

-- 
http://www.hailfinger.org/





More information about the flashrom mailing list