diff mbox

[RFC,1/2] ARM: dts: dra7: remove davinci_mdio hwmod

Message ID 1461001397-25446-1-git-send-email-grygorii.strashko@ti.com (mailing list archive)
State New, archived
Headers show

Commit Message

Grygorii Strashko April 18, 2016, 5:43 p.m. UTC
The Davinci MDIO hwmod perform only one function now - registers "fck"
clock alias for MDIO functional clock. From all other points of view
it's fake: it's part of cpsw and do not have clkctrl or sysc
registers.

Hence, its safe to remove it and add "fck" in DT explicitly.

Signed-off-by: Grygorii Strashko <grygorii.strashko@ti.com>
---
In general, Davinci mdio hwmode can be removed for dra7/am33/am43/dm814x SoCs now
and for omap3/dm816x - after some additional re-working. 

RFC: because it will break DT ABI, mdio will not work on devices with
old DTs and new kernel which still do have [ti,hwmods = "davinci_mdio"]
property defined in DT:
 platform 48485000.mdio: Cannot lookup hwmod 'davinci_mdio'
 davinci_mdio 48485000.mdio: _od_fail_runtime_resume: FIXME: missing hwmod/omap_dev info
 davinci_mdio: probe of 48485000.mdio failed with error -5

 arch/arm/boot/dts/dra7.dtsi | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
diff mbox

Patch

diff --git a/arch/arm/boot/dts/dra7.dtsi b/arch/arm/boot/dts/dra7.dtsi
index 13ac882..e264c14 100644
--- a/arch/arm/boot/dts/dra7.dtsi
+++ b/arch/arm/boot/dts/dra7.dtsi
@@ -1494,7 +1494,8 @@ 
 				compatible = "ti,davinci_mdio";
 				#address-cells = <1>;
 				#size-cells = <0>;
-				ti,hwmods = "davinci_mdio";
+				clocks = <&dpll_gmac_ck>;
+				clock-names = "fck";
 				bus_freq = <1000000>;
 				reg = <0x48485000 0x100>;
 			};