[PATCH] arm64: arch_k3: enable chipid driver

[PATCH] arm64: arch_k3: enable chipid driver

[Grygorii Strashko]

Select TI chip id driver for TI's SoCs based on K3 architecture to provide
this information to user space and Kernel as it is required by other
drivers to determine SoC revision to function properly.

Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>
Reviewed-by: Lokesh Vutla <lokeshvutla@ti.com>
---
 arch/arm64/Kconfig.platforms | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
index 8dd05b2a925c..1d3710e3626a 100644
--- a/arch/arm64/Kconfig.platforms
+++ b/arch/arm64/Kconfig.platforms
@@ -98,6 +98,7 @@ config ARCH_K3
 	select TI_SCI_PROTOCOL
 	select TI_SCI_INTR_IRQCHIP
 	select TI_SCI_INTA_IRQCHIP
+	select TI_K3_SOCINFO
 	help
 	  This enables support for Texas Instruments' K3 multicore SoC
 	  architecture.
-- 
2.17.1

Re: [PATCH] arm64: arch_k3: enable chipid driver

[Grygorii Strashko]

hi All,

On 19/06/2020 19:25, Grygorii Strashko wrote:
> Select TI chip id driver for TI's SoCs based on K3 architecture to provide
> this information to user space and Kernel as it is required by other
> drivers to determine SoC revision to function properly.
> 
> Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>
> Reviewed-by: Lokesh Vutla <lokeshvutla@ti.com>
> ---
>   arch/arm64/Kconfig.platforms | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
> index 8dd05b2a925c..1d3710e3626a 100644
> --- a/arch/arm64/Kconfig.platforms
> +++ b/arch/arm64/Kconfig.platforms
> @@ -98,6 +98,7 @@ config ARCH_K3
>   	select TI_SCI_PROTOCOL
>   	select TI_SCI_INTR_IRQCHIP
>   	select TI_SCI_INTA_IRQCHIP
> +	select TI_K3_SOCINFO
>   	help
>   	  This enables support for Texas Instruments' K3 multicore SoC
>   	  architecture.
> 

Are there any comments?

The driver and dt changes were merged [1][2] and for adding new am654x SoC
revision SR2.0 we need this driver to be enabled always as other drivers are
going to use SOC Bus API intensively.

No dependencies.

[1] https://lore.kernel.org/lkml/20200512123449.16517-1-grygorii.strashko@ti.com/
[2] https://lore.kernel.org/lkml/20200613164346.28852-1-grygorii.strashko@ti.com/


-- 
Best regards,
grygorii

Re: [PATCH] arm64: arch_k3: enable chipid driver

[Peter Ujfalusi]

On 01/07/2020 13.18, Grygorii Strashko wrote:
> hi All,
> 
> On 19/06/2020 19:25, Grygorii Strashko wrote:
>> Select TI chip id driver for TI's SoCs based on K3 architecture to
>> provide
>> this information to user space and Kernel as it is required by other
>> drivers to determine SoC revision to function properly.
>>
>> Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>
>> Reviewed-by: Lokesh Vutla <lokeshvutla@ti.com>
>> ---
>>   arch/arm64/Kconfig.platforms | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
>> index 8dd05b2a925c..1d3710e3626a 100644
>> --- a/arch/arm64/Kconfig.platforms
>> +++ b/arch/arm64/Kconfig.platforms
>> @@ -98,6 +98,7 @@ config ARCH_K3
>>       select TI_SCI_PROTOCOL
>>       select TI_SCI_INTR_IRQCHIP
>>       select TI_SCI_INTA_IRQCHIP
>> +    select TI_K3_SOCINFO
>>       help
>>         This enables support for Texas Instruments' K3 multicore SoC
>>         architecture.
>>
> 
> Are there any comments?

Reviewed-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
Tested-by: Peter Ujfalusi <peter.ujfalusi@ti.com>

> The driver and dt changes were merged [1][2] and for adding new am654x SoC
> revision SR2.0 we need this driver to be enabled always as other drivers
> are
> going to use SOC Bus API intensively.
> 
> No dependencies.
> 
> [1]
> https://lore.kernel.org/lkml/20200512123449.16517-1-grygorii.strashko@ti.com/
> 
> [2]
> https://lore.kernel.org/lkml/20200613164346.28852-1-grygorii.strashko@ti.com/
> 
> 
> 

- Péter

Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

Re: [PATCH] arm64: arch_k3: enable chipid driver

[Grygorii Strashko]

Hi All,

