diff options
author | Alex Elder <elder@linaro.org> | 2021-08-11 16:59:48 +0300 |
---|---|---|
committer | Jakub Kicinski <kuba@kernel.org> | 2021-08-13 00:51:44 +0300 |
commit | 676eec8efd8ed7f051ea84bfa9c1332e656b5c7d (patch) | |
tree | f55688b105f67acca49020add672c1131aacf35b /drivers/net/ethernet/microchip/sparx5/sparx5_mactable.c | |
parent | bed5a942e27e1df67250e27e1f2eb5ea2d4cc362 (diff) | |
download | linux-676eec8efd8ed7f051ea84bfa9c1332e656b5c7d.tar.xz |
net: ipa: always inline ipa_aggr_granularity_val()
It isn't required, but all callers of ipa_aggr_granularity_val()
pass a constant value (IPA_AGGR_GRANULARITY) as the usec argument.
Two of those callers are in ipa_validate_build(), with the result
being passed to BUILD_BUG_ON().
Evidently the "sparc64-linux-gcc" compiler (at least) doesn't always
inline ipa_aggr_granularity_val(), so the result of the function is
not constant at compile time, and that leads to build errors.
Define the function with the __always_inline attribute to avoid the
errors. We can see by inspection that the value passed is never
zero, so we can just remove its WARN_ON() call.
Fixes: 5bc5588466a1f ("net: ipa: use WARN_ON() rather than assertions")
Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: Alex Elder <elder@linaro.org>
Link: https://lore.kernel.org/r/20210811135948.2634264-1-elder@linaro.org
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Diffstat (limited to 'drivers/net/ethernet/microchip/sparx5/sparx5_mactable.c')
0 files changed, 0 insertions, 0 deletions