Rockbox

Tasklist

FS#10128 - Sansa AMS debug cosmetic fix

Attached to Project: Rockbox
Opened by Michael Chicoine (mc2739) - Tuesday, 14 April 2009, 00:10 GMT
Last edited by Thomas Martitz (kugel.) - Monday, 20 April 2009, 20:11 GMT
Task Type Patches
Category Operating System/Drivers
Status Closed
Assigned To No-one
Operating System Another
Severity Low
Priority Normal
Reported Version Version 3.2
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

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)
This task depends upon

Closed by  Thomas Martitz (kugel.)
Monday, 20 April 2009, 20:11 GMT
Reason for closing:  Fixed
Additional comments about closing:  Fixed in r20757.
Comment by Thomas Martitz (kugel.) - Tuesday, 14 April 2009, 01:20 GMT
It should only display 4 character, since dbop is a short :/
Comment by Michael Chicoine (mc2739) - Tuesday, 14 April 2009, 03:39 GMT
Since dbop is a short, then why would it displaying 8 characters? Could there be someplace it is expanded?
Comment by Thomas Martitz (kugel.) - Tuesday, 14 April 2009, 22:33 GMT
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.
   unsigned-dbop.diff (3.3 KiB)
Comment by Michael Chicoine (mc2739) - Wednesday, 15 April 2009, 00:11 GMT
The patch works on the e200v2, although I had to change button-target.h to get it to build. Attached is the updated patch.
   unsigned-dbop-v2.diff (4.3 KiB)

Loading...