media: atomisp: use clamp() in ia_css_eed1_8_encode()

When it needs to get a value within a certain interval, using clamp()
makes the code easier to understand than min(max()).

Signed-off-by: Li Zetao <lizetao1@huawei.com>
Link: https://lore.kernel.org/r/20240830011752.603433-2-lizetao1@huawei.com
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>

diff --git a/drivers/staging/media/atomisp/pci/isp/kernels/eed1_8/ia_css_eed1_8.host.c b/drivers/staging/media/atomisp/pci/isp/kernels/eed1_8/ia_css_eed1_8.host.c
index e4fc90f88e243d584fa2c23c2625820f6164f5fe..55d102b479ac2f58daf8e0a4dd8f2867274ef3b0 100644 (file)
--- a/drivers/staging/media/atomisp/pci/isp/kernels/eed1_8/ia_css_eed1_8.host.c
+++ b/drivers/staging/media/atomisp/pci/isp/kernels/eed1_8/ia_css_eed1_8.host.c
@@ -276,7 +276,7 @@ ia_css_eed1_8_encode(
        for (i = 0; i < (IA_CSS_NUMBER_OF_DEW_ENHANCE_SEGMENTS - 1); i++) {
                min_exp = max(min_exp, from->dew_enhance_seg_exp[i]);
        }
-       to->e_dew_enh_asr = 13 - min(max(min_exp, 0), 13);
+       to->e_dew_enh_asr = 13 - clamp(min_exp, 0, 13);
 
        to->dedgew_max = from->dedgew_max;
 }