Re: [PATCH v8] watchdog: ts4800: add driver for TS-4800 watchdog

2015-12-28 Thread Wim Van Sebroeck
;dev, > + "failed to register watchdog device\n"); > + return ret; > + } > + > + platform_set_drvdata(pdev, wdt); > + > + dev_info(>dev, > + "initialized (timeout = %d sec, nowayout = %d)\n", > + wdd->timeout, nowayout); > + > + return 0; > +} > + > +static int ts4800_wdt_remove(struct platform_device *pdev) > +{ > + struct ts4800_wdt *wdt = platform_get_drvdata(pdev); > + > + watchdog_unregister_device(>wdd); > + > + return 0; > +} > + > +static const struct of_device_id ts4800_wdt_of_match[] = { > + { .compatible = "technologic,ts4800-wdt", }, > + { }, > +}; > +MODULE_DEVICE_TABLE(of, ts4800_wdt_of_match); > + > +static struct platform_driver ts4800_wdt_driver = { > + .probe = ts4800_wdt_probe, > + .remove = ts4800_wdt_remove, > + .driver = { > + .name = "ts4800_wdt", > + .of_match_table = ts4800_wdt_of_match, > + }, > +}; > + > +module_platform_driver(ts4800_wdt_driver); > + > +MODULE_AUTHOR("Damien Riegel "); > +MODULE_LICENSE("GPL v2"); > +MODULE_ALIAS("platform:ts4800_wdt"); > -- > 2.5.0 > This driver has been added to linux-watchdog-next. Kind regards, Wim. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

Re: [PATCH v4 1/2] watchdog: imx2_wdt: add external reset support via 'ext-reset-output' dt prop

2015-12-28 Thread Wim Van Sebroeck
gt; > > >>> > > >>> Hi Guenter, > > >>> > > >>> I don't see why a vendor prefix is necessary - its a feature of the > > >>> IMX6 watchdog supported by this driver to be able to trigger an > > >>> internal chip-

Re: [PATCH v4] watchdog: add support for Sigma Designs SMP86xx/SMP87xx

2015-12-28 Thread Wim Van Sebroeck
P87xx watchdog registered\n"); > + > + return 0; > +} > + > +static int tangox_wdt_remove(struct platform_device *pdev) > +{ > + struct tangox_wdt_device *dev = platform_get_drvdata(pdev); > + > + tangox_wdt_stop(>wdt); > + clk_disable_unprepar

Re: [PATCH v2 0/5] watchdog: Do not use 'dev' from watchdog_device in watchdog drivers

2015-12-28 Thread Wim Van Sebroeck
or changed to use the parent device. > One driver sets the variable during initialization to the watchdog > driver's parent device, which is wrong and was removed. > > v2: Use parent device for dev_XXX messages instead of pr_XXX. This patchset has been added to linux-wat

Re: [PATCH] MAINTAINERS: Add Guenter Roeck as reviewer of watchdog drivers

2015-12-28 Thread Wim Van Sebroeck
/MAINTAINERS > index cba790b42f23..b1e3da7dc393 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -11602,6 +11602,7 @@ F:drivers/input/tablet/wacom_serial4.c > > WATCHDOG DEVICE DRIVERS > M: Wim Van Sebroeck <w...@iguana.be> > +R: Guenter Roeck <li...@roeck-us.ne

Re: [PATCH v4 1/2] watchdog: imx2_wdt: add external reset support via 'ext-reset-output' dt prop

2015-12-28 Thread Wim Van Sebroeck
prefix ? Also, not sure if "-output" > > >>>> has any real value in the property name. "fsl,external-reset", maybe ? > > >>> > > >>> > > >>> Hi Guenter, > > >>> > > >>> I don't see why

Re: [PATCH v8] watchdog: ts4800: add driver for TS-4800 watchdog

2015-12-28 Thread Wim Van Sebroeck
> + ret = watchdog_register_device(wdd); > + if (ret) { > + dev_err(>dev, > + "failed to register watchdog device\n"); > + return ret; > + } > + > + platform_set_drvdata(pdev, wdt); > + > + dev_info(>de

Re: [PATCH 09/10] watchdog: cadence_wdt: use to_platform_device()

2015-12-27 Thread Wim Van Sebroeck
truct platform_device, dev); > + struct platform_device *pdev = to_platform_device(dev); > struct cdns_wdt *wdt = platform_get_drvdata(pdev); > > ret = clk_prepare_enable(wdt->clk); > -- > 2.5.0 > > Patch added to linux-watchdog-next. Kind regards

Re: [PATCH] watchdog: dw_wdt: fix signedness bug in dw_wdt_top_in_seconds()

2015-12-27 Thread Wim Van Sebroeck
is 2 ^ (16 + i) and the watchdog counts down. >*/ > - return (1 << (16 + top)) / clk_get_rate(dw_wdt.clk); > + return (1U << (16 + top)) / clk_get_rate(dw_wdt.clk); > } > > static int dw_wdt_get_top(void) > -- > 2.6.4 > Patch is added

Re: [PATCH 2/2] sp5100_tco: Add AMD Carrizo platform support

2015-12-27 Thread Wim Van Sebroeck
> 1.9.1 > This patch has been added to linux-watchdog-next. Kind regards, Wim. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

Re: [PATCH 1/2] sp5100_tco: fix the device check for SB800 and later chipsets

