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]
Date: Thu, 7 Mar 2024 13:13:43 +0000
From: Volodymyr Babchuk <Volodymyr_Babchuk@...m.com>
To: Sumit Garg <sumit.garg@...aro.org>
CC: Dmitry Baryshkov <dmitry.baryshkov@...aro.org>,
        Bjorn Andersson
	<andersson@...nel.org>,
        Konrad Dybcio <konrad.dybcio@...aro.org>,
        Rob Herring
	<robh+dt@...nel.org>,
        Krzysztof Kozlowski
	<krzysztof.kozlowski+dt@...aro.org>,
        Conor Dooley <conor+dt@...nel.org>,
        "linux-arm-msm@...r.kernel.org" <linux-arm-msm@...r.kernel.org>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] arm64: dts: qcom: sm8150: add reset name for ethernet
 node


Hello Linux Maintainers, Sumit

First, I am terribly sorry about this half-assed patch. Generally I am
doing all the required checks. But this change seemed so
trivial... Anyways, lesson taken, this will not happen anymore.

Sumit Garg <sumit.garg@...aro.org> writes:

> On Thu, 7 Mar 2024 at 12:40, Dmitry Baryshkov
> <dmitry.baryshkov@...aro.org> wrote:
>>
>> On Thu, 7 Mar 2024 at 00:22, Volodymyr Babchuk
>> <Volodymyr_Babchuk@...m.com> wrote:
>> >
>> > Add reset-names property to the ethernet@...00 node. This patch does
>> > not change behavior on Linux, but it is needed for U-Boot, as it tries
>> > to find the reset by name, not by index.
>> >
>> > Signed-off-by: Volodymyr Babchuk <volodymyr_babchuk@...m.com>
>> > ---
>> >  arch/arm64/boot/dts/qcom/sm8150.dtsi | 1 +
>> >  1 file changed, 1 insertion(+)
>> >
>> > diff --git a/arch/arm64/boot/dts/qcom/sm8150.dtsi b/arch/arm64/boot/dts/qcom/sm8150.dtsi
>> > index 761a6757dc26f..c2e65d6a2ac62 100644
>> > --- a/arch/arm64/boot/dts/qcom/sm8150.dtsi
>> > +++ b/arch/arm64/boot/dts/qcom/sm8150.dtsi
>> > @@ -951,6 +951,7 @@ ethernet: ethernet@...00 {
>> >
>> >                         power-domains = <&gcc EMAC_GDSC>;
>> >                         resets = <&gcc GCC_EMAC_BCR>;
>> > +                       resets-names = "emac";
>>
>> According to the snps,dwmac.yaml schema the "emac" is invalid here.
>> Only "stmmaceth" and / or "ahb" are permitted here.
>
> Okay, it looks like earlier the Linux kernel on Qcom SoCs always
> assumed that the EMAC reset signal is deserted by prior boot stages.
> So I suppose we can reuse "stmmaceth" here instead of "emac" with a
> corresponding change to U-Boot driver as well.

Maybe it would be better to access reset in U-Boot by index, in the
same way as linux kernel does? I am not sure that "stmmaceth" will be
correct from the semantic point of view.

As I understand, "stmmac" name is used due to historical reasons in
Linux, as this driver was introduced for STM SoC initially. But the same
IP block is being used in many different SoCs made by different vendors
and there is nothing STM-specific left in it anymore. Especially taking
into account that this IP-block was designed not by STM but by
Synopsys/DesignWare.

-- 
WBR, Volodymyr

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