Received: by 2002:a05:6a10:f347:0:0:0:0 with SMTP id d7csp286452pxu; Wed, 2 Dec 2020 23:20:55 -0800 (PST) X-Google-Smtp-Source: ABdhPJzArDJWTD4BFgbkA9I8B4Mpz3OOc5KsN3P1jS/9jJCWdsccpBx92mHeQn4OPPN8It+BNnb7 X-Received: by 2002:a17:906:4304:: with SMTP id j4mr1308238ejm.304.1606980055495; Wed, 02 Dec 2020 23:20:55 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1606980055; cv=none; d=google.com; s=arc-20160816; b=vTmNwo9v+iwiDrlFbWKcSimUkslGLIu8NjfJi8V8T+8Aj9i0+9ZBpI6XP/jgz8Hr+x qEDpNmIHz3JNFNq5aCkOjUapzM4xsG2t5HWVHOG5ivxcBCruKgVB6bog7fCrkzqphONS fdjCfp9NtOmhCv+ltHMNxWq0LAv5atrXZJ/+roFbIYo3AO7YSAFn+vzcMfYokjXZXyMj hWYViTLcgTqics6MmlTYw5ekClN2ky5OEgfgRnZldPeDUwbKpwMvIUbKouU/5K9GNfSi VYpZEEw69vdPVTJzZQ2QJpdltI7qT7uB5x62NJaeXv3bOqi7riZK1MsL9d+BvfJA/+uU Y5bg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:dkim-signature; bh=EJeEoDEeYHBeUS2YirOFrUkbP35JeO1CRiL+bErAeW0=; b=0GeKn870bQyvH+f+stwRa2Nt/1m0rUfwp4ph5ylr9ya+OlwznubkRf242Qj36Rq6Bk koVUiHMRA2qOlKqv3qZsCDOi3ktdUzSIMLKIIHmgLX2e3s4/NUi2s89r12IEFGkiE5mD 9xz4F1+6ZsDMsno3diK6MzumDnWaOvAl3LbYFxpxF/5jcVLbRiUSFSDvG0gX82zcyr/z ZMSSb44NOyhpZnwuxf/fwfO82Zw/qulRbeTampU0NpshObRHkgDmsvWWTQNk0q8ziOId CU/R3CxnQPwSR/t/x0U1ynfT5UHJiAHhOTjleOk0Mgqj+V4WZpOu606FkA5MFzCPejj0 O5MA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=MMbAufQg; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id dn20si727705ejc.47.2020.12.02.23.20.33; Wed, 02 Dec 2020 23:20:55 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=MMbAufQg; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728245AbgLCHSe (ORCPT + 99 others); Thu, 3 Dec 2020 02:18:34 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57772 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726085AbgLCHSd (ORCPT ); Thu, 3 Dec 2020 02:18:33 -0500 Received: from mail-wm1-x341.google.com (mail-wm1-x341.google.com [IPv6:2a00:1450:4864:20::341]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id DE227C061A4E for ; Wed, 2 Dec 2020 23:17:52 -0800 (PST) Received: by mail-wm1-x341.google.com with SMTP id k10so1745838wmi.3 for ; Wed, 02 Dec 2020 23:17:52 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=EJeEoDEeYHBeUS2YirOFrUkbP35JeO1CRiL+bErAeW0=; b=MMbAufQgXWEWIlC7G8WRrz7dEqThM6CoOO8uQqW+P4myef9EZ7C41l7Clbo7RlvoNX 6UEEx0XDqmBn6aVotTntdsujfXxNyYPeNI4/wZMQ9QXaG01yncMkh5dtO7Si/GXwJ0XF Eig8HtsAX86KYi5uEnNrZ2uwbXdHrKTjNq/8wnvo+k2dSz7qb5OF+OaUALyJcT7gEBoQ NxcLe3+JR2jVTpAnQheo0svi2UpXO1oDgLeod9hCAcwM+tg/DtL0dq3JLeGfzgrLmKiL 0G8s47ZM32iGKDeoGlldCJkVP18KbE7hI5J8Byp956oLmIT+UN7/7vkcwPOYj/jbhaRG vLgg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=EJeEoDEeYHBeUS2YirOFrUkbP35JeO1CRiL+bErAeW0=; b=gl74ayOJjzy2LAN7OJ8zhFPQuiTEGk27Zb7AkZBH4U9zHZA1rdpRYucwLYfbtPwUhb zmY8iylAvvUYkBPCPDBTEqsqIzAeBRWAYcoK1o8+EM26SKoGqLaT5mc8Vg34b1+wJM3e 4l69BcIta14DIGKR5tcIEksF6uOKg88Pv82paHEl/plWwVGpUpkiaas6zKTbig8YhNZl YYW4s2cSzRIgYUpijktocr6piSg1CcmgABXt2g/LWPC+2gIsUuYUtoXKe3fndbz8CbRd ltKAbw20k0nh7U8NZ9zlQ2w7CQwApFS2kmyifPMKYcztTEakt9O2VF1SLZlAbf8POT0W dzCA== X-Gm-Message-State: AOAM531Wvba9Arv3rXPvWq/obqx3DgY9IpKLyUhs4gpdujKUbolGLghR 3MAgeg1tRftaMwez1ExgsIQsJi7eu+8fmw== X-Received: by 2002:a1c:770d:: with SMTP id t13mr1692022wmi.153.1606979871290; Wed, 02 Dec 2020 23:17:51 -0800 (PST) Received: from mai.imgcgcw.net ([2a01:e34:ed2f:f020:1c7d:2d7a:9709:b9a2]) by smtp.gmail.com with ESMTPSA id s13sm230402wmj.28.2020.12.02.23.17.49 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 02 Dec 2020 23:17:50 -0800 (PST) From: Daniel Lezcano To: daniel.lezcano@linaro.org, peter@piie.net, hdegoede@redhat.com, mgross@linux.intel.com Cc: platform-driver-x86@vger.kernel.org (open list:ACER ASPIRE ONE TEMPERATURE AND FAN DRIVER), linux-kernel@vger.kernel.org (open list) Subject: [PATCH v2 1/2] platform/x86/drivers/acerhdf: Use module_param_cb to set/get polling interval Date: Thu, 3 Dec 2020 08:17:37 +0100 Message-Id: <20201203071738.2363701-1-daniel.lezcano@linaro.org> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org The module parameter can be set by using ops to get and set the values. The change will allow to check the correctness of the interval value everytime it is changed instead of checking in the get_temp function. Signed-off-by: Daniel Lezcano --- drivers/platform/x86/acerhdf.c | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/drivers/platform/x86/acerhdf.c b/drivers/platform/x86/acerhdf.c index 44b6bfbd32df..19fc8ff2225c 100644 --- a/drivers/platform/x86/acerhdf.c +++ b/drivers/platform/x86/acerhdf.c @@ -84,8 +84,6 @@ static struct platform_device *acerhdf_dev; module_param(kernelmode, uint, 0); MODULE_PARM_DESC(kernelmode, "Kernel mode fan control on / off"); -module_param(interval, uint, 0600); -MODULE_PARM_DESC(interval, "Polling interval of temperature check"); module_param(fanon, uint, 0600); MODULE_PARM_DESC(fanon, "Turn the fan on above this temperature"); module_param(fanoff, uint, 0600); @@ -824,3 +822,11 @@ MODULE_ALIAS("dmi:*:*Acer*:pnExtensa*5420*:"); module_init(acerhdf_init); module_exit(acerhdf_exit); + +static const struct kernel_param_ops interval_ops = { + .set = param_set_uint, + .get = param_get_uint, +}; + +module_param_cb(interval, &interval_ops, &interval, 0600); +MODULE_PARM_DESC(interval, "Polling interval of temperature check"); -- 2.25.1