On Wed, Oct 25, 2017 at 04:39:13PM +0100, Radu Rendec wrote:
> The minimum, maximum and default values for the watchdog heartbeat
> (timeout) were hardcoded in several places (including module parameter
> description and warning message for invalid module parameter value).
>
> This patch adds macros for the aforementioned values and replaces all
> occurences of hardcoded values by these macros.
>
> Signed-off-by: Radu Rendec <rrendec@xxxxxxxxxx>
> ---
> drivers/watchdog/i6300esb.c | 20 ++++++++++++--------
> 1 file changed, 12 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/watchdog/i6300esb.c b/drivers/watchdog/i6300esb.c
> index 0b4489661a82..3f7f66513dfa 100644
> --- a/drivers/watchdog/i6300esb.c
> +++ b/drivers/watchdog/i6300esb.c
> @@ -81,12 +81,16 @@ static int cards_found;
>
> /* module parameters */
> /* 30 sec default heartbeat (1 < heartbeat < 2*1023) */
> -#define WATCHDOG_HEARTBEAT 30
> +#define ESB_HEARTBEAT_MIN 1
> +#define ESB_HEARTBEAT_MAX 2046
> +#define ESB_HEARTBEAT_DEFAULT 30
Please use tabs above.
#define ESB_HEARTBEAT_MIN 1
#define ESB_HEARTBEAT_MAX 2046
#define ESB_HEARTBEAT_DEFAULT 30
> +#define ESB_HEARTBEAT_RANGE __MODULE_STRING(ESB_HEARTBEAT_MIN) \
> + "<heartbeat<" __MODULE_STRING(ESB_HEARTBEAT_MAX)
> static int heartbeat; /* in seconds */
> module_param(heartbeat, int, 0);
> MODULE_PARM_DESC(heartbeat,
> - "Watchdog heartbeat in seconds. (1<heartbeat<2046, default="
> - __MODULE_STRING(WATCHDOG_HEARTBEAT) ")");
> + "Watchdog heartbeat in seconds. (" ESB_HEARTBEAT_RANGE
> + ", default=" __MODULE_STRING(ESB_HEARTBEAT_DEFAULT) ")");
>
> static bool nowayout = WATCHDOG_NOWAYOUT;
> module_param(nowayout, bool, 0);
> @@ -326,13 +330,13 @@ static int esb_probe(struct pci_dev *pdev,
> /* Initialize the watchdog and make sure it does not run */
> edev->wdd.info = &esb_info;
> edev->wdd.ops = &esb_ops;
> - edev->wdd.min_timeout = 1;
> - edev->wdd.max_timeout = 2 * 0x03ff;
> - edev->wdd.timeout = WATCHDOG_HEARTBEAT;
> + edev->wdd.min_timeout = ESB_HEARTBEAT_MIN;
> + edev->wdd.max_timeout = ESB_HEARTBEAT_MAX;
> + edev->wdd.timeout = ESB_HEARTBEAT_DEFAULT;
> if (watchdog_init_timeout(&edev->wdd, heartbeat, NULL))
> dev_info(&pdev->dev,
> - "heartbeat value must be 1<heartbeat<2046, using %u\n",
> - edev->wdd.timeout);
> + "heartbeat value must be " ESB_HEARTBEAT_RANGE
> + ", using %u\n", edev->wdd.timeout);
> watchdog_set_nowayout(&edev->wdd, nowayout);
> watchdog_stop_on_reboot(&edev->wdd);
> watchdog_stop_on_unregister(&edev->wdd);
> --
> 2.13.6
>
--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
