linux-kernel - Re: [PATCH] arm64: dts: qcom: lemans: Remove unnecessary cells from DSI nodes

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] [day] [month] [year] [list]

Message-ID: <ww5libgz6c6cfisch5um7sanhdvnm6aoz6uaqq4vbiraygmiei@fmxpat6ofd7a>
Date: Tue, 16 Sep 2025 18:24:02 +0300
From: Dmitry Baryshkov <dmitry.baryshkov@....qualcomm.com>
To: Eugen Hristev <eugen.hristev@...aro.org>
Cc: andersson@...nel.org, linux-arm-msm@...r.kernel.org,
        devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] arm64: dts: qcom: lemans: Remove unnecessary cells from
 DSI nodes

On Tue, Sep 16, 2025 at 06:11:07PM +0300, Eugen Hristev wrote:
> 
> 
> On 9/16/25 18:03, Dmitry Baryshkov wrote:
> > On Tue, Sep 16, 2025 at 05:19:31PM +0300, Eugen Hristev wrote:
> >> Fix warnings
> >> Warning (avoid_unnecessary_addr_size): /soc@...isplay-subsystem@...0000/dsi@...4000:
> >> unnecessary #address-cells/#size-cells without "ranges", "dma-ranges" or child "reg" property
> >>
> >> Fixes: 73db32b01c9f ("arm64: dts: qcom: sa8775p: add Display Serial Interface device nodes")
> >> Signed-off-by: Eugen Hristev <eugen.hristev@...aro.org>
> >> ---
> >> I haven't found a pending patch for this, so here it goes, sorry if someone
> >> already sent.
> > 
> > Thanks, but no. The nodes are correct. I don't think we should be having
> > the -cells boilerplate each time somebody adds a panel under the DSI
> > node.
> > 
> 
> I understand your point. But leaving out warnings for a possible future
> node is not ok from my point of view.

Then please fix the compiler not to produce those warnings. This applies
to most if not all DSI hosts, not just DSI controllers on Qualcomm
lemans platform.

> 
> >>
> >>  arch/arm64/boot/dts/qcom/lemans.dtsi | 6 ------
> >>  1 file changed, 6 deletions(-)
> >>
> > 
> 

-- 
With best wishes
Dmitry