RE: [EXTERNAL] Re: [PATCH v1 4/4] ASoc: dt-bindings: Create yaml file for pcm6240 codec driver

["Ding, Shenghao" <shenghao-ding@ti.com>]

> -----Original Message-----
> From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> Sent: Thursday, January 25, 2024 3:50 PM
> To: Ding, Shenghao <shenghao-ding@ti.com>; broonie@kernel.org;
> conor+dt@kernel.org
> Cc: robh+dt@kernel.org; andriy.shevchenko@linux.intel.com; Lu, Kevin
> <kevin-lu@ti.com>; Xu, Baojun <baojun.xu@ti.com>;
> devicetree@vger.kernel.org; lgirdwood@gmail.com; perex@perex.cz;
> pierre-louis.bossart@linux.intel.com; 13916275206@139.com; linux-
> sound@vger.kernel.org; linux-kernel@vger.kernel.org;
> liam.r.girdwood@intel.com; soyer@irl.hu; Huang, Jonathan
> <jkhuang3@ti.com>; tiwai@suse.de; Djuandi, Peter <pdjuandi@ti.com>;
> McPherson, Jeff <j-mcpherson@ti.com>; Navada Kanyana, Mukund
> <navada@ti.com>
> Subject: Re: [EXTERNAL] Re: [PATCH v1 4/4] ASoc: dt-bindings: Create yaml
> file for pcm6240 codec driver
> 
> On 25/01/2024 08: 39, Ding, Shenghao wrote: >> >> Why do you repeat the
> reg constraints? This does not seem needed. >> >>> + interrupts: false >>>
> + >>> + - if: >>> + properties: >>> 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 25/01/2024 08:39, Ding, Shenghao wrote:
> >>
> >> Why do you repeat the reg constraints? This does not seem needed.
> >>
> >>> +        interrupts: false
> >>> +
> >>> +  - if:
> >>> +      properties:
> >>> +        compatible:
> >>> +          contains:
> >>> +            enum:
> >>> +              - ti,pcm3140
> >>> +              - ti,pcm5140
> >>> +              - ti,pcm6140
> >>> +              - ti,pcmd3180
> >>> +    then:
> >>> +      properties:
> >>> +        reg:
> >>> +          description:
> >>> +            I2C address, in multiple pcmdevices case, all the i2c address
> >>> +            aggregate as one Audio Device to support multiple audio slots.
> >>> +          maxItems: 4
> >>> +          minItems: 1
> >>
> 
> You did not respond to any of other comments, therefore I assume you
> agree with them 100% and you will implement them fully.
> 
> 
> >> Drop entire if
> > How to convey ti,pcm1690 does not support interrupt, and others
> > support if I remove this if
> 
> How? There is no pcm1690 here.
How can others know that pcm3140, pcm5140, pcm6140 and pcmd3180
Support i2c address from 0x4c to 0x4f, if this if branch was removed.
> 
> BTW, add missing line breaks, especially before next blocks like
> additionalProperties.
Accepted
> 
> >>
> >>> +          items:
> >>> +            minimum: 0x4c
> >>> +            maximum: 0x4f
> >>> +
> >
> > BR
> > Shenghao Ding
> >
> 
> Best regards,
> Krzysztof