netdev - [PATCH *-next 17/18] dma-mapping: Remove needless return in five void APIs

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20250221-rmv_return-v1-17-cc8dff275827@quicinc.com>
Date: Fri, 21 Feb 2025 05:02:22 -0800
From: Zijun Hu <quic_zijuhu@...cinc.com>
To: Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Will Deacon
	<will@...nel.org>,
        Aneesh Kumar K.V <aneesh.kumar@...nel.org>,
        Andrew Morton
	<akpm@...ux-foundation.org>,
        Nick Piggin <npiggin@...il.com>,
        Peter Zijlstra
	<peterz@...radead.org>, Arnd Bergmann <arnd@...db.de>,
        Thomas Gleixner
	<tglx@...utronix.de>,
        Herbert Xu <herbert@...dor.apana.org.au>,
        "David S.
 Miller" <davem@...emloft.net>,
        Greg Kroah-Hartman
	<gregkh@...uxfoundation.org>,
        "Rafael J. Wysocki" <rafael@...nel.org>,
        "Danilo Krummrich" <dakr@...nel.org>,
        Eric Dumazet <edumazet@...gle.com>, "Jakub Kicinski" <kuba@...nel.org>,
        Paolo Abeni <pabeni@...hat.com>, Simon
 Horman <horms@...nel.org>,
        Johannes Berg <johannes@...solutions.net>,
        "Jamal
 Hadi Salim" <jhs@...atatu.com>,
        Cong Wang <xiyou.wangcong@...il.com>, Jiri
 Pirko <jiri@...nulli.us>,
        Jason Gunthorpe <jgg@...pe.ca>, Leon Romanovsky
	<leon@...nel.org>,
        Linus Walleij <linus.walleij@...aro.org>,
        "Bartosz
 Golaszewski" <brgl@...ev.pl>, Lee Jones <lee@...nel.org>,
        Thomas Graf
	<tgraf@...g.ch>, Christoph Hellwig <hch@....de>,
        Marek Szyprowski
	<m.szyprowski@...sung.com>,
        Robin Murphy <robin.murphy@....com>,
        "Miquel
 Raynal" <miquel.raynal@...tlin.com>,
        Richard Weinberger <richard@....at>,
        Vignesh Raghavendra <vigneshr@...com>
CC: Zijun Hu <zijun_hu@...oud.com>, <linux-arch@...r.kernel.org>,
        <linux-mm@...ck.org>, <linux-kernel@...r.kernel.org>,
        <linux-crypto@...r.kernel.org>, <netdev@...r.kernel.org>,
        <linux-wireless@...r.kernel.org>, <linux-rdma@...r.kernel.org>,
        <linux-gpio@...r.kernel.org>, <linux-pm@...r.kernel.org>,
        <iommu@...ts.linux.dev>, <linux-mtd@...ts.infradead.org>,
        Zijun Hu
	<quic_zijuhu@...cinc.com>
Subject: [PATCH *-next 17/18] dma-mapping: Remove needless return in five
 void APIs

Remove needless 'return' in the following void APIs:

 dma_unmap_single_attrs()
 dma_sync_single_range_for_cpu()
 dma_sync_single_range_for_device()
 dma_free_coherent()
 dma_free_wc()

Since both the API and callee involved are void functions.

Signed-off-by: Zijun Hu <quic_zijuhu@...cinc.com>
---
 include/linux/dma-mapping.h | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/include/linux/dma-mapping.h b/include/linux/dma-mapping.h
index b79925b1c433..a5de6ecaace3 100644
--- a/include/linux/dma-mapping.h
+++ b/include/linux/dma-mapping.h
@@ -388,21 +388,21 @@ static inline dma_addr_t dma_map_single_attrs(struct device *dev, void *ptr,
 static inline void dma_unmap_single_attrs(struct device *dev, dma_addr_t addr,
 		size_t size, enum dma_data_direction dir, unsigned long attrs)
 {
-	return dma_unmap_page_attrs(dev, addr, size, dir, attrs);
+	dma_unmap_page_attrs(dev, addr, size, dir, attrs);
 }
 
 static inline void dma_sync_single_range_for_cpu(struct device *dev,
 		dma_addr_t addr, unsigned long offset, size_t size,
 		enum dma_data_direction dir)
 {
-	return dma_sync_single_for_cpu(dev, addr + offset, size, dir);
+	dma_sync_single_for_cpu(dev, addr + offset, size, dir);
 }
 
 static inline void dma_sync_single_range_for_device(struct device *dev,
 		dma_addr_t addr, unsigned long offset, size_t size,
 		enum dma_data_direction dir)
 {
-	return dma_sync_single_for_device(dev, addr + offset, size, dir);
+	dma_sync_single_for_device(dev, addr + offset, size, dir);
 }
 
 /**
@@ -478,7 +478,7 @@ static inline void *dma_alloc_coherent(struct device *dev, size_t size,
 static inline void dma_free_coherent(struct device *dev, size_t size,
 		void *cpu_addr, dma_addr_t dma_handle)
 {
-	return dma_free_attrs(dev, size, cpu_addr, dma_handle, 0);
+	dma_free_attrs(dev, size, cpu_addr, dma_handle, 0);
 }
 
 
@@ -606,8 +606,8 @@ static inline void *dma_alloc_wc(struct device *dev, size_t size,
 static inline void dma_free_wc(struct device *dev, size_t size,
 			       void *cpu_addr, dma_addr_t dma_addr)
 {
-	return dma_free_attrs(dev, size, cpu_addr, dma_addr,
-			      DMA_ATTR_WRITE_COMBINE);
+	dma_free_attrs(dev, size, cpu_addr, dma_addr,
+		       DMA_ATTR_WRITE_COMBINE);
 }
 
 static inline int dma_mmap_wc(struct device *dev,

-- 
2.34.1