Re: [PATCH v1 3/4] interconnect: qcom: sdm845: Split qnodes into their respective NoCs

From: Bjorn Andersson
Date: Thu Dec 26 2019 - 14:01:03 EST


On Thu 26 Dec 10:48 PST 2019, Rob Herring wrote:

> On Sun, Dec 15, 2019 at 09:58:44PM -0800, David Dai wrote:
> > In order to better represent the hardware and its different Network-On-Chip
> > devices, split the sdm845 provider driver into NoC specific providers.
> > Remove duplicate functionality already provided by the icc rpmh and
> > bcm voter drivers to calculate and commit bandwidth requests to hardware.
>
> This breaks backwards compatibility. Is that okay for all 845 users?
>

Yes, based on our discussions surrounding adding support for the
pre-rpmh platforms, we concluded a while ago that we would like to
attempt to move to this structure instead of the originally agreed upon.

So we haven't merged any consumers using the current 845 binding in the
upstream Linux repository.


I'm not aware of any other public repositories that relies on the
existing binding.

Thanks,
Bjorn

> > Signed-off-by: David Dai <daidavid1@xxxxxxxxxxxxxx>
> > ---
> > drivers/interconnect/qcom/sdm845.c | 1122 ++++++++++--------------
> > include/dt-bindings/interconnect/qcom,sdm845.h | 263 +++---
> > 2 files changed, 606 insertions(+), 779 deletions(-)