[flashrom] [PATCH] Whitelist known working Thinkpads.

[Paul Menzel]

Dear Stefan,


please add T60 and X60 to the commit summary too.

    Whitelist known working Thinkpads T60 and X60.


Am Montag, den 24.09.2012, 23:38 +0200 schrieb Stefan Tauner:
> This should get rid of the need to specify the laptop force parameter
> on T60 and X60 that are using one of the Intel 94x northbridges and
> are known to have no interfering EC.

»Therefore for example the T60’s image can be read with the following
command.

$ sudo flashrom -V -r t60.image

> Signed-off-by: Stefan Tauner <stefan.tauner at student.tuwien.ac.at>
> ---
>  board_enable.c |   13 +++++++++++++
>  1 file changed, 13 insertions(+)
> 
> diff --git a/board_enable.c b/board_enable.c
> index 28a8fcd..e512064 100644
> --- a/board_enable.c
> +++ b/board_enable.c
> @@ -2243,6 +2243,17 @@ static int p2_not_a_laptop(void)
>  	return 0;
>  }
>  
> +/*
> + * Suited for all laptops, which are known to work.
> + */
> +static int p2_whitelist_laptop(void)
> +{
> +	is_laptop = 1;
> +	laptop_ok = 1;
> +	msg_pdbg("Whitelisted laptop detected.\n");
> +	return 0;
> +}
> +
>  #endif
>  
>  /*
> @@ -2396,6 +2407,8 @@ const struct board_match board_matches[] = {
>  	{0x8086, 0x7190,      0,      0,  0x8086, 0x7110,      0,      0, "^SE440BX-2$", NULL, NULL,          P3, "Intel",       "SE440BX-2",             0,   NT, intel_piix4_gpo27_lower},
>  	{0x1022, 0x7468,      0,      0,  0x1022, 0x7460,      0,      0, NULL,         "iwill", "dk8_htx",   P3, "IWILL",       "DK8-HTX",               0,   OK, w83627hf_gpio24_raise_2e},
>  	{0x8086, 0x27A0, 0x8086, 0x27a0,  0x8086, 0x27b8, 0x8086, 0x27b8, NULL,        "kontron", "986lcd-m", P3, "Kontron",     "986LCD-M",              0,   OK, board_kontron_986lcd_m},
> +	{0x8086, 0x27a0, 0x17aa, 0x2017,  0x8086, 0x27b9, 0x17aa, 0x2009, "^ThinkPad T60", NULL, NULL,        P2, "Lenovo",      "T60",                   0,   OK, p2_whitelist_laptop},
> +	{0x8086, 0x27a0, 0x17aa, 0x2017,  0x8086, 0x27b9, 0x17aa, 0x2009, "^ThinkPad X60", NULL, NULL,        P2, "Lenovo",      "X60",                   0,   OK, p2_whitelist_laptop},
>  	{0x8086, 0x2411, 0x8086, 0x2411,  0x8086, 0x7125, 0x0e11, 0xb165, NULL,         NULL, NULL,           P3, "Mitac",       "6513WU",                0,   OK, board_mitac_6513wu},
>  	{0x8086, 0x8186, 0x8086, 0x8186,  0x8086, 0x8800, 0x0000, 0x0000, "^MSC Vertriebs GmbH$", NULL, NULL, P2, "MSC",         "Q7-TCTC",               0,   OK, p2_not_a_laptop},
>  	{0x10DE, 0x005E, 0x1462, 0x7125,  0x10DE, 0x0052, 0x1462, 0x7125, NULL,         NULL, NULL,           P3, "MSI",         "K8N Neo4-F",            0,   OK, nvidia_mcp_gpio2_raise}, /* TODO: Should probably be K8N Neo4 Platinum, see http://www.coreboot.org/pipermail/flashrom/2010-August/004362.html. */

Acked-by: Paul Menzel <paulepanter at users.sourceforge.net>


Thanks,

Paul
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <http://www.flashrom.org/pipermail/flashrom/attachments/20120925/ea406377/attachment.asc>