linux-kernel - Re: [PATCH] arm64: dts: qcom: sc7280: Update SNPS Phy params for SC7280

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: <4d391a78-f524-5bb7-92e0-9778396ed0cb@linaro.org>
Date:   Thu, 29 Sep 2022 11:14:46 +0200
From:   Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
To:     Dmitry Baryshkov <dmitry.baryshkov@...aro.org>,
        Krishna Kurapati PSSNV <quic_kriskura@...cinc.com>
Cc:     Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
        Rob Herring <robh+dt@...nel.org>,
        Andy Gross <agross@...nel.org>,
        Bjorn Andersson <bjorn.andersson@...aro.org>,
        Konrad Dybcio <konrad.dybcio@...ainline.org>,
        devicetree@...r.kernel.org, linux-arm-msm@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH] arm64: dts: qcom: sc7280: Update SNPS Phy params for
 SC7280

On 29/09/2022 11:02, Dmitry Baryshkov wrote:
> On Thu, 29 Sept 2022 at 11:54, Krishna Kurapati PSSNV
> <quic_kriskura@...cinc.com> wrote:
>>
>>
>> On 9/29/2022 1:29 PM, Dmitry Baryshkov wrote:
>>> On Thu, 29 Sept 2022 at 10:14, Krishna Kurapati
>>> <quic_kriskura@...cinc.com> wrote:
>>>>
>>>> Override the SNPS Phy tuning parameters for SC7280 devices. These
>>>> values are common for both trogdor and herobrine variants.
>>>
>>> They are common for trogdor and herobrine, but should these parameters
>>> be a default? In other words, a random new device based on sc7280
>>> would more likely use these overrides or the hardware defaults?
>>>
>> Hi Dmitry,
>>
>>    Currently there are only two platforms, so I made these changes on
>> common dtsi. If a new platform comes (mostly it won't) we can override
>> them in platform specific file is what I thought.
> 
> This is not how it usually works. The 'sc7280.dtsi' is not a 'common
> dtsi' for trogdor and herobrine. It describes the SoC.
> Thus in my opinion if these overrides should be a default to all
> sc7280 platforms, this patch is fine. If these overrides are
> applicable only to the two mentioned platforms, they should go to
> respective platform-specific DT files.

Dmitry's conclusion is correct here. The true question is whether these
are properties of the SoC itself (so do not depend on the board or board
layout) or these depend on design of board.

Best regards,
Krzysztof