Re: Some problems with sysfs patch (was Re: [PATCH v6] watchdog: New watchdog driver for MEN A21 watchdogs)

2013-06-07 Thread Johannes Thumshirn
On Fri, Jun 07, 2013 at 02:14:19AM -0700, Guenter Roeck wrote: > On Fri, Jun 07, 2013 at 09:45:19AM +0200, Johannes Thumshirn wrote: > > On Thu, Jun 06, 2013 at 10:08:28AM -0700, Guenter Roeck wrote: > > > On Thu, Jun 06, 2013 at 03:00:55PM +0200, Johannes Thumshirn wrote: > > > > On Thu, Jun 06,

Re: Some problems with sysfs patch (was Re: [PATCH v6] watchdog: New watchdog driver for MEN A21 watchdogs)

2013-06-07 Thread Guenter Roeck
On Fri, Jun 07, 2013 at 09:45:19AM +0200, Johannes Thumshirn wrote: > On Thu, Jun 06, 2013 at 10:08:28AM -0700, Guenter Roeck wrote: > > On Thu, Jun 06, 2013 at 03:00:55PM +0200, Johannes Thumshirn wrote: > > > On Thu, Jun 06, 2013 at 04:31:10AM -0700, Guenter Roeck wrote: > > > > On Thu, Jun 06,

Re: Some problems with sysfs patch (was Re: [PATCH v6] watchdog: New watchdog driver for MEN A21 watchdogs)

2013-06-07 Thread Johannes Thumshirn
On Thu, Jun 06, 2013 at 10:08:28AM -0700, Guenter Roeck wrote: > On Thu, Jun 06, 2013 at 03:00:55PM +0200, Johannes Thumshirn wrote: > > On Thu, Jun 06, 2013 at 04:31:10AM -0700, Guenter Roeck wrote: > > > On Thu, Jun 06, 2013 at 12:51:03PM +0200, Johannes Thumshirn wrote: > > > > On Mon, Jun 03,

Re: Some problems with sysfs patch (was Re: [PATCH v6] watchdog: New watchdog driver for MEN A21 watchdogs)

2013-06-07 Thread Johannes Thumshirn
On Thu, Jun 06, 2013 at 10:08:28AM -0700, Guenter Roeck wrote: On Thu, Jun 06, 2013 at 03:00:55PM +0200, Johannes Thumshirn wrote: On Thu, Jun 06, 2013 at 04:31:10AM -0700, Guenter Roeck wrote: On Thu, Jun 06, 2013 at 12:51:03PM +0200, Johannes Thumshirn wrote: On Mon, Jun 03, 2013 at

Re: Some problems with sysfs patch (was Re: [PATCH v6] watchdog: New watchdog driver for MEN A21 watchdogs)

2013-06-07 Thread Guenter Roeck
On Fri, Jun 07, 2013 at 09:45:19AM +0200, Johannes Thumshirn wrote: On Thu, Jun 06, 2013 at 10:08:28AM -0700, Guenter Roeck wrote: On Thu, Jun 06, 2013 at 03:00:55PM +0200, Johannes Thumshirn wrote: On Thu, Jun 06, 2013 at 04:31:10AM -0700, Guenter Roeck wrote: On Thu, Jun 06, 2013 at

Re: Some problems with sysfs patch (was Re: [PATCH v6] watchdog: New watchdog driver for MEN A21 watchdogs)

2013-06-07 Thread Johannes Thumshirn
On Fri, Jun 07, 2013 at 02:14:19AM -0700, Guenter Roeck wrote: On Fri, Jun 07, 2013 at 09:45:19AM +0200, Johannes Thumshirn wrote: On Thu, Jun 06, 2013 at 10:08:28AM -0700, Guenter Roeck wrote: On Thu, Jun 06, 2013 at 03:00:55PM +0200, Johannes Thumshirn wrote: On Thu, Jun 06, 2013 at

Re: Some problems with sysfs patch (was Re: [PATCH v6] watchdog: New watchdog driver for MEN A21 watchdogs)

2013-06-06 Thread Guenter Roeck
On Thu, Jun 06, 2013 at 03:00:55PM +0200, Johannes Thumshirn wrote: > On Thu, Jun 06, 2013 at 04:31:10AM -0700, Guenter Roeck wrote: > > On Thu, Jun 06, 2013 at 12:51:03PM +0200, Johannes Thumshirn wrote: > > > On Mon, Jun 03, 2013 at 11:50:26AM +0200, Johannes Thumshirn wrote: > > > > On Fri, May

Re: Some problems with sysfs patch (was Re: [PATCH v6] watchdog: New watchdog driver for MEN A21 watchdogs)

