Commit aae7948d authored by Nícolas F. R. A. Prado's avatar Nícolas F. R. A. Prado Committed by Greg Kroah-Hartman

Revert "dt-bindings: usb: mtk-xhci: Make all clocks required"

This reverts commit ebc4969a. That
commit was supposed to make the binding better reflect the MediaTek XHCI
hardware block by requiring all clocks to be present. But doing that
also causes too much noise in the devicetrees, since it requires
updating old MediaTek DTs to add clock handles for the fixed clocks, and
going forward every new clock added to the binding would require even
more updates.

The commit also didn't update the example to match the changes, causing
additional warnings.

Instead let's keep the clocks optional so that old devicetrees can keep
omitting the fixed clocks, and we'll just add the clocks as required on
new DTs.

Fixes: ebc4969a ("dt-bindings: usb: mtk-xhci: Make all clocks required")
Reviewed-by: default avatarAngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
Reviewed-by: default avatarChunfeng Yun <chunfeng.yun@mediatek.com>
Acked-by: default avatarRob Herring <robh@kernel.org>
Signed-off-by: default avatarNícolas F. R. A. Prado <nfraprado@collabora.com>
Link: https://lore.kernel.org/r/20220708192605.43351-1-nfraprado@collabora.comSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 421c8d9a
...@@ -68,6 +68,7 @@ properties: ...@@ -68,6 +68,7 @@ properties:
maxItems: 1 maxItems: 1
clocks: clocks:
minItems: 1
items: items:
- description: Controller clock used by normal mode - description: Controller clock used by normal mode
- description: Reference clock used by low power mode etc - description: Reference clock used by low power mode etc
...@@ -76,8 +77,9 @@ properties: ...@@ -76,8 +77,9 @@ properties:
- description: controller clock - description: controller clock
clock-names: clock-names:
minItems: 1
items: items:
- const: sys_ck - const: sys_ck # required, the following ones are optional
- const: ref_ck - const: ref_ck
- const: mcu_ck - const: mcu_ck
- const: dma_ck - const: dma_ck
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment