mbox series

[0/2] of: overlay locking fixes

Message ID 1511793987-315-1-git-send-email-geert+renesas@glider.be
Headers show
Series of: overlay locking fixes | expand

Message

Geert Uytterhoeven Nov. 27, 2017, 2:46 p.m. UTC
Hi Pantelis, Rob, Frank,

Here are two locking fixes for OF overlays, found by code inspection.

They were compile-tested only, as I'm still working on getting "real"
(configfs) overlays working again in v4.15-rc1...

Geert Uytterhoeven (2):
  of: overlay: Fix cleanup order in of_overlay_apply()
  of: unittest: Remove bogus overlay mutex release from
    overlay_data_add()

 drivers/of/overlay.c  | 6 +++---
 drivers/of/unittest.c | 1 -
 2 files changed, 3 insertions(+), 4 deletions(-)

Comments

Frank Rowand Nov. 27, 2017, 5:22 p.m. UTC | #1
Hi Geert,

I did not receive patch 1/2, can you please resend it?

-Frank

On 11/27/17 09:46, Geert Uytterhoeven wrote:
> 	Hi Pantelis, Rob, Frank,
> 
> Here are two locking fixes for OF overlays, found by code inspection.
> 
> They were compile-tested only, as I'm still working on getting "real"
> (configfs) overlays working again in v4.15-rc1...
> 
> Geert Uytterhoeven (2):
>   of: overlay: Fix cleanup order in of_overlay_apply()
>   of: unittest: Remove bogus overlay mutex release from
>     overlay_data_add()
> 
>  drivers/of/overlay.c  | 6 +++---
>  drivers/of/unittest.c | 1 -
>  2 files changed, 3 insertions(+), 4 deletions(-)
> 

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Frank Rowand Nov. 27, 2017, 5:49 p.m. UTC | #2
On 11/27/17 09:46, Geert Uytterhoeven wrote:
> 	Hi Pantelis, Rob, Frank,
> 
> Here are two locking fixes for OF overlays, found by code inspection.
> 
> They were compile-tested only, as I'm still working on getting "real"
> (configfs) overlays working again in v4.15-rc1...
> 
> Geert Uytterhoeven (2):
>   of: overlay: Fix cleanup order in of_overlay_apply()
>   of: unittest: Remove bogus overlay mutex release from
>     overlay_data_add()
> 
>  drivers/of/overlay.c  | 6 +++---
>  drivers/of/unittest.c | 1 -
>  2 files changed, 3 insertions(+), 4 deletions(-)
> 

All patches:

Reviewed-by: Frank Rowand <frank.rowand@sony.com>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Rob Herring (Arm) Nov. 28, 2017, 2:59 p.m. UTC | #3
On Mon, Nov 27, 2017 at 03:46:25PM +0100, Geert Uytterhoeven wrote:
> 	Hi Pantelis, Rob, Frank,
> 
> Here are two locking fixes for OF overlays, found by code inspection.
> 
> They were compile-tested only, as I'm still working on getting "real"
> (configfs) overlays working again in v4.15-rc1...
> 
> Geert Uytterhoeven (2):
>   of: overlay: Fix cleanup order in of_overlay_apply()
>   of: unittest: Remove bogus overlay mutex release from
>     overlay_data_add()

Applied.

Rob
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html