Rockbox

Tasklist

FS#9590 - Cowon D2 touchscreen driver registers wrong presses

Attached to Project: Rockbox
Opened by Francesco Rigoni (FranzD2) - Saturday, 29 November 2008, 21:19 GMT
Last edited by Rob Purchase (shotofadds) - Saturday, 26 September 2009, 20:58 GMT
Task Type Patches
Category Drivers
Status Closed
Assigned To No-one
Operating System All players
Severity Low
Priority Normal
Reported Version Release 3.0
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Cowon D2 touchscreen driver registers wrong presses.
This task depends upon

Closed by  Rob Purchase (shotofadds)
Saturday, 26 September 2009, 20:58 GMT
Reason for closing:  Out of Date
Additional comments about closing:  Obsolete as of r22843
Comment by Francesco Rigoni (FranzD2) - Saturday, 29 November 2008, 21:23 GMT
This patch solves the problem, it introduces an error compensation mechanism when reading values from the touchscreen and a busy wait on the "data ready" flag of the corresponding ADC instead of just writing ADCSTART and reading the value immediately.
Thgis patch applies to firmware/target/arm/tcc780x/cowond2/button-cowond2.c .
Comment by Maurus Cuelenaere (mcuelenaere) - Sunday, 30 November 2008, 12:56 GMT
TABs aren't compliant with the Rockbox guidelines ( http://svn.rockbox.org/viewvc.cgi/trunk/docs/CONTRIBUTING?view=markup ), you'll need to replace them with 4 spaces.
Comment by Francesco Rigoni (FranzD2) - Sunday, 30 November 2008, 17:21 GMT
Replaced all tabs with 4 spaces, sorry :)
Comment by Francesco Rigoni (FranzD2) - Sunday, 08 February 2009, 14:13 GMT
Ok, this patch include the calibration fix, hopefully that's the last one :)
Comment by Serg (Shad0w) - Tuesday, 12 May 2009, 11:40 GMT
I can't apply this patch.
Why?

:~/rockbox$ patch -p0 < altTouchScreen_mode_calibration_fix.patch
patching file firmware/target/arm/tcc780x/cowond2/button-cowond2.c
Hunk #3 succeeded at 29 with fuzz 2 (offset -1 lines).
Hunk #4 succeeded at 36 (offset -18 lines).
Hunk #5 succeeded at 52 (offset -18 lines).
Hunk #6 succeeded at 68 (offset -18 lines).
Hunk #7 FAILED at 92.
1 out of 7 hunks FAILED -- saving rejects to file firmware/target/arm/tcc780x/cowond2/button-cowond2.c.rej

P.S. SVN (revision 20916)
Comment by Andrew Beveridge (andrewthecoder) - Sunday, 17 May 2009, 01:40 GMT
The patch fails because that file has changed too much since the patch was made.
Have a look: http://svn.rockbox.org/viewvc.cgi/trunk/firmware/target/arm/tcc780x/cowond2/button-cowond2.c?view=log

Judging by the date of Franz's post above (8th February 2009), I suggest you try revision 18338 of that file.

Wait, by the looks of revision 20527, rob committed part of this patch to SVN already, so this patch is obsolete as far as I understand.

Loading...