soundwire: bandwidth allocation: Use hweight32() to calculate set bits

Browse source

Replace the call to sdw_ch_mask_to_ch() with a call to hweight32().

sdw_ch_mask_to_ch() is counting the number of set bits. The hweight()
family of functions already do this, and they have an advantage of
using a bit-counting instruction if it is available on the target CPU.
This also fixes a potential infinite loop bug in the implementation of
sdw_ch_mask_to_ch().

Signed-off-by: Richard Fitzgerald <rf@opensource.cirrus.com>
Link: https://lore.kernel.org/r/20230315145051.2299822-1-rf@opensource.cirrus.com
Signed-off-by: Vinod Koul <vkoul@kernel.org>

...

This commit is contained in:

Richard Fitzgerald 2023-03-15 14:50:51 +00:00 • committed by Vinod Koul

parent 2367e0ecb4

commit 9ddae9dad0

2 changed files with 2 additions and 12 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Download patch file Download diff file Expand all files Collapse all files

11

drivers/soundwire/bus.h

View file

3

drivers/soundwire/generic_bandwidth_allocation.c

View file