Proxmox-Port/linux-loongson
1
0
Fork 0
mirror of https://git.kernel.org/pub/scm/linux/kernel/git/chenhuacai/linux-loongson synced 2025-08-30 21:52:21 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

staging: fsl-mc: simplify couple of deallocations

Browse Source 
Simplify a couple of deallocations code paths. This also fixes these
checkpatch.pl false positives:
 "WARNING: kfree(NULL) is safe and this check is probably not required"

Signed-off-by: Laurentiu Tudor <laurentiu.tudor@nxp.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
Laurentiu Tudor 2017-06-08 17:28:48 +03:00 committed by Greg Kroah-Hartman
parent 72f3415f81
commit a042fbed02
1 changed files with 3 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
drivers/staging/fsl-mc/bus/fsl-mc-bus.c
View File

@ -420,15 +420,11 @@ bool fsl_mc_is_root_dprc(struct device *dev)
static void fsl_mc_device_release(struct device *dev) static void fsl_mc_device_release(struct device *dev)
{ {
struct fsl_mc_device *mc_dev = to_fsl_mc_device(dev); struct fsl_mc_device *mc_dev = to_fsl_mc_device(dev);
struct fsl_mc_bus *mc_bus = NULL;
kfree(mc_dev->regions); kfree(mc_dev->regions);
if (strcmp(mc_dev->obj_desc.type, "dprc") == 0) if (strcmp(mc_dev->obj_desc.type, "dprc") == 0)
mc_bus = to_fsl_mc_bus(mc_dev); kfree(to_fsl_mc_bus(mc_dev));
if (mc_bus)
kfree(mc_bus);
else else
kfree(mc_dev); kfree(mc_dev);
} }
@ -559,10 +555,8 @@ int fsl_mc_device_add(struct dprc_obj_desc *obj_desc,
error_cleanup_dev: error_cleanup_dev:
kfree(mc_dev->regions); kfree(mc_dev->regions);
if (mc_bus) kfree(mc_bus);
kfree(mc_bus); kfree(mc_dev);
else
kfree(mc_dev);
return error; return error;
} }