Hi. I think .reg_src can be removed? This clock have only one source.
On Thu, Nov 8, 2012 at 2:00 AM, Sylwester Nawrocki <[email protected]> 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 <[email protected]> > Signed-off-by: Sylwester Nawrocki <[email protected]> > --- > 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 [email protected] > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
