summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>2019-08-23 16:32:27 -0400
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>2019-08-26 05:40:24 -0400
commit149f3b87840e7d292ad059f5fc23f1fa2fc98b9e (patch)
tree3044fecf00274e396d3aff05a1c831be788ce542
parent78cd4bf53635d3aeb73435bc89eb6eb39450f315 (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.h9
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)