Add support for MCP23008 (IO expander, I2C)
Closed this issue · 3 comments
Support shall be pretty similar to existing MCP23016 support.
To be determined: create new device or add support for MCP23008 to MCP23016 device?
At first, it will be easier to separate both, probably.
On second thought, I think there is no need to try to factor both MCP23008 and 23017 devices implementation into one generic API.
Having a separate device API is judged preferrable in the end, as I cannot see the true benefits of one generic device handling both chips. Despite duplicate maintenance in case of bugs, the added complexity of source code is probably not worth.
The only remaining open point resides in namespace: shall we use 2 distinct namespaces (one per device), or would it be better to use only one namespace shared by both devices.
In the second option, how should we name this namespace?