netdev - Re: [PATCH v2 1/2] dt-bindings: net: qcom,ethqos: add description for qcs9100

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: <972803d2-eec3-4b67-b541-d3fd68475681@quicinc.com>
Date: Mon, 29 Jul 2024 17:50:50 +0800
From: Tengfei Fan <quic_tengfan@...cinc.com>
To: Vinod Koul <vkoul@...nel.org>, "David S. Miller" <davem@...emloft.net>,
        Eric Dumazet <edumazet@...gle.com>, Jakub Kicinski <kuba@...nel.org>,
        "Paolo
 Abeni" <pabeni@...hat.com>, Rob Herring <robh@...nel.org>,
        Krzysztof
 Kozlowski <krzk+dt@...nel.org>,
        Conor Dooley <conor+dt@...nel.org>,
        Bhupesh
 Sharma <bhupesh.sharma@...aro.org>,
        Alexandre Torgue
	<alexandre.torgue@...s.st.com>,
        Giuseppe Cavallaro <peppe.cavallaro@...com>,
        Jose Abreu <joabreu@...opsys.com>,
        Maxime Coquelin
	<mcoquelin.stm32@...il.com>
CC: <kernel@...cinc.com>, <netdev@...r.kernel.org>,
        <linux-arm-msm@...r.kernel.org>, <devicetree@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>,
        <linux-stm32@...md-mailman.stormreply.com>,
        <linux-arm-kernel@...ts.infradead.org>
Subject: Re: [PATCH v2 1/2] dt-bindings: net: qcom,ethqos: add description for
 qcs9100



On 7/9/2024 10:13 PM, Tengfei Fan wrote:
> Add the compatible for the MAC controller on qcs9100 platforms.
> QCS9100 is drived from SA8775p. Currently, both the QCS9100 and SA8775p
> platform use non-SCMI resource. In the future, the SA8775p platform will
> move to use SCMI resources and it will have new sa8775p-related device
> tree. Consequently, introduce "qcom,qcs9100-ethqos" to describe non-SCMI
> based ethqos.
> 
> Signed-off-by: Tengfei Fan <quic_tengfan@...cinc.com>
> ---
>   Documentation/devicetree/bindings/net/qcom,ethqos.yaml | 1 +
>   Documentation/devicetree/bindings/net/snps,dwmac.yaml  | 2 ++
>   2 files changed, 3 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/net/qcom,ethqos.yaml b/Documentation/devicetree/bindings/net/qcom,ethqos.yaml
> index 6672327358bc..8ab11e00668c 100644
> --- a/Documentation/devicetree/bindings/net/qcom,ethqos.yaml
> +++ b/Documentation/devicetree/bindings/net/qcom,ethqos.yaml
> @@ -20,6 +20,7 @@ properties:
>     compatible:
>       enum:
>         - qcom,qcs404-ethqos
> +      - qcom,qcs9100-ethqos
>         - qcom,sa8775p-ethqos
>         - qcom,sc8280xp-ethqos
>         - qcom,sm8150-ethqos
> diff --git a/Documentation/devicetree/bindings/net/snps,dwmac.yaml b/Documentation/devicetree/bindings/net/snps,dwmac.yaml
> index 0ab124324eec..291252f2f30d 100644
> --- a/Documentation/devicetree/bindings/net/snps,dwmac.yaml
> +++ b/Documentation/devicetree/bindings/net/snps,dwmac.yaml
> @@ -67,6 +67,7 @@ properties:
>           - loongson,ls2k-dwmac
>           - loongson,ls7a-dwmac
>           - qcom,qcs404-ethqos
> +        - qcom,qcs9100-ethqos
>           - qcom,sa8775p-ethqos
>           - qcom,sc8280xp-ethqos
>           - qcom,sm8150-ethqos
> @@ -611,6 +612,7 @@ allOf:
>                 - ingenic,x1830-mac
>                 - ingenic,x2000-mac
>                 - qcom,qcs404-ethqos
> +              - qcom,qcs9100-ethqos
>                 - qcom,sa8775p-ethqos
>                 - qcom,sc8280xp-ethqos
>                 - qcom,sm8150-ethqos
> 

After considering the feedback provided on the subject, We have decided
to keep current SA8775p compatible and ABI compatibility in drivers.
Let's close this session and ignore all the current patches here.
Thank you for your input.

-- 
Thx and BRs,
Tengfei Fan