From: Suzuki K Poulose <suzuki.poul...@arm.com>

Add a helper to check if the given endpoint is input.

Cc: Mathieu Poirier <mathieu.poir...@linaro.org>
Signed-off-by: Suzuki K Poulose <suzuki.poul...@arm.com>
Signed-off-by: Mathieu Poirier <mathieu.poir...@linaro.org>
---
 drivers/hwtracing/coresight/of_coresight.c | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/drivers/hwtracing/coresight/of_coresight.c 
b/drivers/hwtracing/coresight/of_coresight.c
index 2ecdd1432b5c..44903d35009f 100644
--- a/drivers/hwtracing/coresight/of_coresight.c
+++ b/drivers/hwtracing/coresight/of_coresight.c
@@ -45,6 +45,11 @@ of_coresight_get_endpoint_device(struct device_node 
*endpoint)
                               endpoint, of_dev_node_match);
 }
 
+static inline bool of_coresight_ep_is_input(struct device_node *ep)
+{
+       return of_property_read_bool(ep, "slave-mode");
+}
+
 static void of_coresight_get_ports(const struct device_node *node,
                                   int *nr_inport, int *nr_outport)
 {
@@ -56,7 +61,7 @@ static void of_coresight_get_ports(const struct device_node 
*node,
                if (!ep)
                        break;
 
-               if (of_property_read_bool(ep, "slave-mode"))
+               if (of_coresight_ep_is_input(ep))
                        in++;
                else
                        out++;
@@ -213,7 +218,7 @@ of_get_coresight_platform_data(struct device *dev,
                 * No need to deal with input ports, as processing the
                 * output ports connected to them will process the details.
                 */
-               if (of_find_property(ep, "slave-mode", NULL))
+               if (of_coresight_ep_is_input(ep))
                        continue;
 
                ret = of_coresight_parse_endpoint(dev, ep, pdata, i);
-- 
2.7.4

Reply via email to