Dear Ezequiel Garcia,
On Wed, 22 Oct 2014 10:34:44 -0300, Ezequiel Garcia wrote:
> - clocks = <&coreclk 0>;
> + clocks = <&refclk>;
> + clock-names = "fixed";
Why not do like we do on Armada XP, and actually represent the
hardware, by showing that it takes two clocks as input:
timer@20300 {
compatible = "marvell,armada-xp-timer";
clocks = <&coreclk 2>, <&refclk>;
clock-names = "nbclk", "fixed";
};
> watchdog@20300 {
> compatible = "marvell,armada-375-wdt";
> reg = <0x20300 0x34>, <0x20704 0x4>, <0x18254 0x4>;
> - clocks = <&coreclk 0>;
> + clocks = <&refclk>;
> + clock-names = "fixed";
> };
Ditto.
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
