[PATCH] arm64: Kconfig.platforms: remove useless select for ARCH_K3

[PATCH] arm64: Kconfig.platforms: remove useless select for ARCH_K3

[Guillaume La Roque]

After patch done on TI_MESSAGE_MANAGER[1] and TI_SCI_PROTOCOL[2] driver
select on ARCH_K3 are not needed anymore.

Remove it and give possibility to enable this driver in modules.

[1] https://lore.kernel.org/all/20180828005311.8529-1-nm@ti.com/
[2] https://lore.kernel.org/all/20250220-ti-firmware-v2-1-ff26883c6ce9@baylibre.com/

Signed-off-by: Guillaume La Roque <glaroque@baylibre.com>
---
 arch/arm64/Kconfig.platforms | 2 --
 1 file changed, 2 deletions(-)

diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
index 8b76821f190f..5b63a42c4dff 100644
--- a/arch/arm64/Kconfig.platforms
+++ b/arch/arm64/Kconfig.platforms
@@ -138,8 +138,6 @@ config ARCH_K3
 	select PM_GENERIC_DOMAINS if PM
 	select MAILBOX
 	select SOC_TI
-	select TI_MESSAGE_MANAGER
-	select TI_SCI_PROTOCOL
 	select TI_K3_SOCINFO
 	help
 	  This enables support for Texas Instruments' K3 multicore SoC

---
base-commit: e8ab83e34bdc458b5cd77f201e4ed04807978fb1
change-id: 20250504-kconfig-68f139fbf337

Best regards,
-- 
Guillaume La Roque <glaroque@baylibre.com>

Re: [PATCH] arm64: Kconfig.platforms: remove useless select for ARCH_K3

[Nishanth Menon]

On 13:24-20250504, Guillaume La Roque wrote:
> After patch done on TI_MESSAGE_MANAGER[1] and TI_SCI_PROTOCOL[2] driver
> select on ARCH_K3 are not needed anymore.
> 
> Remove it and give possibility to enable this driver in modules.
> 
> [1] https://lore.kernel.org/all/20180828005311.8529-1-nm@ti.com/
> [2] https://lore.kernel.org/all/20250220-ti-firmware-v2-1-ff26883c6ce9@baylibre.com/
> 
> Signed-off-by: Guillaume La Roque <glaroque@baylibre.com>
> ---
>  arch/arm64/Kconfig.platforms | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
> index 8b76821f190f..5b63a42c4dff 100644
> --- a/arch/arm64/Kconfig.platforms
> +++ b/arch/arm64/Kconfig.platforms
> @@ -138,8 +138,6 @@ config ARCH_K3
>  	select PM_GENERIC_DOMAINS if PM
>  	select MAILBOX
>  	select SOC_TI
> -	select TI_MESSAGE_MANAGER
> -	select TI_SCI_PROTOCOL
>  	select TI_K3_SOCINFO
>  	help
>  	  This enables support for Texas Instruments' K3 multicore SoC
> 

While at this, is it possible to remove MAILBOX and PM_GENERIC_DOMAINS
from select and make them as modules?

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D

Re: [PATCH] arm64: Kconfig.platforms: remove useless select for ARCH_K3

[Guillaume La Roque]

Hi,

Le 05/05/2025 à 14:36, Nishanth Menon a écrit :
> On 13:24-20250504, Guillaume La Roque wrote:
>> After patch done on TI_MESSAGE_MANAGER[1] and TI_SCI_PROTOCOL[2] driver
>> select on ARCH_K3 are not needed anymore.
>>
>> Remove it and give possibility to enable this driver in modules.
>>
>> [1] https://lore.kernel.org/all/20180828005311.8529-1-nm@ti.com/
>> [2] https://lore.kernel.org/all/20250220-ti-firmware-v2-1-ff26883c6ce9@baylibre.com/
>>
>> Signed-off-by: Guillaume La Roque <glaroque@baylibre.com>
>> ---
>>   arch/arm64/Kconfig.platforms | 2 --
>>   1 file changed, 2 deletions(-)
>>
>> diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
>> index 8b76821f190f..5b63a42c4dff 100644
>> --- a/arch/arm64/Kconfig.platforms
>> +++ b/arch/arm64/Kconfig.platforms
>> @@ -138,8 +138,6 @@ config ARCH_K3
>>   	select PM_GENERIC_DOMAINS if PM
>>   	select MAILBOX
>>   	select SOC_TI
>> -	select TI_MESSAGE_MANAGER
>> -	select TI_SCI_PROTOCOL
>>   	select TI_K3_SOCINFO
>>   	help
>>   	  This enables support for Texas Instruments' K3 multicore SoC
>>
> While at this, is it possible to remove MAILBOX and PM_GENERIC_DOMAINS
> from select and make them as modules?
>
good point i will confirm  and come back to you.

