Hi Bertrand,

On 21/07/2023 10:07, Bertrand Marquis wrote:
On 21 Jul 2023, at 10:52, Julien Grall <[email protected]> wrote:
+
  config OPTEE
- bool "Enable OP-TEE mediator"
+ bool "Enable OP-TEE mediator (UNSUPPORTED)" if UNSUPPORTED

Given this is under 'TEE mediators', do we still need the 'if UNSUPPORTED'?

As explained to Jan, i used visible if which does not enforce the dependency, 
just a hint for the display.

I thought that it was clearer to keep things like that to make it easier to 
have one of them supported in the future and i added the visible if just so 
that there was not an empty menu.

Indeed.


I could also just use a
if UNSUPPORTED
  ...

endif

to surround everything if that is clearer.
Please tell me and i will push a v2.

I am happy with the existing version.

Cheers,

--
Julien Grall

Reply via email to