diff options
author | Andy Shevchenko <andriy.shevchenko@linux.intel.com> | 2019-08-23 16:32:27 -0400 |
---|---|---|
committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2019-08-26 05:40:24 -0400 |
commit | 149f3b87840e7d292ad059f5fc23f1fa2fc98b9e (patch) | |
tree | 3044fecf00274e396d3aff05a1c831be788ce542 | |
parent | 78cd4bf53635d3aeb73435bc89eb6eb39450f315 (diff) |
device property: Remove duplicate test for NULL
There is no need to check twice for a NULL in fwnode_call_bool_op().
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Reviewed-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
-rw-r--r-- | include/linux/fwnode.h | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/include/linux/fwnode.h b/include/linux/fwnode.h index a11c8c56c78b..ababd6bc82f3 100644 --- a/include/linux/fwnode.h +++ b/include/linux/fwnode.h | |||
@@ -110,10 +110,11 @@ struct fwnode_operations { | |||
110 | (fwnode ? (fwnode_has_op(fwnode, op) ? \ | 110 | (fwnode ? (fwnode_has_op(fwnode, op) ? \ |
111 | (fwnode)->ops->op(fwnode, ## __VA_ARGS__) : -ENXIO) : \ | 111 | (fwnode)->ops->op(fwnode, ## __VA_ARGS__) : -ENXIO) : \ |
112 | -EINVAL) | 112 | -EINVAL) |
113 | #define fwnode_call_bool_op(fwnode, op, ...) \ | 113 | |
114 | (fwnode ? (fwnode_has_op(fwnode, op) ? \ | 114 | #define fwnode_call_bool_op(fwnode, op, ...) \ |
115 | (fwnode)->ops->op(fwnode, ## __VA_ARGS__) : false) : \ | 115 | (fwnode_has_op(fwnode, op) ? \ |
116 | false) | 116 | (fwnode)->ops->op(fwnode, ## __VA_ARGS__) : false) |
117 | |||
117 | #define fwnode_call_ptr_op(fwnode, op, ...) \ | 118 | #define fwnode_call_ptr_op(fwnode, op, ...) \ |
118 | (fwnode_has_op(fwnode, op) ? \ | 119 | (fwnode_has_op(fwnode, op) ? \ |
119 | (fwnode)->ops->op(fwnode, ## __VA_ARGS__) : NULL) | 120 | (fwnode)->ops->op(fwnode, ## __VA_ARGS__) : NULL) |