ADT746X: logical-bitwise & confusion in set_max_duty_at_crit()

Darrick J. Wong djwong at us.ibm.com
Tue Mar 11 05:13:19 EST 2008


On Mon, Mar 10, 2008 at 10:59:43AM +0100, Roel Kluin wrote:

> > The & 0xff here is bogus anyway; temp is only ever used as an u8,
> > so just declare it as that, or do proper overflow/underflow checking
> > on it.  The patch will need testing on hardware too, since it changes
> > behaviour (it should be a bugfix, but who knows).
> 
> Maybe someone can test this?

I did.  No regressions observed and it fixes that bug as well.  Sorry I
didn't catch it earlier... :/

Acked-by: Darrick J. Wong <djwong at us.ibm.com>

--D



More information about the Linuxppc-dev mailing list