Re: [PATCH] arm64: Kconfig.platforms: remove useless select for ARCH_K3

[Guillaume La Roque]

Hi Nishanth,

if i remove MAILBOX it's  enabled by some  non TI drivers and/or 
ARCH_XXX which have it in deps so all TI drivers are still enabled 
properly but not sure it's safe.
and PM_GENERIC_DOMAINS look OK to remove it.

if i try a defconfig based on android defconfig (more simple than 
default defconfig) with only ARCH_K3 enabled i need to set 
CONFIG_MAILBOX flag to still have same TI drivers enabled.
let me know what you want to do.

Guillaume
Le 05/05/2025 à 15:15, Guillaume La Roque a écrit :
> Hi,
>
> Le 05/05/2025 à 14:36, Nishanth Menon a écrit :
>> On 13:24-20250504, Guillaume La Roque wrote:
>>> After patch done on TI_MESSAGE_MANAGER[1] and TI_SCI_PROTOCOL[2] driver
>>> select on ARCH_K3 are not needed anymore.
>>>
>>> Remove it and give possibility to enable this driver in modules.
>>>
>>> [1] https://lore.kernel.org/all/20180828005311.8529-1-nm@ti.com/
>>> [2] 
>>> https://lore.kernel.org/all/20250220-ti-firmware-v2-1-ff26883c6ce9@baylibre.com/
>>>
>>> Signed-off-by: Guillaume La Roque <glaroque@baylibre.com>
>>> ---
>>>   arch/arm64/Kconfig.platforms | 2 --
>>>   1 file changed, 2 deletions(-)
>>>
>>> diff --git a/arch/arm64/Kconfig.platforms 
>>> b/arch/arm64/Kconfig.platforms
>>> index 8b76821f190f..5b63a42c4dff 100644
>>> --- a/arch/arm64/Kconfig.platforms
>>> +++ b/arch/arm64/Kconfig.platforms
>>> @@ -138,8 +138,6 @@ config ARCH_K3
>>>       select PM_GENERIC_DOMAINS if PM
>>>       select MAILBOX
>>>       select SOC_TI
>>> -    select TI_MESSAGE_MANAGER
>>> -    select TI_SCI_PROTOCOL
>>>       select TI_K3_SOCINFO
>>>       help
>>>         This enables support for Texas Instruments' K3 multicore SoC
>>>
>> While at this, is it possible to remove MAILBOX and PM_GENERIC_DOMAINS
>> from select and make them as modules?
>>
> good point i will confirm  and come back to you.

Re: [PATCH] arm64: Kconfig.platforms: remove useless select for ARCH_K3

[Nishanth Menon]

On 15:28-20250507, Guillaume La Roque wrote:
> Hi Nishanth,
> 
> if i remove MAILBOX it's  enabled by some  non TI drivers and/or ARCH_XXX
> which have it in deps so all TI drivers are still enabled properly but not
> sure it's safe.
> and PM_GENERIC_DOMAINS look OK to remove it.
> 
> if i try a defconfig based on android defconfig (more simple than default
> defconfig) with only ARCH_K3 enabled i need to set CONFIG_MAILBOX flag to
> still have same TI drivers enabled.
> let me know what you want to do.


Let us drop both from the select statement - from our K3 perspective
there is no longer a need for them to be built-in. Thank you for working
through this.

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D