mbox series

[v2,0/2] interconnect: qcom: msm8939: Coalesce snoc and snoc_mm

Message ID 20220129032735.2410936-1-bryan.odonoghue@linaro.org
Headers show
Series interconnect: qcom: msm8939: Coalesce snoc and snoc_mm | expand

Message

Bryan O'Donoghue Jan. 29, 2022, 3:27 a.m. UTC 
V2:
Made snoc-mm into a child of snoc - Dmitry
Removes redundant msm8939_snoc_mm_regmap_config - bod

V1:
Booting msm8939 on tip-of-tree I encountered the following error.

[    1.212340] qnoc-msm8939 580000.interconnect_mm: can't request region for resource [mem 0x00580000-0x0059407f]
[    1.212391] qnoc-msm8939 580000.interconnect_mm: Cannot ioremap interconnect bus resource
[    1.221524] qnoc-msm8939: probe of 580000.interconnect_mm failed with error -16

Initially I thought this was a bug with the interconnect driver but,
examining it a bit more I realized the DTS I was working with based on
downstream, declares snoc and snoc_mm as existing at the same address
range.

When we were developing the DTS for 8939 we weren't using the common rpm
interconnect driver so we never saw the ioremap collision.

Taking a hard look at the qcom documentation as well as the downstream code
we see that yes downstream declares snoc and snoc_mm separately but, also
at the same overlapping address.

The qcom documentation for performance points for msm8936/msm8939 snoc,
deliniates snoc_mm as simply two new performance-points i.e. a faster GPLL0
vote associated with new multi-media devices attached to the snoc.

In other words the snoc had two new RPM vote indices added to it, to
represent the higher performance clocks, should one of the multi-media IP
blocks call for it.

We can fix the ioremap collision and still represent the two higher
performance point clock votes by coalsecing snoc and snoc_mm into snoc. The
DTS clock references will take care of the appropriate votes.

Bryan O'Donoghue (2):
  dt-bindings: interconnect: Convert snoc-mm to a sub-node of snoc
  interconnect: qcom: msm8939: Remove snoc_mm specific regmap

 .../bindings/interconnect/qcom,rpm.yaml       | 52 ++++++++++++++++++-
 drivers/interconnect/qcom/msm8939.c           | 10 +---
 2 files changed, 51 insertions(+), 11 deletions(-)