bugAVR Downloader/UploaDEr - Bugs: bug #29837, STK200: stops working with parport...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #29837: STK200: stops working with parport in wtrong state

Submitter:  None
Submitted:  Sun 09 May 2010 01:47:32 PM UTC
   
 
Category:  None Severity:  3 - Normal
Priority:  5 - Normal Item Group:  None
Status:  Duplicate Privacy:  Public
Assigned to:  joerg_wunsch Originator Name:  Jasen Betts
Originator Email:  -email is unavailable- Open/Closed:  Closed
Release:  None Programmer hardware: 
Device type: 

Sun 09 May 2010 02:14:11 PM UTC, comment #1: 

Duplicate for bug #28458 which has already been fixed.

Joerg Wunsch <joerg_wunsch>
Group administrator
Sun 09 May 2010 01:47:32 PM UTC, original submission:  

sometimes "something" causes the parallel port to be left in a state where the reset pin or data pins are driven high by the
STK200 programmer

In this state there is no way to reset the port using avrdude
as the -E parameter is being ignored.

This seems like an oversight, for although the STK200 by design cannot power the target AVR it can interfere with its operation.

it seems there is a "setpin" where there should be "setmany"

here's a patch :)
==== BEGIN
diff --git a/par.c b/par.c
index 269c6c8..6bb064e 100644
--- a/par.c
+++ b/par.c
@@ -285,7 +285,7 @@ static void par_close(PROGRAMMER * pgm)
   ppi_setall(&pgm->fd, PPIDATA, pgm->ppidata);
   ppi_setall(&pgm->fd, PPICTRL, pgm->ppictrl);
 
-  par_setpin(pgm, pgm->pinno[PPI_AVR_BUFF], 1);
+  par_setmany(pgm, pgm->pinno[PPI_AVR_BUFF], 1);
 
   /*
    * Handle exit specs.
==== END

IMO havinf setpin and setmany able to me interchanged
without generating a compiler diagnostic is a mistake.

Anonymous

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attached Files
file #20491:  patch added by None (371B - application/octet-stream - please apply this patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by joerg_wunsch (Posted a comment)
  • -email is unavailable- added by None (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follow 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2010-05-09 joerg_wunsch StatusNone Duplicate
        Assigned toNone joerg_wunsch
        Open/ClosedOpen Closed
    2010-05-09 None Attached File- Added patch, #20491

    Back to the top

    Powered by Savane 3.13-caa5.
    Corresponding source code