fenugrec / nisprog

nisprog Nissan ECU reflashing tool
GNU General Public License v3.0
80 stars 17 forks source link

(breaking): use crc16 for writeblock command instead of cks_add8 #12

Open fenugrec opened 3 years ago

fenugrec commented 3 years ago

cks_add8() is dumb and only used for <SID_FLASH> <SIDFL_WB> ... <CRC> frames. Using the crc16 code we already have anyway would eliminate some code, at the cost of 1 more byte to transfer per chunk. This change would need to be done in nisprog at the same time. (https://github.com/fenugrec/npkern/issues/5)