2015-12-27 Thread Wim Van Sebroeck
PM_ACPI_MMIO_EN+0, SB800_IO_PM_INDEX_REG); > val = val << 8 | inb(SB800_IO_PM_DATA_REG); > - } else { > - /* Read SBResource_MMIO from PCI config(PCI_Reg: 9Ch) */ > - pci_read_config_dword(sp5100_tco_pci, > - SP5100_SB_RESO

Re: [PATCH v3] sp5100_tco: Add AMD Mullins platform support

2015-12-27 Thread Wim Van Sebroeck
/* End of list */ > }; > MODULE_DEVICE_TABLE(pci, sp5100_tco_pci_tbl); This patch has been added to linux-watchdog-next. Kind regards, Wim. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More ma

Re: [PATCH 1/2] sp5100_tco: fix the device check for SB800 and later chipsets

2015-12-27 Thread Wim Van Sebroeck
val = val << 8 | inb(SB800_IO_PM_DATA_REG); > outb(SB800_PM_ACPI_MMIO_EN+0, SB800_IO_PM_INDEX_REG); > val = val << 8 | inb(SB800_IO_PM_DATA_REG); > - } else { > - /* Read SBResource_MMIO from PCI config(PCI_Reg: 9Ch) */ > - pci_read_config_dword(sp

Re: [PATCH v3] sp5100_tco: Add AMD Mullins platform support

2015-12-27 Thread Wim Van Sebroeck
PCI_ANY_ID, }, > { 0, }, /* End of list */ > }; > MODULE_DEVICE_TABLE(pci, sp5100_tco_pci_tbl); This patch has been added to linux-watchdog-next. Kind regards, Wim. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of

Re: [PATCH 2/2] sp5100_tco: Add AMD Carrizo platform support

2015-12-27 Thread Wim Van Sebroeck
*/ > }; > MODULE_DEVICE_TABLE(pci, sp5100_tco_pci_tbl); > -- > 1.9.1 > This patch has been added to linux-watchdog-next. Kind regards, Wim. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.or

Re: [PATCH 09/10] watchdog: cadence_wdt: use to_platform_device()

2015-12-27 Thread Wim Van Sebroeck
v, > - struct platform_device, dev); > + struct platform_device *pdev = to_platform_device(dev); > struct cdns_wdt *wdt = platform_get_drvdata(pdev); > > ret = clk_prepare_enable(wdt->clk); > -- > 2.5.0 > > Patch added to linux-watchdog-nex

Re: [PATCH] watchdog: dw_wdt: fix signedness bug in dw_wdt_top_in_seconds()

2015-12-27 Thread Wim Van Sebroeck
he number of >* cycles is 2 ^ (16 + i) and the watchdog counts down. >*/ > - return (1 << (16 + top)) / clk_get_rate(dw_wdt.clk); > + return (1U << (16 + top)) / clk_get_rate(dw_wdt.clk); > } > > static int dw_wdt_get_top(void) > -- >

Re: [PATCH v8] watchdog: ts4800: add driver for TS-4800 watchdog

2015-12-23 Thread Wim Van Sebroeck
the phandle to the syscon node. > > > > Signed-off-by: Damien Riegel > > Acked-by: Rob Herring > > Reviewed-by: Guenter Roeck > > Hi Guenter, > > > You have reviewed this patch but not picked it up in your tree. Shall I > expect Wim to pick it up direc

Re: [PATCH v8] watchdog: ts4800: add driver for TS-4800 watchdog

2015-12-23 Thread Wim Van Sebroeck
the phandle to the syscon node. > > > > Signed-off-by: Damien Riegel <damien.rie...@savoirfairelinux.com> > > Acked-by: Rob Herring <r...@kernel.org> > > Reviewed-by: Guenter Roeck <li...@roeck-us.net> > > Hi Guenter, > > > You have rev

Re: [RFC/RFT PATCH] watchdog: Move watchdog device creation to watchdog_dev.c

2015-12-15 Thread Wim Van Sebroeck
Hi Pratyush, > On 15/12/2015:06:56:50 PM, Guenter Roeck wrote: > > On 12/14/2015 12:44 PM, Wim Van Sebroeck wrote: > > >On Sun, Dec 13, 2015 at 10:24:35PM -0800, Guenter Roeck wrote: > > > > > >>On 12/13/2015 02:02 PM, Damien Riegel wrote: > > >>

Re: [RFC/RFT PATCH] watchdog: Move watchdog device creation to watchdog_dev.c

2015-12-15 Thread Wim Van Sebroeck
Hi Guenter, > On 12/14/2015 12:44 PM, Wim Van Sebroeck wrote: > >On Sun, Dec 13, 2015 at 10:24:35PM -0800, Guenter Roeck wrote: > > > >>On 12/13/2015 02:02 PM, Damien Riegel wrote: > >>>On Mon, Dec 07, 2015 at 09:41:03PM +0100, Wim Van Sebroeck wrote: > >

Re: [RFC/RFT PATCH] watchdog: Move watchdog device creation to watchdog_dev.c

2015-12-15 Thread Wim Van Sebroeck
Hi Guenter, > On 12/14/2015 12:44 PM, Wim Van Sebroeck wrote: > >On Sun, Dec 13, 2015 at 10:24:35PM -0800, Guenter Roeck wrote: > > > >>On 12/13/2015 02:02 PM, Damien Riegel wrote: > >>>On Mon, Dec 07, 2015 at 09:41:03PM +0100, Wim Van Sebroeck wrote: > >

