Skip to content

Fixed pylint errors #33

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Nov 8, 2022
Merged

Fixed pylint errors #33

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
ceec424
Fixed pylint errors
tekktrik Nov 8, 2022
fbc6a25
Reformatted per pre-commit
tekktrik Nov 8, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
20 changes: 9 additions & 11 deletions adafruit_avrprog.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -114,6 +114,7 @@ def verify_sig(self, chip, verbose=False):
return False
return True

# pylint: disable=too-many-branches
def program_file(self, chip, file_name, verbose=False, verify=True):
"""
Perform a chip erase and program from a file that
Expand Down Expand Up @@ -243,25 +244,22 @@ def read_fuses(self, chip):
self.end()
return (low, high, ext, lock)

# pylint: disable=unused-argument,expression-not-assigned
# pylint: disable=unused-argument,too-many-arguments
def write_fuses(self, chip, low=None, high=None, ext=None, lock=None):
"""
Write any of the 4 fuses. If the kwarg low/high/ext/lock is not
passed in or is None, that fuse is skipped
"""
transaction_comp = (0xE0, 0xA0, 0xA8, 0xA4)
fuses = (lock, low, high, ext)
self.begin(clock=_SLOW_CLOCK)
lock and self._transaction((0xAC, 0xE0, 0, lock))
self._busy_wait()
low and self._transaction((0xAC, 0xA0, 0, low))
self._busy_wait()
high and self._transaction((0xAC, 0xA8, 0, high))
self._busy_wait()
ext and self._transaction((0xAC, 0xA4, 0, ext))
self._busy_wait()
for fuse, comp in zip(fuses, transaction_comp):
if fuse:
self._transaction((0xAC, comp, 0, fuse))
self._busy_wait()
self.end()

# pylint: enable=unused-argument,expression-not-assigned

# pylint: disable=too-many-arguments
def verify_fuses(self, chip, low=None, high=None, ext=None, lock=None):
"""
Verify the 4 fuses. If the kwarg low/high/ext/lock is not
Expand Down