On 15/08/2024 09:15, Chen-Yu Tsai wrote: >> >> tt = kzalloc(sizeof(*tt) * count, GFP_KERNEL); >> - if (!tt) { >> - ret = -ENOMEM; >> - goto out_of_node_put; >> - } >> + if (!tt) >> + return ERR_PTR(-ENOMEM); >> >> *ntrips = count; > > Also drop the "of_node_put(trips);" in the successful path? > Right. Best regards, Krzysztof