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: <B12D1E5C-76DC-4B71-B1E4-39F83B1FFFC5@linaro.org>
Date: Mon, 07 Oct 2024 10:33:49 +0300
From: Dmitry Baryshkov <dmitry.baryshkov@...aro.org>
To: Viken Dadhaniya <quic_vdadhani@...cinc.com>
CC: andersson@...nel.org, konradybcio@...nel.org, robh@...nel.org,
 krzk+dt@...nel.org, conor+dt@...nel.org, linux-arm-msm@...r.kernel.org,
 devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
 quic_msavaliy@...cinc.com, quic_anupkulk@...cinc.com
Subject: Re: [RESEND PATCH v2] arm64: dts: qcom: sa8775p: Populate additional UART DT nodes

On October 7, 2024 9:06:52 AM GMT+03:00, Viken Dadhaniya <quic_vdadhani@...cinc.com> wrote:
>
>
>On 10/7/2024 2:26 AM, Dmitry Baryshkov wrote:
>> On Mon, Sep 30, 2024 at 02:25:33PM GMT, Viken Dadhaniya wrote:
>>> Currently, UART configuration is populated for few
>>> SEs(Serial engine) only in sa8775p DTSI file.
>>> 
>>> Populate UART configurations of remaining SEs for sa8775p soc.
>> 
>> I think this is a long standing tradition of adding UART configuration
>> only to those nodes that actually will be used for UART on one or
>> another device. Is this the case for all these UART nodes?
>
>Recently, we have begun adding all possible UART configurations, similar to what we’ve done with I2C and SPI.
>We will continue this approach for future targets.


Please describe why it is done, not what is being done by the patch. In other words, why do you need all uarts in this case.


>
>> 
>>> 
>>> Co-developed-by: Mukesh Kumar Savaliya <quic_msavaliy@...cinc.com>
>>> Signed-off-by: Mukesh Kumar Savaliya <quic_msavaliy@...cinc.com>
>>> Signed-off-by: Viken Dadhaniya <quic_vdadhani@...cinc.com>
>>> ---
>>> v1 -> v2:
>>> 
>>> - Modifed commit log as requested by Krzysztof.
>>> - Added co-developed-by tag.
>>> 
>>> v1 Link: https://lore.kernel.org/linux-arm-msm/98e7dc28-4413-4247-bad1-98b529f6d62d@kernel.org/T/
>>> ---
>>> ---
>>>   arch/arm64/boot/dts/qcom/sa8775p.dtsi | 231 ++++++++++++++++++++++++++
>>>   1 file changed, 231 insertions(+)
>>> 
>> 


-- 
With best wishes
Dmitry

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