From: Adrian Hunter <adrian.hunter@intel.com>
To: alexandre.belloni@bootlin.com
Cc: Frank.Li@nxp.com, Wolfram Sang <wsa+renesas@sang-engineering.com>,
Aniket <aniketmaurya@google.com>,
linux-i3c@lists.infradead.org
Subject: [PATCH 05/17] i3c: mipi-i3c-hci: Switch DAT bitmap allocation to devm_bitmap_zalloc()
Date: Fri, 19 Dec 2025 16:45:22 +0200 [thread overview]
Message-ID: <20251219144534.84391-6-adrian.hunter@intel.com> (raw)
In-Reply-To: <20251219144534.84391-1-adrian.hunter@intel.com>
The driver already uses managed resources, so convert the Device Address
Table (DAT) bitmap allocation to use devm_bitmap_zalloc(). Remove the
manual cleanup routine.
Signed-off-by: Adrian Hunter <adrian.hunter@intel.com>
---
drivers/i3c/master/mipi-i3c-hci/core.c | 2 --
drivers/i3c/master/mipi-i3c-hci/dat.h | 1 -
drivers/i3c/master/mipi-i3c-hci/dat_v1.c | 11 +++--------
3 files changed, 3 insertions(+), 11 deletions(-)
diff --git a/drivers/i3c/master/mipi-i3c-hci/core.c b/drivers/i3c/master/mipi-i3c-hci/core.c
index 0d3ec674878d..c4b249fde764 100644
--- a/drivers/i3c/master/mipi-i3c-hci/core.c
+++ b/drivers/i3c/master/mipi-i3c-hci/core.c
@@ -185,8 +185,6 @@ static void i3c_hci_bus_cleanup(struct i3c_master_controller *m)
i3c_hci_bus_disable(hci);
hci->io->cleanup(hci);
- if (hci->cmd == &mipi_i3c_hci_cmd_v1)
- mipi_i3c_hci_dat_v1.cleanup(hci);
}
void mipi_i3c_hci_resume(struct i3c_hci *hci)
diff --git a/drivers/i3c/master/mipi-i3c-hci/dat.h b/drivers/i3c/master/mipi-i3c-hci/dat.h
index 1f0f345c3daf..5277c65fc601 100644
--- a/drivers/i3c/master/mipi-i3c-hci/dat.h
+++ b/drivers/i3c/master/mipi-i3c-hci/dat.h
@@ -17,7 +17,6 @@
struct hci_dat_ops {
int (*init)(struct i3c_hci *hci);
- void (*cleanup)(struct i3c_hci *hci);
int (*alloc_entry)(struct i3c_hci *hci);
void (*free_entry)(struct i3c_hci *hci, unsigned int dat_idx);
void (*set_dynamic_addr)(struct i3c_hci *hci, unsigned int dat_idx, u8 addr);
diff --git a/drivers/i3c/master/mipi-i3c-hci/dat_v1.c b/drivers/i3c/master/mipi-i3c-hci/dat_v1.c
index cc5d2deb23ab..c60ef5d77ca3 100644
--- a/drivers/i3c/master/mipi-i3c-hci/dat_v1.c
+++ b/drivers/i3c/master/mipi-i3c-hci/dat_v1.c
@@ -55,8 +55,10 @@ static int hci_dat_v1_init(struct i3c_hci *hci)
}
if (!hci->DAT_data) {
+ struct device *dev = hci->master.dev.parent;
+
/* use a bitmap for faster free slot search */
- hci->DAT_data = bitmap_zalloc(hci->DAT_entries, GFP_KERNEL);
+ hci->DAT_data = devm_bitmap_zalloc(dev, hci->DAT_entries, GFP_KERNEL);
if (!hci->DAT_data)
return -ENOMEM;
@@ -70,12 +72,6 @@ static int hci_dat_v1_init(struct i3c_hci *hci)
return 0;
}
-static void hci_dat_v1_cleanup(struct i3c_hci *hci)
-{
- bitmap_free(hci->DAT_data);
- hci->DAT_data = NULL;
-}
-
static int hci_dat_v1_alloc_entry(struct i3c_hci *hci)
{
unsigned int dat_idx;
@@ -170,7 +166,6 @@ static int hci_dat_v1_get_index(struct i3c_hci *hci, u8 dev_addr)
const struct hci_dat_ops mipi_i3c_hci_dat_v1 = {
.init = hci_dat_v1_init,
- .cleanup = hci_dat_v1_cleanup,
.alloc_entry = hci_dat_v1_alloc_entry,
.free_entry = hci_dat_v1_free_entry,
.set_dynamic_addr = hci_dat_v1_set_dynamic_addr,
--
2.51.0
--
linux-i3c mailing list
linux-i3c@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-i3c
next prev parent reply other threads:[~2025-12-19 14:46 UTC|newest]
Thread overview: 39+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-12-19 14:45 [PATCH 00/17] i3c: mipi-i3c-hci-pci: Add Runtime PM support Adrian Hunter
2025-12-19 14:45 ` [PATCH 01/17] i3c: mipi-i3c-hci: Reset RING_OPERATION1 fields during init Adrian Hunter
2025-12-19 16:12 ` Frank Li
2025-12-19 14:45 ` [PATCH 02/17] i3c: mipi-i3c-hci: Ensure proper bus clean-up Adrian Hunter
2025-12-19 16:22 ` Frank Li
2025-12-19 14:45 ` [PATCH 03/17] i3c: master: Update hot-join flag only on success Adrian Hunter
2025-12-19 16:23 ` Frank Li
2025-12-19 14:45 ` [PATCH 04/17] i3c: master: Replace WARN_ON() with dev_err() in i3c_dev_free_ibi_locked() Adrian Hunter
2025-12-19 16:25 ` Frank Li
2025-12-19 14:45 ` Adrian Hunter [this message]
2025-12-19 16:27 ` [PATCH 05/17] i3c: mipi-i3c-hci: Switch DAT bitmap allocation to devm_bitmap_zalloc() Frank Li
2025-12-19 14:45 ` [PATCH 06/17] i3c: mipi-i3c-hci: Switch PIO data allocation to devm_kzalloc() Adrian Hunter
2025-12-19 16:28 ` Frank Li
2025-12-19 14:45 ` [PATCH 07/17] i3c: mipi-i3c-hci: Manage DMA deallocation via devres action Adrian Hunter
2025-12-19 16:32 ` Frank Li
2026-01-08 8:04 ` Adrian Hunter
2025-12-19 14:45 ` [PATCH 08/17] i3c: mipi-i3c-hci: Cache DAT in memory for Runtime PM restore Adrian Hunter
2025-12-19 16:36 ` Frank Li
2025-12-19 14:45 ` [PATCH 09/17] i3c: mipi-i3c-hci: Introduce helper to restore DAT Adrian Hunter
2025-12-19 16:37 ` Frank Li
2025-12-19 14:45 ` [PATCH 10/17] i3c: mipi-i3c-hci: Extract ring initialization from hci_dma_init() Adrian Hunter
2025-12-19 16:41 ` Frank Li
2025-12-19 14:45 ` [PATCH 11/17] i3c: mipi-i3c-hci: Add DMA suspend and resume support Adrian Hunter
2025-12-19 16:42 ` Frank Li
2025-12-19 14:45 ` [PATCH 12/17] i3c: mipi-i3c-hci: Refactor PIO register initialization Adrian Hunter
2025-12-19 16:46 ` Frank Li
2025-12-19 14:45 ` [PATCH 13/17] i3c: mipi-i3c-hci: Add PIO suspend and resume support Adrian Hunter
2025-12-19 16:46 ` Frank Li
2025-12-19 14:45 ` [PATCH 14/17] i3c: mipi-i3c-hci: Factor out software reset into helper Adrian Hunter
2025-12-19 16:49 ` Frank Li
2025-12-19 14:45 ` [PATCH 15/17] i3c: master: Introduce optional Runtime PM support Adrian Hunter
2025-12-19 17:08 ` Frank Li
2025-12-19 20:11 ` Frank Li
2025-12-20 14:50 ` Adrian Hunter
2025-12-19 14:45 ` [PATCH 16/17] i3c: mipi-i3c-hci: Add " Adrian Hunter
2025-12-19 17:16 ` Frank Li
2025-12-19 14:45 ` [PATCH 17/17] i3c: mipi-i3c-hci-pci: Add " Adrian Hunter
2025-12-19 17:21 ` Frank Li
2025-12-20 15:10 ` Adrian Hunter
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20251219144534.84391-6-adrian.hunter@intel.com \
--to=adrian.hunter@intel.com \
--cc=Frank.Li@nxp.com \
--cc=alexandre.belloni@bootlin.com \
--cc=aniketmaurya@google.com \
--cc=linux-i3c@lists.infradead.org \
--cc=wsa+renesas@sang-engineering.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.