Re: [RFC/RFT PATCH] watchdog: Move watchdog device creation to watchdog_dev.c

2015-12-15 Thread Wim Van Sebroeck
Hi Pratyush, > On 15/12/2015:06:56:50 PM, Guenter Roeck wrote: > > On 12/14/2015 12:44 PM, Wim Van Sebroeck wrote: > > >On Sun, Dec 13, 2015 at 10:24:35PM -0800, Guenter Roeck wrote: > > > > > >>On 12/13/2015 02:02 PM, Damien Riegel wrote: > > >>

Re: [RFC/RFT PATCH] watchdog: Move watchdog device creation to watchdog_dev.c

2015-12-14 Thread Wim Van Sebroeck
On Sun, Dec 13, 2015 at 10:24:35PM -0800, Guenter Roeck wrote: > On 12/13/2015 02:02 PM, Damien Riegel wrote: > >On Mon, Dec 07, 2015 at 09:41:03PM +0100, Wim Van Sebroeck wrote: > >>Hi All, > >> > >>>On 12/07/2015 08:15 AM, Damien Riegel wrote: > &g

Re: [RFC/RFT PATCH] watchdog: Move watchdog device creation to watchdog_dev.c

2015-12-14 Thread Wim Van Sebroeck
On Sun, Dec 13, 2015 at 10:24:35PM -0800, Guenter Roeck wrote: > On 12/13/2015 02:02 PM, Damien Riegel wrote: > >On Mon, Dec 07, 2015 at 09:41:03PM +0100, Wim Van Sebroeck wrote: > >>Hi All, > >> > >>>On 12/07/2015 08:15 AM, Damien Riegel wrote: > &g

[PATCH v3] staging: gdm72xx: add userspace data struct

2015-12-11 Thread Wim de With
This fixes the sparse warnings about dereferencing a userspace pointer. Once I updated the sparse annotations, I noticed a bug in gdm_wimax_ioctl() where we pass a user space pointer to gdm_update_fsm() which dereferences it. I fixed this. Signed-off-by: Wim de With --- drivers/staging/gdm72xx

[PATCH v3] staging: gdm72xx: add userspace data struct

2015-12-11 Thread Wim de With
This fixes the sparse warnings about dereferencing a userspace pointer. Once I updated the sparse annotations, I noticed a bug in gdm_wimax_ioctl() where we pass a user space pointer to gdm_update_fsm() which dereferences it. I fixed this. Signed-off-by: Wim de With <nauxu...@wimdewith.

Re: [PATCH v2] staging: gdm72xx: add userspace data struct

