Some updates

Author: wagiminator

picoISP_Programmer/README.md

@@ -19,7 +19,7 @@ The CH552E is a low-cost, enhanced E8051 core microcontroller compatible with th
 The picoisp firmware is an enhanced port of the USBtinyISP firmware originally developed by Dick Streefland and Ladyada for ATtiny microcontrollers. This port is based on the conversion for CH55x microcontroller by DeqingSun. It includes the [auto-clock extension](https://github.com/nerdralph/usbasp) by Ralph Doncaster, which automatically selects the maximum programming speed, and a [Windows Compatible ID](https://github.com/pbatard/libwdi/wiki/WCID-Devices) (WCID) for automated driver installation on Windows. Since it is compatible with the original USBtinyISP, it works fine with AVRdude (avrdude -c usbtiny) and the Arduino IDE (Tools -> Programmer -> USBtinyISP).
 
 ### picoasp for USBasp Compatibility
-The picoasp firmware is an enhanced port of the USBasp firmware originally developed by Thomas Fischl for ATmega8 microcontrollers. It includes the auto-clock extension by Ralph Doncaster, which automatically selects the maximum programming speed, and a Windows Compatible ID (WCID) for automated driver installation on Windows. Since it is compatible with the original USBasp, it works fine with AVRdude (avrdude -c usbasp) and the Arduino IDE (Tools -> Programmer -> USBasp).
+The picoasp firmware is an enhanced port of the USBasp firmware originally developed by Thomas Fischl for ATmega8 microcontrollers. It includes the auto-clock extension by Ralph Doncaster, which automatically selects the maximum programming speed, and a Windows Compatible ID (WCID) for automated driver installation on Windows. It also supports the TPI protocol, enabling the programming of devices such as the ATtiny10. Since it is compatible with the original USBasp, it works fine with AVRdude (avrdude -c usbasp) and the Arduino IDE (Tools -> Programmer -> USBasp).
 
 ### picostk for STK500 Compatibility
 The picostk firmware is an improved port of the ArduinoISP (or Arduino as ISP) firmware originally developed by Randall Bohn for the Arduino Uno. It is compatible with the STK500v1 programmer. In addition, Ralph Doncaster's auto-clock extension is included, which automatically selects the maximum programming speed. It works fine with AVRdude (avrdude -c stk500v1 or avrdude -c arduino) and the Arduino IDE (Tools -> Programmer -> Arduino as ISP). Don't forget to choose the right serial port.

picoISP_Programmer/software/picoasp/picoasp.bin

@@ -1,16 +1,13 @@
 // ===================================================================================
 // Project:   picoASP AVR ISP Programmer based on CH551, CH552, CH554
-// Version:   v0.4
+// Version:   v1.0
 // Year:      2023
 // Author:    Stefan Wagner
 // Github:    https://github.com/wagiminator
 // EasyEDA:   https://easyeda.com/wagiminator
 // License:   http://creativecommons.org/licenses/by-sa/3.0/
 // ===================================================================================
 //
-// !!! This firmware version is experimental and still in development !!!
-// ToDo: make TPI work
-//
 // Description:
 // ------------
 // The CH55x-based picoASP is an ISP/TPI programmer for AVR microcontrollers compatible 

picoISP_Programmer/software/picoasp/picoasp.c

@@ -1,20 +1,20 @@
 // ===================================================================================
-// TPI Functions for for CH551, CH552 and CH554
+// TPI Functions for for CH551, CH552 and CH554                               * v1.1 *
 // ===================================================================================
 
 #include "tpi.h"
 
-__xdata uint16_t TPI_dly_cnt; 
+__xdata uint8_t TPI_dly_cnt; 
 
 // Transmit/receive one bit via TPI; b must be 1 when receiving
 __bit TPI_clockBit(__bit b) {
   __bit result;
-  uint16_t cnt;
+  uint8_t cnt;
   PIN_write(PIN_MOSI, b);                 // set DATA according to bit
   cnt = TPI_dly_cnt; while(cnt--);        // delay
   PIN_high(PIN_SCK);                      // CLK high
-  result = PIN_read(PIN_MOSI);            // read DATA
   cnt = TPI_dly_cnt; while(cnt--);        // delay
+  result = PIN_read(PIN_MOSI);            // read DATA
   PIN_low(PIN_SCK);                       // CLK low
   return result;
 }
@@ -29,28 +29,28 @@ void TPI_connect(void) {
 
   PIN_high(PIN_RESET);                    // RST high
   PIN_output(PIN_RESET);                  // RST to output
-  DLY_us(1000);                           // wait a bit (128ms ???)
+  DLY_us(1000);                           // wait a bit
   PIN_low(PIN_RESET);                     // RST low
+  DLY_us(5600);                           // wait a bit
   PIN_high(PIN_MOSI);                     // DATA high
   PIN_input_PU(PIN_MOSI);                 // DATA input pullup, open-drain output
-  DLY_us(5600);                           // wait a bit (2000ns ???)
   PIN_low(PIN_SCK);                       // CLK low
   PIN_output(PIN_SCK);                    // CLK to output
 
-  for(i=32; i; i--) TPI_clockBit(1);      // DATA high for 32 TPI clock cycles (16 ???)
+  for(i=32; i; i--) TPI_clockBit(1);      // DATA high for 32 TPI clock cycles
 }
 
 // Disconnect TPI bus
 void TPI_disconnect(void) {
   TPI_writeByte(TPI_OP_SSTCS(TPISR));
   TPI_writeByte(0);
   DLY_us(3200);
-  PIN_input(PIN_SCK);                     // CLK to input, high-impedance
-  PIN_input(PIN_MOSI);                    // DATA to input, high-impedance
   PIN_high(PIN_RESET);                    // RST high
   DLY_us(1600);                           // wait a bit
   PIN_low(PIN_RESET);                     // RST low
   DLY_us(1600);                           // wait a bit
+  PIN_input(PIN_SCK);                     // CLK to input, high-impedance
+  PIN_input(PIN_MOSI);                    // DATA to input, high-impedance
   PIN_input(PIN_RESET);                   // RST to input
 
   #ifdef PIN_LED_PRG
@@ -97,7 +97,7 @@ uint8_t TPI_readByte(void) {
   // Read data bits
   for(i=8; i; i--) {                      // 8 bits
     result >>= 1;                         // LSB first
-    if(TPI_clockBit(1)) result &= 0x80;   // get data bit
+    if(TPI_clockBit(1)) result |= 0x80;   // get data bit
   }
 
   // Receive and check parity

picoISP_Programmer/software/picoasp/src/tpi.c

@@ -55,7 +55,7 @@
 #define NVMCMD_WORD_WRITE       0x1D
 
 // Globals
-extern __xdata uint16_t TPI_dly_cnt;    // number of iterations in tpi_delay loop
+extern __xdata uint8_t TPI_dly_cnt;     // number of iterations in tpi_delay loop
 
 // Functions
 void TPI_connect(void);                 // connect TPI bus

picoISP_Programmer/software/picoasp/src/tpi.h

@@ -90,7 +90,7 @@ uint8_t ASP_control(void) {
       return 1;
 
     case USBASP_FUNC_TPI_CONNECT:
-      TPI_dly_cnt = *(uint16_t*) &EP0_buffer[2];
+      TPI_dly_cnt = EP0_buffer[2]; //*(uint16_t*) &EP0_buffer[2];
       TPI_connect();
       return 0;