Re: [PATCH v5 2/3] ARM: dts: add dts files for exynos5260 SoC

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

 



On 13.03.2014 06:06, Rahul Sharma wrote:
On 13 March 2014 06:28, Pankaj Dubey <pankaj.dubey@xxxxxxxxxxx> wrote:
On 03/13/2014 12:16 AM, Rahul Sharma wrote:

[snip]

+               clocks {
+                       compatible = "simple-bus";
+                       #address-cells = <1>;
+                       #size-cells = <0>;


Even though you added  #address-cells property I can not see "reg" property
in any of "fixed-clock".
Isn't it better either we can remove this or add "reg" property?
Also all these fixed-clock are missing #clock-cells property.


Since it is a soc level addition, I enabled provision for adding reg
property for
fixed clocks in boards files (or in SoC file).


All sub-nodes should follow the same pattern.

After recent discussion with DT people, the conclusion is that simple-bus should be used only for MMIO platform devices with "reg" values mapped directly into address space in which the simple-bus node resides.

So for now I'd remove the grouping and keep the clocks in "soc" node directly.

[snip]

+
+                       dptx_phy_ch0: phyclk_dptx_phy_ch0_txd {
+                               compatible = "fixed-clock";
+                               clock-frequency = <270000000>;
+                               clock-output-names =
"phyclk_dptx_phy_ch0_txd_clk";
+                       };

I'm not sure whether these clocks are really fixed clocks. They are output from certain PHY blocks which are not always-on, while using the fixed clock binding would suggest otherwise. IMHO they should be hidden inside the clock driver, without DT IDs assigned as a temporary hack, until proper support gets added for them (e.g. proper clock provider from appropriate PHY).

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




[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux