[Nut-upsdev] SMS and Megatec protocol Diffs

[Marcio Gomes]

Carlos Rodrigues escreveu:
> On Feb 11, 2008 5:30 PM, Marcio Gomes <mpg at microlink.com.br> wrote:
>   
>> I change the if in [OFF] detect state to, in original way, if you send a
>> reset.watchdog, the UPS goes
>> to OFF state in clients, but it is not correct, because UPS is OL and
>> Shutdown are in progress, but
>> the power is not OFF
>>     
>
> First of all, thanks for the patch. But I have a few comments:
>
> Are you saying the UPS raises the "OFF" flag when a timed shutdown
> command is sent to it, before actually shutting down? If so, that is a
> violation of the megatec protocol. The megatec version in the
> development trunk has an option "ignoreoff" to accomodate UPSes that
> exibit this behavior.
>   

Ok I will fetch the trunk and test.
>   
>> Know working SMS Models  :
>>
>> SINUS Single : Only versions
>> Manager III : Only serial versions
>>     
>
> I'll add this to the compatibility list. 
>   

Please add this text  _/*Only serial versions without "alerta24horas 
feature*/_  the
"alerta24horas" ups use a proprietary protcol. I am working in it, but 
without sucess.


>   
>>  /* Known battery types must be in ascending order by "nominal" first, and then by "max". */
>> -static BatteryVolts_t batteries[] = {{ 12.0,  9.0, 16.0,  9.7, 13.7,  0.0 },   /* Mustek PowerMust 600VA Plus (LB unknown) */
>> +static BatteryVolts_t batteries[] = {{ 12.0,  9.0, 16.0,  9.6 ,13.3 , 9.7 }, /* SMS Ltda confirmed to Sinus Single 2Kva , Manager III 600 and 1300 Va */
>> +                                     { 12.0,  9.0, 16.0,  9.7, 13.7,  0.0 },   /* Mustek PowerMust 600VA Plus (LB unknown) */
>>     
>
> With this change, the Mustek line will always be ignored, making the
> charge calculation wrong for them (and other models). This is actually
> an impossible situation, so the latest versions of megatec have a new
> option "battvolts" to let the user specify these values if needed.
> Anyway, I'm now adding the values for this option in the compatibility
> list when I have them (i.e. for that model it will be
> "battvolts=9.6:13.3").
>
>   
Please wait some days, i am working in a fine adjust, the above strings 
have some
problems between various SMS hardware versions. I will send the 
paramters asp.

>> -static char watchdog_timeout = 1;  /* in minutes */
>> +static char watchdog_timeout = 5;  /* in minutes */
>>     
>
> The frequency at which the timer is reset is much higher than once per
> minute, so it won't slip and accidentaly let the UPS shutdown.
> However, I guess some UPSes may be innacurate with values <= 1 minute.
> This may be a good idea.
>   
I have some problems in past. This is the main motivation to change
this value to 5.

>   
>> -       if (query.flags[FL_LOAD_OFF] == '1') {
>> -               status_set("OFF");
>> -       } else if (query.flags[FL_ON_BATT] == '1' || query.flags[FL_BATT_TEST] == '1') {
>> +//     } else
>> +       if (query.flags[FL_ON_BATT] == '1' || query.flags[FL_BATT_TEST] == '1') {
>>                 status_set("OB");
>>         } else {
>>                 status_set("OL");
>> @@ -579,6 +579,10 @@
>>                         }
>>                 }
>>         }
>> +
>> +        if (query.flags[FL_LOAD_OFF] == '1') {
>> +                status_set("SHUTDOWN");
>> +        }
>>     
>
> Please check the megatec version in the development trunk*. This
> section has changed and I guess it works similarly to what you wanted
> here.
>
>   


> * http://svn.debian.org/wsvn/nut/trunk/drivers/megatec.c?op=file&rev=0&sc=0
>
>   
Ok. I will fetch it, and try help in SMS and TSHARA adjusts to thi driver.

Carlos -> Do you speak portuguese, I am brazilian !


Regards,

Marcio Gomes