diff mbox series

[v4] dma-buf: heap: Replace nested max() with single max3()

Message ID 20250317103702708UdayAw742BADL4gzNYcle@zte.com.cn (mailing list archive)
State New
Headers show
Series [v4] dma-buf: heap: Replace nested max() with single max3() | expand

Commit Message

feng.wei8@zte.com.cn March 17, 2025, 2:37 a.m. UTC 
From: FengWei <feng.wei8@zte.com.cn>

Use max3() macro instead of nesting max() to simplify the return
statement.

Signed-off-by: FengWei <feng.wei8@zte.com.cn>
---
v3 -> v4
fix the format of this patch.
 drivers/dma-buf/dma-heap.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Krzysztof Kozlowski March 17, 2025, 8:24 a.m. UTC | #1 
On 17/03/2025 03:37, feng.wei8@zte.com.cn wrote:
> From: FengWei <feng.wei8@zte.com.cn>
> 
> Use max3() macro instead of nesting max() to simplify the return
> statement.
> 
> Signed-off-by: FengWei <feng.wei8@zte.com.cn>
> ---
> v3 -> v4
> fix the format of this patch.
>  drivers/dma-buf/dma-heap.c | 2 +-
You sent five versions per day of the same patch.

Look what was in v3:

	v2 -> v3
	fix the format of this patch

So you are doing the same over and over and sending it to us?

Srsly, ZTE, slow down and be sure you follow the process BEFORE you send
flood of patches like that and learn on the go.

Best regards,
Krzysztof
diff mbox series

Patch

diff --git a/drivers/dma-buf/dma-heap.c b/drivers/dma-buf/dma-heap.c
index 3cbe87d4a464..96cb9ab5731a 100644
--- a/drivers/dma-buf/dma-heap.c
+++ b/drivers/dma-buf/dma-heap.c
@@ -147,7 +147,7 @@  static long dma_heap_ioctl(struct file *file, unsigned int ucmd,
 		in_size = 0;
 	if ((ucmd & kcmd & IOC_OUT) == 0)
 		out_size = 0;
-	ksize = max(max(in_size, out_size), drv_size);
+	ksize = max3(in_size, out_size, drv_size);

 	/* If necessary, allocate buffer for ioctl argument */
 	if (ksize > sizeof(stack_kdata)) {