[PATCH 1/1] ASoC: Intel: Skylake: change list empty check to warning

[PATCH 1/1] ASoC: Intel: Skylake: change list empty check to warning

[Patrick Waterlander]

NULL is a valid state for the module list when querying unique ID.
Downgrade the empty condition from error to warning.

Signed-off-by: Patrick Waterlander <patrick.waterlander@intel.com>

diff --git a/sound/soc/intel/skylake/skl-sst.c b/sound/soc/intel/skylake/skl-sst.c
index 5951bbdf1f1a..95e6a23538cd 100644
--- a/sound/soc/intel/skylake/skl-sst.c
+++ b/sound/soc/intel/skylake/skl-sst.c
@@ -324,7 +324,7 @@ static struct skl_module_table *skl_module_get_from_id(
 	struct skl_module_table *module;
 
 	if (list_empty(&ctx->module_list)) {
-		dev_err(ctx->dev, "Module list is empty\n");
+		dev_warn(ctx->dev, "Module list is empty\n");
 		return NULL;
 	}
 
-- 
2.17.1

Re: [PATCH 1/1] ASoC: Intel: Skylake: change list empty check to warning

[Pierre-Louis Bossart]

On 4/25/19 4:57 PM, Patrick Waterlander wrote:
> NULL is a valid state for the module list when querying unique ID.
> Downgrade the empty condition from error to warning.
> 
> Signed-off-by: Patrick Waterlander <patrick.waterlander@intel.com>
> 
> diff --git a/sound/soc/intel/skylake/skl-sst.c b/sound/soc/intel/skylake/skl-sst.c
> index 5951bbdf1f1a..95e6a23538cd 100644
> --- a/sound/soc/intel/skylake/skl-sst.c
> +++ b/sound/soc/intel/skylake/skl-sst.c
> @@ -324,7 +324,7 @@ static struct skl_module_table *skl_module_get_from_id(
>   	struct skl_module_table *module;
>   
>   	if (list_empty(&ctx->module_list)) {
> -		dev_err(ctx->dev, "Module list is empty\n");
> +		dev_warn(ctx->dev, "Module list is empty\n");

Not sure what you are trying to fix here? Either it's an error and we 
can keep the message as is or it's not and that trace should go away. 
Changing the level doesn't really add much value, does it?

You should run this by Filip's team (CC:ed) and copy maintainers.

>   		return NULL;
>   	}
>   
>