Fixed READ EEPROM in stk500v2 bootloader

[mlasevich]

As per #6455

CMD_READ_EEPROM_ISP was addressing EEPROM as 16bit instead of 8bit data.
Changed to mimic CMD_PROGRAM_EEPROM_ISP workaround.

[mlasevich]

What is the status of this? Will this be merged into the official bootloader or is there an issue with it?

Thanks.

[facchinm]

@mlasevich we usually update the bootloaders as soon as they are used in production. Any bugfix is well accepted but will be merged only if the generated binary has been validated

[mlasevich]

Thanks for the reply. I mostly wanted to know if I needed to do anything else to get this accepted and it sounds like that is not the case. It is no rush, we can perfectly well use the custom built bootloader for our purposes, but it always feels better to use the official build :-)

For my education, what does "used in production" mean - does that mean it will be tested once a new board using it will be released or something else?

Thanks :-)

[facchinm]

Exactly, as soon as a board is being factory-flashed with a given bootloader we update the corresponding source and binary 😉

[mlasevich]

Closing as the relevant file is no longer in this repo. Opening new PR here: arduino/ArduinoCore-avr#24