Hi Sai,[..]
On 01/22/2019 01:37 PM, Sai Prakash Ranjan wrote:
Add coresight components found on Qualcomm SDM845 SoC.
Signed-off-by: Sai Prakash Ranjan <saiprakash.ranjan@xxxxxxxxxxxxxx>
Sorry, but I hadn't noticed the PID override strings below. Please
find the question.
+ÂÂÂÂÂÂÂ /*
+ÂÂÂÂÂÂÂÂ * On QCOM SDM845, we bypass the normal AMBA bus discovery
+ÂÂÂÂÂÂÂÂ * method by forcing the peripheral ID because of the wrong
+ÂÂÂÂÂÂÂÂ * value read from ETM PID registers.
+ÂÂÂÂÂÂÂÂ */
What is the value read back from the ETM PIDx registers ? Do they
provide inconsistent or incompatible value w.r.t the ETM/Coresight
architecture ? If it is an unsupported CPU with proper values,
you must add them to the table in etm4x driver.
+ÂÂÂÂÂÂÂ etm@7040000 {
+ÂÂÂÂÂÂÂÂÂÂÂ compatible = "arm,coresight-etm4x", "arm,primecell";
+ÂÂÂÂÂÂÂÂÂÂÂ arm,primecell-periphid = <0x000bb95d> > +ÂÂÂÂÂÂÂÂÂÂÂ reg = <0 0x07040000 0 0x1000>;
+
+ÂÂÂÂÂÂÂÂÂÂÂ cpu = <&CPU0>;
+
You seem to be specifying the PID of A53 ETM all over, while at least
one of your cores is ETMv4.2 (from the other patch) and A53 is not
ETMv4.2. As above, it would be good to add the PID to the table.