From: AngeloGioacchino Del Regno <angelogioacchino.delregno@somainline.org>
To: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>,
Andy Gross <agross@kernel.org>,
Bjorn Andersson <bjorn.andersson@linaro.org>,
Georgi Djakov <djakov@kernel.org>
Cc: Shawn Guo <shawn.guo@linaro.org>,
Yassine Oudjana <y.oudjana@protonmail.com>,
linux-arm-msm@vger.kernel.org, linux-pm@vger.kernel.org
Subject: Re: [PATCH v2 03/11] interconnect: sdm660: drop default/unused values
Date: Sat, 4 Sep 2021 12:59:14 +0200 [thread overview]
Message-ID: <6791b29a-db47-0280-5178-c2277498e14e@somainline.org> (raw)
In-Reply-To: <20210903232421.1384199-4-dmitry.baryshkov@linaro.org>
Il 04/09/21 01:24, Dmitry Baryshkov ha scritto:
> Simplify qnode setup by removing unused/default values.
>
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Tested on Sony Xperia XA2 (sdm630-pioneer)
Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@somainline.org>
> ---
> drivers/interconnect/qcom/sdm660.c | 407 +++++------------------------
> 1 file changed, 64 insertions(+), 343 deletions(-)
>
> diff --git a/drivers/interconnect/qcom/sdm660.c b/drivers/interconnect/qcom/sdm660.c
> index 652f7cc9cad6..4a72f9677d4e 100644
> --- a/drivers/interconnect/qcom/sdm660.c
> +++ b/drivers/interconnect/qcom/sdm660.c
> @@ -35,6 +35,7 @@
> #define M_BKE_EN_EN_BMASK 0x1
>
> /* Valid for both NoC and BIMC */
> +#define NOC_QOS_MODE_INVALID -1
> #define NOC_QOS_MODE_FIXED 0x0
> #define NOC_QOS_MODE_LIMITER 0x1
> #define NOC_QOS_MODE_BYPASS 0x2
> @@ -279,10 +280,7 @@ static struct qcom_icc_node mas_cnoc_a2noc = {
> .mas_rpm_id = 146,
> .slv_rpm_id = -1,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> .num_links = ARRAY_SIZE(mas_cnoc_a2noc_links),
> .links = mas_cnoc_a2noc_links,
> };
> @@ -297,11 +295,6 @@ static struct qcom_icc_node mas_sdcc_1 = {
> .buswidth = 8,
> .mas_rpm_id = 33,
> .slv_rpm_id = -1,
> - .qos.ap_owned = false,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> .num_links = ARRAY_SIZE(mas_sdcc_1_links),
> .links = mas_sdcc_1_links,
> };
> @@ -316,11 +309,6 @@ static struct qcom_icc_node mas_sdcc_2 = {
> .buswidth = 8,
> .mas_rpm_id = 35,
> .slv_rpm_id = -1,
> - .qos.ap_owned = false,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> .num_links = ARRAY_SIZE(mas_sdcc_2_links),
> .links = mas_sdcc_2_links,
> };
> @@ -335,11 +323,6 @@ static struct qcom_icc_node mas_blsp_1 = {
> .buswidth = 4,
> .mas_rpm_id = 41,
> .slv_rpm_id = -1,
> - .qos.ap_owned = false,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> .num_links = ARRAY_SIZE(mas_blsp_1_links),
> .links = mas_blsp_1_links,
> };
> @@ -354,11 +337,6 @@ static struct qcom_icc_node mas_blsp_2 = {
> .buswidth = 4,
> .mas_rpm_id = 39,
> .slv_rpm_id = -1,
> - .qos.ap_owned = false,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> .num_links = ARRAY_SIZE(mas_blsp_2_links),
> .links = mas_blsp_2_links,
> };
> @@ -511,11 +489,6 @@ static struct qcom_icc_node mas_snoc_bimc = {
> .buswidth = 4,
> .mas_rpm_id = 3,
> .slv_rpm_id = -1,
> - .qos.ap_owned = false,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> .num_links = ARRAY_SIZE(mas_snoc_bimc_links),
> .links = mas_snoc_bimc_links,
> };
> @@ -583,10 +556,7 @@ static struct qcom_icc_node mas_snoc_cnoc = {
> .mas_rpm_id = 52,
> .slv_rpm_id = -1,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> .num_links = ARRAY_SIZE(mas_snoc_cnoc_links),
> .links = mas_snoc_cnoc_links,
> };
> @@ -635,10 +605,7 @@ static struct qcom_icc_node mas_qdss_dap = {
> .mas_rpm_id = 49,
> .slv_rpm_id = -1,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> .num_links = ARRAY_SIZE(mas_qdss_dap_links),
> .links = mas_qdss_dap_links,
> };
> @@ -655,10 +622,7 @@ static struct qcom_icc_node mas_apss_proc = {
> .mas_rpm_id = 0,
> .slv_rpm_id = -1,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> .num_links = ARRAY_SIZE(mas_apss_proc_links),
> .links = mas_apss_proc_links,
> };
> @@ -685,10 +649,7 @@ static struct qcom_icc_node mas_cnoc_mnoc_mmss_cfg = {
> .mas_rpm_id = 4,
> .slv_rpm_id = -1,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> .num_links = ARRAY_SIZE(mas_cnoc_mnoc_mmss_cfg_links),
> .links = mas_cnoc_mnoc_mmss_cfg_links,
> };
> @@ -704,10 +665,7 @@ static struct qcom_icc_node mas_cnoc_mnoc_cfg = {
> .mas_rpm_id = 5,
> .slv_rpm_id = -1,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> .num_links = ARRAY_SIZE(mas_cnoc_mnoc_cfg_links),
> .links = mas_cnoc_mnoc_cfg_links,
> };
> @@ -880,11 +838,6 @@ static struct qcom_icc_node mas_snoc_cfg = {
> .buswidth = 4,
> .mas_rpm_id = 20,
> .slv_rpm_id = -1,
> - .qos.ap_owned = false,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> .num_links = ARRAY_SIZE(mas_snoc_cfg_links),
> .links = mas_snoc_cfg_links,
> };
> @@ -907,11 +860,6 @@ static struct qcom_icc_node mas_bimc_snoc = {
> .buswidth = 8,
> .mas_rpm_id = 21,
> .slv_rpm_id = -1,
> - .qos.ap_owned = false,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> .num_links = ARRAY_SIZE(mas_bimc_snoc_links),
> .links = mas_bimc_snoc_links,
> };
> @@ -934,11 +882,6 @@ static struct qcom_icc_node mas_gnoc_snoc = {
> .buswidth = 8,
> .mas_rpm_id = 150,
> .slv_rpm_id = -1,
> - .qos.ap_owned = false,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> .num_links = ARRAY_SIZE(mas_gnoc_snoc_links),
> .links = mas_gnoc_snoc_links,
> };
> @@ -962,11 +905,6 @@ static struct qcom_icc_node mas_a2noc_snoc = {
> .buswidth = 16,
> .mas_rpm_id = 112,
> .slv_rpm_id = -1,
> - .qos.ap_owned = false,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> .num_links = ARRAY_SIZE(mas_a2noc_snoc_links),
> .links = mas_a2noc_snoc_links,
> };
> @@ -981,11 +919,6 @@ static struct qcom_icc_node slv_a2noc_snoc = {
> .buswidth = 16,
> .mas_rpm_id = -1,
> .slv_rpm_id = 143,
> - .qos.ap_owned = false,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> .num_links = ARRAY_SIZE(slv_a2noc_snoc_links),
> .links = slv_a2noc_snoc_links,
> };
> @@ -996,11 +929,6 @@ static struct qcom_icc_node slv_ebi = {
> .buswidth = 4,
> .mas_rpm_id = -1,
> .slv_rpm_id = 0,
> - .qos.ap_owned = false,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> };
>
> static struct qcom_icc_node slv_hmss_l3 = {
> @@ -1009,11 +937,6 @@ static struct qcom_icc_node slv_hmss_l3 = {
> .buswidth = 4,
> .mas_rpm_id = -1,
> .slv_rpm_id = 160,
> - .qos.ap_owned = false,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> };
>
> static const u16 slv_bimc_snoc_links[] = {
> @@ -1026,11 +949,6 @@ static struct qcom_icc_node slv_bimc_snoc = {
> .buswidth = 4,
> .mas_rpm_id = -1,
> .slv_rpm_id = 2,
> - .qos.ap_owned = false,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> .num_links = ARRAY_SIZE(slv_bimc_snoc_links),
> .links = slv_bimc_snoc_links,
> };
> @@ -1046,10 +964,7 @@ static struct qcom_icc_node slv_cnoc_a2noc = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 208,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> .num_links = ARRAY_SIZE(slv_cnoc_a2noc_links),
> .links = slv_cnoc_a2noc_links,
> };
> @@ -1061,10 +976,7 @@ static struct qcom_icc_node slv_mpm = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 62,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_pmic_arb = {
> @@ -1074,10 +986,7 @@ static struct qcom_icc_node slv_pmic_arb = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 59,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_tlmm_north = {
> @@ -1087,10 +996,7 @@ static struct qcom_icc_node slv_tlmm_north = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 214,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_tcsr = {
> @@ -1100,10 +1006,7 @@ static struct qcom_icc_node slv_tcsr = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 50,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_pimem_cfg = {
> @@ -1113,10 +1016,7 @@ static struct qcom_icc_node slv_pimem_cfg = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 167,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_imem_cfg = {
> @@ -1126,10 +1026,7 @@ static struct qcom_icc_node slv_imem_cfg = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 54,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_message_ram = {
> @@ -1139,10 +1036,7 @@ static struct qcom_icc_node slv_message_ram = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 55,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_glm = {
> @@ -1152,10 +1046,7 @@ static struct qcom_icc_node slv_glm = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 209,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_bimc_cfg = {
> @@ -1165,10 +1056,7 @@ static struct qcom_icc_node slv_bimc_cfg = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 56,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_prng = {
> @@ -1178,10 +1066,7 @@ static struct qcom_icc_node slv_prng = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 44,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_spdm = {
> @@ -1191,10 +1076,7 @@ static struct qcom_icc_node slv_spdm = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 60,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_qdss_cfg = {
> @@ -1204,10 +1086,7 @@ static struct qcom_icc_node slv_qdss_cfg = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 63,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static const u16 slv_cnoc_mnoc_cfg_links[] = {
> @@ -1221,10 +1100,7 @@ static struct qcom_icc_node slv_cnoc_mnoc_cfg = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 66,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> .num_links = ARRAY_SIZE(slv_cnoc_mnoc_cfg_links),
> .links = slv_cnoc_mnoc_cfg_links,
> };
> @@ -1236,10 +1112,7 @@ static struct qcom_icc_node slv_snoc_cfg = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 70,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_qm_cfg = {
> @@ -1249,10 +1122,7 @@ static struct qcom_icc_node slv_qm_cfg = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 212,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_clk_ctl = {
> @@ -1262,10 +1132,7 @@ static struct qcom_icc_node slv_clk_ctl = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 47,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_mss_cfg = {
> @@ -1275,10 +1142,7 @@ static struct qcom_icc_node slv_mss_cfg = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 48,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_tlmm_south = {
> @@ -1288,10 +1152,7 @@ static struct qcom_icc_node slv_tlmm_south = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 217,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_ufs_cfg = {
> @@ -1301,10 +1162,7 @@ static struct qcom_icc_node slv_ufs_cfg = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 92,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_a2noc_cfg = {
> @@ -1314,10 +1172,7 @@ static struct qcom_icc_node slv_a2noc_cfg = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 150,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_a2noc_smmu_cfg = {
> @@ -1327,10 +1182,7 @@ static struct qcom_icc_node slv_a2noc_smmu_cfg = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 152,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_gpuss_cfg = {
> @@ -1340,10 +1192,7 @@ static struct qcom_icc_node slv_gpuss_cfg = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 11,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_ahb2phy = {
> @@ -1353,10 +1202,7 @@ static struct qcom_icc_node slv_ahb2phy = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 163,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_blsp_1 = {
> @@ -1366,10 +1212,7 @@ static struct qcom_icc_node slv_blsp_1 = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 39,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_sdcc_1 = {
> @@ -1379,10 +1222,7 @@ static struct qcom_icc_node slv_sdcc_1 = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 31,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_sdcc_2 = {
> @@ -1392,10 +1232,7 @@ static struct qcom_icc_node slv_sdcc_2 = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 33,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_tlmm_center = {
> @@ -1405,10 +1242,7 @@ static struct qcom_icc_node slv_tlmm_center = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 218,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_blsp_2 = {
> @@ -1418,10 +1252,7 @@ static struct qcom_icc_node slv_blsp_2 = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 37,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_pdm = {
> @@ -1431,10 +1262,7 @@ static struct qcom_icc_node slv_pdm = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 41,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static const u16 slv_cnoc_mnoc_mmss_cfg_links[] = {
> @@ -1448,10 +1276,7 @@ static struct qcom_icc_node slv_cnoc_mnoc_mmss_cfg = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 58,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> .num_links = ARRAY_SIZE(slv_cnoc_mnoc_mmss_cfg_links),
> .links = slv_cnoc_mnoc_mmss_cfg_links,
> };
> @@ -1463,10 +1288,7 @@ static struct qcom_icc_node slv_usb_hs = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 40,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_usb3_0 = {
> @@ -1476,10 +1298,7 @@ static struct qcom_icc_node slv_usb3_0 = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 22,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_srvc_cnoc = {
> @@ -1489,10 +1308,7 @@ static struct qcom_icc_node slv_srvc_cnoc = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 76,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static const u16 slv_gnoc_bimc_links[] = {
> @@ -1506,10 +1322,7 @@ static struct qcom_icc_node slv_gnoc_bimc = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 210,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> .num_links = ARRAY_SIZE(slv_gnoc_bimc_links),
> .links = slv_gnoc_bimc_links,
> };
> @@ -1525,10 +1338,7 @@ static struct qcom_icc_node slv_gnoc_snoc = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 211,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> .num_links = ARRAY_SIZE(slv_gnoc_snoc_links),
> .links = slv_gnoc_snoc_links,
> };
> @@ -1540,10 +1350,7 @@ static struct qcom_icc_node slv_camera_cfg = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 3,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_camera_throttle_cfg = {
> @@ -1553,10 +1360,7 @@ static struct qcom_icc_node slv_camera_throttle_cfg = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 154,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_misc_cfg = {
> @@ -1566,10 +1370,7 @@ static struct qcom_icc_node slv_misc_cfg = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 8,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_venus_throttle_cfg = {
> @@ -1579,10 +1380,7 @@ static struct qcom_icc_node slv_venus_throttle_cfg = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 178,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_venus_cfg = {
> @@ -1592,10 +1390,7 @@ static struct qcom_icc_node slv_venus_cfg = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 10,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_mmss_clk_xpu_cfg = {
> @@ -1605,10 +1400,7 @@ static struct qcom_icc_node slv_mmss_clk_xpu_cfg = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 13,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_mmss_clk_cfg = {
> @@ -1618,10 +1410,7 @@ static struct qcom_icc_node slv_mmss_clk_cfg = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 12,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_mnoc_mpu_cfg = {
> @@ -1631,10 +1420,7 @@ static struct qcom_icc_node slv_mnoc_mpu_cfg = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 14,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_display_cfg = {
> @@ -1644,10 +1430,7 @@ static struct qcom_icc_node slv_display_cfg = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 4,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_csi_phy_cfg = {
> @@ -1657,10 +1440,7 @@ static struct qcom_icc_node slv_csi_phy_cfg = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 224,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_display_throttle_cfg = {
> @@ -1670,10 +1450,7 @@ static struct qcom_icc_node slv_display_throttle_cfg = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 156,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_smmu_cfg = {
> @@ -1683,10 +1460,7 @@ static struct qcom_icc_node slv_smmu_cfg = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 205,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static const u16 slv_mnoc_bimc_links[] = {
> @@ -1700,10 +1474,7 @@ static struct qcom_icc_node slv_mnoc_bimc = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 16,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> .num_links = ARRAY_SIZE(slv_mnoc_bimc_links),
> .links = slv_mnoc_bimc_links,
> };
> @@ -1715,10 +1486,7 @@ static struct qcom_icc_node slv_srvc_mnoc = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 17,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_hmss = {
> @@ -1728,10 +1496,7 @@ static struct qcom_icc_node slv_hmss = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 20,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_lpass = {
> @@ -1741,10 +1506,7 @@ static struct qcom_icc_node slv_lpass = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 21,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_wlan = {
> @@ -1753,11 +1515,6 @@ static struct qcom_icc_node slv_wlan = {
> .buswidth = 4,
> .mas_rpm_id = -1,
> .slv_rpm_id = 206,
> - .qos.ap_owned = false,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> };
>
> static struct qcom_icc_node slv_cdsp = {
> @@ -1767,10 +1524,7 @@ static struct qcom_icc_node slv_cdsp = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 221,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static struct qcom_icc_node slv_ipa = {
> @@ -1780,10 +1534,7 @@ static struct qcom_icc_node slv_ipa = {
> .mas_rpm_id = -1,
> .slv_rpm_id = 183,
> .qos.ap_owned = true,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> + .qos.qos_mode = NOC_QOS_MODE_INVALID,
> };
>
> static const u16 slv_snoc_bimc_links[] = {
> @@ -1796,11 +1547,6 @@ static struct qcom_icc_node slv_snoc_bimc = {
> .buswidth = 16,
> .mas_rpm_id = -1,
> .slv_rpm_id = 24,
> - .qos.ap_owned = false,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> .num_links = ARRAY_SIZE(slv_snoc_bimc_links),
> .links = slv_snoc_bimc_links,
> };
> @@ -1815,11 +1561,6 @@ static struct qcom_icc_node slv_snoc_cnoc = {
> .buswidth = 8,
> .mas_rpm_id = -1,
> .slv_rpm_id = 25,
> - .qos.ap_owned = false,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> .num_links = ARRAY_SIZE(slv_snoc_cnoc_links),
> .links = slv_snoc_cnoc_links,
> };
> @@ -1830,11 +1571,6 @@ static struct qcom_icc_node slv_imem = {
> .buswidth = 8,
> .mas_rpm_id = -1,
> .slv_rpm_id = 26,
> - .qos.ap_owned = false,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> };
>
> static struct qcom_icc_node slv_pimem = {
> @@ -1843,11 +1579,6 @@ static struct qcom_icc_node slv_pimem = {
> .buswidth = 8,
> .mas_rpm_id = -1,
> .slv_rpm_id = 166,
> - .qos.ap_owned = false,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> };
>
> static struct qcom_icc_node slv_qdss_stm = {
> @@ -1856,11 +1587,6 @@ static struct qcom_icc_node slv_qdss_stm = {
> .buswidth = 4,
> .mas_rpm_id = -1,
> .slv_rpm_id = 30,
> - .qos.ap_owned = false,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> };
>
> static struct qcom_icc_node slv_srvc_snoc = {
> @@ -1869,11 +1595,6 @@ static struct qcom_icc_node slv_srvc_snoc = {
> .buswidth = 16,
> .mas_rpm_id = -1,
> .slv_rpm_id = 29,
> - .qos.ap_owned = false,
> - .qos.qos_mode = -1,
> - .qos.areq_prio = 0,
> - .qos.prio_level = 0,
> - .qos.qos_port = -1,
> };
>
> static struct qcom_icc_node *sdm660_a2noc_nodes[] = {
> @@ -2113,7 +1834,7 @@ static int qcom_icc_set_bimc_qos(struct icc_node *src, u64 max_bw,
> provider = src->provider;
> qp = to_qcom_provider(provider);
>
> - if (qn->qos.qos_mode != -1)
> + if (qn->qos.qos_mode != NOC_QOS_MODE_INVALID)
> mode = qn->qos.qos_mode;
>
> /* QoS Priority: The QoS Health parameters are getting considered
> @@ -2171,7 +1892,7 @@ static int qcom_icc_set_noc_qos(struct icc_node *src, u64 max_bw)
> return 0;
> }
>
> - if (qn->qos.qos_mode != -1)
> + if (qn->qos.qos_mode != NOC_QOS_MODE_INVALID)
> mode = qn->qos.qos_mode;
>
> if (mode == NOC_QOS_MODE_FIXED) {
> @@ -2264,7 +1985,7 @@ static int qcom_icc_set(struct icc_node *src, struct icc_node *dst)
> ret = qcom_icc_rpm_set(qn->mas_rpm_id, qn->slv_rpm_id, sum_bw);
> if (ret)
> return ret;
> - } else if (qn->qos.qos_mode != -1) {
> + } else if (qn->qos.qos_mode != NOC_QOS_MODE_INVALID) {
> /* set bandwidth directly from the AP */
> ret = qcom_icc_qos_set(src, sum_bw);
> if (ret)
>
next prev parent reply other threads:[~2021-09-04 10:59 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-09-03 23:24 [PATCH v2 0/11] interconnect: merge AP-owned support into icc-rpm Dmitry Baryshkov
2021-09-03 23:24 ` [PATCH v2 01/11] interconnect: icc-rpm: move bus clocks handling into qnoc_probe Dmitry Baryshkov
2021-09-04 10:58 ` AngeloGioacchino Del Regno
2021-09-04 11:04 ` Marijn Suijten
2021-10-04 11:49 ` Georgi Djakov
2021-09-03 23:24 ` [PATCH v2 02/11] interconnect: sdm660: expand DEFINE_QNODE macros Dmitry Baryshkov
2021-09-04 10:58 ` AngeloGioacchino Del Regno
2021-09-04 10:58 ` AngeloGioacchino Del Regno
2021-09-04 11:05 ` Marijn Suijten
2021-09-03 23:24 ` [PATCH v2 03/11] interconnect: sdm660: drop default/unused values Dmitry Baryshkov
2021-09-04 10:59 ` AngeloGioacchino Del Regno [this message]
2021-09-04 11:05 ` Marijn Suijten
2021-09-03 23:24 ` [PATCH v2 04/11] interconnect: sdm660: merge common code into icc-rpm Dmitry Baryshkov
2021-09-04 10:59 ` AngeloGioacchino Del Regno
2021-09-04 11:05 ` Marijn Suijten
2021-10-04 11:54 ` Georgi Djakov
2021-09-03 23:24 ` [PATCH v2 05/11] interconnect: icc-rpm: add support for QoS reg offset Dmitry Baryshkov
2021-09-04 10:59 ` AngeloGioacchino Del Regno
2021-09-03 23:24 ` [PATCH v2 06/11] interconnect: msm8916: expand DEFINE_QNODE macros Dmitry Baryshkov
2021-09-04 10:59 ` AngeloGioacchino Del Regno
2021-09-03 23:24 ` [PATCH v2 07/11] interconnect: msm8916: add support for AP-owned nodes Dmitry Baryshkov
2021-09-04 11:00 ` AngeloGioacchino Del Regno
2021-09-03 23:24 ` [PATCH v2 08/11] interconnect: msm8939: expand DEFINE_QNODE macros Dmitry Baryshkov
2021-09-04 11:00 ` AngeloGioacchino Del Regno
2021-09-03 23:24 ` [PATCH v2 09/11] interconnect: msm8939: add support for AP-owned nodes Dmitry Baryshkov
2021-09-04 11:00 ` AngeloGioacchino Del Regno
2021-09-03 23:24 ` [PATCH v2 10/11] interconnect: qcs404: expand DEFINE_QNODE macros Dmitry Baryshkov
2021-09-04 11:00 ` AngeloGioacchino Del Regno
2021-09-03 23:24 ` [PATCH v2 11/11] interconnect: qcom: drop DEFINE_QNODE macro Dmitry Baryshkov
2021-09-04 11:00 ` AngeloGioacchino Del Regno
2021-09-04 11:05 ` Marijn Suijten
2021-09-06 5:42 ` [PATCH v2 0/11] interconnect: merge AP-owned support into icc-rpm Shawn Guo
2021-09-25 19:40 ` Dmitry Baryshkov
2021-10-04 11:56 ` Georgi Djakov
2021-10-04 12:37 ` Dmitry Baryshkov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=6791b29a-db47-0280-5178-c2277498e14e@somainline.org \
--to=angelogioacchino.delregno@somainline.org \
--cc=agross@kernel.org \
--cc=bjorn.andersson@linaro.org \
--cc=djakov@kernel.org \
--cc=dmitry.baryshkov@linaro.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=shawn.guo@linaro.org \
--cc=y.oudjana@protonmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).