summaryrefslogtreecommitdiffstats
path: root/drivers/firmware
diff options
context:
space:
mode:
authorMatt Fleming <matt.fleming@intel.com>2012-10-23 12:35:43 +0100
committerMatt Fleming <matt.fleming@intel.com>2012-10-30 10:39:27 +0000
commit07b1c5bc64cff9c880261a1fef562ef7ea7f6575 (patch)
treec055ffdac6ab14f8920baba8976fa26cd242fc73 /drivers/firmware
parent91716322d834cba34f4a7ed5e4a39673eb90862b (diff)
downloadop-kernel-dev-07b1c5bc64cff9c880261a1fef562ef7ea7f6575.zip
op-kernel-dev-07b1c5bc64cff9c880261a1fef562ef7ea7f6575.tar.gz
efivarfs: Make 'datasize' unsigned long
There's no reason to declare 'datasize' as an int, since the majority of the functions it's passed to expect an unsigned long anyway. Plus, this way we avoid any sign problems during arithmetic. Acked-by: Jeremy Kerr <jeremy.kerr@canonical.com> Signed-off-by: Matt Fleming <matt.fleming@intel.com>
Diffstat (limited to 'drivers/firmware')
-rw-r--r--drivers/firmware/efivars.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/firmware/efivars.c b/drivers/firmware/efivars.c
index 3b0cf9a..6a858d1 100644
--- a/drivers/firmware/efivars.c
+++ b/drivers/firmware/efivars.c
@@ -692,7 +692,7 @@ static ssize_t efivarfs_file_write(struct file *file,
void *data;
u32 attributes;
struct inode *inode = file->f_mapping->host;
- int datasize = count - sizeof(attributes);
+ unsigned long datasize = count - sizeof(attributes);
unsigned long newdatasize;
if (count < sizeof(attributes))
OpenPOWER on IntegriCloud