[bug #60753] Patch #1436 breaks multiple programmer/device combinations on MacOS BigSur

[anonymous]

URL:
  <https://savannah.nongnu.org/bugs/?60753>

                 Summary: Patch #1436 breaks multiple programmer/device
combinations on MacOS BigSur
                 Project: AVR Downloader/UploaDEr
            Submitted by: None
            Submitted on: Wed 09 Jun 2021 03:26:22 PM UTC
                Category: None
                Severity: 3 - Normal
                Priority: 5 - Normal
              Item Group: None
                  Status: None
                 Privacy: Public
             Assigned to: None
         Originator Name: Melanie Thielker
        Originator Email: melanie@t-data.com
             Open/Closed: Open
                 Release: SVN snapshot
         Discussion Lock: Any
     Programmer hardware: Various
             Device type: Various

    _______________________________________________________

Details:

Tested with stk500, avrispmkii, with ATMega328P, ATTiny861, ATTiny13

Programmer stops and waits forever after reporting the chip initialized and
ready to receive data.

         Programmer Type : STK500V2
         Description     : Atmel STK500
         Programmer Model: AVRISP
         Hardware Version: 2
         Firmware Version Master : 2.20
         Vtarget         : 5.0 V
         SCK period      : 0.1 us

avrdude: AVR device initialized and ready to accept instructions

Reading |                                                    | 0% 0.00s

Removing avrpart.c:257, make it work again.

The line in question is this:

m->page_size = 1; // ensure not 0

With it removed, the result is as expected:

         Programmer Type : STK500V2
         Description     : Atmel STK500
         Programmer Model: AVRISP
         Hardware Version: 2
         Firmware Version Master : 2.20
         Vtarget         : 5.0 V
         SCK period      : 0.1 us

avrdude: AVR device initialized and ready to accept instructions

Reading | ################################################## | 100% 0.10s

avrdude: Device signature = 0x1e950f (probably m328p)
avrdude: safemode: lfuse reads as C7
avrdude: safemode: hfuse reads as D9
avrdude: safemode: efuse reads as FC

avrdude: safemode: lfuse reads as C7
avrdude: safemode: hfuse reads as D9
avrdude: safemode: efuse reads as FC
avrdude: safemode: Fuses OK (E:FC, H:D9, L:C7)

avrdude done.  Thank you.

Strangely, changing it to:

if (!m->page_size)
  m->page_size = 1

also breaks it, suggesting that a page size of 0 is actually valid.





    _______________________________________________________

Reply to this item at:

  <https://savannah.nongnu.org/bugs/?60753>

_______________________________________________
  Message sent via Savannah
  https://savannah.nongnu.org/