ASoC: SDCA: Move most of the messages from info to debug

The SDCA code is very spammy on boot as it prints a lot of parsing
details using info prints. Now primary development is complete move
these to debug prints to reduce the spam.

Tested-by: Bard Liao <yung-chuan.liao@linux.intel.com>
Reviewed-by: Maciej Strozek <mstrozek@opensource.cirrus.com>
Reviewed-by: Peter Ujfalusi <peter.ujfalusi@linux.intel.com>
Tested-by: Richard Fitzgerald <rf@opensource.cirrus.com>
Signed-off-by: Charles Keepax <ckeepax@opensource.cirrus.com>
Link: https://patch.msgid.link/20251120153023.2105663-7-ckeepax@opensource.cirrus.com
Reviewed-by: Vinod Koul <vkoul@kernel.org>
Signed-off-by: Mark Brown <broonie@kernel.org>

diff --git a/sound/soc/sdca/sdca_functions.c b/sound/soc/sdca/sdca_functions.c
index 4cc8420af517a8ff42d22d41d81df946f0bbaea9..e4b3efeb30f0baa16137276e72e004aa5dc1c5d0 100644 (file)
--- a/sound/soc/sdca/sdca_functions.c
+++ b/sound/soc/sdca/sdca_functions.c
@@ -6,8 +6,6 @@
  * https://www.mipi.org/mipi-sdca-v1-0-download
  */
 
-#define dev_fmt(fmt) "%s: " fmt, __func__
-
 #include <linux/acpi.h>
 #include <linux/byteorder/generic.h>
 #include <linux/cleanup.h>
@@ -1013,10 +1011,10 @@ static int find_sdca_entity_control(struct device *dev, struct sdca_entity *enti
        control->type = find_sdca_control_datatype(entity, control);
        control->nbits = find_sdca_control_bits(entity, control);
 
-       dev_info(dev, "%s: %s: control %#x mode %#x layers %#x cn %#llx int %d %s\n",
-                entity->label, control->label, control->sel,
-                control->mode, control->layers, control->cn_list,
-                control->interrupt_position, control->deferrable ? "deferrable" : "");
+       dev_dbg(dev, "%s: %s: control %#x mode %#x layers %#x cn %#llx int %d %s\n",
+               entity->label, control->label, control->sel,
+               control->mode, control->layers, control->cn_list,
+               control->interrupt_position, control->deferrable ? "deferrable" : "");
 
        return 0;
 }
@@ -1137,9 +1135,9 @@ static int find_sdca_entity_iot(struct device *dev,
        if (!ret)
                terminal->num_transducer = tmp;
 
-       dev_info(dev, "%s: terminal type %#x ref %#x conn %#x count %d\n",
-                entity->label, terminal->type, terminal->reference,
-                terminal->connector, terminal->num_transducer);
+       dev_dbg(dev, "%s: terminal type %#x ref %#x conn %#x count %d\n",
+               entity->label, terminal->type, terminal->reference,
+               terminal->connector, terminal->num_transducer);
 
        return 0;
 }
@@ -1165,8 +1163,8 @@ static int find_sdca_entity_cs(struct device *dev,
        if (!ret)
                clock->max_delay = tmp;
 
-       dev_info(dev, "%s: clock type %#x delay %d\n", entity->label,
-                clock->type, clock->max_delay);
+       dev_dbg(dev, "%s: clock type %#x delay %d\n", entity->label,
+               clock->type, clock->max_delay);
 
        return 0;
 }
@@ -1217,8 +1215,8 @@ static int find_sdca_entity_pde(struct device *dev,
                delays[i].to_ps = delay_list[j++];
                delays[i].us = delay_list[j++];
 
-               dev_info(dev, "%s: from %#x to %#x delay %dus\n", entity->label,
-                        delays[i].from_ps, delays[i].to_ps, delays[i].us);
+               dev_dbg(dev, "%s: from %#x to %#x delay %dus\n", entity->label,
+                       delays[i].from_ps, delays[i].to_ps, delays[i].us);
        }
 
        power->num_max_delay = num_delays;
