| Message ID | 20200827083045.76356-4-tomi.valkeinen@ti.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series | None | expand |
diff --git a/arch/arm64/boot/dts/ti/k3-am654-base-board.dts b/arch/arm64/boot/dts/ti/k3-am654-base-board.dts index 611e66207010..99b32038a453 100644 --- a/arch/arm64/boot/dts/ti/k3-am654-base-board.dts +++ b/arch/arm64/boot/dts/ti/k3-am654-base-board.dts @@ -55,7 +55,7 @@ sw6 { }; }; - clk_ov5640_fixed: clock { + clk_ov5640_fixed: fixed-clock-ov5640 { compatible = "fixed-clock"; #clock-cells = <0>; clock-frequency = <24000000>;
The fixed clock for OV5640 is named 'clock' which is a very generic name and easily leads to conflicts. I encountered this with a similarly named fixed-clock node in k3-am654-evm-tc358876.dtso, which then overrode the OV5640 fixed clock, causing OV5640 not to work when tc358876 overlay had been loaded. Rename the node to 'fixed-clock-ov5640'. Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com> --- arch/arm64/boot/dts/ti/k3-am654-base-board.dts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)