[PATCH v1 3/6] watchdog: register watchdog virtual device with short name wdog
Sascha Hauer
s.hauer at pengutronix.de
Mon Mar 12 03:35:26 PDT 2018
Hi Oleksij,
On Thu, Mar 08, 2018 at 12:05:12PM +0100, Oleksij Rempel wrote:
> the watchdog hwdev is usually named with devicetree schema
> which is not practical for CLI.
>
> On device registration "wdog" will be extended with some index number
> extracted from devicetree (if awailable) or automatically assigned
> first available number. End result will be "wdog0" .. etc.
>
> Signed-off-by: Oleksij Rempel <o.rempel at pengutronix.de>
> ---
> drivers/watchdog/wd_core.c | 6 ++++++
> include/watchdog.h | 1 +
> 2 files changed, 7 insertions(+)
>
> diff --git a/drivers/watchdog/wd_core.c b/drivers/watchdog/wd_core.c
> index 1d0f0de7d..97e2ab568 100644
> --- a/drivers/watchdog/wd_core.c
> +++ b/drivers/watchdog/wd_core.c
> @@ -33,6 +33,11 @@ static const char *watchdog_name(struct watchdog *wd)
>
> int watchdog_register(struct watchdog *wd)
> {
> +
Please drop that blank line.
> + strcpy(wd->dev.name, "wdog");
I would prefer a of_alias_get() as first try and "wdog" only as
fallback.
For the hardcoded string wd->dev.id needs to be initialized with
DEVICE_ID_DYNAMIC, otherwise only a single watchdog can be successfully
registered.
> + wd->dev.parent = wd->hwdev;
> + register_device(&wd->dev);
Check return value.
Sascha
--
Pengutronix e.K. | |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
More information about the barebox
mailing list