@@ -1447,8 +1445,8 @@ static int find_sdca_entity(struct device *dev, struct sdw_slave *sdw,
 
        entity->type = tmp;
 
-       dev_info(dev, "%s: entity %#x type %#x\n",
-                entity->label, entity->id, entity->type);
+       dev_dbg(dev, "%s: entity %#x type %#x\n",
+               entity->label, entity->id, entity->type);
 
        switch (entity->type) {
        case SDCA_ENTITY_TYPE_IT:
@@ -1623,7 +1621,7 @@ static int find_sdca_entity_connection_iot(struct device *dev,
 
        terminal->clock = clock_entity;
 
-       dev_info(dev, "%s -> %s\n", clock_entity->label, entity->label);
+       dev_dbg(dev, "%s -> %s\n", clock_entity->label, entity->label);
 
        fwnode_handle_put(clock_node);
        return 0;
@@ -1673,7 +1671,7 @@ static int find_sdca_entity_connection_pde(struct device *dev,
                        return -EINVAL;
                }
 
-               dev_info(dev, "%s -> %s\n", managed[i]->label, entity->label);
+               dev_dbg(dev, "%s -> %s\n", managed[i]->label, entity->label);
        }
 
        power->num_managed = num_managed;
@@ -1808,7 +1806,7 @@ static int find_sdca_entity_connection(struct device *dev,
 
                pins[i] = connected_entity;
 
-               dev_info(dev, "%s -> %s\n", connected_entity->label, entity->label);
+               dev_dbg(dev, "%s -> %s\n", connected_entity->label, entity->label);
 
                i++;
                fwnode_handle_put(connected_node);
@@ -1893,8 +1891,8 @@ static int find_sdca_cluster_channel(struct device *dev,
 
        channel->relationship = tmp;
 
-       dev_info(dev, "cluster %#x: channel id %#x purpose %#x relationship %#x\n",
-                cluster->id, channel->id, channel->purpose, channel->relationship);
+       dev_dbg(dev, "cluster %#x: channel id %#x purpose %#x relationship %#x\n",
+               cluster->id, channel->id, channel->purpose, channel->relationship);
 
        return 0;
 }
@@ -2065,7 +2063,7 @@ static int find_sdca_filesets(struct device *dev, struct sdw_slave *sdw,
                        return -EINVAL;
                }
 
-               dev_info(dev, "fileset: %#x\n", filesets_list[i]);
+               dev_dbg(dev, "fileset: %#x\n", filesets_list[i]);
 
                files = devm_kcalloc(dev, num_entries / mult_fileset,
                                     sizeof(struct sdca_fdl_file), GFP_KERNEL);
@@ -2086,8 +2084,8 @@ static int find_sdca_filesets(struct device *dev, struct sdw_slave *sdw,
                        file->file_id = fileset_entries[j++];
                        file->fdl_offset = fileset_entries[j++];
 
-                       dev_info(dev, "file: %#x, vendor: %#x, offset: %#x\n",
-                                file->file_id, file->vendor_id, file->fdl_offset);
+                       dev_dbg(dev, "file: %#x, vendor: %#x, offset: %#x\n",
+                               file->file_id, file->vendor_id, file->fdl_offset);
                }
 
                set->id = filesets_list[i];
@@ -2130,9 +2128,9 @@ int sdca_parse_function(struct device *dev, struct sdw_slave *sdw,
        if (!ret)
                function->reset_max_delay = tmp;
 
-       dev_info(dev, "%pfwP: name %s busy delay %dus reset delay %dus\n",
-                function->desc->node, function->desc->name,
-                function->busy_max_delay, function->reset_max_delay);
+       dev_dbg(dev, "%pfwP: name %s busy delay %dus reset delay %dus\n",
+               function->desc->node, function->desc->name,
+               function->busy_max_delay, function->reset_max_delay);
 
        ret = find_sdca_init_table(dev, function_desc->node, function);
        if (ret)