The patch title prefix should say drm/nouveau.
Alex
On Fri, Aug 30, 2024 at 3:15 AM Li Zetao <lizetao1@xxxxxxxxxx> wrote:
>
> 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@xxxxxxxxxx>
> ---
> drivers/gpu/drm/nouveau/nvkm/subdev/volt/base.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/nouveau/nvkm/subdev/volt/base.c b/drivers/gpu/drm/nouveau/nvkm/subdev/volt/base.c
> index a17a6dd8d3de..803b98df4858 100644
> --- a/drivers/gpu/drm/nouveau/nvkm/subdev/volt/base.c
> +++ b/drivers/gpu/drm/nouveau/nvkm/subdev/volt/base.c
> @@ -142,7 +142,7 @@ nvkm_volt_map(struct nvkm_volt *volt, u8 id, u8 temp)
> return -ENODEV;
> }
>
> - result = min(max(result, (s64)info.min), (s64)info.max);
> + result = clamp(result, (s64)info.min, (s64)info.max);
>
> if (info.link != 0xff) {
> int ret = nvkm_volt_map(volt, info.link, temp);
> --
> 2.34.1
>
