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

Add applet gpio.pca953x #159

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

electroniceel
Copy link
Member

I wanted to further verify the test jig board works. Although I know that this applet can't be used to control the test jig in production (as that will be connected to the internal i2c of the Glasgow), I thought writing an applet for it would be the easiest way to test the i2c io extender on it. Also I wanted to write an applet to get a feeling for how easy that is.

So here is my first applet. I tried to match the style I saw in some of the other applets, hope I got that right.

I thought creating the new "gpio" applet group was the best way to group this applet. Similar applets, like for the pcf8574, mcp23017 or the ports on glasgow itself, would also fit here. Another option might be the "control" group, but I think gpio and io extenders are specific enough to warrant their own group.

@marcan
Copy link
Member

marcan commented Sep 27, 2019

I agree with the new group. I thought a "logic probe" applet frontend that just exposes a useful UI (curses?) for poking/prodding the glasgow ports would be useful, and it should fit within that category. This sounds like it could be useful to generalize over all backends that support GPIO operation (be it the raw Glasgow pins or an I2C expander or anything else, e.g. JTAG boundary scan anyone?), but that can happen later.

@electroniceel
Copy link
Member Author

incorporated all the review from irc

Copy link
Member

@whitequark whitequark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As discussed, I would like this to not use bitarray.

@electroniceel
Copy link
Member Author

adapted to current glasgow upstream (like the change from i2c_master to i2c_initiator)

Base automatically changed from master to main February 27, 2021 09:59
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.

None yet

3 participants