diff options
| author | Stefan Richter <stefanr@s5r6.in-berlin.de> | 2013-04-29 00:16:14 +0200 |
|---|---|---|
| committer | Stefan Richter <stefanr@s5r6.in-berlin.de> | 2013-07-30 15:46:18 +0200 |
| commit | 0a41981803fcd4107fff4e943afb72940ba653d2 (patch) | |
| tree | 0874a8c79880ce2c0f6e13d6afb4fa9441b84aca /drivers | |
| parent | firewire: WQ_NON_REENTRANT is meaningless and going away (diff) | |
| download | linux-0a41981803fcd4107fff4e943afb72940ba653d2.tar.xz linux-0a41981803fcd4107fff4e943afb72940ba653d2.zip | |
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')
| -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) static int add_client_resource(struct client *client, struct client_resource *resource, gfp_t gfp_mask) { - bool preload = gfp_mask & __GFP_WAIT; + bool preload = !!(gfp_mask & __GFP_WAIT); unsigned long flags; int ret; |