On 24 August 2016 at 10:47, Stephen Boyd <sboyd@xxxxxxxxxxxxxx> wrote: > On 08/23, Rafał Miłecki wrote: >> +static int bcm53573_ilp_enable(struct clk_hw *hw) >> +{ >> + struct bcm53573_ilp *ilp = container_of(hw, struct bcm53573_ilp, hw); >> + >> + writel(0x10199, ilp->pmu + PMU_SLOW_CLK_PERIOD); >> + writel(0x10000, ilp->pmu + 0x674); > > Is there a name for 0x674? No in any sources published by Broadcom. I was experimenting with bit 0x10000 in PMU_SLOW_CLK_PERIOD and 0x10000 in 0x674. Both have to be set. I was suspecting that maybe one of them is just some trigger and it can be switched back to 0, but it's not the case. I definitely need to set both of them to get clock working. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html