linux-kernel - [PATCH] i3c: Correct the macro module_i3c_i2c

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-next>] [day] [month] [year] [list]

Message-ID: <20221205105413.937704-1-naveenkrishna.chatradhi@amd.com>
Date:   Mon, 5 Dec 2022 10:54:13 +0000
From:   Naveen Krishna Chatradhi <naveenkrishna.chatradhi@....com>
To:     <linux-i3c@...ts.infradead.org>, <alexandre.belloni@...tlin.com>
CC:     <linux-kernel@...r.kernel.org>, <boris.brezillon@...labora.com>,
        "Naveen Krishna Chatradhi" <nchatrad@....com>,
        Akshay Gupta <Akshay.Gupta@....com>
Subject: [PATCH] i3c: Correct the macro module_i3c_i2c_driver

From: Naveen Krishna Chatradhi <nchatrad@....com>

Present definition for module_i3c_i2c_driver uses only the
1st argument i.e., struct i3c_driver.
Irrespective of CONFIG_I3C being enabled/disabled,
struct i2c_driver is never passed to module_driver()

Passing struct i2c_driver as the 4th argument works.

Signed-off-by: Akshay Gupta <Akshay.Gupta@....com>
Signed-off-by: Naveen Krishna Chatradhi <nchatrad@....com>
---
 include/linux/i3c/device.h | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/include/linux/i3c/device.h b/include/linux/i3c/device.h
index 8242e13e7b0b..419192b5cc4d 100644
--- a/include/linux/i3c/device.h
+++ b/include/linux/i3c/device.h
@@ -287,7 +287,8 @@ static inline void i3c_i2c_driver_unregister(struct i3c_driver *i3cdrv,
 #define module_i3c_i2c_driver(__i3cdrv, __i2cdrv)	\
 	module_driver(__i3cdrv,				\
 		      i3c_i2c_driver_register,		\
-		      i3c_i2c_driver_unregister)
+		      i3c_i2c_driver_unregister,	\
+		      __i2cdrv)
 
 int i3c_device_do_priv_xfers(struct i3c_device *dev,
 			     struct i3c_priv_xfer *xfers,
-- 
2.25.1