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. Signed-off-by: Simon Glass <s...@chromium.org> Reported-by: Coverity (CID: 312951) --- Changes in v2: - Add a standard comment instead of a Coverity annotation drivers/core/regmap.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/core/regmap.c b/drivers/core/regmap.c index b51ce108c14..982c2ee9fab 100644 --- a/drivers/core/regmap.c +++ b/drivers/core/regmap.c @@ -293,6 +293,7 @@ struct regmap *devm_regmap_init(struct udevice *dev, int rc; struct regmap **mapp, *map; + /* this looks like a leak, but devres takes care of it */ mapp = devres_alloc(devm_regmap_release, sizeof(struct regmap *), __GFP_ZERO); if (unlikely(!mapp)) -- 2.31.1.751.gd2f1c929bd-goog