Re: [PATCH v2] firmware: config: ti-sci: Default set to ARCH_K3 for the ti sci driver

[Guillaume La Roque]

Le 19/03/2025 à 22:45, Arnd Bergmann a écrit :
> On Thu, Feb 20, 2025, at 14:31, Guillaume La Roque wrote:
> > With ARCH_K3=y we cannot enable TI_SCI_PROTOCOL=m because
> > ARCH_K3 selects TI_SCI_PROTOCOL.
> >
> > Modify the logic to enable TI_SCI_PROTOCOL by default when ARCH_K3=y
> > allowing us to submit a future patch to remove select on ARCH_K3 without
> > breaking existing users.
> >
> > Signed-off-by: Guillaume La Roque <glaroque@xxxxxxxxxxxx>
> > ---
> > Link to comment done on last serie [1] we come back on first version of
> > series[2] to not use imply but set deps on ARCH_K3 in driver directly.
> > An other patch will be sent to update Kconfig.platform when this patch
> > is merged.
> I just merged the patch and hadn't seen the earlier discussion,
> just two small notes from me:
>
> - thanks for doing it this way instead of the backwards 'imply'
> - there is really no need to split it into two steps, since the
>    change to the Kconfig.platforms file logically goes with this
>    one and I would merge both through the SoC tree, having it
>    as a single patch would make sense.
>
>          Arnd

Hi,

thanks for merged.

i do what we ask me in review in fact or perhaps i misunderstand sorry 
for that.

Guillaume