|View Issue Details [ Jump to Notes ]||[ Issue History ] [ Print ]|
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0005155||WHDLoad||[All Projects] General||public||2021-04-23 23:19||2021-04-24 23:05|
|Assigned To||Wepl||Project Info||HD-Installer for OS-Killer|
|Target Version||Fixed in Version|
|Summary||0005155: Allow all vampires cards users to use joypad with more than 2 buttons|
|Description||The apollo vampire team made a choice to patch the lowlevel.library for allowing more than 2 buttons (instead of using the CD32 patch register technique for example). See informations here : https://github.com/flype44/VControl/blob/master/DOCUMENTATION.md#vcontrol-joyport|
As a result:
- all games using directly the hardware will never be able to use more than 2 buttons
- but all games using the lowlevel.library to read the input control can work.
But right now, they don't work with WHDLoad because when WHDload starts, it loads a non patched version of the lowlevel.library.
Would it be possible to:
- either add a "configuration hook" to allow vampire users to inject a patched version of the lowlevel.library
- or, if not possible, to provide a dedicated version of the whdload executable for the vampire ?
|Steps To Reproduce||On a apollo vampire card, launch any games that requires more than 2 buttons with whdload|
|Tags||No tags attached.|
|KickROM||40 - Kick 3.1|
that isn't so simple
all slaves use dedicated joypad read code. whdload doesn't have any control read code (maybe time to add some...)
then, the code for this patch isn't public right?
finally would need some testing but none of us has access to Vampire
I didn't want to imply that my request is easy to implement, sorry if I have not been clear about that.
I don't know much about how whdload is built, but if some controllers read code could be added, it would be great and probably would serve more purposes than just the one I based my request on.
It would really be a whdload new feature, not specific to the vampire.
Now, from the vampire perspective, unless I'm mistaken, the patch of the lowlevel.library published by the apollo team is open source (https://github.com/flype44/VControl/blob/master/V_JOYPORT.asm)
for the testing, I guess it's not ideal if none of the developers have access to a vampire... Correct me if I'm wrong, but the "vampire specific part" of the new feature would be minimal ? Maybe I could help with the final testing of this vampire part... let me know what you think.
Last edited: 2021-04-24 15:51
With the source, yes, it's very easy to implement. But I can't test. And @wepl is the whdload developper not me.
Of course it would require refitting ALL installs using the joypad read routine, but that's not really a problem with some scripting.
I am against adding vampire only features for several reasons:
As far as I'm aware, none of the currently active people involved in creating patches and neither Bert (the WHDLoad coder) has such hardware. This means, whenever someone reports a bug, no one will be able to debug/reproduce it. And I for one will never ever buy a Vampire card.
A WHDLoad version for Vampire only cards doesn't make much sense to me either.
In the end, Bert has to decide, but I doubt such features will be added anytime soon, if ever.
I completely agree with you, that's why my request is to be able to "inject" a custom joypad read routine in whdload.
the implementation of the routine (to make it work with the vampire) would be out of scope of whdload so you won't have to test it.
Some other users may implement different versions of the routine if they want (for example to replace the up direction by a button press).
That's why I think this feature would be very useful, way beyond the vampire need.
As JOTD already noted every install would need to be adapted to support this. This can't expected and I myself would not spend time on adapting my installs for this.
Nevertheless a resload function to check the controller states would be useful and was already thought about. This would allow to encapsulate this specific in WHDLoad.
I will try to discuss such a API extension with the other developers.
|2021-04-23 23:19||quack75||New Issue|
|2021-04-24 00:05||JOTD||Note Added: 0010087|
|2021-04-24 15:45||quack75||Note Added: 0010089|
|2021-04-24 15:50||JOTD||Note Added: 0010090|
|2021-04-24 15:50||JOTD||Assigned To||=> Wepl|
|2021-04-24 15:50||JOTD||Status||new => assigned|
|2021-04-24 15:51||JOTD||Note Edited: 0010090||View Revisions|
|2021-04-24 16:34||StingRay||Note Added: 0010091|
|2021-04-24 17:10||quack75||Note Added: 0010092|
|2021-04-24 23:05||Wepl||Note Added: 0010094|