diff mbox series

[07/10] arm64: Kconfig: expand STM32 Armv8 SoC with STM32MP21 SoCs family

Message ID 20250210-b4-stm32mp2_new_dts-v1-7-e8ef1e666c5e@foss.st.com (mailing list archive)
State Not Applicable
Headers show
Series Expand STM32MP2 family with new SoC and boards | expand

Checks

Context Check Description
netdev/tree_selection success Not a local patch

Commit Message

Amelie Delaunay Feb. 10, 2025, 3:21 p.m. UTC
Expand config ARCH_STM32 with the new STM32MP21 SoCs family which is
composed of STM32MP211, STM32MP213 and STM32MP215 SoCs.

Signed-off-by: Amelie Delaunay <amelie.delaunay@foss.st.com>
---
 arch/arm64/Kconfig.platforms | 2 ++
 1 file changed, 2 insertions(+)

Comments

Krzysztof Kozlowski Feb. 13, 2025, 9:02 a.m. UTC | #1
On Mon, Feb 10, 2025 at 04:21:01PM +0100, Amelie Delaunay wrote:
> Expand config ARCH_STM32 with the new STM32MP21 SoCs family which is
> composed of STM32MP211, STM32MP213 and STM32MP215 SoCs.
> 
> Signed-off-by: Amelie Delaunay <amelie.delaunay@foss.st.com>
> ---
>  arch/arm64/Kconfig.platforms | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
> index 844a39620cfea8bfc031a545d85e33894ef20994..f788dbc09c9eb6f5801758ccf6b0ffe50a96090e 100644
> --- a/arch/arm64/Kconfig.platforms
> +++ b/arch/arm64/Kconfig.platforms
> @@ -325,6 +325,8 @@ config ARCH_STM32
>  			- STM32MP251, STM32MP253, STM32MP255 and STM32MP257.
>  		- STM32MP23:
>  			- STM32MP231, STM32MP233, STM32MP235.
> +		- STM32MP21:

Squash it with previous patch and keep some sort of order.

> +			- STM32MP211, STM32MP213, STM32MP215.
>  
>  config ARCH_SYNQUACER
>  	bool "Socionext SynQuacer SoC Family"
> 
> -- 
> 2.25.1
>
Amelie Delaunay Feb. 14, 2025, 3:06 p.m. UTC | #2
On 2/13/25 10:02, Krzysztof Kozlowski wrote:
> On Mon, Feb 10, 2025 at 04:21:01PM +0100, Amelie Delaunay wrote:
>> Expand config ARCH_STM32 with the new STM32MP21 SoCs family which is
>> composed of STM32MP211, STM32MP213 and STM32MP215 SoCs.
>>
>> Signed-off-by: Amelie Delaunay <amelie.delaunay@foss.st.com>
>> ---
>>   arch/arm64/Kconfig.platforms | 2 ++
>>   1 file changed, 2 insertions(+)
>>
>> diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
>> index 844a39620cfea8bfc031a545d85e33894ef20994..f788dbc09c9eb6f5801758ccf6b0ffe50a96090e 100644
>> --- a/arch/arm64/Kconfig.platforms
>> +++ b/arch/arm64/Kconfig.platforms
>> @@ -325,6 +325,8 @@ config ARCH_STM32
>>   			- STM32MP251, STM32MP253, STM32MP255 and STM32MP257.
>>   		- STM32MP23:
>>   			- STM32MP231, STM32MP233, STM32MP235.
>> +		- STM32MP21:
> 
> Squash it with previous patch and keep some sort of order.
>

Ok for squashing with patch 3.
Do you mean to keep the current chronological order used here or to 
change the order because "chronological" is not an appropriate order? In 
this case, would the alphanumeric order be fine?

>> +			- STM32MP211, STM32MP213, STM32MP215.
>>   
>>   config ARCH_SYNQUACER
>>   	bool "Socionext SynQuacer SoC Family"
>>
>> -- 
>> 2.25.1
>>

Regards,
Amelie
Krzysztof Kozlowski Feb. 15, 2025, 11:14 a.m. UTC | #3
On 14/02/2025 16:06, Amelie Delaunay wrote:
> On 2/13/25 10:02, Krzysztof Kozlowski wrote:
>> On Mon, Feb 10, 2025 at 04:21:01PM +0100, Amelie Delaunay wrote:
>>> Expand config ARCH_STM32 with the new STM32MP21 SoCs family which is
>>> composed of STM32MP211, STM32MP213 and STM32MP215 SoCs.
>>>
>>> Signed-off-by: Amelie Delaunay <amelie.delaunay@foss.st.com>
>>> ---
>>>   arch/arm64/Kconfig.platforms | 2 ++
>>>   1 file changed, 2 insertions(+)
>>>
>>> diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
>>> index 844a39620cfea8bfc031a545d85e33894ef20994..f788dbc09c9eb6f5801758ccf6b0ffe50a96090e 100644
>>> --- a/arch/arm64/Kconfig.platforms
>>> +++ b/arch/arm64/Kconfig.platforms
>>> @@ -325,6 +325,8 @@ config ARCH_STM32
>>>   			- STM32MP251, STM32MP253, STM32MP255 and STM32MP257.
>>>   		- STM32MP23:
>>>   			- STM32MP231, STM32MP233, STM32MP235.
>>> +		- STM32MP21:
>>
>> Squash it with previous patch and keep some sort of order.
>>
> 
> Ok for squashing with patch 3.
> Do you mean to keep the current chronological order used here or to 

chronological of what? Adding it? That's the worse of possible orders,
because it is basically random invitation to conflicts.

If chronological of market release, that's tricky to any contributor to
figure out.

> change the order because "chronological" is not an appropriate order? In 
> this case, would the alphanumeric order be fine?

Many lists go alphanumerical because it is most obvious and avoids
conflicts.

Best regards,
Krzysztof
diff mbox series

Patch

diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
index 844a39620cfea8bfc031a545d85e33894ef20994..f788dbc09c9eb6f5801758ccf6b0ffe50a96090e 100644
--- a/arch/arm64/Kconfig.platforms
+++ b/arch/arm64/Kconfig.platforms
@@ -325,6 +325,8 @@  config ARCH_STM32
 			- STM32MP251, STM32MP253, STM32MP255 and STM32MP257.
 		- STM32MP23:
 			- STM32MP231, STM32MP233, STM32MP235.
+		- STM32MP21:
+			- STM32MP211, STM32MP213, STM32MP215.
 
 config ARCH_SYNQUACER
 	bool "Socionext SynQuacer SoC Family"