Re: [RFC PATCH 2/2] ARM: OMAP2: dra7: remove davinci mdio hwmod

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 06/21/2016 02:56 PM, Tony Lindgren wrote:
* Tony Lindgren <tony@xxxxxxxxxxx> [160418 17:15]:
* Tony Lindgren <tony@xxxxxxxxxxx> [160418 16:34]:
* Grygorii Strashko <grygorii.strashko@xxxxxx> [160418 10:44]:
The Davinci MDIO hwod 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 now, because "fck" clock is added
in DT for Davinci MDIO node explicitly.

I think you're right here. At least dra7 has just a single module
for cpsw at 0x48484000, ap 3 10.0, size 0x4000. I'll check the
other SoCs too.

Yeah it seems we only have one module for all the cpsw related
IP blocks.

So are these safe to apply in whatever order?


Patch 1 should go first.

but do you agree with comment I've added in patch 1:
>>>

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
<<<

--
regards,
-grygorii
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux