[openwrt/openwrt] ramips: 5.15: add patch fixing compilation warning for hsdma-mt7621

LEDE Commits lede-commits at lists.infradead.org
Thu May 11 18:25:00 PDT 2023


ansuel pushed a commit to openwrt/openwrt.git, branch master:
https://git.openwrt.org/da2f5e8776da6ca7ca5ae9646243dc0f04ce24b9

commit da2f5e8776da6ca7ca5ae9646243dc0f04ce24b9
Author: Christian Marangi <ansuelsmth at gmail.com>
AuthorDate: Sun May 7 01:13:45 2023 +0200

    ramips: 5.15: add patch fixing compilation warning for hsdma-mt7621
    
    Add patch fixing compilation warning for hsdma-mt7621:
    drivers/staging/mt7621-dma/hsdma-mt7621.c: In function 'mtk_hsdma_probe':
    drivers/staging/mt7621-dma/hsdma-mt7621.c:685:9: error: ignoring return value of 'device_reset' declared with attribute 'warn_unused_result' [-Werror=unused-result]
      685 |         device_reset(&pdev->dev);
          |         ^~~~~~~~~~~~~~~~~~~~~~~~
    cc1: all warnings being treated as errors
    
    Signed-off-by: Christian Marangi <ansuelsmth at gmail.com>
---
 ...mt7621-dma-handle-error-from-device_reset.patch | 31 ++++++++++++++++++++++
 1 file changed, 31 insertions(+)

diff --git a/target/linux/ramips/patches-5.15/109-drivers-mt7621-dma-handle-error-from-device_reset.patch b/target/linux/ramips/patches-5.15/109-drivers-mt7621-dma-handle-error-from-device_reset.patch
new file mode 100644
index 0000000000..a4543337ff
--- /dev/null
+++ b/target/linux/ramips/patches-5.15/109-drivers-mt7621-dma-handle-error-from-device_reset.patch
@@ -0,0 +1,31 @@
+From 3f6dfa25128e428acfba20792bc7506d58806baa Mon Sep 17 00:00:00 2001
+From: Christian Marangi <ansuelsmth at gmail.com>
+Date: Sun, 7 May 2023 01:06:17 +0200
+Subject: [PATCH] drivers: mt7621-dma: handle error from device_reset
+
+Handle error from device reset to fix compilation warning.
+Fix compilation warning:
+drivers/staging/mt7621-dma/hsdma-mt7621.c: In function 'mtk_hsdma_probe':
+drivers/staging/mt7621-dma/hsdma-mt7621.c:685:9: error: ignoring return value of 'device_reset' declared with attribute 'warn_unused_result' [-Werror=unused-result]
+  685 |         device_reset(&pdev->dev);
+      |         ^~~~~~~~~~~~~~~~~~~~~~~~
+cc1: all warnings being treated as errors
+
+Signed-off-by: Christian Marangi <ansuelsmth at gmail.com>
+---
+ drivers/staging/mt7621-dma/hsdma-mt7621.c | 4 +++-
+ 1 file changed, 3 insertions(+), 1 deletion(-)
+
+--- a/drivers/staging/mt7621-dma/hsdma-mt7621.c
++++ b/drivers/staging/mt7621-dma/hsdma-mt7621.c
+@@ -682,7 +682,9 @@ static int mtk_hsdma_probe(struct platfo
+ 		return ret;
+ 	}
+ 
+-	device_reset(&pdev->dev);
++	ret = device_reset(&pdev->dev);
++	if (ret)
++		dev_err(&pdev->dev, "failed to reset device\n");
+ 
+ 	dd = &hsdma->ddev;
+ 	dma_cap_set(DMA_MEMCPY, dd->cap_mask);




More information about the lede-commits mailing list