Sequent Smart Fan V6 #61
Comments
|
Hmmm. That device doesn’t use gpio triggers it uses I2c. Does it show the reported version? |
|
Just started playing with REM and I also have the v6 Board. In my case, it will also continually crash REM with the following error Sequents tool reports hardware version 4, even though the silkscreen says 7 |
|
I have been working on getting the v6 working. I need to keep cleaning this up before I submit a PR back to the original repo, but in the meantime, you can pull from https://github.com/sberkovitz/relayEquipmentManager/tree/smartfan-v6. The v6 unit is very different than the original with far less capabilities. Note there is also an GPIO mapping change in here to accommodate the new Bookworm OS in pinouts/raspi.json; I am not yet sure how to properly generalize this so it remains compatibile with all units, ideally REM needs to transition to libgpiod as well |
|
I took a look at the mapping for gpio and I think you may be mistaken as there is no GPIO539 on a Broadcom chipset. What you are lookin at is a conversion to a 4byte integer where the upper short is actually a mask. In the file you changed it simply has bit 2 set on the upper byte so GPIO539 is actually Broadcom 27. Perhaps the issue is that bookworm needs a 32bit value. I will post up some changes for the above unit calculations. The error you are receiving is because the code is trying to normalize the temperature into a common unit and unfortunately it does not have the source units set in the configuration. I think that originally came from the board. |
|
I did some more work on this (sberkovitz@3638327) - it seems these values are different on the Pi 5, as well as different in Bookworm vs "Legacy OS" (as they seem to be calling it). I think you're right about it needing the full 32 bit value, but it also seems like newer devices will not be consistent. Ultimately this is why they want us to use libgpiod instead of the depricated approach the onoff module takes. In the meantime, I added a new board configuration instead of updating the existing one, should ensure it remains backwards compatible. |
|
I pushed up some changes for the smartfan that fixes the missing units issue.
I have found why your GPIO numbers seem so strange. The pi engineers are now identifying gpios by touple which includes the chip id (2) in the gpio. Honestly, this is the same approach as other boards such as the orange pi and probably what raspberry should have been doing all along. Perhaps there is a future for multiple gpio headers on the same board. For now the problem is that the pi 5 does not have the gpio chip embedded (chip id 0). It is external to the cpu on chip id (2). Frankly, gpio has been a debate for a long time. If you rewind a few years ago these same folk yapping about libgpiod now were pushing the bcm libs and goofball wiring pi schemes mapping to the pin number. That route would have expired a long time ago. I am not quite sure where to go from here since there are no interrupts supported in the node modules. |
|
I have an update which provides support for the v6 board on both versions of raspi-os. It is relatively low risk (I think) and should not break anything existing. It's not necessarily the most elegant and there is room for code re-use between the new module and the previous, but it's working. Do you want me to create a PR to submit this back? |
This PR adds support for Sequent Systems Smart Fan v6, along with support for Raspi OS Bookworm on a 4B. The v6 Smart Fan uses a different GPIO pin from previous generations, and also lacks the capabilities drop from v4 onward. As I don't have an older device to test with, I opted to create a new module for this generation instead of sharing the existing one. This new module only contains the capabilities of the v4 and onward, and supports the changed GPIO pin in v6 (?). Some other general fixes as well. Raspberry Pi Bookworm changes how GPIO pins are addressed. Per @rstrouse in rstrouse#61 (comment) I have found why your GPIO numbers seem so strange. The pi engineers are now identifying gpios by touple which includes the chip id (2) in the gpio. Honestly, this is the same approach as other boards such as the orange pi and probably what raspberry should have been doing all along. Perhaps there is a future for multiple gpio headers on the same board. For now the problem is that the pi 5 does not have the gpio chip embedded (chip id 0). It is external to the cpu on chip id (2). To address this, I have included a new pinout profile that I have tested with the Raspberry 4B. I am almost certain the 5 will need a different pinout, but unsure of other 4 variants and earlier.
|
Thank you for this contribution. |
|
I still have issue with the V6 fan. |
|
Are you running Raspberry Pi Bookworm? If so, edit your controllerConfig.json and set controllerType |
I just received three modules from Sequent, the BAS, 8 relays board and the Smart Fan V6. Both BAS and 8 relay board work fine. The Smart Fan however does not work on V4 or <V4. All signs show that it should work. Displays CPU temp, show fan power output but the fan does not turn on. I did test the fan separately and it works fine. The Sequent documentation for the V6 is nonexistent on their website but I did notice this note under the description which I suspect is probably the problem "(Ver. 6.0 uses also GPIO 12)".
If it run the code from Sequent let say "fan 0 pwr 100"
I get: "Failed to open gpio direction for writing!"
Any ideas where to go from here?
The text was updated successfully, but these errors were encountered: