2022-04-12 20:26:32

by Frank Rowand

[permalink] [raw]
Subject: [PATCH v2 0/2] of: overlay: rework overlay apply and remove kfree()s

From: Frank Rowand <[email protected]>

Fix various kfree() issues related to of_overlay_apply().

The fixes revealed inconsist variable names for the same variable
across functions, resulting in difficulty understanding the code
that was being modified. Doing both variable renaming and the
fixes results in a hard to review patch, so split into two patches.

The first patch in the series contains only variable renaming.
The second patch contains the kfree() related fixes.

Frank Rowand (2):
of: overlay: rename variables to be consistent
of: overlay: rework overlay apply and remove kfree()s

Documentation/devicetree/overlay-notes.rst | 23 ++-
drivers/of/overlay.c | 175 +++++++++++----------
2 files changed, 115 insertions(+), 83 deletions(-)

--
Frank Rowand <[email protected]>


2022-04-12 23:49:12

by Frank Rowand

[permalink] [raw]
Subject: Re: [PATCH v2 0/2] of: overlay: rework overlay apply and remove kfree()s

adding cc: Jan Kiszka <[email protected]>

On 4/10/22 16:08, [email protected] wrote:
> From: Frank Rowand <[email protected]>
>
> Fix various kfree() issues related to of_overlay_apply().
>
> The fixes revealed inconsist variable names for the same variable
> across functions, resulting in difficulty understanding the code
> that was being modified. Doing both variable renaming and the
> fixes results in a hard to review patch, so split into two patches.
>
> The first patch in the series contains only variable renaming.
> The second patch contains the kfree() related fixes.
>
> Frank Rowand (2):
> of: overlay: rename variables to be consistent
> of: overlay: rework overlay apply and remove kfree()s
>
> Documentation/devicetree/overlay-notes.rst | 23 ++-
> drivers/of/overlay.c | 175 +++++++++++----------
> 2 files changed, 115 insertions(+), 83 deletions(-)
>