> -----Original Message----- > From: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> > Sent: Wednesday, February 7, 2024 5:53 PM > To: Ding, Shenghao <shenghao-ding@xxxxxx>; broonie@xxxxxxxxxx; > conor+dt@xxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; robh+dt@xxxxxxxxxx; > andriy.shevchenko@xxxxxxxxxxxxxxx; linux-sound@xxxxxxxxxxxxxxx; > liam.r.girdwood@xxxxxxxxx; lgirdwood@xxxxxxxxx; linux- > kernel@xxxxxxxxxxxxxxx > Cc: Lu, Kevin <kevin-lu@xxxxxx>; Xu, Baojun <baojun.xu@xxxxxx>; P O, Vijeth > <v-po@xxxxxx>; Navada Kanyana, Mukund <navada@xxxxxx>; > perex@xxxxxxxx; McPherson, Jeff <j-mcpherson@xxxxxx>; pierre- > louis.bossart@xxxxxxxxxxxxxxx; 13916275206@xxxxxxx; Chawla, Mohit > <mohit.chawla@xxxxxx>; soyer@xxxxxx; Huang, Jonathan > <jkhuang3@xxxxxx>; tiwai@xxxxxxx; Djuandi, Peter <pdjuandi@xxxxxx>; > Agrawal, Manisha <manisha.agrawal@xxxxxx>; Hari, Raj <s-hari@xxxxxx>; > Yashar, Avi <aviel@xxxxxx>; Nagalla, Hari <hnagalla@xxxxxx>; Bajjuri, > Praneeth <praneeth@xxxxxx> > Subject: Re: [EXTERNAL] Re: [PATCH v3 4/4] ASoc: dt-bindings: PCM6240: > Add initial DT binding > > On 07/02/2024 10: 47, Ding, Shenghao wrote: > > >> -----Original Message--- > -- >> From: Krzysztof Kozlowski <krzysztof. kozlowski@ linaro. org> >> Sent: > Tuesday, February 6, 2024 9: 53 PM >> To: Ding, Shenghao > 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 07/02/2024 10:47, Ding, Shenghao wrote: > > > > > >> -----Original Message----- > >> From: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> > >> Sent: Tuesday, February 6, 2024 9:53 PM > >> To: Ding, Shenghao <shenghao-ding@xxxxxx>; broonie@xxxxxxxxxx; > >> conor+dt@xxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; robh+dt@xxxxxxxxxx; > >> andriy.shevchenko@xxxxxxxxxxxxxxx; linux-sound@xxxxxxxxxxxxxxx; > >> liam.r.girdwood@xxxxxxxxx; lgirdwood@xxxxxxxxx; linux- > >> kernel@xxxxxxxxxxxxxxx > >> Cc: Lu, Kevin <kevin-lu@xxxxxx>; Xu, Baojun <baojun.xu@xxxxxx>; P O, > >> Vijeth <v-po@xxxxxx>; Navada Kanyana, Mukund <navada@xxxxxx>; > >> perex@xxxxxxxx; McPherson, Jeff <j-mcpherson@xxxxxx>; pierre- > >> louis.bossart@xxxxxxxxxxxxxxx; 13916275206@xxxxxxx; Chawla, Mohit > >> <mohit.chawla@xxxxxx>; soyer@xxxxxx; Huang, Jonathan > >> <jkhuang3@xxxxxx>; tiwai@xxxxxxx; Djuandi, Peter <pdjuandi@xxxxxx>; > >> Agrawal, Manisha <manisha.agrawal@xxxxxx>; Hari, Raj <s-hari@xxxxxx>; > >> Yashar, Avi <aviel@xxxxxx>; Nagalla, Hari <hnagalla@xxxxxx>; Bajjuri, > >> Praneeth <praneeth@xxxxxx> > >> 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. > > Hm, it is a bit surprising to see some PCM6240-compatible devices 100% > different from other PCM6240-compatible. PCM6240 is ADC. DIX4192 is not > ADC, not even DAC. How can it be compatible with PCM6240 in such case? > They all belong to audio convertor family. > Best regards, > Krzysztof >