2015-12-10 Thread Wim de With
On Thu, Dec 10, 2015 at 02:44:45PM +, One Thousand Gnomes wrote: > (except that you mean sizeof(struct fsm_s) and it doesn't compile at the > moment! Oops, sloppy mistake. > data_s can just be modified to be __user. All uses of it follow that > rule. What do you mean? The data still needs

[PATCH v2] staging: gdm72xx: add userspace data struct

2015-12-10 Thread Wim de With
This fixes the sparse warnings about dereferencing a userspace pointer. Once I updated the sparse annotations, I noticed a bug in gdm_wimax_ioctl() where we pass a user space pointer to gdm_update_fsm() which dereferences it. I fixed this. Signed-off-by: Wim de With --- drivers/staging/gdm72xx

Re: [PATCH] staging: gdm72xx: add userspace data struct

2015-12-10 Thread Wim de With
On 10-12-2015 10:37, Dan Carpenter wrote: > On Thu, Dec 10, 2015 at 10:11:12AM +0100, Wim de With wrote: >> @@ -482,8 +483,16 @@ static int gdm_wimax_ioctl(struct net_device *dev, >> struct ifreq *ifr, int cmd) >> /* NOTE: gdm_update

[PATCH] staging: gdm72xx: add userspace data struct

2015-12-10 Thread Wim de With
this, and am not sure what to do about it. Signed-off-by: Wim de With --- drivers/staging/gdm72xx/gdm_wimax.c | 17 + drivers/staging/gdm72xx/wm_ioctl.h | 7 ++- 2 files changed, 19 insertions(+), 5 deletions(-) diff --git a/drivers/staging/gdm72xx/gdm_wimax.c b/drivers/staging

Re: [PATCH] staging: gdm72xx: add userspace data struct

2015-12-10 Thread Wim de With
On 10-12-2015 10:37, Dan Carpenter wrote: > On Thu, Dec 10, 2015 at 10:11:12AM +0100, Wim de With wrote: >> @@ -482,8 +483,16 @@ static int gdm_wimax_ioctl(struct net_device *dev, >> struct ifreq *ifr, int cmd) >> /* NOTE: gdm_update

[PATCH] staging: gdm72xx: add userspace data struct

2015-12-10 Thread Wim de With
this, and am not sure what to do about it. Signed-off-by: Wim de With <nauxu...@wimdewith.com> --- drivers/staging/gdm72xx/gdm_wimax.c | 17 + drivers/staging/gdm72xx/wm_ioctl.h | 7 ++- 2 files changed, 19 insertions(+), 5 deletions(-) diff --git a/drivers/staging/gdm72xx/gdm_w

Re: [PATCH v2] staging: gdm72xx: add userspace data struct

2015-12-10 Thread Wim de With
On Thu, Dec 10, 2015 at 02:44:45PM +, One Thousand Gnomes wrote: > (except that you mean sizeof(struct fsm_s) and it doesn't compile at the > moment! Oops, sloppy mistake. > data_s can just be modified to be __user. All uses of it follow that > rule. What do you mean? The data still needs

[PATCH v2] staging: gdm72xx: add userspace data struct

2015-12-10 Thread Wim de With
This fixes the sparse warnings about dereferencing a userspace pointer. Once I updated the sparse annotations, I noticed a bug in gdm_wimax_ioctl() where we pass a user space pointer to gdm_update_fsm() which dereferences it. I fixed this. Signed-off-by: Wim de With <nauxu...@wimdewith.

[PATCH] staging: lustre: add __user attributes to llite/file.c

2015-12-08 Thread Wim de With
/file.c:2483:36:expected void const [noderef] * drivers/staging/lustre/lustre/llite/file.c:2483:36:got void * It simply casts pointers to __user pointers in most cases, and changes a few pointers to __user pointers. Signed-off-by: Wim de With --- drivers/staging/lustre/lustre/llite

[PATCH] staging: lustre: add __user attributes to llite/file.c

2015-12-08 Thread Wim de With
/file.c:2483:36:expected void const [noderef] * drivers/staging/lustre/lustre/llite/file.c:2483:36:got void * It simply casts pointers to __user pointers in most cases, and changes a few pointers to __user pointers. Signed-off-by: Wim de With <nauxu...@wimdewith.com> --- drivers/s

Re: [RFC/RFT PATCH] watchdog: Move watchdog device creation to watchdog_dev.c

2015-12-07 Thread Wim Van Sebroeck
sysfs code in watchdog_dev.c . It belongs in watchdog_sysfs.c imho. If someone has a better idea, I'll be glad to listen to it and see what the benefits are. But I want a clean system for excluding both /dev/ (current watchdog_dev.c) and/or sysfs (watchdog_sysfs.c) in the future. Off-course the current behaviour is to have the /dev/ interface and have the option to add sysfs attributes. Kind regards, Wim. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

Re: [RFC/RFT PATCH] watchdog: Move watchdog device creation to watchdog_dev.c

2015-12-07 Thread Wim Van Sebroeck
fdef-ery that becomes unreadable). So I do not agree to have sysfs code in watchdog_dev.c . It belongs in watchdog_sysfs.c imho. If someone has a better idea, I'll be glad to listen to it and see what the benefits are. But I want a clean system for excluding both /dev/ (current watchdog_de

[GIT PULL REQUEST] watchdog - v4.4-rc3 Fixes

2015-11-27 Thread Wim Van Sebroeck
watchdog mode. Add it to mtk_wdt_stop function, so that the watchdog can be stopped (e.g. during suspend). Signed-off-by: Nicolas Boichat Acked-by: Matthias Brugger Reviewed-by: Guenter Roeck Signed-off-by: Wim Van Sebroeck commit

[GIT PULL REQUEST] watchdog - v4.4-rc3 Fixes

2015-11-27 Thread Wim Van Sebroeck
Guenter Roeck <li...@roeck-us.net> Signed-off-by: Wim Van Sebroeck <w...@iguana.be> commit 646251a59c73253a7a2c7b1d6adbedd2292390a0 Author: Shaohui Xie <shaohui@freescale.com> Date: Tue Nov 17 17:53:19 2015 +0800 watchdog: Add support for Freescale Layerscape platforms

[GIT PULL REQUEST] watchdog - v4.4 Merge window

2015-11-09 Thread Wim Van Sebroeck
values in kerneldoc Signed-off-by: Wolfram Sang Reviewed-by: Guenter Roeck Signed-off-by: Wim Van Sebroeck commit 80220fa72b917c64675f3ba4008d2c5a7b50b281 Author: Wolfram Sang Date: Tue Nov 3 09:00:15 2015 +0100 watchdog: include: fix some typos Signed-off

[GIT PULL REQUEST] watchdog - v4.4 Merge window

2015-11-09 Thread Wim Van Sebroeck
tchdog: include: add units for timeout values in kerneldoc Signed-off-by: Wolfram Sang <wsa+rene...@sang-engineering.com> Reviewed-by: Guenter Roeck <li...@roeck-us.net> Signed-off-by: Wim Van Sebroeck <w...@iguana.be> commit 80220fa72b917c64675f3ba4008d2c5a7b50b281 A

Re: [PATCH] watchdog_dev: Use device tree alias for naming watchdogs

2015-10-27 Thread Wim Van Sebroeck
possible */ > + if (wdd->parent) { > + ret = of_alias_get_id(wdd->parent->of_node, "watchdog"); > + if (ret >= 0) > + id = ida_simple_get(_ida, ret, > + ret + 1, GFP_KERNEL); > + } > + > + if

Re: [PATCH v3 0/2] watchdog: driver for BCM7038 and newer chips.

2015-10-27 Thread Wim Van Sebroeck
100644 drivers/watchdog/bcm7038_wdt.c Patches added to linux-watchdog-next. Kind regards, Wim. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

Re: [PATCH] watchdog_dev: Use device tree alias for naming watchdogs

2015-10-27 Thread Wim Van Sebroeck
id if possible */ > + if (wdd->parent) { > + ret = of_alias_get_id(wdd->parent->of_node, "watchdog"); > + if (ret >= 0) > + id = ida_simple_get(_ida, ret, > + ret + 1, GFP_KERNEL); >

Re: [PATCH v3 0/2] watchdog: driver for BCM7038 and newer chips.

2015-10-27 Thread Wim Van Sebroeck
100644 drivers/watchdog/bcm7038_wdt.c Patches added to linux-watchdog-next. Kind regards, Wim. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

[GIT PULL REQUEST] watchdog - v4.3-rc3 Fixes

2015-09-30 Thread Wim Van Sebroeck
point") Reviewed-by: Matt Fleming Cc: Lee Jones Signed-off-by: Guenter Roeck Signed-off-by: Wim Van Sebroeck commit 898e6861ff7cfc9f539b57859a27fbd1fe4298ae Author: Noralf Trønnes Date: Wed Jun 17 16:04:04 2015 +0200 watchdog: bcm2835: Fix poweroff behaviour

[GIT PULL REQUEST] watchdog - v4.3-rc3 Fixes

2015-09-30 Thread Wim Van Sebroeck
CO on Intel Sunrisepoint") Reviewed-by: Matt Fleming <matt.flem...@intel.com> Cc: Lee Jones <lee.jo...@linaro.org> Signed-off-by: Guenter Roeck <li...@roeck-us.net> Signed-off-by: Wim Van Sebroeck <w...@iguana.be> commit 898e6861ff7cfc9f539b57859a27

Re: [GIT PULL REQUEST] watchdog - v4.3-rc1 merge window

2015-09-10 Thread Wim Van Sebroeck
tal: read error: Connection reset by peer > > Sorry for not mentioning it earlier. The tree hasn't changed since 1.5 weeks, so on content level nothing has changed. Thanks, Wim. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message

Re: [GIT PULL REQUEST] watchdog - v4.3-rc1 merge window

2015-09-10 Thread Wim Van Sebroeck
ull. I just restarted the git-daemon. You should be able to pull again. Thanks for reporting it. Kind regards, Wim. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org

Re: [GIT PULL REQUEST] watchdog - v4.3-rc1 merge window

2015-09-10 Thread Wim Van Sebroeck
ot just when trying to pull. I just restarted the git-daemon. You should be able to pull again. Thanks for reporting it. Kind regards, Wim. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More major

Re: [GIT PULL REQUEST] watchdog - v4.3-rc1 merge window

2015-09-10 Thread Wim Van Sebroeck
tal: read error: Connection reset by peer > > Sorry for not mentioning it earlier. The tree hasn't changed since 1.5 weeks, so on content level nothing has changed. Thanks, Wim. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message

[GIT PULL REQUEST] watchdog - v4.3-rc1 merge window

2015-09-09 Thread Wim Van Sebroeck
-by: Lubomir Rintel Acked-by: Maxime Coquelin Acked-by: Thierry Reding Acked-by: Viresh Kumar Acked-by: Linus Walleij Signed-off-by: Wim Van Sebroeck commit bf5125d5e0759a0f513b1bcd33c15edc0cf4c17b Author: Bjorn Andersson Date: Mon Aug 17 09:19:03 2015 -0700 watchdog

[GIT PULL REQUEST] watchdog - v4.3-rc1 merge window

2015-09-09 Thread Wim Van Sebroeck
.@nvidia.com> Acked-by: Viresh Kumar <viresh.ku...@linaro.org> Acked-by: Linus Walleij <linus.wall...@linaro.org> Signed-off-by: Wim Van Sebroeck <w...@iguana.be> commit bf5125d5e0759a0f513b1bcd33c15edc0cf4c17b Author: Bjorn Andersson <bjorn.anders...@sonym

[GIT PULL REQUEST] watchdog - v4.2 merge window

2015-07-01 Thread Wim Van Sebroeck
not change the behavior if the parameter is not given. Signed-off-by: Lars Poeschel Reviewed-by: Guenter Roeck Signed-off-by: Wim Van Sebroeck commit 5e53c8ed813d6100d2cebbf83f2192b201f70772 Author: Jean-Baptiste Theou Date: Tue Jun 9 09:55:03 2015 -0700 watchdog: gpio_wdt

[GIT PULL REQUEST] watchdog - v4.2 merge window

2015-07-01 Thread Wim Van Sebroeck
- which also does not change the behavior if the parameter is not given. Signed-off-by: Lars Poeschel poesc...@lemonage.de Reviewed-by: Guenter Roeck li...@roeck-us.net Signed-off-by: Wim Van Sebroeck w...@iguana.be commit 5e53c8ed813d6100d2cebbf83f2192b201f70772 Author: Jean

Re: [PATCH v2] watchdog: omap_wdt: early_enable module parameter

2015-06-30 Thread Wim Van Sebroeck
sskb.ellerman.id.au/kisskb/buildresult/12452335/ > > I see a patch has already been posted to arm-kernel but not > Cc'd to all originally involved so figured I'd mention it. It's fixed. The reason why we din't catch it was because of interference with 'watchdog: omap: put struct watchdog

Re: [PATCH v2] watchdog: omap_wdt: early_enable module parameter

2015-06-30 Thread Wim Van Sebroeck
but not Cc'd to all originally involved so figured I'd mention it. It's fixed. The reason why we din't catch it was because of interference with 'watchdog: omap: put struct watchdog_device into driver data'. Kind regards, Wim. -- To unsubscribe from this list: send the line unsubscribe linux-kernel

[PATCH] staging: dgnc: fix line length over 80 chars in dgnc_sysfs.c

2015-05-20 Thread Wim de With
This patch fixes most of the lines over 80 characters long in dgnc_sysfs.c. I couldn't find a way to break line 202-207 in a sensible way. If there is a way, let me know. Signed-off-by: Wim de With --- drivers/staging/dgnc/dgnc_sysfs.c | 110 +- 1 file

[PATCH] staging: dgnc: fix line length over 80 chars in dgnc_sysfs.c

2015-05-20 Thread Wim de With
This patch fixes most of the lines over 80 characters long in dgnc_sysfs.c. I couldn't find a way to break line 202-207 in a sensible way. If there is a way, let me know. Signed-off-by: Wim de With nauxu...@wimdewith.com --- drivers/staging/dgnc/dgnc_sysfs.c | 110

Re: [PATCH] Staging: unisys: fix function declaration format in visorchipset.c

2015-05-18 Thread Wim de With
On 18-5-2015 12:45, Jes Sorensen wrote: > Wim de With writes: >> This is a patch that fixes the function declarations in >> visorbus/visorchipset.c by removing newlines after the function return >> type > > This patch doesn't fix things, it makes things worse! >

[PATCH] Staging: unisys: fix function declaration format in visorchipset.c

2015-05-18 Thread Wim de With
This is a patch that fixes the function declarations in visorbus/visorchipset.c by removing newlines after the function return type Signed-off-by: Wim de With --- drivers/staging/unisys/visorbus/visorchipset.c | 225 ++--- 1 file changed, 85 insertions(+), 140 deletions

Re: [PATCH] Staging: unisys: fix function declaration format in visorchipset.c

2015-05-18 Thread Wim de With
On 18-5-2015 12:45, Jes Sorensen wrote: Wim de With nauxu...@wimdewith.com writes: This is a patch that fixes the function declarations in visorbus/visorchipset.c by removing newlines after the function return type This patch doesn't fix things, it makes things worse! If you want to post

[PATCH] Staging: unisys: fix function declaration format in visorchipset.c

2015-05-18 Thread Wim de With
This is a patch that fixes the function declarations in visorbus/visorchipset.c by removing newlines after the function return type Signed-off-by: Wim de With nauxu...@wimdewith.com --- drivers/staging/unisys/visorbus/visorchipset.c | 225 ++--- 1 file changed, 85 insertions

[GIT PULL REQUEST] watchdog - v4.1 merge window

2015-04-22 Thread Wim Van Sebroeck
address is not valid anymore, redirect people to the Pengutronix kernel team. Reported-by: Harald Geyer Signed-off-by: Wolfram Sang Acked-by: Robert Schwebel Signed-off-by: Wim Van Sebroeck commit e8cc53665728860a56dbbd56bfc32ac934efb70a Author: Wolfram Sang Date

[GIT PULL REQUEST] watchdog - v4.1 merge window

2015-04-22 Thread Wim Van Sebroeck
My Pengutronix address is not valid anymore, redirect people to the Pengutronix kernel team. Reported-by: Harald Geyer har...@ccbib.org Signed-off-by: Wolfram Sang w...@the-dreams.de Acked-by: Robert Schwebel r.schwe...@pengutronix.de Signed-off-by: Wim Van Sebroeck w

Re: [PATCH 21/27] watchdog: bcm281xx: Remove use of seq_printf return value

2015-03-27 Thread Wim Van Sebroeck
"Current: %d s / %d (%#x) ticks\n" > +"Busy count: %lu\n", > +res, wdt->resolution, > + ctl_sec, ctl, ctl, > +cur_sec, cur, cur, > +wdt->busy

[GIT PULL REQUEST] watchdog - v4.0-rc5 Fixes

2015-03-27 Thread Wim Van Sebroeck
an Cc: Ezequiel Garcia Cc: Naidu Tellapati Cc: Jude Abraham Cc: linux-watch...@vger.kernel.org Reviewed-by: Guenter Roeck Signed-off-by: Wim Van Sebroeck commit a629c08fdb98ebb184d745553af9dda4f05941bf Author: James Hogan Date: Fri Feb 20 23:45:44 2015 + watchd

Re: [PATCH] watchdog: cadence: Remove Kconfig dependency on ARCH

2015-03-27 Thread Wim Van Sebroeck
;Cadence Watchdog Timer" > - depends on ARM > select WATCHDOG_CORE > help > Say Y here if you want to include support for the watchdog This patch has been added to linux-watchdog-next. Kind regards, Wim. -- To unsubscribe from this list: send the line "

Re: [PATCH 0/3] arm: msm: Use timer DT node for qcom watchdog config

2015-03-27 Thread Wim Van Sebroeck
++-- > 3 files changed, 41 insertions(+), 10 deletions(-) This patchset has been added to linux-watchdog-next. Kind regards, Wim. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More m

Re: [PATCH 21/27] watchdog: bcm281xx: Remove use of seq_printf return value

2015-03-27 Thread Wim Van Sebroeck
*file) -- 2.1.2 This patch was applied to linux-watchdog-next. Kind regards, Wim. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ

Re: [PATCH 0/3] arm: msm: Use timer DT node for qcom watchdog config

2015-03-27 Thread Wim Van Sebroeck
(-) This patchset has been added to linux-watchdog-next. Kind regards, Wim. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http

[GIT PULL REQUEST] watchdog - v4.0-rc5 Fixes

2015-03-27 Thread Wim Van Sebroeck
-by: James Hogan james.ho...@imgtec.com Cc: Ezequiel Garcia ezequiel.gar...@imgtec.com Cc: Naidu Tellapati naidu.tellap...@imgtec.com Cc: Jude Abraham jude.abra...@imgtec.com Cc: linux-watch...@vger.kernel.org Reviewed-by: Guenter Roeck li...@roeck-us.net Signed-off-by: Wim Van

Re: [PATCH] watchdog: cadence: Remove Kconfig dependency on ARCH

2015-03-27 Thread Wim Van Sebroeck
WATCHDOG_CORE help Say Y here if you want to include support for the watchdog This patch has been added to linux-watchdog-next. Kind regards, Wim. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More

[GIT PULL REQUEST] watchdog - v3.20 or 4.00 Merge window

2015-02-17 Thread Wim Van Sebroeck
the same SoC bus driver (bcma) as BCM47XX but based on ARM instead of MIPS. Signed-off-by: Rafał Miłecki Reviewed-by: Guenter Roeck Signed-off-by: Wim Van Sebroeck commit 6b96c72279cd73c1a03e97265548ce067128203a Author: Zubair Lutfullah Kakakhel Date: Tue Feb 3 10:25:48 2015

[GIT PULL REQUEST] watchdog - v3.20 or 4.00 Merge window

2015-02-17 Thread Wim Van Sebroeck
architecture using the same SoC bus driver (bcma) as BCM47XX but based on ARM instead of MIPS. Signed-off-by: Rafał Miłecki zaj...@gmail.com Reviewed-by: Guenter Roeck li...@roeck-us.net Signed-off-by: Wim Van Sebroeck w...@iguana.be commit 6b96c72279cd73c1a03e97265548ce067128203a

Re: [PATCH_V3 0/2] watchdog: jz4740: Add DT support

2015-02-03 Thread Wim Van Sebroeck
ndings/watchdog/ingenic,jz4740-wdt.txt | 12 > > drivers/watchdog/jz4740_wdt.c| 10 ++ > 2 files changed, 22 insertions(+) > create mode 100644 > Documentation/devicetree/bindings/watchdog/ingenic,jz4740-wdt.txt Both patches have been

Re: [PATCH v3 2/2] watchdog: dw_wdt: Try to get a 30 second watchdog by default

2015-02-03 Thread Wim Van Sebroeck
e considered a bit of a stopgap solution. > > Signed-off-by: Doug Anderson > Reviewed-by: Guenter Roeck This one has also been added to linux-watchdog-next. Kind regards, Wim. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a

Re: [PATCH v3 1/2] watchdog: dw_wdt: pat the watchdog before enabling it

2015-02-03 Thread Wim Van Sebroeck
he watchdog manually. > > Signed-off-by: Doug Anderson Patch has been added to linux-watchdog-next. Kind regards, Wim. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

Re: [PATCH] watchdog: w83627hf_wdt: Add support for NCT6791 and NCT6792

2015-02-03 Thread Wim Van Sebroeck
Hi Guenter, > The watchdog functionality in both chips is almost identical to NCT6779. > > Signed-off-by: Guenter Roeck Patch has been added to linux-watchdog-next. Thanks for all the other reviews again. Kind regards, Wim. -- To unsubscribe from this list: send the line "uns

Re: [PATCH v4] gpio_wdt: Add "always_running" feature to GPIO watchdog

2015-02-03 Thread Wim Van Sebroeck
armed" member keeps > track of whether a timeout must also cause a reset. > > Signed-off-by: Mike Looijmans Patch has been added to linux-watchdog-next. Kind regards, Wim. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to maj

Re: [PATCH v4] gpio_wdt: Add always_running feature to GPIO watchdog

2015-02-03 Thread Wim Van Sebroeck
a timeout must also cause a reset. Signed-off-by: Mike Looijmans mike.looijm...@topic.nl Patch has been added to linux-watchdog-next. Kind regards, Wim. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info

Re: [PATCH] watchdog: w83627hf_wdt: Add support for NCT6791 and NCT6792

2015-02-03 Thread Wim Van Sebroeck
Hi Guenter, The watchdog functionality in both chips is almost identical to NCT6779. Signed-off-by: Guenter Roeck li...@roeck-us.net Patch has been added to linux-watchdog-next. Thanks for all the other reviews again. Kind regards, Wim. -- To unsubscribe from this list: send the line

Re: [PATCH_V3 0/2] watchdog: jz4740: Add DT support

2015-02-03 Thread Wim Van Sebroeck
| 10 ++ 2 files changed, 22 insertions(+) create mode 100644 Documentation/devicetree/bindings/watchdog/ingenic,jz4740-wdt.txt Both patches have been added to linux-watchdog-next. Kind regards, Wim. -- To unsubscribe from this list: send the line unsubscribe

Re: [PATCH v3 2/2] watchdog: dw_wdt: Try to get a 30 second watchdog by default

2015-02-03 Thread Wim Van Sebroeck
Reviewed-by: Guenter Roeck li...@roeck-us.net This one has also been added to linux-watchdog-next. Kind regards, Wim. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org

Re: [PATCH v3 1/2] watchdog: dw_wdt: pat the watchdog before enabling it

2015-02-03 Thread Wim Van Sebroeck
to linux-watchdog-next. Kind regards, Wim. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

Re: [PATCH v5 0/2] ARM: mediatek: Add watchdog driver

2015-02-02 Thread Wim Van Sebroeck
k-wdt.txt > create mode 100644 drivers/watchdog/mtk_wdt.c > Both patches have been added to linux-watchdog-next. I made a small change: I removed the .owner field in the platform_driver struct. This is now normally set by the core. Kind regards, Wim. -- To unsubscribe from this list: sen

Re: [PATCH] watchdog: hpwdt: Fix initialization message in hpwdt.c

2015-02-02 Thread Wim Van Sebroeck
Timer Driver: NMI decoding initialized, > allow kernel dump: ON (default = 0/OFF)<= > > This "default = 0/OFF" message may confuse users. > Fix it as "default = 1/ON". > > Signed-off-by: Masanari Iida Patch has been added to linux-watchdog-next.

Re: [PATCH] watchdog: imx2: Constify struct regmap_config and watchdog_ops

2015-02-02 Thread Wim Van Sebroeck
Hi Krzysztof, > The regmap_config struct may be const because it is not modified by the > driver and regmap_init() accepts pointer to const. Make struct > watchdog_ops const as well. > > Signed-off-by: Krzysztof Kozlowski Patch added to linux-watchdog-next. Ki

Re: [PATCH] watchdog: imx2: Constify struct regmap_config and watchdog_ops

2015-02-02 Thread Wim Van Sebroeck
Hi Krzysztof, The regmap_config struct may be const because it is not modified by the driver and regmap_init() accepts pointer to const. Make struct watchdog_ops const as well. Signed-off-by: Krzysztof Kozlowski k.kozlow...@samsung.com Patch added to linux-watchdog-next. Kind regards, Wim

Re: [PATCH v5 0/2] ARM: mediatek: Add watchdog driver

2015-02-02 Thread Wim Van Sebroeck
the .owner field in the platform_driver struct. This is now normally set by the core. Kind regards, Wim. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH] watchdog: hpwdt: Fix initialization message in hpwdt.c

