diff options
author | Stefan Richter <stefanr@s5r6.in-berlin.de> | 2013-04-28 18:16:14 -0400 |
---|---|---|
committer | Stefan Richter <stefanr@s5r6.in-berlin.de> | 2013-07-30 09:46:18 -0400 |
commit | 0a41981803fcd4107fff4e943afb72940ba653d2 (patch) | |
tree | 0874a8c79880ce2c0f6e13d6afb4fa9441b84aca /drivers/firewire | |
parent | 4e6b9319bce7a2be878d79bcbe2cb558b619b360 (diff) |
firewire: core: typecast from gfp_t to bool more safely
An idr related patch introduced the following sparse warning:
drivers/firewire/core-cdev.c:488:33: warning: incorrect type in initializer (different base types)
drivers/firewire/core-cdev.c:488:33: expected bool [unsigned] [usertype] preload
drivers/firewire/core-cdev.c:488:33: got restricted gfp_t
So let's convert from gfp_t bitfield to Boolean explicitly and safely.
Signed-off-by: Stefan Richter <stefanr@s5r6.in-berlin.de>
Diffstat (limited to 'drivers/firewire')
-rw-r--r-- | drivers/firewire/core-cdev.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/firewire/core-cdev.c b/drivers/firewire/core-cdev.c index ac1b43a04285..d7d5c8af92b9 100644 --- a/drivers/firewire/core-cdev.c +++ b/drivers/firewire/core-cdev.c | |||
@@ -486,7 +486,7 @@ static int ioctl_get_info(struct client *client, union ioctl_arg *arg) | |||
486 | static int add_client_resource(struct client *client, | 486 | static int add_client_resource(struct client *client, |
487 | struct client_resource *resource, gfp_t gfp_mask) | 487 | struct client_resource *resource, gfp_t gfp_mask) |
488 | { | 488 | { |
489 | bool preload = gfp_mask & __GFP_WAIT; | 489 | bool preload = !!(gfp_mask & __GFP_WAIT); |
490 | unsigned long flags; | 490 | unsigned long flags; |
491 | int ret; | 491 | int ret; |
492 | 492 | ||