Hi + Luis from Synopsys.
Sorry the delay, I was out of office last week. Comment below. On 2/21/19 6:10 PM, Gareth Williams wrote:
From: Phil Edworthy <[email protected]> The driver requires an undocumented clock property, so detail it. Add documentation for a separate, optional, peripheral clock. Signed-off-by: Phil Edworthy <[email protected]> Acked-by: Wolfram Sang <[email protected]> --- v4: - Updated commit message to reference "peripheral clock" instead of "bus clock" - Added Wolfram's Acked-by v3: - Changed clocks and clock-names sections to use term "peripheral clock" (pclk) instead of "bus clock" (busclk).
...
Optional properties : + + - clock-names : Contains the names of the clocks: + "ic_clk", for the core clock used to generate the external I2C clock. + "pclk", the peripheral clock, required for register accesses. +
Actually it looks there is need to revert back to bus clock (or better) in comments but keep the "pclk" property.
The specification I have tells the ic_clk is the peripheral clock which runs the logic and the pclk (exactly pclk) is for bus interface and where registers are.
Luis: did I interpret it right? -- Jarkko