2015-02-02 Thread Wim Van Sebroeck
initialized, allow kernel dump: ON (default = 0/OFF)= This default = 0/OFF message may confuse users. Fix it as default = 1/ON. Signed-off-by: Masanari Iida standby2...@gmail.com Patch has been added to linux-watchdog-next. Kind regards, Wim. -- To unsubscribe from this list: send the line

[GIT PULL REQUEST] watchdog - v3.19-rc5 Fixes

2015-01-24 Thread Wim Van Sebroeck
assignment from platform_drivers This platform_driver does not need to set an owner, it will be populated by the driver core. Signed-off-by: Wolfram Sang Signed-off-by: Wim Van Sebroeck commit 5fe65ce7ccbb47b16e17a88bcdac73cffadb80fa Author: Markus Pargmann Date: Mon Sep 8

[GIT PULL REQUEST] watchdog - v3.19-rc5 Fixes

2015-01-24 Thread Wim Van Sebroeck
: drop owner assignment from platform_drivers This platform_driver does not need to set an owner, it will be populated by the driver core. Signed-off-by: Wolfram Sang w...@the-dreams.de Signed-off-by: Wim Van Sebroeck w...@iguana.be commit

Re: linux-next: error while fetching the watchdog tree

2014-12-28 Thread Wim Van Sebroeck
orry for the delay (was abroad), but should be fixed now. Kind regards, Wim. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

