Skip to content

Commit 2c96956

Browse files
coleleavittvinodkoul
authored andcommitted
soundwire: bus: demote UNATTACHED state warnings to dev_dbg()
The dev_warn() messages in sdw_handle_slave_status() for UNATTACHED transitions were added in commit d1b3285 ("soundwire: bus: add dev_warn() messages to track UNATTACHED devices") to debug attachment failures with dynamic debug enabled. These warnings fire during normal operation -- for example when a codec driver triggers a hardware reset after firmware download, causing the device to momentarily go UNATTACHED before re-attaching -- producing misleading noise on every boot. Demote the messages to dev_dbg() so they remain available via dynamic debug for diagnosing real attachment failures without alarming users during expected initialization sequences. Fixes: d1b3285 ("soundwire: bus: add dev_warn() messages to track UNATTACHED devices") Signed-off-by: Cole Leavitt <cole@unwrap.rs> Reviewed-by: Richard Fitzgerald <rf@opensource.cirrus.com> Link: https://patch.msgid.link/20260218180210.9263-1-cole@unwrap.rs Signed-off-by: Vinod Koul <vkoul@kernel.org>
1 parent fee12f3 commit 2c96956

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

drivers/soundwire/bus.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1899,8 +1899,8 @@ int sdw_handle_slave_status(struct sdw_bus *bus,
18991899

19001900
if (status[i] == SDW_SLAVE_UNATTACHED &&
19011901
slave->status != SDW_SLAVE_UNATTACHED) {
1902-
dev_warn(&slave->dev, "Slave %d state check1: UNATTACHED, status was %d\n",
1903-
i, slave->status);
1902+
dev_dbg(&slave->dev, "Slave %d state check1: UNATTACHED, status was %d\n",
1903+
i, slave->status);
19041904
sdw_modify_slave_status(slave, SDW_SLAVE_UNATTACHED);
19051905

19061906
/* Ensure driver knows that peripheral unattached */
@@ -1951,8 +1951,8 @@ int sdw_handle_slave_status(struct sdw_bus *bus,
19511951
if (slave->status == SDW_SLAVE_UNATTACHED)
19521952
break;
19531953

1954-
dev_warn(&slave->dev, "Slave %d state check2: UNATTACHED, status was %d\n",
1955-
i, slave->status);
1954+
dev_dbg(&slave->dev, "Slave %d state check2: UNATTACHED, status was %d\n",
1955+
i, slave->status);
19561956

19571957
sdw_modify_slave_status(slave, SDW_SLAVE_UNATTACHED);
19581958
break;

0 commit comments

Comments
 (0)