On Thu 05 Nov 07:52 CST 2020, Georgi Djakov wrote:
> The QUP nodes are currently defined just as entries in the topology,
> but they are not referenced by any of the NoCs. Let's fix this and
> "attach" them to their NoCs, so that the QUP drivers are able to use
> them as path endpoints and scale their bandwidth.
>
> This is based on the information from the downstream msm-4.9 kernel.
>
> Signed-off-by: Georgi Djakov <georgi.djakov@xxxxxxxxxx>
Reviewed-by: Bjorn Andersson <bjorn.andersson@xxxxxxxxxx>
Georgi, would you mind if I take the series through my tree, to avoid
conflicts in sdm845.dtsi?
Regards,
Bjorn
> ---
> drivers/interconnect/qcom/sdm845.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/drivers/interconnect/qcom/sdm845.c b/drivers/interconnect/qcom/sdm845.c
> index 5304aea3b058..366870150cbd 100644
> --- a/drivers/interconnect/qcom/sdm845.c
> +++ b/drivers/interconnect/qcom/sdm845.c
> @@ -177,6 +177,7 @@ DEFINE_QBCM(bcm_sn15, "SN15", false, &qnm_memnoc);
>
> static struct qcom_icc_bcm *aggre1_noc_bcms[] = {
> &bcm_sn9,
> + &bcm_qup0,
> };
>
> static struct qcom_icc_node *aggre1_noc_nodes[] = {
> @@ -190,6 +191,7 @@ static struct qcom_icc_node *aggre1_noc_nodes[] = {
> [SLAVE_A1NOC_SNOC] = &qns_a1noc_snoc,
> [SLAVE_SERVICE_A1NOC] = &srvc_aggre1_noc,
> [SLAVE_ANOC_PCIE_A1NOC_SNOC] = &qns_pcie_a1noc_snoc,
> + [MASTER_QUP_1] = &qhm_qup1,
> };
>
> static const struct qcom_icc_desc sdm845_aggre1_noc = {
> @@ -218,6 +220,7 @@ static struct qcom_icc_node *aggre2_noc_nodes[] = {
> [SLAVE_A2NOC_SNOC] = &qns_a2noc_snoc,
> [SLAVE_ANOC_PCIE_SNOC] = &qns_pcie_snoc,
> [SLAVE_SERVICE_A2NOC] = &srvc_aggre2_noc,
> + [MASTER_QUP_2] = &qhm_qup2,
> };
>
> static const struct qcom_icc_desc sdm845_aggre2_noc = {
