On 11/09/2012 08:42 AM, Andrey Gusakov wrote:
Hi.
I think .reg_src can be removed? This clock have only one source.
Yes, good point. I'll repost with reg_src removed.
Can you test that patch then ?
Thanks,
Sylwester
On Thu, Nov 8, 2012 at 2:00 AM, Sylwester Nawrocki
<sylvester.nawro...@gmail.com> 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.
Reported-by: In-Bae Jeong<kukya...@gmail.com>
Signed-off-by: Sylwester Nawrocki<sylvester.nawro...@gmail.com>
---
arch/arm/mach-s3c64xx/clock.c | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/arch/arm/mach-s3c64xx/clock.c b/arch/arm/mach-s3c64xx/clock.c
index 28041e8..85b9cf1 100644
--- a/arch/arm/mach-s3c64xx/clock.c
+++ b/arch/arm/mach-s3c64xx/clock.c
@@ -744,6 +744,7 @@ 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 },
--
1.7.4.1
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
