On Tue, Jun 05, 2018 at 10:43:19PM +0100, Suzuki K Poulose wrote:
The coresight drivers relied on default bindings for graph
in DT, while reusing the "reg" field of the "ports" to indicate
the actual hardware port number for the connections. However,
with the rules getting stricter w.r.t to the address mismatch
with the label, it is no longer possible to use the port address
field for the hardware port number. Hence, we add an explicit
property to denote the hardware port number, "coresight,hwid"
which must be specified for each "endpoint".
Cc: Mathieu Poirier <mathieu.poirier@xxxxxxxxxx>
Cc: Sudeep Holla <sudeep.holla@xxxxxxx>
Cc: Rob Herring <robh@xxxxxxxxxx>
Signed-off-by: Suzuki K Poulose <suzuki.poulose@xxxxxxx>
---
.../devicetree/bindings/arm/coresight.txt | 29 ++++++++++---
drivers/hwtracing/coresight/of_coresight.c | 49 +++++++++++++++++-----
2 files changed, 62 insertions(+), 16 deletions(-)
diff --git a/Documentation/devicetree/bindings/arm/coresight.txt b/Documentation/devicetree/bindings/arm/coresight.txt
index ed6b555..bf75ab3 100644
--- a/Documentation/devicetree/bindings/arm/coresight.txt
+++ b/Documentation/devicetree/bindings/arm/coresight.txt
@@ -108,8 +108,13 @@ following properties to uniquely identify the connection details.
"slave-mode"
};
For the binding part:
Reviewed-by: Mathieu Poirier <mathieu.poirier@xxxxxxxxxx>
diff --git a/drivers/hwtracing/coresight/of_coresight.c b/drivers/hwtracing/coresight/of_coresight.c
index d01a9ce..d23d7dd 100644
--- a/drivers/hwtracing/coresight/of_coresight.c
+++ b/drivers/hwtracing/coresight/of_coresight.c
+/*
* of_coresight_parse_endpoint : Parse the given output endpoint @ep
* and fill the connection information in *@pconn.
*
@@ -109,11 +134,11 @@ EXPORT_SYMBOL_GPL(of_coresight_get_cpu);
* 0 - If the parsing completed without any fatal errors.
* -Errno - Fatal error, abort the scanning.
*/
-static int of_coresight_parse_endpoint(struct device_node *ep,
+static int of_coresight_parse_endpoint(struct device *dev,
+ struct device_node *ep,
struct coresight_connection **pconn)
{
- int ret = 0;
- struct of_endpoint endpoint, rendpoint;
+ int ret = 0, local_port, child_port;
struct device_node *rparent = NULL;
struct device_node *rep = NULL;
struct device *rdev = NULL;
@@ -128,7 +153,8 @@ static int of_coresight_parse_endpoint(struct device_node *ep,
break;
/* Parse the local port details */
- if (of_graph_parse_endpoint(ep, &endpoint))
+ local_port = of_coresight_endpoint_get_port_id(dev, ep);
+ if (local_port < 0)
break;
/*
* Get a handle on the remote endpoint and the device it is
@@ -140,9 +166,6 @@ static int of_coresight_parse_endpoint(struct device_node *ep,
rparent = of_graph_get_port_parent(rep);
if (!rparent)
break;
- if (of_graph_parse_endpoint(rep, &rendpoint))
- break;
-
/* If the remote device is not available, defer probing */
rdev = of_coresight_get_endpoint_device(rparent);
if (!rdev) {
@@ -150,9 +173,15 @@ static int of_coresight_parse_endpoint(struct device_node *ep,
break;
}
- conn->outport = endpoint.port;
+ child_port = of_coresight_endpoint_get_port_id(rdev, rep);
+ if (child_port < 0) {
+ ret = 0;
Why returning '0' on an error condition? Same for 'local_port' above.