Re: [PATCH 1/2 v2] device-tree: nexus7-flo: Remove power gpio key entry and use pmic8xxx-pwrkey

From: Dmitry Torokhov
Date: Sun Apr 17 2016 - 08:24:32 EST


On Fri, Apr 15, 2016 at 03:01:06PM -0700, Bjorn Andersson wrote:
> On Fri 15 Apr 11:59 PDT 2016, Stephen Boyd wrote:
>
> [..]
>
> > ----8<-----
> > From: Stephen Boyd <sboyd@xxxxxxxxxxxxxx>
> > Subject: [PATCH] Input: pmic8xxx-pwrkey: Fix algorithm for converting trigger
> > delay
> >
> > The trigger delay algorithm that converts from microseconds to
> > the register value looks incorrect. According to most of the PMIC
> > documentation, the equation is
> >
> > delay (Seconds) = (1 / 1024) * 2 ^ (x + 4)
> >
> > except for one case where the documentation looks to have a
> > formatting issue and the equation looks like
> >
> > delay (Seconds) = (1 / 1024) * 2 x + 4
> >
> > Most likely this driver was written with the improper
> > documentation to begin with. According to the downstream sources
> > the valid delays are from 2 seconds to 1/64 second, and the
> > latter equation just doesn't make sense for that. Let's fix the
> > algorithm and the range check to match the documentation and the
> > downstream sources.
> >
> > Reported-by: Bjorn Andersson <bjorn.andersson@xxxxxxxxxx>
>
> Acked-by: Bjorn Andersson <bjorn.andersson@xxxxxxxxxx>

Applied, thank you.

--
Dmitry