Commit bdf2ffb6 authored by Deepak R Varma's avatar Deepak R Varma Committed by Hans de Goede
Browse files

platform/x86: dell-smo8800: Use min_t() for comparison and assignment 



Simplify code by using min_t helper macro for logical evaluation
and value assignment. Use the _t variant of min macro since the
variable types are not same.
This issue is identified by coccicheck using the minmax.cocci file.

Signed-off-by: default avatarDeepak R Varma <drv@mailo.com>
Acked-by: default avatarPali Rohár <pali@kernel.org>
Link: https://lore.kernel.org/r/Y9P8debIztOZXazW@ubun2204.myguest.virtualbox.org


Reviewed-by: default avatarHans de Goede <hdegoede@redhat.com>
Signed-off-by: default avatarHans de Goede <hdegoede@redhat.com>
parent cf2cc541

drivers/platform/x86/dell/dell-smo8800.c

+1 −4
Original line number Diff line number Diff line
@@ -67,10 +67,7 @@ static ssize_t smo8800_misc_read(struct file *file, char __user *buf, 


	retval = 1;



	if (data < 255)

		byte_data = data;

	else

		byte_data = 255;

	byte_data = min_t(u32, data, 255);



	if (put_user(byte_data, buf))

		retval = -EFAULT;