[PATCH 2/2] Staging: olpc_dcon.c: obsolete use of strict_stroul; resumeline
Valentin Rothberg
valentinrothberg at googlemail.com
Sun Dec 4 11:17:33 UTC 2011
As Dan mentioned, dcon_write() will only write u16 values. The
appropriate parts have been changed. As a result of module_param()
not accepting u16 as a valid data type, unsigned short is used.
Signed-off-by: Valentin Rothberg <valentinrothberg at googlemail.com>
---
drivers/staging/olpc_dcon/olpc_dcon.c | 8 ++++----
1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/staging/olpc_dcon/olpc_dcon.c b/drivers/staging/olpc_dcon/olpc_dcon.c
index 1e64992..1b3847b 100644
--- a/drivers/staging/olpc_dcon/olpc_dcon.c
+++ b/drivers/staging/olpc_dcon/olpc_dcon.c
@@ -34,8 +34,8 @@
/* Module definitions */
-static int resumeline = 898;
-module_param(resumeline, int, 0444);
+static unsigned short resumeline = 898;
+module_param(resumeline, unsigned short, 0444);
/* Default off since it doesn't work on DCON ASIC in B-test OLPC board */
static int useaa = 1;
@@ -498,10 +498,10 @@ static ssize_t dcon_freeze_store(struct device *dev,
static ssize_t dcon_resumeline_store(struct device *dev,
struct device_attribute *attr, const char *buf, size_t count)
{
- unsigned long rl;
+ unsigned short rl;
int rc;
- rc = strict_strtoul(buf, 10, &rl);
+ rc = kstrtou16(buf, 10, &rl);
if (rc)
return rc;
--
1.7.5.4
More information about the devel
mailing list