On Tue, Sep 15, 2015 at 12:05:37PM +0300, Laurentiu Palcu wrote: > On Tue, Sep 15, 2015 at 04:56:52PM +0900, Krzysztof Kozlowski wrote: [...] > > In this case - the 2XTMR_EN - I assume that for certain batteries or > > boards you want to extend the time of charging in case of special > > conditions? > > FWIW, I don't see any real benefit in manually changing the 2X timer at > all. The default setting is decent enough. The chip will automatically > switch to 2X timer in certain conditions: for example, when the > temperature threshold is exceeded and the charge current is > automatically lowered. This makes sense since it'll take the battery > more time to charge at a lower current. Laurentiu, your comment just made me go double-check the applicable datasheets and realize that the timer 2x extension feature is by default enabled (not disabled as I originally thought) which is great. With that, we do have good coverage for the "special" charging conditions and don't need to expose access to this feature at this time. Will remove it. > It would probably make more sense to add a DT property to change the > 'Safety Timer Time Limit' device property (which can be 0.75hrs, 6hrs, > 9hrs or totally disabled) than the 2XTMR. > > I omitted both when I originally added support for this chip since the > 6hrs default safety timer seemed more than enough to charge a nowadays > battery, even at 500mA (provided by a standard USB port). But, future > batteries might (and probably will) have higher capacities and will take > longer to charge. Agreed, 6hrs should cover most use cases. And it'll be easy enough to add on this property when needed. Regards, -- Andreas Dannenberg Texas Instruments Inc -- 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
