Skip to content

ADP: Servicetastatur #13768

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

ADP: Servicetastatur #13768

wants to merge 6 commits into from

Conversation

stonedDiscord
Copy link
Contributor

@stonedDiscord stonedDiscord commented Jun 2, 2025 •
edited
Loading

New machines marked as NOT_WORKING

Merkur Service Testgerät (ADP)

@stonedDiscord
Squashed commit of the following:
8c0d96c 
commit f672616
Author: stonedDiscord <[email protected]>
Date:   Sat May 31 16:41:23 2025 +0200

    cleanup

commit a281068
Author: stonedDiscord <[email protected]>
Date:   Sat May 31 15:05:06 2025 +0200

    doesnt work

commit e5a0981
Author: stonedDiscord <[email protected]>
Date:   Sat May 31 14:01:26 2025 +0200

    screen works

commit 7426aa4
Author: stonedDiscord <[email protected]>
Date:   Sat May 31 12:53:39 2025 +0200

    LCD read

commit 42439a9
Author: stonedDiscord <[email protected]>
Date:   Sat May 31 12:33:41 2025 +0200

    comments and lcd work

commit 7040399
Author: stonedDiscord <[email protected]>
Date:   Sun May 11 19:14:08 2025 +0200

    entire address

commit 9537c6b
Author: stonedDiscord <[email protected]>
Date:   Sun Apr 6 14:06:55 2025 +0200

    i2c and attempt at lcd

commit 4a3d89a
Author: stonedDiscord <[email protected]>
Date:   Sat Apr 5 20:26:10 2025 +0200

    servicetastatur
@stonedDiscord
correct info at top
e7459e8
@stonedDiscord
umlaut
c547b1b
@stonedDiscord
swap l an r
235f14e
@stonedDiscord
bus is not just lcd
e0b06a7
@stonedDiscord
Copy link
Contributor Author

This is the service keyboard for all the machines in the adp folder.
It starts up but complains about the i2c eeprom being wrong/missing.

Picture, schematic and ROM (not by me):
https://archive.org/details/schaltplan-forum

@stonedDiscord stonedDiscord marked this pull request as ready for review June 10, 2025 13:50
@rb6502
Copy link
Contributor

rb6502 commented Jun 10, 2025

Can it be made to write out a valid EEPROM? If so, we'd want to include that in the ROM set.

Also, how does this interact with the actual machines?

@stonedDiscord
Copy link
Contributor Author

stonedDiscord commented Jun 10, 2025 •
edited
Loading

It plugs into the machine with a 6pin connector on a cable.
Wiping the EEPROM on a real one doesn't show the error message so I assume it can't detect it.
The EEPROM is used to save settings from one machine and apply them to others for when you have to set up an entire arcade hall.
If there is a way to write it out it's probably some key combination during startup, I tried to press some keys but I don't think they registered correctly.

@angelosa
Copy link
Member

Perhaps the eeprom is expected to be 1 filled to make it initialize?

@stonedDiscord
Copy link
Contributor Author 

	ROM_REGION(0x800, "eeprom", 0)
	ROM_FILL(0x000, 0x7ff, 0xff )

sadly didn't change anything

@angelosa
Copy link
Member

Just to be sure:

  1. you need to delete your existing eeprom file, in nvram directory.
  2. ROM_REGION(0x800, "eeprom", ROMREGION_ERASEFF) has same effect.

@stonedDiscord
Copy link
Contributor Author

same result

@rb6502
Copy link
Contributor

rb6502 commented Jun 13, 2025

It's likely the I2C comms aren't right - it might need a pull-up resistor on SCL and/or SDA or something of that nature.

@stonedDiscord
Copy link
Contributor Author

There are no resistors (or other components) on the lines between the CPU and the EEPROM.
My suspicion is that I did something wrong in servicet_state::port3_r() but I can't find out what and would like to submit it in this state anyway.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@stonedDiscord @rb6502 @angelosa