diff options
| author | Richard Fitzgerald <rf@opensource.cirrus.com> | 2023-03-15 14:50:51 +0000 |
|---|---|---|
| committer | Vinod Koul <vkoul@kernel.org> | 2023-04-12 15:30:35 +0530 |
| commit | 9ddae9dad09e8bdaa33909bdcbb062c883c3c332 (patch) | |
| tree | 81f67cde2e1f44a79aa66fb7a241727fa027e83e /drivers/soundwire/generic_bandwidth_allocation.c | |
| parent | 2367e0ecb498764e95cfda691ff0828f7d25f9a4 (diff) | |
| download | linux-9ddae9dad09e8bdaa33909bdcbb062c883c3c332.tar.gz linux-9ddae9dad09e8bdaa33909bdcbb062c883c3c332.tar.bz2 linux-9ddae9dad09e8bdaa33909bdcbb062c883c3c332.zip | |
soundwire: bandwidth allocation: Use hweight32() to calculate set bits
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>
Diffstat (limited to 'drivers/soundwire/generic_bandwidth_allocation.c')
| -rw-r--r-- | drivers/soundwire/generic_bandwidth_allocation.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/soundwire/generic_bandwidth_allocation.c b/drivers/soundwire/generic_bandwidth_allocation.c index f7c66083a4dd..ea3e8ef408e4 100644 --- a/drivers/soundwire/generic_bandwidth_allocation.c +++ b/drivers/soundwire/generic_bandwidth_allocation.c @@ -6,6 +6,7 @@ * */ +#include <linux/bitops.h> #include <linux/device.h> #include <linux/module.h> #include <linux/mod_devicetable.h> @@ -54,7 +55,7 @@ static void sdw_compute_slave_ports(struct sdw_master_runtime *m_rt, slave_total_ch = 0; list_for_each_entry(p_rt, &s_rt->port_list, port_node) { - ch = sdw_ch_mask_to_ch(p_rt->ch_mask); + ch = hweight32(p_rt->ch_mask); sdw_fill_xport_params(&p_rt->transport_params, p_rt->num, false, |