linux-kernel - Re: [PATCH v2 2/2] interconnect: qcom: Add Milos interconnect provider driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <0a09fbc8-8835-4e94-862b-0baaea5ee251@oss.qualcomm.com>
Date: Tue, 8 Jul 2025 13:30:16 +0200
From: Konrad Dybcio <konrad.dybcio@....qualcomm.com>
To: Luca Weiss <luca.weiss@...rphone.com>, Georgi Djakov <djakov@...nel.org>,
        Rob Herring <robh@...nel.org>,
        Krzysztof Kozlowski <krzk+dt@...nel.org>,
        Conor Dooley <conor+dt@...nel.org>
Cc: ~postmarketos/upstreaming@...ts.sr.ht, phone-devel@...r.kernel.org,
        linux-arm-msm@...r.kernel.org, linux-pm@...r.kernel.org,
        devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 2/2] interconnect: qcom: Add Milos interconnect
 provider driver

On 7/8/25 12:20 PM, Luca Weiss wrote:
> Add driver for the Qualcomm interconnect buses found in Milos based
> platforms. The topology consists of several NoCs that are controlled by
> a remote processor that collects the aggregated bandwidth for each
> master-slave pairs.
> 
> Signed-off-by: Luca Weiss <luca.weiss@...rphone.com>
> ---

[...]

> +static struct qcom_icc_bcm * const aggre1_noc_bcms[] = {
> +};

You can remove the empty bcm arrays and .(num_)bcms assignments
for them

[...]

> +static const struct of_device_id qnoc_of_match[] = {
> +	{ .compatible = "qcom,milos-aggre1-noc", .data = &milos_aggre1_noc},
> +	{ .compatible = "qcom,milos-aggre2-noc", .data = &milos_aggre2_noc},
> +	{ .compatible = "qcom,milos-clk-virt", .data = &milos_clk_virt},
> +	{ .compatible = "qcom,milos-cnoc-cfg", .data = &milos_cnoc_cfg},
> +	{ .compatible = "qcom,milos-cnoc-main", .data = &milos_cnoc_main},
> +	{ .compatible = "qcom,milos-gem-noc", .data = &milos_gem_noc},
> +	{ .compatible = "qcom,milos-lpass-ag-noc", .data = &milos_lpass_ag_noc},
> +	{ .compatible = "qcom,milos-mc-virt", .data = &milos_mc_virt},
> +	{ .compatible = "qcom,milos-mmss-noc", .data = &milos_mmss_noc},
> +	{ .compatible = "qcom,milos-nsp-noc", .data = &milos_nsp_noc},
> +	{ .compatible = "qcom,milos-pcie-anoc", .data = &milos_pcie_anoc},
> +	{ .compatible = "qcom,milos-system-noc", .data = &milos_system_noc},
> +	{ }

a space before '}' would be neat

Konrad

> +};
> +MODULE_DEVICE_TABLE(of, qnoc_of_match);
> +
> +static struct platform_driver qnoc_driver = {
> +	.probe = qcom_icc_rpmh_probe,
> +	.remove = qcom_icc_rpmh_remove,
> +	.driver = {
> +		.name = "qnoc-milos",
> +		.of_match_table = qnoc_of_match,
> +		.sync_state = icc_sync_state,

Are there any issues with sync_state? (hopefully not)

Konrad