2013-06-06 Thread Johannes Thumshirn
On Thu, Jun 06, 2013 at 04:31:10AM -0700, Guenter Roeck wrote: > On Thu, Jun 06, 2013 at 12:51:03PM +0200, Johannes Thumshirn wrote: > > On Mon, Jun 03, 2013 at 11:50:26AM +0200, Johannes Thumshirn wrote: > > > On Fri, May 31, 2013 at 09:15:23PM -0700, Guenter Roeck wrote: > > > > On Fri, May 31,

Re: Some problems with sysfs patch (was Re: [PATCH v6] watchdog: New watchdog driver for MEN A21 watchdogs)

2013-06-06 Thread Guenter Roeck
On Thu, Jun 06, 2013 at 12:51:03PM +0200, Johannes Thumshirn wrote: > On Mon, Jun 03, 2013 at 11:50:26AM +0200, Johannes Thumshirn wrote: > > On Fri, May 31, 2013 at 09:15:23PM -0700, Guenter Roeck wrote: > > > On Fri, May 31, 2013 at 02:55:19PM +0200, Johannes Thumshirn wrote: > > > > Hi

Some problems with sysfs patch (was Re: [PATCH v6] watchdog: New watchdog driver for MEN A21 watchdogs)

2013-06-06 Thread Johannes Thumshirn
On Mon, Jun 03, 2013 at 11:50:26AM +0200, Johannes Thumshirn wrote: > On Fri, May 31, 2013 at 09:15:23PM -0700, Guenter Roeck wrote: > > On Fri, May 31, 2013 at 02:55:19PM +0200, Johannes Thumshirn wrote: > > > Hi Guenther, > > > On Fri, May 31, 2013 at 04:40:37AM -0700, Guenter Roeck wrote: > > >

Some problems with sysfs patch (was Re: [PATCH v6] watchdog: New watchdog driver for MEN A21 watchdogs)

2013-06-06 Thread Johannes Thumshirn
On Mon, Jun 03, 2013 at 11:50:26AM +0200, Johannes Thumshirn wrote: On Fri, May 31, 2013 at 09:15:23PM -0700, Guenter Roeck wrote: On Fri, May 31, 2013 at 02:55:19PM +0200, Johannes Thumshirn wrote: Hi Guenther, On Fri, May 31, 2013 at 04:40:37AM -0700, Guenter Roeck wrote: +#define

Re: Some problems with sysfs patch (was Re: [PATCH v6] watchdog: New watchdog driver for MEN A21 watchdogs)

2013-06-06 Thread Guenter Roeck
On Thu, Jun 06, 2013 at 12:51:03PM +0200, Johannes Thumshirn wrote: On Mon, Jun 03, 2013 at 11:50:26AM +0200, Johannes Thumshirn wrote: On Fri, May 31, 2013 at 09:15:23PM -0700, Guenter Roeck wrote: On Fri, May 31, 2013 at 02:55:19PM +0200, Johannes Thumshirn wrote: Hi Guenther, On

Re: Some problems with sysfs patch (was Re: [PATCH v6] watchdog: New watchdog driver for MEN A21 watchdogs)

2013-06-06 Thread Johannes Thumshirn
On Thu, Jun 06, 2013 at 04:31:10AM -0700, Guenter Roeck wrote: On Thu, Jun 06, 2013 at 12:51:03PM +0200, Johannes Thumshirn wrote: On Mon, Jun 03, 2013 at 11:50:26AM +0200, Johannes Thumshirn wrote: On Fri, May 31, 2013 at 09:15:23PM -0700, Guenter Roeck wrote: On Fri, May 31, 2013 at

Re: Some problems with sysfs patch (was Re: [PATCH v6] watchdog: New watchdog driver for MEN A21 watchdogs)

2013-06-06 Thread Guenter Roeck
On Thu, Jun 06, 2013 at 03:00:55PM +0200, Johannes Thumshirn wrote: On Thu, Jun 06, 2013 at 04:31:10AM -0700, Guenter Roeck wrote: On Thu, Jun 06, 2013 at 12:51:03PM +0200, Johannes Thumshirn wrote: On Mon, Jun 03, 2013 at 11:50:26AM +0200, Johannes Thumshirn wrote: On Fri, May 31, 2013

Re: [PATCH v6] watchdog: New watchdog driver for MEN A21 watchdogs

2013-06-03 Thread Johannes Thumshirn
On Fri, May 31, 2013 at 09:15:23PM -0700, Guenter Roeck wrote: > On Fri, May 31, 2013 at 02:55:19PM +0200, Johannes Thumshirn wrote: > > Hi Guenther, > > On Fri, May 31, 2013 at 04:40:37AM -0700, Guenter Roeck wrote: > > > > +#define GPIO_WD_ENAB 169 > > > > +#define GPIO_WD_FAST 170 > > > >

