search

ShendoXT / memcarduino

Arduino PlayStation 1 Memory Card reader
GNU General Public License v3.0
258 stars 39 forks source link

Added optional multi-memory card support #9

Closed soulgriever closed 3 years ago

soulgriever commented 6 years ago

Disabled by default on arduino but easily configurable multi-memory card support should someone connect one arduino to multiple memory cards and use a separate ATT pin for each card.

Also added simple example to memcarduino.py -h

TheBlueTroll commented 6 years ago

Soo, i cant seem to get it to work im using the python script provided to communicate, but i keep getting "error: mcduino communication error, got "8" as identifier (should be "MCDINO")"

im not sure exactly how thats happening, but i load in the older memcarduino.ino and it works fine.

Update: in true murphy fashion, as soon as i comment it starts fuukin working.

TheBlueTroll commented 6 years ago

instead of having 1 hardcoded att pin, you made it so its 2 hardcoded att pins. and a command to pick one. by loading the hardcoded value into "attPin". instead why not rewrite : case MCSELECT1: delay(5); AttPin = AttPin1; break;

into

  case MCSELECT1:
  delay(5);
  AttPin = Serial.read();
    break;

and have the python send the pin that the att is on via a commandline option?

in addition to the setup of the pins "pinMode(AttPin, OUTPUT);" and "digitalWrite(AttPin,HIGH);"

soulgriever commented 6 years ago

Tried passing a command line option initially. No matter what I did, I couldn't get it to work. Might take another stab at it in the future.