Messages in this thread | | | Subject | Re: [PATCH] drivers: ipa: use devm_kzalloc for simplicity | From | Alex Elder <> | Date | Mon, 18 May 2020 10:03:52 -0500 |
| |
On 5/14/20 2:47 PM, David Miller wrote: > From: Jakub Kicinski <kuba@kernel.org> > Date: Thu, 14 May 2020 10:15:16 -0700 > >> On Wed, 13 May 2020 20:55:20 -0700 Wang Wenhu wrote: >>> Make a substitution of kzalloc with devm_kzalloc to simplify the >>> ipa_probe() process. >>> >>> Signed-off-by: Wang Wenhu <wenhu.wang@vivo.com> >> >> The code is perfectly fine as is. What problem are you trying to solve? > > I agree, these kinds of transformations are kind of excessive.
I have considered using the devm_*() functions, but if I were to make such a switch it would be done comprehensively, throughout the driver. It is not something I plan to do in the near term though.
-Alex
| |