diff options
| author | Tiezhu Yang | 2020-07-07 10:12:45 +0800 |
|---|---|---|
| committer | Marc Zyngier | 2020-07-17 13:41:42 +0100 |
| commit | f90fafecf4880b9785da85feb9b3e6d85fbf2287 (patch) | |
| tree | 7c4f85350e577c82e282eebd75717b563048c37f /drivers/irqchip/irq-loongson-htpic.c | |
| parent | 52b350cbc94721d87f087d1a5800f9531c2d682c (diff) | |
irqchip/loongson-htpic: Remove redundant kfree operation
In the function htpic_of_init(), when kzalloc htpic fails, it should
return -ENOMEM directly, no need to execute "goto" to kfree.
Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
Signed-off-by: Marc Zyngier <maz@kernel.org>
Link: https://lore.kernel.org/r/1594087972-21715-2-git-send-email-yangtiezhu@loongson.cn
Diffstat (limited to 'drivers/irqchip/irq-loongson-htpic.c')
| -rw-r--r-- | drivers/irqchip/irq-loongson-htpic.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/irqchip/irq-loongson-htpic.c b/drivers/irqchip/irq-loongson-htpic.c index dd018c22ea83..63f72803c8c4 100644 --- a/drivers/irqchip/irq-loongson-htpic.c +++ b/drivers/irqchip/irq-loongson-htpic.c @@ -93,10 +93,8 @@ int __init htpic_of_init(struct device_node *node, struct device_node *parent) } htpic = kzalloc(sizeof(*htpic), GFP_KERNEL); - if (!htpic) { - err = -ENOMEM; - goto out_free; - } + if (!htpic) + return -ENOMEM; htpic->base = of_iomap(node, 0); if (!htpic->base) { |