On 07/07/2020 10:02, Peter Ujfalusi wrote:
> On 01/07/2020 13.18, Grygorii Strashko wrote:
>> On 19/06/2020 19:25, Grygorii Strashko wrote:
>>> Select TI chip id driver for TI's SoCs based on K3 architecture to
>>> provide
>>> this information to user space and Kernel as it is required by other
>>> drivers to determine SoC revision to function properly.
>>>
>>> Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>
>>> Reviewed-by: Lokesh Vutla <lokeshvutla@ti.com>
>>> ---
>>>    arch/arm64/Kconfig.platforms | 1 +
>>>    1 file changed, 1 insertion(+)
>>>
>>> diff --git a/arch/arm64/Kconfig.platforms b/arch/arm64/Kconfig.platforms
>>> index 8dd05b2a925c..1d3710e3626a 100644
>>> --- a/arch/arm64/Kconfig.platforms
>>> +++ b/arch/arm64/Kconfig.platforms
>>> @@ -98,6 +98,7 @@ config ARCH_K3
>>>        select TI_SCI_PROTOCOL
>>>        select TI_SCI_INTR_IRQCHIP
>>>        select TI_SCI_INTA_IRQCHIP
>>> +    select TI_K3_SOCINFO
>>>        help
>>>          This enables support for Texas Instruments' K3 multicore SoC
>>>          architecture.
>>>
>>
>> Are there any comments?
> 
> Reviewed-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
> Tested-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
> 
>> The driver and dt changes were merged [1][2] and for adding new am654x SoC
>> revision SR2.0 we need this driver to be enabled always as other drivers
>> are
>> going to use SOC Bus API intensively.
>>
>> No dependencies.
>>
>> [1]
>> https://lore.kernel.org/lkml/20200512123449.16517-1-grygorii.strashko@ti.com/
>>
>> [2]
>> https://lore.kernel.org/lkml/20200613164346.28852-1-grygorii.strashko@ti.com/
>>

Any chances it can be applied for 5.9?


Thank you.
-- 
Best regards,
grygorii

Re: [PATCH] arm64: arch_k3: enable chipid driver

[Tero Kristo]

On 15/07/2020 13:03, Grygorii Strashko wrote:
> Hi All,
> 
> On 07/07/2020 10:02, Peter Ujfalusi wrote:
>> On 01/07/2020 13.18, Grygorii Strashko wrote:
>>> On 19/06/2020 19:25, Grygorii Strashko wrote:
>>>> Select TI chip id driver for TI's SoCs based on K3 architecture to
>>>> provide
>>>> this information to user space and Kernel as it is required by other
>>>> drivers to determine SoC revision to function properly.
>>>>
>>>> Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>
>>>> Reviewed-by: Lokesh Vutla <lokeshvutla@ti.com>
>>>> ---
>>>>    arch/arm64/Kconfig.platforms | 1 +
>>>>    1 file changed, 1 insertion(+)
>>>>
>>>> diff --git a/arch/arm64/Kconfig.platforms 
>>>> b/arch/arm64/Kconfig.platforms
>>>> index 8dd05b2a925c..1d3710e3626a 100644
>>>> --- a/arch/arm64/Kconfig.platforms
>>>> +++ b/arch/arm64/Kconfig.platforms
>>>> @@ -98,6 +98,7 @@ config ARCH_K3
>>>>        select TI_SCI_PROTOCOL
>>>>        select TI_SCI_INTR_IRQCHIP
>>>>        select TI_SCI_INTA_IRQCHIP
>>>> +    select TI_K3_SOCINFO
>>>>        help
>>>>          This enables support for Texas Instruments' K3 multicore SoC
>>>>          architecture.
>>>>
>>>
>>> Are there any comments?
>>
>> Reviewed-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
>> Tested-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
>>
>>> The driver and dt changes were merged [1][2] and for adding new 
>>> am654x SoC
>>> revision SR2.0 we need this driver to be enabled always as other drivers
>>> are
>>> going to use SOC Bus API intensively.
>>>
>>> No dependencies.
>>>
>>> [1]
>>> https://lore.kernel.org/lkml/20200512123449.16517-1-grygorii.strashko@ti.com/ 
>>>
>>>
>>> [2]
>>> https://lore.kernel.org/lkml/20200613164346.28852-1-grygorii.strashko@ti.com/ 
>>>
>>>
> 
> Any chances it can be applied for 5.9?

Queuing up for 5.9, thanks.

Sorry there has been some confusion on my behalf regarding the config 
changes.

-Tero
--
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki