linux-kernel - Re: [PATCH v3] HID: logitech-hidpp: battery: provide CAPACITY property for newer devices

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Sat, 31 Jul 2021 23:08:07 +0100
From:   Filipe Laíns <lains@...eup.net>
To:     Hamza Mahfooz <someguy@...ective-light.com>
Cc:     linux-kernel@...r.kernel.org, Bastien Nocera <hadess@...ess.net>,
        Jiri Kosina <jikos@...nel.org>,
        Benjamin Tissoires <benjamin.tissoires@...hat.com>,
        linux-input@...r.kernel.org
Subject: Re: [PATCH v3] HID: logitech-hidpp: battery: provide CAPACITY
 property for newer devices

On Fri, 2021-07-23 at 17:39 -0400, Hamza Mahfooz wrote:
> 
> On Fri, Jul 23 2021 at 08:42:32 PM +0100, Filipe Laíns 
> <lains@...eup.net> wrote:
> > That said, I think we should definitely have a comment here nothing 
> > that, and
> > possible have some bounds checks for the reported voltage value 
> > hinting that
> > there may be bug.
> 
> Hey Filipe,
> 
> Do you have any thoughts on what the bounds ought to be?
> 3500 mV seems like a rather safe lower bound, however the upper bound
> seems much more fuzzy.
> 
> 

Hi Hamza,

Sorry for the delay getting back to you! The most relevant bound would be the
lower one, but I think 5000mV would be a good value.

Cheers,
Filipe Laíns

Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)