Re: [PATCH] platform/surface: aggregator: fix a bit test

2021-04-20 Thread Hans de Goede
Hi, On 4/20/21 10:44 AM, Dan Carpenter wrote: > The "funcs" variable is a u64. If "func" is more than 31 then the > BIT() shift will wrap instead of testing the high bits. > > Fixes: c167b9c7e3d6 ("platform/surface: Add Surface Aggregator subsystem") > Reported-by: kernel test robot >

Re: [PATCH] platform/surface: aggregator: fix a bit test

2021-04-20 Thread Maximilian Luz
On 4/20/21 10:44 AM, Dan Carpenter wrote: The "funcs" variable is a u64. If "func" is more than 31 then the BIT() shift will wrap instead of testing the high bits. Fixes: c167b9c7e3d6 ("platform/surface: Add Surface Aggregator subsystem") Reported-by: kernel test robot Signed-off-by: Dan

[PATCH] platform/surface: aggregator: fix a bit test

2021-04-20 Thread Dan Carpenter
The "funcs" variable is a u64. If "func" is more than 31 then the BIT() shift will wrap instead of testing the high bits. Fixes: c167b9c7e3d6 ("platform/surface: Add Surface Aggregator subsystem") Reported-by: kernel test robot Signed-off-by: Dan Carpenter ---