On Mon, Sep 08, 2014 at 03:58:25PM +0100, ext Mark Rutland wrote: > On Mon, Sep 08, 2014 at 02:52:42PM +0100, Pavel Machek wrote: > > > I would suggest we have two properties that describe the resistor's > > > rating and whether or not there is a diode: > > > > > > trickle-resistor-ohms = <250> > > > diode-connected; > > > > > > That's easy for a human to write and/or validate, we can easily extend > > > it in future, requires no proliferation of macros, and describes the > > > hardware rather than telling software what to do. > > > > > > The driver becomes a little more complicated, but gains sanity checking, > > > which is a good thing. > > > > Certainly looks better than register bit defines. > > > > As rtc-bq32k has similar options, I'm interested, too. I believe we > > should add > > > > trickle-charge-enable; I am unsure about this. It makes sense for devices where we cannot select resistor but just enable or disable charger. Is there any such devices? For devices like ds1339 this makes no sense. For them it is simpler to just enable the charger if resistor value is specified, and default the charger to be off. But I do not see a problem in supporting the trickle-charge-enable; for devices needing this (independently from the ds1307 driver). > > > > (we may not want to charge at all), and I believe the diode should be > > > > disconnect-diode; > > > > ... With diode connected, charge is slower, and that probably should > > be the default value. We don't want to give too much current in > > default case. (bq32k has hard-coded resistor value for > > diode/not-diode case). > > I agree that if one case is less likely to be problematic / damaging > that should be the default. _Maybe_ diode-connected = <1>; or diode-connected = <0>; or both diode-connected; and diode-disconnected; could be used and let the driver to choose the defaults? But yes, I have no problem with inverting the default to be diode-connected. Then one question regarding the "process". Now if I cook up fourth patch with inverted diode-connected default for ds1339, should I collect new acks for this new patch? (V3 was acked by Alessandro and Jason.) I assume yes. > > Mark. -- ============================================= Matti Vaittinen Senile SW Specialist FINLAND ~~ When things go utterly wrong vim users can always type :help! ~~ -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
