From: Wolfram Sang <w.sang@pengutronix.de>
To: Sascha Hauer <s.hauer@pengutronix.de>
Cc: linux-kernel@vger.kernel.org,
Samuel Ortiz <sameo@linux.intel.com>,
Jean Delvare <khali@linux-fr.org>,
linux-i2c@vger.kernel.org, linux-watchdog@vger.kernel.org,
Ben Dooks <ben-linux@fluff.org>,
kernel@pengutronix.de
Subject: Re: [PATCH 3/3] watchdog: Add Congatec CGEB watchdog driver
Date: Tue, 7 Feb 2012 16:08:56 +0100 [thread overview]
Message-ID: <20120207150856.GI2539@pengutronix.de> (raw)
In-Reply-To: <1328102793-4313-4-git-send-email-s.hauer@pengutronix.de>
[-- Attachment #1: Type: text/plain, Size: 3028 bytes --]
Hi Sascha,
> +struct cgeb_watchdog_stage {
> + unsigned long timeout;
> + unsigned long event;
> +};
> +
> +struct cgeb_watchdog_config {
> + unsigned long size;
> + unsigned long timeout; /* not used in staged mode */
> + unsigned long delay;
> + unsigned long mode;
> + /* optional parameters for staged watchdog */
> + unsigned long op_mode;
> + unsigned long stage_count;
> + struct cgeb_watchdog_stage stages[CGOS_WDOG_EVENT_MAX_STAGES];
> +};
> +
There is some unused stuff in here.
> +struct cgeb_watchdog_priv {
> + struct cgeb_board_data *board;
> + struct watchdog_device wdd;
> + unsigned int timeout_s;
Is wdd->timeout not sufficent?
> + int unit;
> +};
> +
> +static int cgeb_watchdog_set_timeout(struct watchdog_device *wdd,
> + unsigned int timeout_s)
> +{
> + struct cgeb_watchdog_priv *priv = watchdog_get_drvdata(wdd);
> +
> + if (!timeout_s)
> + return -EINVAL;
Is this possible? You have min_timeout = 1.
> +static int __devinit cgeb_watchdog_probe(struct platform_device *pdev)
> +{
> + struct cgeb_watchdog_priv *priv;
> + struct cgeb_pdata *pdata = pdev->dev.platform_data;
> + int ret;
> +
> + dev_info(&pdev->dev, "registering\n");
"registered" on success would be more useful?
> +
> + priv = devm_kzalloc(&pdev->dev, sizeof(*priv), GFP_KERNEL);
> + if (!priv)
> + return -ENOMEM;
> +
> + priv->wdd.ops = &cgeb_watchdog_ops;
> + priv->wdd.info = &cgeb_wdd_info;
> + priv->wdd.min_timeout = 1;
> + priv->wdd.max_timeout = 3600;
> + priv->board = pdata->board;
> + priv->unit = pdata->unit;
> +
> + watchdog_set_drvdata(&priv->wdd, priv);
> + platform_set_drvdata(pdev, priv);
> +
> + ret = watchdog_register_device(&priv->wdd);
> + if (ret)
> + return ret;
> +
> + return 0;
return watchdog_register_device();
> +}
> +
> +static int __devexit cgeb_watchdog_remove(struct platform_device *pdev)
> +{
> + struct cgeb_watchdog_priv *priv = platform_get_drvdata(pdev);
> +
> + watchdog_unregister_device(&priv->wdd);
> +
> + return 0;
> +}
> +
> +static struct platform_driver cgeb_watchdog_driver = {
> + .probe = cgeb_watchdog_probe,
> + .remove = __exit_p(cgeb_watchdog_remove),
__devexit_p
> + .driver = {
> + .name = "cgeb-watchdog",
> + .owner = THIS_MODULE,
> + },
> +};
> +
> +static int __init cgeb_watchdog_driver_init(void)
> +{
> + return platform_driver_register(&cgeb_watchdog_driver);
> +}
> +
> +static void __exit cgeb_watchdog_driver_exit(void)
> +{
> + platform_driver_unregister(&cgeb_watchdog_driver);
> +}
> +
> +module_init(cgeb_watchdog_driver_init);
> +module_exit(cgeb_watchdog_driver_exit);
module_platform_driver()
> +
> +MODULE_AUTHOR("Sascha Hauer <s.hauer@pengutronix.de>");
> +MODULE_DESCRIPTION("cgeb watchdog driver");
> +MODULE_LICENSE("GPL");
GPL v2 according to header
--
Pengutronix e.K. | Wolfram Sang |
Industrial Linux Solutions | http://www.pengutronix.de/ |
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 198 bytes --]
next prev parent reply other threads:[~2012-02-07 15:09 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-02-01 13:26 [RESEND] Congatec CGEB base, i2c and watchdog driver support Sascha Hauer
2012-02-01 13:26 ` [PATCH 1/3] mfd: Add basic support for the Congatec CGEB BIOS interface Sascha Hauer
2012-02-06 19:07 ` Christian Gmeiner
2012-02-20 16:39 ` Samuel Ortiz
2012-02-20 18:07 ` Sascha Hauer
2012-02-27 13:58 ` Samuel Ortiz
2012-02-27 17:24 ` Sascha Hauer
2012-02-28 14:32 ` Mark Brown
2012-02-01 13:26 ` [PATCH 2/3] i2c: Add Congatec CGEB I2C driver Sascha Hauer
2012-02-07 8:38 ` Christian Gmeiner
2012-02-07 10:10 ` Wolfram Sang
2012-02-15 13:54 ` Sascha Hauer
2012-02-13 23:22 ` Ben Dooks
2012-02-01 13:26 ` [PATCH 3/3] watchdog: Add Congatec CGEB watchdog driver Sascha Hauer
2012-02-07 15:08 ` Wolfram Sang [this message]
2012-02-15 13:41 ` Sascha Hauer
2012-02-06 10:58 ` [RESEND] Congatec CGEB base, i2c and watchdog driver support Christian Gmeiner
-- strict thread matches above, loose matches on Subject: below --
2013-02-12 10:02 [PATCH] " Sascha Hauer
2013-02-12 10:02 ` [PATCH 3/3] watchdog: Add Congatec CGEB watchdog driver Sascha Hauer
2013-03-26 10:16 ` Christian Gmeiner
2013-03-26 14:33 ` Guenter Roeck
2013-04-03 15:09 ` Sascha Hauer
2013-04-03 15:30 ` Guenter Roeck
2013-04-03 17:06 ` Sascha Hauer
2013-04-03 15:16 ` Sascha Hauer
2012-01-16 15:32 [RFC] Congatec CGEB base, i2c and watchdog driver support Sascha Hauer
2012-01-16 15:32 ` [PATCH 3/3] watchdog: Add Congatec CGEB watchdog driver Sascha Hauer
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20120207150856.GI2539@pengutronix.de \
--to=w.sang@pengutronix.de \
--cc=ben-linux@fluff.org \
--cc=kernel@pengutronix.de \
--cc=khali@linux-fr.org \
--cc=linux-i2c@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-watchdog@vger.kernel.org \
--cc=s.hauer@pengutronix.de \
--cc=sameo@linux.intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).