[Nut-upsdev] [PATCH/RFC v2 0/3] Updates to ACP smart driver

Arjen de Korte nut+devel at de-korte.org
Fri Feb 18 09:49:02 UTC 2011


Citeren Michal Soltys <soltys op ziu.info>:

> So, any +/- comments about this approach ?

The reason why it is taking me so long to adopt this, is the reason  
that (although you separated this is three chunks) the changes to the  
apcsmart driver require to change the LB processing at the same time.  
I'd prefer to delay that part of the patch until we have had time to  
really dig into this. As told before, I'm OK with the remainder of the  
changes you made. I'd prefer it if you could keep the LB out of this  
patch and let me handle this part (since this goes deep into the  
internals of drivers). One thing that I found out already, is that  
main.c is not the proper place to do this. This should really be  
handled in the status_commit() in dstate.c to prevent toggling between  
two states (this isn't pretty).

Best regards, Arjen
-- 
Please keep list traffic on the list (off-list replies will be rejected)




More information about the Nut-upsdev mailing list