mirror of
				https://github.com/torvalds/linux.git
				synced 2025-11-04 10:40:15 +02:00 
			
		
		
		
	ASoC: topology: Do not ignore route checks when parsing graphs
One of the framework responsibilities is to ensure that the enumerated DPCMs are valid i.e.: a valid BE is connected to a valid FE DAI. While the are checks in soc-core.c and soc-pcm.c that verify this, a component driver may attempt to workaround this by loading an invalid graph through the topology file. Be strict and fail topology loading when invalid graph is encountered. Signed-off-by: Cezary Rojewski <cezary.rojewski@intel.com> Link: https://msgid.link/r/20240308090502.2136760-3-cezary.rojewski@intel.com Reviewed-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com> Signed-off-by: Mark Brown <broonie@kernel.org>
This commit is contained in:
		
							parent
							
								
									0cb3b7fd53
								
							
						
					
					
						commit
						6974857c2b
					
				
					 1 changed files with 9 additions and 2 deletions
				
			
		| 
						 | 
				
			
			@ -1083,8 +1083,15 @@ static int soc_tplg_dapm_graph_elems_load(struct soc_tplg *tplg,
 | 
			
		|||
			break;
 | 
			
		||||
		}
 | 
			
		||||
 | 
			
		||||
		/* add route, but keep going if some fail */
 | 
			
		||||
		snd_soc_dapm_add_routes(dapm, route, 1);
 | 
			
		||||
		ret = snd_soc_dapm_add_routes(dapm, route, 1);
 | 
			
		||||
		if (ret) {
 | 
			
		||||
			if (!dapm->card->disable_route_checks) {
 | 
			
		||||
				dev_err(tplg->dev, "ASoC: dapm_add_routes failed: %d\n", ret);
 | 
			
		||||
				break;
 | 
			
		||||
			}
 | 
			
		||||
			dev_info(tplg->dev,
 | 
			
		||||
				 "ASoC: disable_route_checks set, ignoring dapm_add_routes errors\n");
 | 
			
		||||
		}
 | 
			
		||||
	}
 | 
			
		||||
 | 
			
		||||
	return ret;
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in a new issue