linux-kernel - Re: Re: [PATCH 5/7] arm64: dts: ti: k3-am62-verdin: Drop McASP AFIFOs

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: <yd4doskkq443xkvc4ahnxexyi7f5oteoh5egiv3plcduyl7dfr@dgtdwd3dbdkg>
Date: Wed, 5 Jun 2024 14:28:22 +0530
From: Jai Luthra <j-luthra@...com>
To: Francesco Dolcini <francesco@...cini.it>
CC: Nishanth Menon <nm@...com>, Vignesh Raghavendra <vigneshr@...com>,
        Tero
 Kristo <kristo@...nel.org>, Rob Herring <robh@...nel.org>,
        Krzysztof
 Kozlowski <krzk+dt@...nel.org>,
        Conor Dooley <conor+dt@...nel.org>,
        Jayesh
 Choudhary <j-choudhary@...com>,
        Devarsh Thakkar <devarsht@...com>, Bryan
 Brattlof <bb@...com>,
        Aradhya Bhatia <a-bhatia1@...com>,
        Francesco Dolcini
	<francesco.dolcini@...adex.com>,
        <linux-arm-kernel@...ts.infradead.org>, <devicetree@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>
Subject: Re: Re: [PATCH 5/7] arm64: dts: ti: k3-am62-verdin: Drop McASP AFIFOs

Hi Francesco,

Thanks for the review.

On Jun 05, 2024 at 10:47:13 +0200, Francesco Dolcini wrote:
> Hello Jai, thanks for the patch.
> 
> On Tue, Jun 04, 2024 at 03:11:06PM +0530, Jai Luthra wrote:
> > McASP AFIFOs are not necessary with UDMA-P/BCDMA as there is buffering
> > on the DMA IP. Drop these for better audio latency.
> > 
> > Fixes: 316b80246b16 ("arm64: dts: ti: add verdin am62")
> > Signed-off-by: Jai Luthra <j-luthra@...com>
> > ---
> >  arch/arm64/boot/dts/ti/k3-am62-verdin.dtsi | 8 ++++----
> >  1 file changed, 4 insertions(+), 4 deletions(-)
> > 
> > diff --git a/arch/arm64/boot/dts/ti/k3-am62-verdin.dtsi b/arch/arm64/boot/dts/ti/k3-am62-verdin.dtsi
> > index 2038c5e04639..27e5220e1dc7 100644
> > --- a/arch/arm64/boot/dts/ti/k3-am62-verdin.dtsi
> > +++ b/arch/arm64/boot/dts/ti/k3-am62-verdin.dtsi
> > @@ -1364,8 +1364,8 @@ &mcasp0 {
> >  	       0 0 0 0
> >  	>;
> >  	tdm-slots = <2>;
> > -	rx-num-evt = <32>;
> > -	tx-num-evt = <32>;
> > +	rx-num-evt = <0>;
> > +	tx-num-evt = <0>;
> 
> From my understanding of the dt-binding having these properties set to
> <0> is equivalent to not having those properties at all.
> My suggestion would be to just remove those, therefore.
> 

I was following the existing convention of setting it to 0 for 
disabling, as done in k3-j721e-common-proc-board.dts

IMO it is better to keep it explicit, but I don't feel strongly against 
dropping it either (as long as we do the same for j721e). Will defer the 
decision to the tree maintainers.

> With that said, in any case, whatever you do my suggested change or 
> not,
> 
> Acked-by: Francesco Dolcini <francesco.dolcini@...adex.com>
> 
> Francesco
> 

-- 
Thanks,
Jai

GPG Fingerprint: 4DE0 D818 E5D5 75E8 D45A AFC5 43DE 91F9 249A 7145