Re: [PATCH v3 1/4] dt-bindings: firmware: qcom,scm: support indicating SDI default state

From: Brian Norris
Date: Wed Aug 16 2023 - 23:36:31 EST


On Wed, Aug 16, 2023 at 06:45:38PM +0200, Robert Marko wrote:
> IPQ5018 has SDI (Secure Debug Image) enabled by TZ by default, and that
> means that WDT being asserted or just trying to reboot will hang the board
> in the debug mode and only pulling the power and repowering will help.
> Some IPQ4019 boards like Google WiFI have it enabled as well.
>
> So, lets add a boolean property to indicate that SDI is enabled by default
> and thus needs to be disabled by the kernel.
>
> Signed-off-by: Robert Marko <robimarko@xxxxxxxxx>

The series looks good to me. Thanks for doing this!

Reviewed-by: Brian Norris <computersforpeace@xxxxxxxxx>