Add power-domains property for asrc and spdif.

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 Documentation/devicetree/bindings/sound/fsl,imx-asrc.yaml | 3 +++
 Documentation/devicetree/bindings/sound/fsl,spdif.yaml    | 3 +++
 2 files changed, 6 insertions(+)

diff --git a/Documentation/devicetree/bindings/sound/fsl,imx-asrc.yaml b/Documentation/devicetree/bindings/sound/fsl,imx-asrc.yaml
index bfef2fcb75b14..98616507e3c10 100644
--- a/Documentation/devicetree/bindings/sound/fsl,imx-asrc.yaml
+++ b/Documentation/devicetree/bindings/sound/fsl,imx-asrc.yaml
@@ -74,6 +74,9 @@ properties:
       - const: asrck_f
       - const: spba
 
+  power-domains:
+    maxItems: 1
+
   fsl,asrc-rate:
     $ref: /schemas/types.yaml#/definitions/uint32
     description: The mutual sample rate used by DPCM Back Ends
diff --git a/Documentation/devicetree/bindings/sound/fsl,spdif.yaml b/Documentation/devicetree/bindings/sound/fsl,spdif.yaml
index 1d64e8337aa4b..82430f1d5e5a2 100644
--- a/Documentation/devicetree/bindings/sound/fsl,spdif.yaml
+++ b/Documentation/devicetree/bindings/sound/fsl,spdif.yaml
@@ -86,6 +86,9 @@ properties:
       registers. Set this flag for HCDs with big endian descriptors and big
       endian registers.
 
+  power-domains:
+    maxItems: 1
+
 required:
   - compatible
   - reg

-- 
2.34.1

On 28/02/2024 20:14, Frank Li wrote:
> Add power-domains property for asrc and spdif.

Why?

You keep adding this all over the tree without explanations. Why
suddenly old hardware became part of power domain?

Best regards,
Krzysztof

On Thu, Feb 29, 2024 at 10:44:20AM +0100, Krzysztof Kozlowski wrote:
> On 28/02/2024 20:14, Frank Li wrote:
> > Add power-domains property for asrc and spdif.
> 
> Why?

i.MX8QXP i.MX8DXL and i.MX8QM need it.

> 
> You keep adding this all over the tree without explanations. Why
> suddenly old hardware became part of power domain?

This part upstreamed earlier than dts part. Because dependency eDMAv3 have
not upstreamed for the long time. I just upstreamed eDMAv3 support
recently. So I can start upstream dts part. So I find binding doc missed it
when do dtb_check.

Frank

> 
> Best regards,
> Krzysztof
>

On 01/03/2024 05:01, Frank Li wrote:
> On Thu, Feb 29, 2024 at 10:44:20AM +0100, Krzysztof Kozlowski wrote:
>> On 28/02/2024 20:14, Frank Li wrote:
>>> Add power-domains property for asrc and spdif.
>>
>> Why?
> 
> i.MX8QXP i.MX8DXL and i.MX8QM need it.

Where is it explained in the commit msg? What about other devices/variants?


Best regards,
Krzysztof

On Fri, Mar 01, 2024 at 07:30:10AM +0100, Krzysztof Kozlowski wrote:
> On 01/03/2024 05:01, Frank Li wrote:
> > On Thu, Feb 29, 2024 at 10:44:20AM +0100, Krzysztof Kozlowski wrote:
> >> On 28/02/2024 20:14, Frank Li wrote:
> >>> Add power-domains property for asrc and spdif.
> >>
> >> Why?
> > 
> > i.MX8QXP i.MX8DXL and i.MX8QM need it.
> 
> Where is it explained in the commit msg? What about other devices/variants?

I will add it. These patches are for imx8qxp\imx8dxl\imx8qm only. If other
platform have problem, it should be fixed by new patch. 

> 
> 
> Best regards,
> Krzysztof
>

On 01/03/2024 16:42, Frank Li wrote:
> On Fri, Mar 01, 2024 at 07:30:10AM +0100, Krzysztof Kozlowski wrote:
>> On 01/03/2024 05:01, Frank Li wrote:
>>> On Thu, Feb 29, 2024 at 10:44:20AM +0100, Krzysztof Kozlowski wrote:
>>>> On 28/02/2024 20:14, Frank Li wrote:
>>>>> Add power-domains property for asrc and spdif.
>>>>
>>>> Why?
>>>
>>> i.MX8QXP i.MX8DXL and i.MX8QM need it.
>>
>> Where is it explained in the commit msg? What about other devices/variants?
> 
> I will add it. These patches are for imx8qxp\imx8dxl\imx8qm only. If other
> platform have problem, it should be fixed by new patch. 

I would assume more responsible approach from NXP, not just "their
problem", but anyway that's not what the comment is. Why do you say all
the variants have power domains in one commit, but now in email says
that they might not have? All your patches are vague in this aspect.

Best regards,
Krzysztof

On Mon, Mar 04, 2024 at 04:05:07PM +0100, Krzysztof Kozlowski wrote:
> On 01/03/2024 16:42, Frank Li wrote:
> > On Fri, Mar 01, 2024 at 07:30:10AM +0100, Krzysztof Kozlowski wrote:
> >> On 01/03/2024 05:01, Frank Li wrote:
> >>> On Thu, Feb 29, 2024 at 10:44:20AM +0100, Krzysztof Kozlowski wrote:
> >>>> On 28/02/2024 20:14, Frank Li wrote:
> >>>>> Add power-domains property for asrc and spdif.
> >>>>
> >>>> Why?
> >>>
> >>> i.MX8QXP i.MX8DXL and i.MX8QM need it.
> >>
> >> Where is it explained in the commit msg? What about other devices/variants?
> > 
> > I will add it. These patches are for imx8qxp\imx8dxl\imx8qm only. If other
> > platform have problem, it should be fixed by new patch. 
> 
> I would assume more responsible approach from NXP, not just "their
> problem", but anyway that's not what the comment is. Why do you say all
> the variants have power domains in one commit, but now in email says
> that they might not have? All your patches are vague in this aspect.

I am not means "their problem". What's I said is one patch just fix one
thing. If need additional fix,  I can create a new patches. I have not said
all need power domains specifically. If there are some words that let me
think all variant need power domain, please let me where, I can improve it.

Frank
> 
> Best regards,
> Krzysztof
>