patchAVR Downloader/UploaDEr - Patches: patch #7703, AT89S5x support, stk500 spi...

 
 
Show feedback again

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

patch #7703: AT89S5x support, stk500 spi frequency bugfix, other minor fixes

Submitted by:  Jan Waclawek <wek>
Submitted on:  Mon 16 Jan 2012 05:25:31 PM UTC  
Votes:  20  
 
Category: NonePriority: 5 - Normal
Status: NonePrivacy: Public
Assigned to: NoneOpen/Closed: Open

Fri 10 Aug 2012 06:30:46 AM UTC, comment #2:

Are there any chances to get stk500v2_set_sck_period() function fixed? This patch seems to contain correct implementation of it. As of now, stk500v2_set_sck_period() produces values which are absolutely inconsistent with either, protocol specification or stk500.exe program.

Boris <bxm>
Mon 16 Jan 2012 05:43:18 PM UTC, comment #1:

I'm happy with about everything you are telling, except one:

> I tend to set my editor to strip extra spaces from line
> ends and convert tabs to spaces; sorry.


There's one general rule:

Don't mess with other people's style. Style changes easily
obfuscate actual code changes in the version history, and
thus make it hard to track down bugs later on.

If you really have to mess with it, strictly separate the
style-only changes from actual code changes, so both can be
separated in two commits.

No, your editor's preferences don't serve as an excuse here. My
editor's preferences frequently don't match the files' style here
either (and AVRDUDE's files, unfortunately, don't follow a consistent
style at all), so I have to adjust my editor settings anew for each
file, so resulting changes will follow the existing style in the file
I'm working on as best as possible.

Joerg Wunsch <joerg_wunsch>
Project Administrator
Mon 16 Jan 2012 05:25:31 PM UTC, original submission:

I apologize for the mess, but this is the result of attempting to add a more thorough AT89S5x support (including the 'S8253/'Sx051 subgroup (for which the "sck_polarity" config keyword was added) and accordingly modified STK500 firmware, to be published later); and while working on it, I could not resist to fix other bugs I came across...

This is a work in progress, so the '51-specific stuff is not polished up to expectations (including no documentation YET); I am submitting it as it is now as I have no idea when will I have the opportunity to get back to it. Incorporating them should make no harm to the "mainstream" though, IMO. Please note that this patch incorporates https://savannah.nongnu.org/patch/?7538 although with a different, more generic keyword ("reset_polarity" rather than "is_at89s", as the newer AT89LP family for which I plan to add support later too, has members with "normal" reset polarity).

On the other hand, items 1, 3 and 4 may be of general interest.

Adding to the mess, I tend to set my editor to strip extra spaces from line ends and convert tabs to spaces; sorry.

List of changes:

1. sck frequency setting/printout in terminal mode for stk500V2
- fixed sck frequency reading
- fixed correct calculation of the frequency (based on AVR068 rather than AVR061 which would be correct for stk500v1)
- both sck period and frequency printed out
- correct xtal frequency for AVRISP used
- sck frequency can be entered as frequency in Hz/kHz/MHz now

2. 'S5x programming support
- created a configuration file containing 'S52
- reset_polarity and sck_polarity (normal/invert) added to part configuration
- ** TODO ** add documentation
- reset pin for bitbanged-style programmers is inverted based on reset_polarity
- programming enable in bitbang.c now correctly checks returned value according to pollindex/pollvalue
- stk500v2 now issues the correct value for PARAM_RESET_POLARITY
- for paged memories, _byteblock and block (or any other name) mode added to stk500v2.c

3. default retry_pulse changed to RESET, as SCK is used for retry_pulse only in the historical AT90S parts
- ** todo ** set explicit retry_pulse in conf file for those parts where this applies

4. attempt to "spi" in terminal mode for stk500 crashed avrdude
- fixed (hopefully)

Jan Waclawek

Jan Waclawek <wek>

 

Attached Files
file #24838:  patch1.zip added by wek (20kB - application/zip)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by bxm (Voted in favor of this item)
  • -unavailable- added by bxm (Posted a comment)
  • -unavailable- added by joerg_wunsch (Posted a comment)
  • -unavailable- added by wek (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 20 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 2 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Fri 10 Aug 2012 06:39:12 AM UTCbxmCarbon-Copy-=>Added bxm
    Mon 16 Jan 2012 05:25:31 PM UTCwekAttached File-=>Added patch1.zip, #24838
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup