- Status Closed
- Percent Complete
- Task Type Patches
- Category Operating System/Drivers
- Assigned To No-one
- Operating System Another
- Severity Low
- Priority Very Low
- Reported Version Version 3.2
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
Attached to Project: Rockbox
Opened by Michael Chicoine - 2009-04-14
Last edited by Thomas Martitz - 2009-04-20
Opened by Michael Chicoine - 2009-04-14
Last edited by Thomas Martitz - 2009-04-20
FS#10128 - Sansa AMS debug cosmetic fix
This is a cosmetic fix for the Sansa AMS ports debug screen. DBOP_DIN displays 8 hex characters until the record button is pressed, then it displays only 4 hex characters (first 4 charactersare "0"). Changed the sprintf parameter from %4x to %8x so that the leading zeros are displayed.
debug-as3525.c.diff
(0.5 KiB)
Closed by Thomas Martitz
2009-04-20 20:11
Reason for closing: Fixed
Additional comments about closing:
2009-04-20 20:11
Reason for closing: Fixed
Additional comments about closing:
Fixed in r20757.
It should only display 4 character, since dbop is a short :/
Since dbop is a short, then why would it displaying 8 characters? Could there be someplace it is expanded?
According to amiconn it's expanded due to the sign bit. Currently dbop_din is a signed short. Since the sign bit changes due to button presses, the sign bit gets added again before the actual short data. The attached patch just changes it to unsigned. Please test and report back.
The patch works on the e200v2, although I had to change button-target.h to get it to build. Attached is the updated patch.