iidxio BIO2 driver backed implementation - [merged]

icex2 commented 3 years ago

Merges iidxio-bio2 -> master

Summary

Implements the iidxio API using the iidxio bio2 driver. This allows you to play any game that supports BT5's iidxio API.

Description

Not a lot to add, summary covers that.

Related Issue

https://dev.s-ul.eu/djhackers/bemanitools/-/issues/8

How Has This Been Tested?

Tested on my own cabinet with an original Konami BIO2 from a IIDX 25 PCB + sub IO to hook it up to the C02 wiring of the cabinet.

Checklist

Tested with the following games:
- [x] iidx 27
[x] Followed the developer (style) guidelines.
[x] Updated existing doc of or add new doc to README file(s).
[x] Updated development documentation.

icex2 commented 3 years ago

added 3 commits

f8989429 - bio2: Add BIO2_BI2A_CMD_INIT command
4edb3616 - bi2a: Clarify data field usage on init command
c111adbb - bi2a-iidx: Fix incorrect init command data

Compare with previous version

icex2 commented 3 years ago

added 4 commits

571890f5 - bio2: Add BIO2_BI2A_CMD_INIT command
b5164f4e - bi2a: Clarify data field usage on init command
7c592cd3 - bi2a-iidx: Fix incorrect init command data
bf90b20c - wip

Compare with previous version

icex2 commented 3 years ago

marked this merge request as draft from bf90b20c049d7d9fc614fc761fcd1ec1fca3ed70

icex2 commented 3 years ago

added 1 commit

f7ea0309 - hardware doc: ezusb RJ45 to DIN8 connector pinout

Compare with previous version

icex2 commented 3 years ago

added 7 commits

42923764 - hardware doc: ezusb RJ45 to DIN8 connector pinout
df4c41aa - doc: Add vigem-iidxio documentation
bdb7feed - doc: Collect vigem related documentation in own subfolder in doc
a5c1f991 - doc: Add index README for vigem stuff, add vigem info to root readme
7c2622d7 - doc: Add iidxio-bio2 documentation
b251b1ce - doc: Improve visibility of iidxio API implementations
e5d52cf4 - wip

Compare with previous version

icex2 commented 3 years ago

added 2 commits

ecf2e315 - doc/iidxhook9: Add notes about BIO2 driver usage
68b822b0 - wip

Compare with previous version

icex2 commented 3 years ago

added 1 commit

46ba2dc7 - doc/hardware: Add information about bio2 turntable bypass cable

Compare with previous version

icex2 commented 3 years ago

marked the checklist item Updated existing doc of or add new doc to README file(s). as completed

icex2 commented 3 years ago

marked the checklist item Updated development documentation. as completed

icex2 commented 3 years ago

added 14 commits

466901d6 - iidxiotool: Fix incorrect top lamp order for setting output
aa977fa0 - bio2/iidx: Deduplicate node command, use common acio base command
1def3b21 - bio2/iidx: Add enums to easily map top lamps and panel buttons
37f25788 - bio2drv: Add implementation for iidx
6be5ffb9 - iidxio-bio2: Add iidxio implementation using bio2 iidx driver
fd78f62f - dev doc: Documented BIO2 package dumps used for debugging
0a3d9357 - bio2: Add BIO2_BI2A_CMD_INIT command
7982b9ca - bi2a: Clarify data field usage on init command
a5feed72 - bi2a-iidx: Fix incorrect init command data
d91228cb - hardware doc: ezusb RJ45 to DIN8 connector pinout
dc0f712a - doc: Add iidxio-bio2 documentation
b3c6be92 - doc: Improve visibility of iidxio API implementations
49340b67 - doc/iidxhook9: Add notes about BIO2 driver usage
b31cca92 - doc/hardware: Add information about bio2 turntable bypass cable

Compare with previous version

icex2 commented 3 years ago

marked this merge request as ready

icex2 commented 3 years ago

Updated, moved ViGEm relevant commits to its own MR !68

icex2 commented 3 years ago

added 16 commits