Re: linux-next: error while fetching the watchdog tree

2014-12-28 Thread Wim Van Sebroeck
), but should be fixed now. Kind regards, Wim. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/

Re: [PATCH 22/28] watchdog: drop owner assignment from platform_drivers

2014-12-22 Thread Wim Van Sebroeck
Hi Wolfram, > This platform_driver does not need to set an owner, it will be populated by > the > driver core. > > Signed-off-by: Wolfram Sang Added to linux-watchdog-next. Kind regards, Wim. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel"

Re: [PATCH 22/28] watchdog: drop owner assignment from platform_drivers

2014-12-22 Thread Wim Van Sebroeck
Hi Wolfram, This platform_driver does not need to set an owner, it will be populated by the driver core. Signed-off-by: Wolfram Sang w...@the-dreams.de Added to linux-watchdog-next. Kind regards, Wim. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body

[GIT PULL REQUEST] watchdog - v3.19 merge window

2014-12-18 Thread Wim Van Sebroeck
drivers/watchdog/imx2_wdt.c:38:0: include/linux/watchdog.h:104:20: note: expected 'struct watchdog_device *' but argument is of type 'struct watchdog_device **' Reported-by: Olof's autobuilder Signed-off-by: Fabio Estevam Reviewed-by: Guenter Roeck Signed-off-by: Wim Van Sebroeck

<    1   2   3   4   5   6   7   8   9   10   >