| Message ID | 20230621051358.3905149-1-u-kumar1@ti.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | arm64: defconfig: Enable K3 RTI Watchdog | expand |
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 >
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 >>
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 --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
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(+)