github-mirror/linux
1
0
Fork 0
mirror of https://github.com/torvalds/linux.git synced 2026-06-04 04:23:35 +02:00
Code Issues Actions Packages Projects Releases Wiki Activity Graph

ASoC: SDCA: functions: Fix confusing cleanup.h syntax

Browse Source 
Initializing automatic __free variables to NULL without need (e.g.
branches with different allocations), followed by actual allocation is
in contrary to explicit coding rules guiding cleanup.h:

"Given that the "__free(...) = NULL" pattern for variables defined at
the top of the function poses this potential interdependency problem the
recommendation is to always define and assign variables in one statement
and not group variable definitions at the top of the function when
__free() is used."

Code does not have a bug, but is less readable and uses discouraged
coding practice, so fix that by moving declaration to the place of
assignment.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
Reviewed-by: Charles Keepax <ckeepax@opensource.cirrus.com>
Link: https://patch.msgid.link/20251203-asoc-wrong-cleanup-h-continued-v1-3-5142be4874fb@oss.qualcomm.com
Signed-off-by: Mark Brown <broonie@kernel.org>
This commit is contained in:
Krzysztof Kozlowski 2025-12-03 17:12:40 +01:00 committed by Mark Brown
parent bafd5cf04b
commit e39011184f
No known key found for this signature in database
GPG Key ID: 24D68B725D5487D0
1 changed files with 14 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
sound/soc/sdca/sdca_functions.c
View File

@ -1184,7 +1184,6 @@ static int find_sdca_entity_pde(struct device *dev,
{
static const int mult_delay = 3;
struct sdca_entity_pde *power = &entity->pde;
u32 *delay_list __free(kfree) = NULL;
struct sdca_pde_delay *delays;
int num_delays;
int i, j;
@ -1205,7 +1204,8 @@ static int find_sdca_entity_pde(struct device *dev,
return -EINVAL;
}
delay_list = kcalloc(num_delays, sizeof(*delay_list), GFP_KERNEL);
u32 *delay_list __free(kfree) = kcalloc(num_delays, sizeof(*delay_list),
GFP_KERNEL);
if (!delay_list)
return -ENOMEM;
@ -1250,7 +1250,6 @@ static int find_sdca_entity_ge(struct device *dev,
struct sdca_entity *entity)
{
struct sdca_entity_ge *group = &entity->ge;
u8 *affected_list __free(kfree) = NULL;
u8 *affected_iter;
int num_affected;
int i, j;
@ -1269,7 +1268,8 @@ static int find_sdca_entity_ge(struct device *dev,
return -EINVAL;
}
affected_list = kcalloc(num_affected, sizeof(*affected_list), GFP_KERNEL);
u8 *affected_list __free(kfree) = kcalloc(num_affected, sizeof(*affected_list),
GFP_KERNEL);
if (!affected_list)
return -ENOMEM;
@ -1495,7 +1495,6 @@ static int find_sdca_entities(struct device *dev, struct sdw_slave *sdw,
struct fwnode_handle *function_node,
struct sdca_function_data *function)
{
u32 *entity_list __free(kfree) = NULL;
struct sdca_entity *entities;
int num_entities;
int i, ret;
@ -1517,7 +1516,8 @@ static int find_sdca_entities(struct device *dev, struct sdw_slave *sdw,
if (!entities)
return -ENOMEM;
entity_list = kcalloc(num_entities, sizeof(*entity_list), GFP_KERNEL);
u32 *entity_list __free(kfree) = kcalloc(num_entities, sizeof(*entity_list),
GFP_KERNEL);
if (!entity_list)
return -ENOMEM;
@ -1642,7 +1642,6 @@ static int find_sdca_entity_connection_pde(struct device *dev,
struct sdca_entity *entity)
{
struct sdca_entity_pde *power = &entity->pde;
u32 *managed_list __free(kfree) = NULL;
struct sdca_entity **managed;
int num_managed;
int i;
@ -1664,7 +1663,8 @@ static int find_sdca_entity_connection_pde(struct device *dev,
if (!managed)
return -ENOMEM;
managed_list = kcalloc(num_managed, sizeof(*managed_list), GFP_KERNEL);
u32 *managed_list __free(kfree) = kcalloc(num_managed, sizeof(*managed_list),
GFP_KERNEL);
if (!managed_list)
return -ENOMEM;
@ -1961,7 +1961,6 @@ static int find_sdca_clusters(struct device *dev,
struct fwnode_handle *function_node,
struct sdca_function_data *function)
{
u32 *cluster_list __free(kfree) = NULL;
struct sdca_cluster *clusters;
int num_clusters;
int i, ret;
@ -1982,7 +1981,8 @@ static int find_sdca_clusters(struct device *dev,
if (!clusters)
return -ENOMEM;
cluster_list = kcalloc(num_clusters, sizeof(*cluster_list), GFP_KERNEL);
u32 *cluster_list __free(kfree) = kcalloc(num_clusters, sizeof(*cluster_list),
GFP_KERNEL);
if (!cluster_list)
return -ENOMEM;
@ -2026,7 +2026,6 @@ static int find_sdca_filesets(struct device *dev, struct sdw_slave *sdw,
{
static const int mult_fileset = 3;
char fileset_name[SDCA_PROPERTY_LENGTH];
u32 *filesets_list __free(kfree) = NULL;
struct sdca_fdl_set *sets;
int num_sets;
int i, j;
@ -2041,7 +2040,8 @@ static int find_sdca_filesets(struct device *dev, struct sdw_slave *sdw,
return num_sets;
}
filesets_list = kcalloc(num_sets, sizeof(u32), GFP_KERNEL);
u32 *filesets_list __free(kfree) = kcalloc(num_sets, sizeof(u32),
GFP_KERNEL);
if (!filesets_list)
return -ENOMEM;
@ -2053,7 +2053,6 @@ static int find_sdca_filesets(struct device *dev, struct sdw_slave *sdw,
return -ENOMEM;
for (i = 0; i < num_sets; i++) {
u32 *fileset_entries __free(kfree) = NULL;
struct sdca_fdl_set *set = &sets[i];
struct sdca_fdl_file *files;
int num_files, num_entries;
@ -2079,7 +2078,8 @@ static int find_sdca_filesets(struct device *dev, struct sdw_slave *sdw,
if (!files)
return -ENOMEM;
fileset_entries = kcalloc(num_entries, sizeof(u32), GFP_KERNEL);
u32 *fileset_entries __free(kfree) = kcalloc(num_entries, sizeof(u32),
GFP_KERNEL);
if (!fileset_entries)
return -ENOMEM;