- Status Closed
- Percent Complete
- Task Type Patches
- Category Battery/Charging
- Assigned To No-one
- Operating System All players
- Severity Low
- Priority Very Low
- Reported Version Version 3.3
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
FS#10397 - mkamsboot: little code rewrite to have better output
This rewrites the code to extend the output of the (standalone) mkamsboot, so that OF version and model version (e.g. v4 for m200v4) are shown.
Closed by kugel.
2009-07-05 02:33
Reason for closing: Accepted
Additional comments about closing: Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407
2009-07-05 02:33
Reason for closing: Accepted
Additional comments about closing: Warning: Undefined array key "typography" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 371 Warning: Undefined array key "camelcase" in /home/rockbox/flyspray/plugins/dokuwiki/inc/parserutils.php on line 407
Committed as of r21648. Thanks
Rafael.
Loading...
Available keyboard shortcuts
- Alt + ⇧ Shift + l Login Dialog / Logout
- Alt + ⇧ Shift + a Add new task
- Alt + ⇧ Shift + m My searches
- Alt + ⇧ Shift + t focus taskid search
Tasklist
- o open selected task
- j move cursor down
- k move cursor up
Task Details
- n Next task
- p Previous task
- Alt + ⇧ Shift + e ↵ Enter Edit this task
- Alt + ⇧ Shift + w watch task
- Alt + ⇧ Shift + y Close Task
Task Editing
- Alt + ⇧ Shift + s save task
See, I mixed fw_revision and hw_revision in 1 place :(
s/clop/clip/ in mkamsboot.h
fw_version and hw_version should go in separate arrays since they don't change between different firmwares for a same model
If you want I can do it when i find some time
I thought we agreed on doing this sort of cosmetics later :)
This patch is about giving more output (including OF version) which needs slight code changes.
- Adds the changes mentioned.
- Change a bit the text of a string (Beginning to patch the firmware ⇒ Firmware patching has begun!)
- Add newlines after this string, and before last line of output
Is it ok for you?
Surely it is ok, just make sure it still works! :)
[INFO] Original firmware MD5 checksum match
[INFO] Model: Sansa Clip v2 (v2) - Firmware version: 2.01.32
[ERR] Model name "clv2" not found in rockbox.sansa
[ERR] Could not load rockbox.sansa
Normal since it was scrambled with "clip", but perhaps the message could be better.
Also perhaps the hw_revision could be merged into the model name ? "Sansa Clip v2 (v2)"
I'd drop the v2 actually.