diff mbox series

arm64: defconfig: Enable K3 RTI Watchdog

Message ID 20230621051358.3905149-1-u-kumar1@ti.com
State New
Headers
Series arm64: defconfig: Enable K3 RTI Watchdog |

Commit Message

Kumar, Udit June 21, 2023, 5:13 a.m. UTC 
  K3 Family has RTI watchdog. This patch is
needed to enable WDT functions.

Signed-off-by: Udit Kumar <u-kumar1@ti.com>
---
 arch/arm64/configs/defconfig | 1 +
 1 file changed, 1 insertion(+)

Comments

Nishanth Menon June 21, 2023, 5:15 a.m. UTC | #1 
On 10:43-20230621, Udit Kumar wrote:
> K3 Family has RTI watchdog. This patch is

For the benefit of others, please provide context of what K3 and RTI
watchdog is.

> needed to enable WDT functions.

Why not a module?

Also please indicate specific products that can benefit with the change.

> 
> Signed-off-by: Udit Kumar <u-kumar1@ti.com>
> ---
>  arch/arm64/configs/defconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> index 57c6b7bb88d4..4567645abe35 100644
> --- a/arch/arm64/configs/defconfig
> +++ b/arch/arm64/configs/defconfig
> @@ -686,6 +686,7 @@ CONFIG_UNIPHIER_WATCHDOG=y
>  CONFIG_PM8916_WATCHDOG=m
>  CONFIG_BCM2835_WDT=y
>  CONFIG_BCM7038_WDT=m
> +CONFIG_K3_RTI_WATCHDOG=y
>  CONFIG_MFD_ALTERA_SYSMGR=y
>  CONFIG_MFD_BD9571MWV=y
>  CONFIG_MFD_AXP20X_I2C=y
> -- 
> 2.34.1
>
Kumar, Udit June 21, 2023, 5:29 a.m. UTC | #2 
On 6/21/2023 10:45 AM, Nishanth Menon wrote:
> On 10:43-20230621, Udit Kumar wrote:
>> K3 Family has RTI watchdog. This patch is
> For the benefit of others, please provide context of what K3 and RTI
> watchdog is.

ok


>
>> needed to enable WDT functions.
> Why not a module?

For ease of use, i used as part of kernel.

No issues having as module

>
> Also please indicate specific products that can benefit with the change.

Sure.


>> Signed-off-by: Udit Kumar <u-kumar1@ti.com>
>> ---
>>   arch/arm64/configs/defconfig | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
>> index 57c6b7bb88d4..4567645abe35 100644
>> --- a/arch/arm64/configs/defconfig
>> +++ b/arch/arm64/configs/defconfig
>> @@ -686,6 +686,7 @@ CONFIG_UNIPHIER_WATCHDOG=y
>>   CONFIG_PM8916_WATCHDOG=m
>>   CONFIG_BCM2835_WDT=y
>>   CONFIG_BCM7038_WDT=m
>> +CONFIG_K3_RTI_WATCHDOG=y
>>   CONFIG_MFD_ALTERA_SYSMGR=y
>>   CONFIG_MFD_BD9571MWV=y
>>   CONFIG_MFD_AXP20X_I2C=y
>> -- 
>> 2.34.1
>>
Nishanth Menon June 21, 2023, 5:36 a.m. UTC | #3 
On 10:59-20230621, Kumar, Udit wrote:
[..]
> For ease of use, i used as part of kernel.
> 
> No issues having as module

Please keep it as a module as a rule, we don't want to blow up the kernel
Image size unless we have a very very strong need to have it in kernel
(example: for boot)
diff mbox series

Patch

diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
index 57c6b7bb88d4..4567645abe35 100644
--- a/arch/arm64/configs/defconfig
+++ b/arch/arm64/configs/defconfig
@@ -686,6 +686,7 @@  CONFIG_UNIPHIER_WATCHDOG=y
 CONFIG_PM8916_WATCHDOG=m
 CONFIG_BCM2835_WDT=y
 CONFIG_BCM7038_WDT=m
+CONFIG_K3_RTI_WATCHDOG=y
 CONFIG_MFD_ALTERA_SYSMGR=y
 CONFIG_MFD_BD9571MWV=y
 CONFIG_MFD_AXP20X_I2C=y