Menu

#373 Fix Cardco Cardkey 1 Keypad

v3.x
closed-accepted
gpz
None
2024-06-16
2024-06-11
No

This fixes the Cardco Cardkey keypad so it works with the official driver (attached).

When a key is pressed, pin 6 (fire) is connected to ground, not pin 5 (POTX). Also, which key is pressed was encoded wrong.

2 Attachments

Discussion

  • Dieter Baron

    Dieter Baron - 2024-06-11

    Sorry, attached the wrong diff.

     
  • gpz

    gpz - 2024-06-15

    applied in r45187 - thanks!

     
  • gpz

    gpz - 2024-06-15
    • status: open --> closed-accepted
    • assigned_to: gpz
     
  • Querino

    Querino - 2024-06-15

    but now according to "joyride 1.4b" the "0" key seems to be pressed all the time. is that correct?

     
  • gpz

    gpz - 2024-06-15

    Is the VICE test program also wrong?

     
    • Querino

      Querino - 2024-06-16

      Is the VICE test program also wrong?

      that doesn't seem to work at all. it doesn't show any key presses, except the permanently pressed "0" (still a ? for me).


      ah well... i also wonder how the atari CX85 is supposed to wor, the keyboard doesn't work at all if i select this one. also, the KEYPAD_KEYS_NUM 20 doesn^t match the actual number of defined keys. not sure if this is a real issue or if i am doing something wrong.
      i get it this isn't the proper place here if it's an issue.

       
  • gpz

    gpz - 2024-06-15

    And as for the "wrong keys" - are you sure this is not the result of a wrong keymap?

     
  • Dieter Baron

    Dieter Baron - 2024-06-16

    Yes, see my comment on issue #2039.

     
  • gpz

    gpz - 2024-06-16
    • status: closed-accepted --> open
     
  • gpz

    gpz - 2024-06-16

    reopening this - the patch seems buggy. I noticed this while trying to fix the test program

    a) please next time also update all comments. after your patch the comments do not match the code anymore (which is super confusing really). I have done that now in r45191
    b) the encoding of the values is still wrong, some values are used twice, some never - have a look at the comment in the code. Please tell what is correct :)

    edit: OK even your new comments were wrong :) The code is fine apparently.

     

    Last edit: gpz 2024-06-16
  • gpz

    gpz - 2024-06-16

    Also fixed cardkey-c64.prg to work with the current implementation. Please confirm

     
  • gpz

    gpz - 2024-06-16
    • status: open --> closed-accepted
     

Log in to post a comment.