Re: [PATCH] sfc: replace min/max nesting with clamp()

From
patchwork-bot+netdevbpf@kernel.org
Date
2025-08-15 00:59:59
Also in
lkml
Reply to
Xichao Zhao, 2025-08-12
Hello:

This patch was applied to netdev/net-next.git (main)
by Jakub Kicinski [off-list ref]:

On Tue, 12 Aug 2025 14:50:26 +0800 you wrote:
The clamp() macro explicitly expresses the intent of constraining
a value within bounds.Therefore, replacing min(max(a, b), c) with
clamp(val, lo, hi) can improve code readability.

Signed-off-by: Xichao Zhao <redacted>
---
 drivers/net/ethernet/sfc/efx_channels.c       | 4 ++--
 drivers/net/ethernet/sfc/falcon/efx.c         | 5 ++---
 drivers/net/ethernet/sfc/siena/efx_channels.c | 4 ++--
 3 files changed, 6 insertions(+), 7 deletions(-)
Here is the summary with links:
  - sfc: replace min/max nesting with clamp()
    https://git.kernel.org/netdev/net-next/c/6398d8a856fb

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html

Thread (5 messages) 5 messages, 5 authors, last reply 2025-08-17
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help