ramoops: fix use of rounddown_pow_of_two()
authorMarco Stornelli <[email protected]>
Fri, 13 Jan 2012 01:20:58 +0000 (17:20 -0800)
committerLinus Torvalds <[email protected]>
Fri, 13 Jan 2012 04:13:13 +0000 (20:13 -0800)
The return value of rounddown_pow_of_two wasn't evaluated, so the
operation was a no-op.

Signed-off-by: Marco Stornelli <[email protected]>
Reported-by: Andrew Morton <[email protected]>
Reviewed-by: WANG Cong <[email protected]>
Signed-off-by: Andrew Morton <[email protected]>
Signed-off-by: Linus Torvalds <[email protected]>
drivers/char/ramoops.c

index feda90cdac9f2b265d217cb78a1b7850d235ca5d..9c152ab2bfa77d879ffe9953fcc04f40b9b2622d 100644 (file)
@@ -125,8 +125,8 @@ static int __init ramoops_probe(struct platform_device *pdev)
                goto fail3;
        }
 
-       rounddown_pow_of_two(pdata->mem_size);
-       rounddown_pow_of_two(pdata->record_size);
+       pdata->mem_size = rounddown_pow_of_two(pdata->mem_size);
+       pdata->record_size = rounddown_pow_of_two(pdata->record_size);
 
        /* Check for the minimum memory size */
        if (pdata->mem_size < MIN_MEM_SIZE &&