[flashrom] PID ID for TUMPA Lite Edition in FT2232

[Jadran Puharic]

I don't mind, glad that I could help, because flashrom saved me many times.
Didn't know about signed-off tag & docs... next time will try to save
You a hassle.

BRGDS, Jadran Puharic

2014-04-27 7:04 GMT+02:00, Stefan Tauner <stefan.tauner at alumni.tuwien.ac.at>:
> On Fri, 24 Jan 2014 15:01:40 +0100
> Jadran Puharic <jpuharic at gmail.com> wrote:
>
>> I have a TUMPA Lite edition it is FTDI Based Board similar to
>> TUMPA/FT232H, but single channel:
>> http://www.tiaowiki.com/w/TIAO_USB_Multi_Protocol_Adapter_Lite_User's_Manual
>>
>> You can build one from upper link, or buy cheap on ebay.
>>
>> And I added this to ft2232_spi.c
>>
>> --- ../flashrom/ft2232_spi.c	2013-09-17 19:43:16.000000000 +0200
>> +++ ft2232_spi.c	2014-01-24 11:57:49.000000000 +0100
>> @@ -43,6 +43,7 @@
>>  #define FTDI_FT4232H_PID	0x6011
>>  #define FTDI_FT232H_PID		0x6014
>>  #define TIAO_TUMPA_PID		0x8a98
>> +#define TIAO_TUMPA_LITE_PID   0x8a99
>>  #define AMONTEC_JTAGKEY_PID	0xCFF8
>>
>>  #define GOEPEL_VID		0x096C
>> @@ -62,6 +63,7 @@
>>  	{FTDI_VID, FTDI_FT4232H_PID, OK, "FTDI", "FT4232H"},
>>  	{FTDI_VID, FTDI_FT232H_PID, OK, "FTDI", "FT232H"},
>>  	{FTDI_VID, TIAO_TUMPA_PID, OK, "TIAO", "USB Multi-Protocol Adapter"},
>> +        {FTDI_VID, TIAO_TUMPA_LITE_PID, OK, "TIAO", "USB Multi-Protocol
>> Adapter Lite"},
>>  	{FTDI_VID, AMONTEC_JTAGKEY_PID, OK, "Amontec", "JTAGkey"},
>>  	{GOEPEL_VID, GOEPEL_PICOTAP_PID, OK, "GOEPEL", "PicoTAP"},
>>  	{FIC_VID, OPENMOKO_DBGBOARD_PID, OK, "FIC", "OpenMoko Neo1973 Debug
>> board (V2+)"},
>> @@ -208,6 +210,10 @@
>>  			/* Interface A is SPI1, B is SPI2. */
>>  			ft2232_type = TIAO_TUMPA_PID;
>>  			channel_count = 2;
>> +		} else if (!strcasecmp(arg, "tumpalite")) {
>> +			/* Only one channel is used on lite edition */
>> +			ft2232_type = TIAO_TUMPA_LITE_PID;
>> +			channel_count = 1;
>>  		} else if (!strcasecmp(arg, "busblaster")) {
>>  			/* In its default configuration it is a jtagkey clone */
>>  			ft2232_type = FTDI_FT2232H_PID;
>>
>> works as advertised.
>>
>
> Hello Jadran,
>
> thanks for your patch.
> It is missing a proper signed-off tag though (see
> http://www.coreboot.org/Development_Guidelines#Sign-off_Procedure) and
> since I had to correct the whitespace and add documentation myself
> anyway, I will commit it with my own sign-off. I hope you don't mind.
>
> --
> Kind regards/Mit freundlichen Grüßen, Stefan Tauner
>