Re: [PATCH 1/1] memory: tegra: fix fdc test bug

From: Thierry Reding
Date: Fri Apr 07 2017 - 13:43:14 EST


On Fri, Apr 07, 2017 at 06:28:20PM +0100, Ben Dooks wrote:
> The fdc variable is a bool, taking the address of this will generally
> produce true, thus skewing any users of the tegra_mc_set_latency_allowance
> code.
>
> Note, this does not seem to be producing a compiler warning. This was only
> spotted as it meant a caller was seeing 3 times higher than expected
> register results.
>
> Signed-off-by: Ben Dooks <ben.dooks@xxxxxxxxxxxxxxx>
> ---
> drivers/memory/tegra/mc.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)

What tree is this based on? I don't see this code anywhere in mainline.

Thierry

Attachment: signature.asc
Description: PGP signature