Re: [PATCH] xen/arm: Move TEE mediators in a kconfig submenu

[Julien Grall <julien@xxxxxxx>]

Hi Bertrand,

On 21/07/2023 10:07, Bertrand Marquis wrote:
> > On 21 Jul 2023, at 10:52, Julien Grall <julien@xxxxxxx> 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