diff options
| author | Jorge Marques <jorge.marques@analog.com> | 2025-11-12 22:30:00 +0100 |
|---|---|---|
| committer | Alexandre Belloni <alexandre.belloni@bootlin.com> | 2025-11-29 00:25:40 +0100 |
| commit | 5b9481a4157198b93cd6a5e7ad2603682202149d (patch) | |
| tree | 2d17d6233ac42df658d0dcbb6c6f689f21061bdd | |
| parent | a54b1aeb61de63250ee608040a89bed8d1aa1e20 (diff) | |
i3c: master: Remove i3c_device_free_ibi from i3c_device_remove
i3c_device_disable_ibi should be called before i3c_device_free_ibi,
however, a driver using devm actions cannot yield the call before the
bus_type.remove(), requiring to use a .remove method that is usually
discouraged for drivers that uses resources already manage. Since the
only consumer mctp-i3c.c of this method calls both
i3c_device_disable_ibi then i3c_device_free_ibi, remove the call from
the i3c_device_remove (bus_type.remove()).
Signed-off-by: Jorge Marques <jorge.marques@analog.com>
Link: https://patch.msgid.link/20251112-ibi-unsafe-v1-1-d8454db22613@analog.com
Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
| -rw-r--r-- | drivers/i3c/master.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/i3c/master.c b/drivers/i3c/master.c index 66513a27e6e7..a0fe00e2487c 100644 --- a/drivers/i3c/master.c +++ b/drivers/i3c/master.c @@ -334,8 +334,6 @@ static void i3c_device_remove(struct device *dev) if (driver->remove) driver->remove(i3cdev); - - i3c_device_free_ibi(i3cdev); } const struct bus_type i3c_bus_type = { |