Re: [PATCH v6] watchdog: New watchdog driver for MEN A21 watchdogs

2013-06-03 Thread Johannes Thumshirn
On Fri, May 31, 2013 at 09:15:23PM -0700, Guenter Roeck wrote: On Fri, May 31, 2013 at 02:55:19PM +0200, Johannes Thumshirn wrote: Hi Guenther, On Fri, May 31, 2013 at 04:40:37AM -0700, Guenter Roeck wrote: +#define GPIO_WD_ENAB 169 +#define GPIO_WD_FAST 170 +#define

Re: [PATCH v6] watchdog: New watchdog driver for MEN A21 watchdogs

2013-05-31 Thread Guenter Roeck
On Fri, May 31, 2013 at 02:55:19PM +0200, Johannes Thumshirn wrote: > Hi Guenther, > On Fri, May 31, 2013 at 04:40:37AM -0700, Guenter Roeck wrote: > > > +#define GPIO_WD_ENAB 169 > > > +#define GPIO_WD_FAST 170 > > > +#define GPIO_WD_TRIG 171 > > > + > > > +#define GPIO_RST_CAUSE_BASE

Re: [PATCH v6] watchdog: New watchdog driver for MEN A21 watchdogs

2013-05-31 Thread Johannes Thumshirn
Hi Guenther, On Fri, May 31, 2013 at 04:40:37AM -0700, Guenter Roeck wrote: > > +#define GPIO_WD_ENAB 169 > > +#define GPIO_WD_FAST 170 > > +#define GPIO_WD_TRIG 171 > > + > > +#define GPIO_RST_CAUSE_BASE 166 > > + > > I think I asked that before ... as you are supporting

Re: [PATCH v6] watchdog: New watchdog driver for MEN A21 watchdogs

2013-05-31 Thread Guenter Roeck
On Fri, May 31, 2013 at 01:08:52PM +0200, Johannes Thumshirn wrote: > This patch adds the driver for the watchdog devices found on MEN Mikro > Elektronik A21 VMEbus CPU Carrier Boards. It has DT-support and uses the > watchdog framework. > > Revision 2: > * Removed unneeded open flag in struct

[PATCH v6] watchdog: New watchdog driver for MEN A21 watchdogs

2013-05-31 Thread Johannes Thumshirn
This patch adds the driver for the watchdog devices found on MEN Mikro Elektronik A21 VMEbus CPU Carrier Boards. It has DT-support and uses the watchdog framework. Revision 2: * Removed unneeded open flag in struct a21_wdt_drv * Corrected 3bit reason code from gpio * Additional sysfs files are

[PATCH v6] watchdog: New watchdog driver for MEN A21 watchdogs

2013-05-31 Thread Johannes Thumshirn
This patch adds the driver for the watchdog devices found on MEN Mikro Elektronik A21 VMEbus CPU Carrier Boards. It has DT-support and uses the watchdog framework. Revision 2: * Removed unneeded open flag in struct a21_wdt_drv * Corrected 3bit reason code from gpio * Additional sysfs files are

Re: [PATCH v6] watchdog: New watchdog driver for MEN A21 watchdogs

2013-05-31 Thread Guenter Roeck
On Fri, May 31, 2013 at 01:08:52PM +0200, Johannes Thumshirn wrote: This patch adds the driver for the watchdog devices found on MEN Mikro Elektronik A21 VMEbus CPU Carrier Boards. It has DT-support and uses the watchdog framework. Revision 2: * Removed unneeded open flag in struct

Re: [PATCH v6] watchdog: New watchdog driver for MEN A21 watchdogs

2013-05-31 Thread Johannes Thumshirn
Hi Guenther, On Fri, May 31, 2013 at 04:40:37AM -0700, Guenter Roeck wrote: +#define GPIO_WD_ENAB 169 +#define GPIO_WD_FAST 170 +#define GPIO_WD_TRIG 171 + +#define GPIO_RST_CAUSE_BASE 166 + I think I asked that before ... as you are supporting devicetree, gpio pins

Re: [PATCH v6] watchdog: New watchdog driver for MEN A21 watchdogs

2013-05-31 Thread Guenter Roeck
On Fri, May 31, 2013 at 02:55:19PM +0200, Johannes Thumshirn wrote: Hi Guenther, On Fri, May 31, 2013 at 04:40:37AM -0700, Guenter Roeck wrote: +#define GPIO_WD_ENAB 169 +#define GPIO_WD_FAST 170 +#define GPIO_WD_TRIG 171 + +#define GPIO_RST_CAUSE_BASE 166 + I