Re: [PATCH 04/10] omap_hwspinlock: Replace "hweight_long(i & 0xf) != 1" with "!is_power_of_2(i & 0xf)"

From: Ohad Ben-Cohen
Date: Mon Dec 07 2015 - 06:08:49 EST


Hi,

On Sun, Dec 6, 2015 at 12:33 PM, Zhaoxiu Zeng <zhaoxiu.zeng@xxxxxxxxx> wrote:
>
> From: Zeng Zhaoxiu <zhaoxiu.zeng@xxxxxxxxx>
>
> Signed-off-by: Zeng Zhaoxiu <zhaoxiu.zeng@xxxxxxxxx>

Please explain why do you think we should make this change.

Btw, the original code used is_power_of_2, but we thought hweight is
more explicit so it was adopted.

Thanks,
Ohad.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/