linux-kernel - RE: [EXTERNAL] Re: [PATCH v3 4/4] ASoc: dt-bindings: PCM6240: Add initial DT binding

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: <0a1f20ae254746b1adbfbdf7b4027518@ti.com>
Date: Wed, 7 Feb 2024 09:50:51 +0000
From: "Ding, Shenghao" <shenghao-ding@...com>
To: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
CC: "Lu, Kevin" <kevin-lu@...com>, "Xu, Baojun" <baojun.xu@...com>,
        "P O,
 Vijeth" <v-po@...com>, "Navada Kanyana, Mukund" <navada@...com>,
        "perex@...ex.cz" <perex@...ex.cz>,
        "McPherson, Jeff" <j-mcpherson@...com>,
        "pierre-louis.bossart@...ux.intel.com"
	<pierre-louis.bossart@...ux.intel.com>,
        "13916275206@....com"
	<13916275206@....com>,
        "Chawla, Mohit" <mohit.chawla@...com>, "soyer@....hu"
	<soyer@....hu>,
        "Huang, Jonathan" <jkhuang3@...com>, "tiwai@...e.de"
	<tiwai@...e.de>,
        "Djuandi, Peter" <pdjuandi@...com>,
        "Agrawal, Manisha"
	<manisha.agrawal@...com>,
        "Hari, Raj" <s-hari@...com>, "Yashar, Avi"
	<aviel@...com>,
        "Nagalla, Hari" <hnagalla@...com>,
        "Bajjuri, Praneeth"
	<praneeth@...com>,
        "broonie@...nel.org" <broonie@...nel.org>,
        "conor+dt@...nel.org" <conor+dt@...nel.org>,
        "devicetree@...r.kernel.org"
	<devicetree@...r.kernel.org>,
        "robh+dt@...nel.org" <robh+dt@...nel.org>,
        "andriy.shevchenko@...ux.intel.com" <andriy.shevchenko@...ux.intel.com>,
        "linux-sound@...r.kernel.org" <linux-sound@...r.kernel.org>,
        "liam.r.girdwood@...el.com" <liam.r.girdwood@...el.com>,
        "lgirdwood@...il.com" <lgirdwood@...il.com>,
        "linux-kernel@...r.kernel.org"
	<linux-kernel@...r.kernel.org>
Subject: RE: [EXTERNAL] Re: [PATCH v3 4/4] ASoc: dt-bindings: PCM6240: Add
 initial DT binding



