[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <A874F61F95741C4A9BA573A70FE3998FEBCF@DQHE02.ent.ti.com>
Date: Tue, 7 Aug 2012 08:03:45 +0000
From: "Kim, Milo" <Milo.Kim@...com>
To: Axel Lin <axel.lin@...il.com>, Bryan Wu <bryan.wu@...onical.com>
CC: Richard Purdie <rpurdie@...ys.net>,
"linux-leds@...r.kernel.org" <linux-leds@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [PATCH RFT] leds: lp8788: Fix updating scale configuration bits
> We need to do left shift (cfg->num + LP8788_ISINK_SCALE_OFFSET) bits
> for
> updating scale configuration.
>
> Signed-off-by: Axel Lin <axel.lin@...il.com>
> ---
> Hi Milo,
> Current code of updating scale configuration bits looks wrong to me
> because the mask does not match the val.
> I don't have this hardware, can you test this patch?
>
> Thanks,
The scale bits can be never updated without this patch.
This patch should be applied.
Thanks for fixing this bug !
Best Regards,
Milo
Powered by blists - more mailing lists