Thread (37 messages) 37 messages, 7 authors, 2020-06-03

Re: [PATCHv1 03/19] power: supply: core: add manufacture date properties

From: Emil Velikov <hidden>
Date: 2020-05-15 16:04:22
Also in: linux-pm, lkml

On 2020/05/15, Sebastian Reichel wrote:
Hi,

On Fri, May 15, 2020 at 03:47:32PM +0100, Emil Velikov wrote:
quoted
On 2020/05/13, Sebastian Reichel wrote:
quoted
Some smart batteries store their manufacture date, which is
useful to identify the battery and/or to know about the cell
quality.
Have you considered exposing this as a single file? Say following
the ISO8601 format - YYYY-MM-DD.
Yes. My initial implementation was exactly that. The thing is, that
I suspect some fuel gauge implementations may only expose the year
or just year + month. I chose 3 files, since receiving 'YYYY-MM'
instead of full ISO code might be more unexpected than not having
the DAY file available. But I don't have a strong opinion on this.
Fwiw the ISO 8601 allows for YYYY-MM, although you're spot on. Having
the three fields separate makes perfect sense.

Thanks
Emil
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help