Re: add bcm47xx watchdog driver

[matthieu castet]

Hi,

Wim Van Sebroeck wrote:
> Hi Matthieu, Florian,
>
> > It works very well on my Netgear WGT634U, thanks !
> >
> > Tested-by: Florian Fainelli <florian@xxxxxxxxxxx>
>
> I incorporated the changes of Andrew and did a clean-up here and there to keep checkpatch happy.
> Could you test the attached diff to see that it still works.
>
> Thanks,
> Wim.

> +
> +#define WDT_DEFAULT_TIME	30	/* seconds */
> +#define WDT_MAX_TIME		255	/* seconds */
Why have changed this from 256 to 255 ?

> +}
> +
> +static int bcm47xx_wdt_notify_sys(struct notifier_block *this,
> +	nsigned long code, void *unused)
       ^^^^
Does this build ?


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