Patch 1 is just a constification path. It should be straighforward. Patch 2 is COMPLETELY SPECULATIVE. Some code and memory allocation *LOOK* useless. So review with care! Looking are lore does not give any information about this code to me. v1: https://lore.kernel.org/all/20200413164440.1138178-1-iskren.chernev@xxxxxxxxx/ v2: https://lore.kernel.org/all/20200414172250.2363235-1-iskren.chernev@xxxxxxxxx/ It *looks* like code written to release some resources in a .remove() function, maybe used before switchwing to devm_ function. Christophe JAILLET (2): regulator: max77826: Constify struct regulator_desc regulator: max77826: Simplify max77826_i2c_probe() drivers/regulator/max77826-regulator.c | 20 +------------------- 1 file changed, 1 insertion(+), 19 deletions(-) -- 2.46.0
