Hi, On 11/09/2012 10:31 AM, Andrey Gusakov wrote:
On Fri, Nov 9, 2012 at 12:32 PM, Sylwester Nawrocki <sylvester.nawrocki@xxxxxxxxx> wrote:The "camera" clock defined in arch/arm/mach-s3c64xx/clock.c has null clock source mux control register as it can have only one parent clock. In such cases there is a need to configure the parent clock statically, otherwise s3c_set_clksrc() bails out with an error message "no parent clock specified" leaving the parent clock not configured. Define statically the parent clock so it is possible to get or set rate of the "camera" clock. While at it remove the unneded null reg_src definition. Reported-by: In-Bae Jeong<kukyakya@xxxxxxxxx> Signed-off-by: Sylwester Nawrocki<sylvester.nawrocki@xxxxxxxxx> --- arch/arm/mach-s3c64xx/clock.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/arch/arm/mach-s3c64xx/clock.c b/arch/arm/mach-s3c64xx/clock.c index 28041e8..2e6d7f9 100644 --- a/arch/arm/mach-s3c64xx/clock.c +++ b/arch/arm/mach-s3c64xx/clock.c @@ -744,9 +744,9 @@ static struct clksrc_clk clksrcs[] = { .name = "camera", .ctrlbit = S3C_CLKCON_SCLK_CAM, .enable = s3c64xx_sclk_ctrl, + .parent =&clk_h2, }, .reg_div = { .reg = S3C_CLK_DIV0, .shift = 20, .size = 4 }, - .reg_src = { .reg = NULL, .shift = 0, .size = 0 }, .sources =&clkset_camif,Just figure out that .sources can be removed to. And seems "clkset_camif" and "clkset_camif_list" can be removed as unused.
OK, you're right. I think it could be done as a separate patch, depending on this one to avoid conflicts.
I'll try to test it tiday.
Thanks. -- 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
