Re: [flashrom] [PATCH 4/5] dediprog: Support new communication protocol.

[David Woodhouse]

On Mon, 2015-06-22 at 03:37 +0200, Stefan Tauner wrote:
> +/* Returns true if firmware (and thus hardware) supports the "new" protocol */
> +static bool is_new_prot(void)
>  {
>   size_t i;
>   /* if (SF100) */
>   return dediprog_firmwareversion >= FIRMWARE_VERSION(5, 5, 0);
>   /* else if (SF600)
>   return dediprog_firmwareversion >= FIRMWARE_VERSION(6, 9, 0); 
>
> */

... and in dediprog_check_devicestring() we still have:

msg_pdbg("Found a %s\n", buf);
    if (memcmp(buf, "SF100", 0x5) != 0) {
    	msg_perr("Device not a SF100!\n");
    	return 1;
    }

Is this patch set actually intended to support SF600? I had assumed so,
so I've rounded up some potential testers for that, and was just trying
to put together something for them to test...