28 Jul
2023
28 Jul
'23
2:51 p.m.
On 28/07/2023 11:08, Maso Huang wrote:
Add mt7986 audio afe document.
Signed-off-by: Maso Huang maso.huang@mediatek.com
Thank you for your patch. There is something to discuss/improve.
+required:
- compatible
- reg
- interrupts
- clocks
- clock-names
- assigned-clocks
- assigned-clock-parents
You should constrain your clocks per variants. I doubt that they are really so flexible/optional on each SoC... or maybe missing clocks are result of unimplemented parts in the driver? But then this should not really affect bindings. Bindings still should require such clocks. Your DTS can always provide a <0>, if needed.
+additionalProperties: false
Best regards, Krzysztof