linux-kernel - RE: [PATCH v3 RESEND 3/5] watchdog: s3c2410_wdt: Increase max timeout value of watchdog

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <2b5e01dbfc5f$e298d0c0$a7ca7240$@samsung.com>
Date: Thu, 24 Jul 2025 11:27:47 +0530
From: "Alim Akhtar" <alim.akhtar@...sung.com>
To: "'Sangwook Shin'" <sw617.shin@...sung.com>, <krzk@...nel.org>,
	<wim@...ux-watchdog.org>, <linux@...ck-us.net>
Cc: <linux-arm-kernel@...ts.infradead.org>,
	<linux-samsung-soc@...r.kernel.org>, <linux-watchdog@...r.kernel.org>,
	<linux-kernel@...r.kernel.org>
Subject: RE: [PATCH v3 RESEND 3/5] watchdog: s3c2410_wdt: Increase max
 timeout value of watchdog

Hi Sangwook

> -----Original Message-----
> From: Sangwook Shin <sw617.shin@...sung.com>
> Sent: Monday, July 14, 2025 11:25 AM
> To: krzk@...nel.org; alim.akhtar@...sung.com; wim@...ux-watchdog.org;
> linux@...ck-us.net
> Cc: linux-arm-kernel@...ts.infradead.org; linux-samsung-
> soc@...r.kernel.org; linux-watchdog@...r.kernel.org; linux-
> kernel@...r.kernel.org; Sangwook Shin <sw617.shin@...sung.com>
> Subject: [PATCH v3 RESEND 3/5] watchdog: s3c2410_wdt: Increase max
> timeout value of watchdog
> 
> Increase max_timeout value from 55s to 3664647s (1017h 57min 27s) with
> 38400000 frequency system if the system has 32-bit WTCNT register.
> 
> cat
> /sys/devices/platform/10060000.watchdog_cl0/watchdog/watchdog0/max_t
> imeout
> 3664647
> 
> [    0.302473] s3c2410-wdt 10060000.watchdog_cl0: Heartbeat:
> count=1099394100000, timeout=3664647, freq=300000
> [    0.302479] s3c2410-wdt 10060000.watchdog_cl0: Heartbeat:
> timeout=3664647, divisor=256, count=1099394100000 (fff8feac)
> [    0.302510] s3c2410-wdt 10060000.watchdog_cl0: starting watchdog timer
> [    0.302722] s3c2410-wdt 10060000.watchdog_cl0: watchdog active, reset
> enabled, irq disabled
> 
> If system has 32-bit WTCNT, add QUIRK_HAS_32BIT_MAXCNT to its quirk
> flags, then it will operation with 32-bit counter. If not, with 16-bit counter like
> previous.
> 
As Krzysztof commented, please merge this patch with the next patch so that Quirk and it consumer is part of the same patch. 
Once you have done that, feel free to add
Reviewed-by: Alim Akhtar <alim.akhtar@...sung.com>