hardware: add support for Gosund P1 Power Strip

[alextircovnicu]

No description provided.

[mcspr]

Thanks!

I was wondering about the Tasmota implementation doing multiple analogRead calls and taking an average shifting the sum N times:
https://github.com/arendst/Tasmota/blob/3b432a58a3f0de1c1affedf9f894c2f0f4e1b8e1/tasmota/xsns_02_analog.ino#L234-L248
Buttons in the interface do it twice, rules example from the templates page does it 32 times every 0.25seconds. If what we do right now works, might not be a problem :) My guess the precision might be slightly off, sometimes (or never, hopefully).

(and, technically, Tasmota implementation is also missing out on the SDK function doing this specific thing... system_adc_read_fast, but it might not be really suited for using wifi at the same time)

[alextircovnicu]

Sorry for being so late - you know, it is hard with two kids abroad while Europe is more or less in lock-down.
So, to the topic: Like I previously mentioned I had Tasmota running for a week, configured as explained on the template page.
Every 10 buttons presses it was behaving odd: turning ON two relays at once, turning ON another relay while I was turning off one. Also the button click it was not natural. I had to keep the button pressed more than a usual "button click" . Seeing now the implementation I guess all these behaviors make sense.

I think your implementation is the way to go here. Yes, for analog readings, I guess, getting more samples and shifting the sum may be a more logical approach. For this resistor ladder, you already know the expected values (which are about 20 ohms smaller than Tasmota uses). I have activated the analog sensor, to see the button values and the readings are pretty constant (never noticed a deviation bigger than 7 ohms).

I will update the Wiki in the upcoming days.

P.S. Tuya convert never worked for me with this Gosund devices and, which I think it is a little bit strange, the Tasmota minimal firmware (uploaded after espurna) always bricked my devices - I had use the uart programmer again on them.

[mcspr]

I meant to point out the buttons are seem to be configured similar to ours, through the GPIO config as analog button, although it is only available in the not-yet-released Tasmota 9.x version. So the script is kind of deprecated.
But, I have no idea what to do with the adcparam or whatever is used to configure them :)
About the minimal fw... that will probably never work, if this issue still applies:
#1663

Analog sensor does more than one analogRead btw, ref. ANALOG_SAMPLES defaulting to 10 times every ~1ms which adds each value then divides. If it is ok for ladder though, this can be just one read (meaning... two, including the debounce check that the value is still in range)

Stay safe! No worries.