Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

More pigpiod features (SPI, I2C, File) #19

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

mathiasbietz
Copy link

I made some additions for the missing features directly in pigpio-client.js.
In case that way is not intended, let me know, please, so I can change that in order to let the difference to master stay at a minimum in the future.

All function additions have been tested with real hardware on my Raspberry Pi 3B (latest Raspbian and pigpiod).

Adds client functions to make respective calls to the pigpio daemon for
- SPI open, close and write methods
- File open, close, read and seek methods

Includes updated documentation.
Adds client functions to make respective calls to the pigpio daemon for
- I2C open, close read and write methods

Includes updated documentation.
@guymcswain
Copy link
Owner

@mathiasbietz ,
Thank you for the contributions. I'm just now in the process of merging I2C APIs of my own, along with BSC I2C slave peripheral. So I request that you wait until after I make my merge then re-submit your PR for the SPI and File APIs.
Thanks!

@guymcswain
Copy link
Owner

I have pushed my I2C APIs and created v1.5.0. You may use this to create your new PR. Thanks.

implementation and all socket commands in SIF.js.
Comprises changes since 1.4.0 (so also includes 1.5.0).
@townsen
Copy link

townsen commented Jul 4, 2021

Any chance these conflicts can be resolved and the new functionality merged? I need the File APIs...

providing SMBUS read operations allowing to specify a register.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants