Am 11.08.2012 04:56 schrieb Stefan Tauner:
> And remove the unused force parameter.
>
> Signed-off-by: Stefan Tauner <[email protected]>
> ---
> i forgot to move the declaration in the header, and i noticed that the
> force parameter is useless because we use the global variable instead.
> this should actually change eventually, but for now just kill the
> parameter because it is doubtful that it will be part of the solution.
Can you please rename show_id to check_image_matches_hardware or
something like that (e.g. check_image_matches_cbtable_id)? show_id is a
totally misleading name.
AFAICS the rest is a pure code move, and makes a lot of sense.
Oh, and please downgrade the following message to pdbg:
msg_pinfo("Flash image seems to be a legacy BIOS. Disabling coreboot-related
checks.\n");
Maybe downgrade the following message as well:
msg_pinfo("Flash image seems to have garbage in the ID location. Disabling
checks.\n");
Acked-by: Carl-Daniel Hailfinger <[email protected]>
Regards,
Carl-Daniel
--
http://www.hailfinger.org/
_______________________________________________
flashrom mailing list
[email protected]
http://www.flashrom.org/mailman/listinfo/flashrom
