diff options
Diffstat (limited to 'meta-amd-bsp/recipes-kernel/linux/linux-yocto-4.14.71/2041-drm-amd-powerplay-Don-t-cast-kzalloc-return-value.patch')
-rw-r--r-- | meta-amd-bsp/recipes-kernel/linux/linux-yocto-4.14.71/2041-drm-amd-powerplay-Don-t-cast-kzalloc-return-value.patch | 109 |
1 files changed, 109 insertions, 0 deletions
diff --git a/meta-amd-bsp/recipes-kernel/linux/linux-yocto-4.14.71/2041-drm-amd-powerplay-Don-t-cast-kzalloc-return-value.patch b/meta-amd-bsp/recipes-kernel/linux/linux-yocto-4.14.71/2041-drm-amd-powerplay-Don-t-cast-kzalloc-return-value.patch new file mode 100644 index 00000000..3e0efc7b --- /dev/null +++ b/meta-amd-bsp/recipes-kernel/linux/linux-yocto-4.14.71/2041-drm-amd-powerplay-Don-t-cast-kzalloc-return-value.patch @@ -0,0 +1,109 @@ +From 5218b4f7edada9686af421ad1164207918dbe1bf Mon Sep 17 00:00:00 2001 +From: Georgiana Chelu <georgiana.chelu93@gmail.com> +Date: Tue, 17 Oct 2017 23:22:07 +0300 +Subject: [PATCH 2041/4131] drm/amd/powerplay: Don't cast kzalloc() return + value + +The kzalloc function returns a void pointer and the assignment +operator converts it to the type of pointer it is assigned to. +Therefore, there is no need to cast. + +Issue found by alloc_cast.cocci: +* WARNING: casting value returned by memory allocation function +to <struct type> is useless. + +Path to the cocci script: scripts/coccinelle/api/alloc/alloc_cast.cocci + +Signed-off-by: Georgiana Chelu <georgiana.chelu93@gmail.com> +Signed-off-by: Alex Deucher <alexander.deucher@amd.com> +--- + .../amd/powerplay/hwmgr/vega10_processpptables.c | 24 ++++++++-------------- + 1 file changed, 8 insertions(+), 16 deletions(-) + +diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_processpptables.c b/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_processpptables.c +index e343df1..d968c383 100644 +--- a/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_processpptables.c ++++ b/drivers/gpu/drm/amd/powerplay/hwmgr/vega10_processpptables.c +@@ -291,8 +291,7 @@ static int get_mm_clock_voltage_table( + table_size = sizeof(uint32_t) + + sizeof(phm_ppt_v1_mm_clock_voltage_dependency_record) * + mm_dependency_table->ucNumEntries; +- mm_table = (phm_ppt_v1_mm_clock_voltage_dependency_table *) +- kzalloc(table_size, GFP_KERNEL); ++ mm_table = kzalloc(table_size, GFP_KERNEL); + + if (!mm_table) + return -ENOMEM; +@@ -519,8 +518,7 @@ static int get_socclk_voltage_dependency_table( + sizeof(phm_ppt_v1_clock_voltage_dependency_record) * + clk_dep_table->ucNumEntries; + +- clk_table = (phm_ppt_v1_clock_voltage_dependency_table *) +- kzalloc(table_size, GFP_KERNEL); ++ clk_table = kzalloc(table_size, GFP_KERNEL); + + if (!clk_table) + return -ENOMEM; +@@ -554,8 +552,7 @@ static int get_mclk_voltage_dependency_table( + sizeof(phm_ppt_v1_clock_voltage_dependency_record) * + mclk_dep_table->ucNumEntries; + +- mclk_table = (phm_ppt_v1_clock_voltage_dependency_table *) +- kzalloc(table_size, GFP_KERNEL); ++ mclk_table = kzalloc(table_size, GFP_KERNEL); + + if (!mclk_table) + return -ENOMEM; +@@ -596,8 +593,7 @@ static int get_gfxclk_voltage_dependency_table( + sizeof(phm_ppt_v1_clock_voltage_dependency_record) * + clk_dep_table->ucNumEntries; + +- clk_table = (struct phm_ppt_v1_clock_voltage_dependency_table *) +- kzalloc(table_size, GFP_KERNEL); ++ clk_table = kzalloc(table_size, GFP_KERNEL); + + if (!clk_table) + return -ENOMEM; +@@ -663,8 +659,7 @@ static int get_pix_clk_voltage_dependency_table( + sizeof(phm_ppt_v1_clock_voltage_dependency_record) * + clk_dep_table->ucNumEntries; + +- clk_table = (struct phm_ppt_v1_clock_voltage_dependency_table *) +- kzalloc(table_size, GFP_KERNEL); ++ clk_table = kzalloc(table_size, GFP_KERNEL); + + if (!clk_table) + return -ENOMEM; +@@ -728,8 +723,7 @@ static int get_dcefclk_voltage_dependency_table( + sizeof(phm_ppt_v1_clock_voltage_dependency_record) * + num_entries; + +- clk_table = (struct phm_ppt_v1_clock_voltage_dependency_table *) +- kzalloc(table_size, GFP_KERNEL); ++ clk_table = kzalloc(table_size, GFP_KERNEL); + + if (!clk_table) + return -ENOMEM; +@@ -772,8 +766,7 @@ static int get_pcie_table(struct pp_hwmgr *hwmgr, + sizeof(struct phm_ppt_v1_pcie_record) * + atom_pcie_table->ucNumEntries; + +- pcie_table = (struct phm_ppt_v1_pcie_table *) +- kzalloc(table_size, GFP_KERNEL); ++ pcie_table = kzalloc(table_size, GFP_KERNEL); + + if (!pcie_table) + return -ENOMEM; +@@ -1026,8 +1019,7 @@ static int get_vddc_lookup_table( + table_size = sizeof(uint32_t) + + sizeof(phm_ppt_v1_voltage_lookup_record) * max_levels; + +- table = (phm_ppt_v1_voltage_lookup_table *) +- kzalloc(table_size, GFP_KERNEL); ++ table = kzalloc(table_size, GFP_KERNEL); + + if (NULL == table) + return -ENOMEM; +-- +2.7.4 + |