[PATCH v2 1/1] ARM : omap3 : fix wrong container_of in clock36xx.c
jp.francois at cynove.com
Thu Jun 6 03:56:47 EDT 2013
2013/6/5 Paul Walmsley <paul at pwsan.com>:
> On Thu, 30 May 2013, Jean-Philippe Francois wrote:
>> omap36xx_pwrdn_clk_enable_with_hsdiv_restore expects the parent hw of the clock
>> to be a clk_hw_omap. However, looking at cclock3xxx_data.c, all concerned clock
>> have parent defined as clk_divider. Instead of using container_of to eventually get
>> to the register and directly mess with the divider, change freq via clk_set_rate,
>> and let the clock framework toggle the divider value.
>> Tested with 3.9 on dm3730.
>> Signed-off-by: Jean-Philippe Fran??ois <jp.francois at cynove.com>
> Tested this patch before applying, and noticed that it causes the
> retention dynamic idle power management test to fail here on the
> Not sure at this point if this is caused by the patch, or if the patch is
> just unmasking another bug.
> Jean, Mike, Rajendra, care to take a look at this?
I am booting with nohlt on my board, because I otherwise have memory
I never looked into it because power consumption is currently a non
issue for us, and the board
will always be working anyway.
However I have a beaglexm too, so I can probably help with testing.
Does the first version  of the patch, that only touch the MSB of
the divider also trigger the
> - Paul
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
More information about the linux-arm-kernel