linux-kernel - Re: [PATCH] Add watchdog driver for w90p910

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: <e68bb3470908050344u73cdb1c0o3257febee77d9f9f@mail.gmail.com>
Date:	Wed, 5 Aug 2009 18:44:57 +0800
From:	Wan ZongShun <mcuos.com@...il.com>
To:	Wim Van Sebroeck <wim@...ana.be>
Cc:	linux-arm-kernel <linux-arm-kernel@...ts.arm.linux.org.uk>,
	linux-kernel <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] Add watchdog driver for w90p910

Dear Wim,


2009/8/5, Wim Van Sebroeck <wim@...ana.be>:
> Hi Wan,
>
> > > I have an additional question: which timeout values can you set? is it 0, 1, 2 and 3 seconds?
> >
> > The 0, 1, 2 and 3 is not seconds, rather than 'level' , it means my
> > watchdog controller can only set four grades,
> > When WTIS bit of REG_WTCR was set one value of  the 0, 1, 2 and 3, the
> >  real time interval can be calculated via following formula .
> >
> > WTIS                 real time interval (formula)
> > 0x00                   ((2^ 14 ) * ((external crystal freq) / 256))seconds
> > 0x01                   ((2^ 16 ) * ((external crystal freq) / 256))seconds
> > 0x02                   ((2^ 18 ) * ((external crystal freq) / 256))seconds
> > 0x03                   ((2^ 20 ) * ((external crystal freq) / 256))seconds
> >
> > The external crystal freq is 15Mhz in w90p910 evaluation board. so I
> > set the '0x00' as a default value, it means the real time interval is
> > 0.28 second approximately.
>
> Hmm, the watchdog API works in seconds.
> A timer will need to be used like in similar watchdog drivers (like at91sam9_wdt.c, pcwd.c).

Do you mean that we should provide userspace a setting seconds api?
it like following:

ioctl(fd, WDIOC_SETTIMEOUT,&value);

the value must be seconds?

Hmm, but the w90p910 only provides four grades seconds setting, rather
than all seconds time can be set, if so, how to write the wdt driver
so that the patch can be accepted for you?any idea?

>
> Kind regards,
> Wim.
>
>


-- 
Wan z.s
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/