> -----Original Message-----
> From: Ding, Shenghao <shenghao-ding@...com>
> Sent: Wednesday, February 7, 2024 5:48 PM
> To: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>;
> broonie@...nel.org; conor+dt@...nel.org; devicetree@...r.kernel.org;
> robh+dt@...nel.org; andriy.shevchenko@...ux.intel.com; linux-
> sound@...r.kernel.org; liam.r.girdwood@...el.com; lgirdwood@...il.com;
> linux-kernel@...r.kernel.org
> Cc: Lu, Kevin <kevin-lu@...com>; Xu, Baojun <baojun.xu@...com>; P O, Vijeth
> <v-po@...com>; Navada Kanyana, Mukund <navada@...com>;
> perex@...ex.cz; McPherson, Jeff <j-mcpherson@...com>; pierre-
> louis.bossart@...ux.intel.com; 13916275206@....com; Chawla, Mohit
> <mohit.chawla@...com>; soyer@....hu; Huang, Jonathan
> <jkhuang3@...com>; tiwai@...e.de; Djuandi, Peter <pdjuandi@...com>;
> Agrawal, Manisha <manisha.agrawal@...com>; Hari, Raj <s-hari@...com>;
> Yashar, Avi <aviel@...com>; Nagalla, Hari <hnagalla@...com>; Bajjuri,
> Praneeth <praneeth@...com>
> Subject: RE: [EXTERNAL] Re: [PATCH v3 4/4] ASoc: dt-bindings: PCM6240:
> Add initial DT binding
> 
> > -----Original Message----- > From: Krzysztof Kozlowski <krzysztof. 
> > kozlowski@ linaro. org> > Sent: Tuesday, February 6, 2024 9: 53 PM >
> > To: Ding, Shenghao <shenghao-ding@ ti. com>; broonie@ kernel. org; >
> > conor+dt@ kernel. org;
> ZjQcmQRYFpfptBannerStart
> This message was sent from outside of Texas Instruments.
> Do not click links or open attachments unless you recognize the source of
> this email and know the content is safe.
> 
> ZjQcmQRYFpfptBannerEnd
> 
> 
> > -----Original Message-----
> > From: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
> > Sent: Tuesday, February 6, 2024 9:53 PM
> > To: Ding, Shenghao <shenghao-ding@...com>; broonie@...nel.org;
> > conor+dt@...nel.org; devicetree@...r.kernel.org; robh+dt@...nel.org;
> > andriy.shevchenko@...ux.intel.com; linux-sound@...r.kernel.org;
> > liam.r.girdwood@...el.com; lgirdwood@...il.com; linux-
> > kernel@...r.kernel.org
> > Cc: Lu, Kevin <kevin-lu@...com>; Xu, Baojun <baojun.xu@...com>; P O,
> > Vijeth <v-po@...com>; Navada Kanyana, Mukund <navada@...com>;
> > perex@...ex.cz; McPherson, Jeff <j-mcpherson@...com>; pierre-
> > louis.bossart@...ux.intel.com; 13916275206@....com; Chawla, Mohit
> > <mohit.chawla@...com>; soyer@....hu; Huang, Jonathan
> > <jkhuang3@...com>; tiwai@...e.de; Djuandi, Peter <pdjuandi@...com>;
> > Agrawal, Manisha <manisha.agrawal@...com>; Hari, Raj <s-hari@...com>;
> > Yashar, Avi <aviel@...com>; Nagalla, Hari <hnagalla@...com>; Bajjuri,
> > Praneeth <praneeth@...com>
> > Subject: [EXTERNAL] Re: [PATCH v3 4/4] ASoc: dt-bindings: PCM6240: Add
> > initial DT binding
> >
> > On 03/02/2024 04: 05, Shenghao Ding wrote: > + > + ti,tad5212:
> > Low-power stereo audio DAC with 120-dB dynamic range. > + oneOf: > + -
> > items: > + -
> > enum: > + - ti,adc3120 > + - ti,adc5120 > + - ti,pcm3120 > + -
> > ti,pcm5120 ZjQcmQRYFpfptBannerStart This message was sent from
> outside
> > of Texas Instruments.
> > Do not click links or open attachments unless you recognize the source
> > of this email and know the content is safe.
> >
> > ZjQcmQRYFpfptBannerEnd
> > On 03/02/2024 04:05, Shenghao Ding wrote:
> > > +
> > > +      ti,tad5212: Low-power stereo audio DAC with 120-dB dynamic range.
> > > +    oneOf:
> > > +      - items:
> > > +          - enum:
> > > +              - ti,adc3120
> > > +              - ti,adc5120
> > > +              - ti,pcm3120
> > > +              - ti,pcm5120
> > > +              - ti,pcm6120
> > > +          - const: ti,adc6120
> > > +      - items:
> > > +          - enum:
> > > +              - ti,pcm6260
> > > +              - ti,pcm6140
> > > +              - ti,pcm3140
> > > +              - ti,pcm5140
> > > +          - const: ti,pcm6240
> > > +      - items:
> > > +          - const: ti,dix4192
> > > +          - const: ti,pcm6240
> >
> > Why dix4192 is not part of previous enum?
> 
> dix4192 is not traditional ADC or DAC, but an Integrated Digital Audio
> Interface Receiver and Transmitter, like an audio bridge to connect different
> digital audio protocol, compatible with the AES3, S/PDIF, IEC 60958, and EIAJ
> CP-1201 interface standards, Left-Justified, Right-Justified, and Philips I2S™
> Data Formats.
> So keep it alone for professional purpose.
I will add detail dix4191 description in next patch.
> 
> >
> > > +      - items:
> > > +          - const: ti,adc6120
> > > +          - const: ti,pcmd512x
> > > +      - items:
> > > +          - const: ti,pcm1690
> > > +          - const: ti,pcm9211
> > > +      - items:
> > > +          - enum:
> > > +              - ti,pcmd3180
> > > +          - const: ti,pcmd3140
> > > +      - items:
> > > +          - enum:
> > > +              - ti,taa5412
> > > +          - const: ti,taa5212
> > > +      - items:
> > > +          - enum:
> > > +              - ti,tad5412
> > > +          - const: ti,tad5212
> > > +      - enum:
> > > +          - ti,pcm6240
> > > +          - ti,pcmd3140
> > > +          - ti,taa5212
> > > +          - ti,tad5212
> > > +          - ti,pcmd3180
> >
> > This one is duplicated.
> accept
> >
> > > +
> >
> >
> > Best regards,
> > Krzysztof