kerrydwong / mcp3903 Goto Github PK

MCP3903.cpp

original:
writeRegister(REG_CONFIG, cmd1);
writeRegister(REG_CONFIG, cmd2);

correct:
writeRegister(REG_STATUS_COMM, cmd1);
writeRegister(REG_CONFIG, cmd2);

Hello, I couldn't get this library to work, until it hit me:
When using 5V Arduino, you NEED!!!! to add resistors for the MISO, MOSI, SCK lines. Otherwise it doesn't work and you risk damaging your chip. Your blog post is flooded with people with this issue. I will go post on your blog too.

Hello, the title is self descriptive. There is an overflow error in your code. The fix is easy; I will push a pull request shortly.

Hi @kerrydwong, fantastic project!!

We were able to make a demo board and adapt and extend some of your work to "straight" C for an ARM/FPGA application on Microsemi SmartFusion2 hardware. We also extended the Arduino example to all the channels and tested on the ESP8266. If this work we contributed looks interesting to you and you are still maintaining this project, please feel free to merge back in these additions and I can pull our derivative project and reference your work fully.

https://github.com/CalPlug/MCP3903_ADC/tree/master/Library

Best wishes,
Mike K.