26a1831a - doc/api: Add missing sdvxio implementations to list
617bd2d0 - bt5 api/iidxio: Add missing enum to easily identify top lamp order
6fd6eea6 - iidxiotool: Fix incorrect top lamp order for setting output
705d9b1a - bio2/iidx: Deduplicate node command, use common acio base command
6d983026 - bio2/iidx: Add enums to easily map top lamps and panel buttons
812bd1bc - bio2drv: Add implementation for iidx
4d17b9a8 - iidxio-bio2: Add iidxio implementation using bio2 iidx driver
c635cecf - dev doc: Documented BIO2 package dumps used for debugging
13016da9 - bio2: Add BIO2_BI2A_CMD_INIT command
cdc2bfd6 - bi2a: Clarify data field usage on init command
83e0430e - bi2a-iidx: Fix incorrect init command data
f6758fa8 - hardware doc: ezusb RJ45 to DIN8 connector pinout
7f8c76e7 - doc: Add iidxio-bio2 documentation
b44efa62 - doc: Improve visibility of iidxio API implementations
e42a0c2e - doc/iidxhook9: Add notes about BIO2 driver usage
121a9d0c - doc/hardware: Add information about bio2 turntable bypass cable

Compare with previous version

icex2 commented 3 years ago

In GitLab by @xyen on Dec 19, 2020, 23:19

Commented on doc/iidxhook/iidxhook9.md line 111

(which is the crucial part for automatic detection). -> (which is the crucial part for the stock game's automatic detection).

icex2 commented 3 years ago

In GitLab by @xyen on Dec 19, 2020, 23:19

Commented on src/main/bio2drv/bi2a-iidx.c line 15

A comment here describing what happens if we use the wrong value would be nice.

icex2 commented 3 years ago

In GitLab by @xyen on Dec 19, 2020, 23:19

Commented on src/main/bio2emu-iidx/bi2a.c line 88

CLEAR -> BIO2_BI2A_CMD_INIT

icex2 commented 3 years ago

In GitLab by @xyen on Dec 19, 2020, 23:22

Commented on src/main/bio2drv/bi2a-iidx.c line 15

nit: better yet move both this one and the one in bi2a-sdvx to the header.

icex2 commented 3 years ago

In GitLab by @xyen on Dec 19, 2020, 23:22

approving aside from some minor changes

icex2 commented 3 years ago

In GitLab by @xyen on Dec 19, 2020, 23:22

approved this merge request

icex2 commented 3 years ago

I wouldn't move that to the header since, at least with current knowledge, this is an implementation detail specific to that usage of the IO, e.g. sdvx vs. iidx. Added clarifying comment because that's a good point.

icex2 commented 3 years ago

changed this line in version 10 of the diff

icex2 commented 3 years ago

changed this line in version 10 of the diff

icex2 commented 3 years ago

added 7 commits

849684a9 - bi2a: Clarify data field usage on init command
46cd8b36 - bi2a-iidx: Fix incorrect init command data
24cd02e0 - hardware doc: ezusb RJ45 to DIN8 connector pinout
630b8394 - doc: Add iidxio-bio2 documentation
07655ae4 - doc: Improve visibility of iidxio API implementations
d513d245 - doc/iidxhook9: Add notes about BIO2 driver usage
c240fe2c - doc/hardware: Add information about bio2 turntable bypass cable

Compare with previous version

icex2 commented 3 years ago

resolved all threads

icex2 commented 3 years ago

added 26 commits

c240fe2c...3199a4fa - 6 commits from branch master
c7a84fbd - bio2: Update unknown commands
8bfa8917 - bio2/iidx: Swap in and out struct names to align with sdvx definitions
15911177 - Add missing iidx 27 avs version info to list
69614a30 - Module.mk: Apply lex sort to includes
ba3a1ce2 - doc/api: Add missing sdvxio implementations to list
e7ed8e21 - bt5 api/iidxio: Add missing enum to easily identify top lamp order
b4cd23d5 - iidxiotool: Fix incorrect top lamp order for setting output
b7fe7232 - bio2/iidx: Deduplicate node command, use common acio base command
b832ad74 - bio2/iidx: Add enums to easily map top lamps and panel buttons
287b3705 - bio2drv: Add implementation for iidx
7740e5e0 - iidxio-bio2: Add iidxio implementation using bio2 iidx driver
f0bd33bc - dev doc: Documented BIO2 package dumps used for debugging
f42c5ace - bio2: Add BIO2_BI2A_CMD_INIT command
63064d34 - bi2a: Clarify data field usage on init command
f41c9995 - bi2a-iidx: Fix incorrect init command data
ec5102f5 - hardware doc: ezusb RJ45 to DIN8 connector pinout
30d346a3 - doc: Add iidxio-bio2 documentation
7d0cb64a - doc: Improve visibility of iidxio API implementations
10e085c1 - doc/iidxhook9: Add notes about BIO2 driver usage
e49b1e05 - doc/hardware: Add information about bio2 turntable bypass cable

Compare with previous version

djhackersdev / bemanitools