random-mirrors/smaeul-u-boot
1
0
Fork 0
mirror of https://github.com/smaeul/u-boot.git synced 2025-11-03 21:48:15 +00:00
Code Issues Packages Projects Releases Wiki Activity

reset: Avoid a warning in devm_reset_bulk_get_by_node()

Browse Source 
The devres_alloc() function is intended to avoid the need for freeing
memory, although in practice it may not be enabled, thus leading to a true
leak.

Nevertheless this is intended. Add a comment to explain this.

Signed-off-by: Simon Glass <sjg@chromium.org>
Reported-by: Coverity (CID: 312952)
This commit is contained in:
Simon Glass 2021-05-13 19:39:21 -06:00 committed by Tom Rini
parent 7f0f4e1825
commit ca4c24509c
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
drivers/reset/reset-uclass.c
View File

@ -325,6 +325,8 @@ struct reset_ctl_bulk *devm_reset_bulk_get_by_node(struct udevice *dev,
bulk = devres_alloc(devm_reset_bulk_release, bulk = devres_alloc(devm_reset_bulk_release,
sizeof(struct reset_ctl_bulk), sizeof(struct reset_ctl_bulk),
__GFP_ZERO); __GFP_ZERO);
/* this looks like a leak, but devres takes care of it */
if (unlikely(!bulk)) if (unlikely(!bulk))
return ERR_PTR(-ENOMEM); return ERR_PTR(-ENOMEM);