On Fri, 2022-12-09 at 15:56 +0100, Krzysztof Kozlowski wrote:
On 09/12/2022 11:56, Trevor Wu (吳文良) wrote:
+patternProperties:
- "^mediatek,etdm-in[1-2]-chn-disabled$":
- $ref: /schemas/types.yaml#/definitions/uint8-array
- minItems: 1
- maxItems: 16
- description:
By default, all data received from ETDM pins will beoutputed to
memory. etdm in supports disable_out in direct mode(w/ointerconn).
User can specify the channel ID which they hope droppingand then
the specified channel won't be seen on memory.So we know what are the IDs but it's a mystery what are the values. Especially with unique values - how any of these should case that channel "won't be seen in memory"?
For example, FE can support 14ch, but BE(etdm) can't support 14ch(it can support 16ch). In the case, we can configure 16ch to etdm and make use of the property to disable the last two channels.
mediatek,etdm-in1-chn-disabled = /bits/ 8 <0xE 0xF>;
Your description should explain that this is a list of channel IDs which should be disabled.
Hi Krzysztof,
Thanks for your suggestion. I'll put it into the description.
- uniqueItems: true
- items:
minimum: 0maximum: 15- "^mediatek,etdm-in[1-2]-mclk-always-on-rate-hz$":
- description: Specify etdm in mclk output rate for always
on case.
How is it different than assigned-clock-rates?
This includes clock enabling at init stage.
assigned-clock-rates are also at init stage. I asked what is different.
If the property is used, there are three parts included in dai driver probe function.
1. set clock parent (which APLL) 2. set clock rate (MCLK rate) 3. enable clock (MCLK On)
The first two parts can be done by existing dts clock properties, but the last one can't. When MCLK should be enabled at boot time and kept on, this property is used. That's why I say the property is designed for always-on case.
Thanks, Trevor.
- "^mediatek,etdm-out[1-3]-mclk-always-on-rate-hz$":
- description: Specify etdm out mclk output rate for always
on case.
- "^mediatek,etdm-in[1-2]-multi-pin-mode$":
- type: boolean
- description: if present, the etdm data mode is I2S.
- "^mediatek,etdm-out[1-3]-multi-pin-mode$":
- type: boolean
- description: if present, the etdm data mode is I2S.
- "^mediatek,etdm-in[1-2]-cowork-source$":
- $ref: /schemas/types.yaml#/definitions/uint32
- description:
etdm modules can share the same external clock pin.Specify
which etdm clock source is required by this etdm inmoudule.
typo: module
Best regards, Krzysztof