Re: [PATCH 2/7] clk: intel: Add clock driver for GRX500 SoC

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

 





On 6/14/2018 10:09 PM, Rob Herring wrote:
On Thu, Jun 14, 2018 at 2:40 AM, yixin zhu <yixin.zhu@xxxxxxxxxxxxxxx> wrote:


On 6/13/2018 6:37 AM, Rob Herring wrote:

On Tue, Jun 12, 2018 at 01:40:29PM +0800, Songjun Wu wrote:

From: Yixin Zhu <yixin.zhu@xxxxxxxxxxxxxxx>

PLL of GRX500 provide clock to DDR, CPU, and peripherals as show below

[...]

+Example:
+       clkgate0: clkgate0 {
+               #clock-cells = <1>;
+               compatible = "intel,grx500-gate0-clk";
+               reg = <0x114>;
+               clock-output-names = "gate_xbar0", "gate_xbar1",
"gate_xbar2",
+               "gate_xbar3", "gate_xbar6", "gate_xbar7";
+       };


We generally don't do a clock node per clock or few clocks but rather 1
clock node per clock controller block. See any recent clock bindings.

Rob

Do you mean only one example is needed per clock controller block?
cpuclk is not needed in the document?

No, I mean generally we have 1 DT node for the h/w block with all the
clock control registers rather than nodes with a single register and 1
or a couple of clocks. Sometimes the clock registers are mixed with
other functions which complicates things a bit. But I can't tell that
here because you haven't documented what's in the rest of the register
space.

Rob

Thanks
Will update to use one DT node for the whole clock controller.
--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux PPP]     [Linux FS]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Linmodem]     [Device Mapper]     [Linux Kernel for ARM]

  Powered by Linux