SmokeMonsterPacks / Nt-Mini-Noir-Jailbreak

Custom "Jailbreak" firmware for the Analogue Nt Mini V2 "Noir"
142 stars 14 forks source link

Clarify support for Famicom data pad vs NTT datapad #8

Closed itsumimario closed 3 years ago

itsumimario commented 3 years ago

The documentation as it is currently suggests in multiple places that the famicom data pad (aka the famicom network controller HVC-051) is a viable controller choice for several of the cores that make use of the number pad. This doesn’t seem to be the case because the cores that could potentially make use of the number pad don’t read input from the famicom expansion port. So perhaps clarify that it’s the NTT Datapad SNES controller (with SNES to NES adapter) plugged into the NES port that ought to be used, rather than a famicom data pad plugged into the famicom expansion port.

frederic-mahe commented 3 years ago

@itsumimario the README says:

Only the NTT Data Pad or the Famicom Network Controller can fully map the numberpads of the Intellivision controller.

Do you confirm that the Famicom Network Controller (HVC-051) does not work with the Intellivision? and that only the SNES NTT Data Pad (NDK10) does?

itsumimario commented 3 years ago

I can confirm that the HVC-051 does not work with Intellivision. I do not yet own the SNES NTT Data Pad, so I cannot confirm myself that it works, but I have heard from others that it does work.

From what I’ve heard, the issue is that the controller button-press signals coming from the famicom expansion port get mapped to player 3, which isn’t useful at all for most of the cores that could potentially find the number pad on an HVC-051 useful, since those cores are expecting input by a player 1. This is in contrast to the SNES NTT datapad, which when plugged in to the NES port 1 (via a SNES to NES controller adapter), simply sends its button signals as player 1 as expected. Thus the NTT datapad works seamlessly with all cores, I would imagine.

frederic-mahe commented 3 years ago

Thanks @itsumimario the README has been updated.

itsumimario commented 3 years ago

Thanks for updating the documentation. I’m gonna be a tad obnoxious and point out that there still remains a reference to using the HVC-051 under the Colecovision section of the readme that I think you forgot to strikeout or remove. :)

frederic-mahe commented 3 years ago

you are right @itsumimario (fixed in 25f3ada4a15a0dfadae8c135b2b3dec60dca3a20).