[PATCH v2 04/22] firmware: ti_sci: Remove out of place RM debug messages

From: Beleswar Padhi

Date: Wed Sep 30 2026 - 12:21:27 EST


From: Andrew Davis <afd@xxxxxx>

These debug messages only print out the index param that was passed in
from the calling drivers. This does not add any additional info that
wouldn't work better in the calling drivers themselves (k3-ringacc.c and
k3-udma.c). These also make later cleanup around device fetching more
difficult, remove these debug prints.

Signed-off-by: Andrew Davis <afd@xxxxxx>
Co-developed-by: Beleswar Padhi <b-padhi@xxxxxx>
Signed-off-by: Beleswar Padhi <b-padhi@xxxxxx>
---
v2: Changelog:
1. None to this patch.

Link to v1:
https://lore.kernel.org/all/20260929201746.4078803-5-b-padhi@xxxxxx/

drivers/firmware/ti_sci.c | 14 ++++----------
1 file changed, 4 insertions(+), 10 deletions(-)

diff --git a/drivers/firmware/ti_sci.c b/drivers/firmware/ti_sci.c
index 6ec16fe57f132..a91db923c8939 100644
--- a/drivers/firmware/ti_sci.c
+++ b/drivers/firmware/ti_sci.c
@@ -2405,14 +2405,12 @@ static int ti_sci_cmd_rm_ring_cfg(const struct ti_sci_handle *handle,
struct ti_sci_msg_hdr *resp;
struct ti_sci_xfer *xfer;
struct ti_sci_info *info;
- struct device *dev;
int ret = 0;

if (IS_ERR_OR_NULL(handle))
return -EINVAL;

info = handle_to_ti_sci_info(handle);
- dev = info->dev;

xfer = ti_sci_get_one_xfer(info, TI_SCI_MSG_RM_RING_CFG,
TI_SCI_FLAG_REQ_ACK_ON_PROCESSED,
@@ -2439,7 +2437,7 @@ static int ti_sci_cmd_rm_ring_cfg(const struct ti_sci_handle *handle,

fail:
ti_sci_put_one_xfer(&info->minfo, xfer);
- dev_dbg(dev, "RM_RA:config ring %u ret:%d\n", params->index, ret);
+
return ret;
}

@@ -2555,14 +2553,12 @@ static int ti_sci_cmd_rm_udmap_tx_ch_cfg(const struct ti_sci_handle *handle,
struct ti_sci_msg_hdr *resp;
struct ti_sci_xfer *xfer;
struct ti_sci_info *info;
- struct device *dev;
int ret = 0;

if (IS_ERR_OR_NULL(handle))
return -EINVAL;

info = handle_to_ti_sci_info(handle);
- dev = info->dev;

xfer = ti_sci_get_one_xfer(info, TISCI_MSG_RM_UDMAP_TX_CH_CFG,
TI_SCI_FLAG_REQ_ACK_ON_PROCESSED,
@@ -2598,7 +2594,7 @@ static int ti_sci_cmd_rm_udmap_tx_ch_cfg(const struct ti_sci_handle *handle,

fail:
ti_sci_put_one_xfer(&info->minfo, xfer);
- dev_dbg(dev, "TX_CH_CFG: chn %u ret:%u\n", params->index, ret);
+
return ret;
}

@@ -2620,14 +2616,12 @@ static int ti_sci_cmd_rm_udmap_rx_ch_cfg(const struct ti_sci_handle *handle,
struct ti_sci_msg_hdr *resp;
struct ti_sci_xfer *xfer;
struct ti_sci_info *info;
- struct device *dev;
int ret = 0;

if (IS_ERR_OR_NULL(handle))
return -EINVAL;

info = handle_to_ti_sci_info(handle);
- dev = info->dev;

xfer = ti_sci_get_one_xfer(info, TISCI_MSG_RM_UDMAP_RX_CH_CFG,
TI_SCI_FLAG_REQ_ACK_ON_PROCESSED,
@@ -2660,7 +2654,7 @@ static int ti_sci_cmd_rm_udmap_rx_ch_cfg(const struct ti_sci_handle *handle,

fail:
ti_sci_put_one_xfer(&info->minfo, xfer);
- dev_dbg(dev, "RX_CH_CFG: chn %u ret:%d\n", params->index, ret);
+
return ret;
}

@@ -2725,7 +2719,7 @@ static int ti_sci_cmd_rm_udmap_rx_flow_cfg(const struct ti_sci_handle *handle,

fail:
ti_sci_put_one_xfer(&info->minfo, xfer);
- dev_dbg(info->dev, "RX_FL_CFG: %u ret:%d\n", params->flow_index, ret);
+
return ret;
}

--
2.34.1