Message ID | 1428826758-484-1-git-send-email-hs@denx.de |
---|---|
State | Superseded |
Delegated to: | Anatolij Gustschin |
Headers | show |
Hi Heiko, On 04/12/2015 01:19 AM, Heiko Schocher wrote: > make the ldb_clock configurable through the new define > CONFIG_SYS_LDB_CLOCK. This is needed as the ldb clock is not > always 650000000, for example on the aristainetos2 board, > where the ldb clock derives from PLL5 clock. > > Signed-off-by: Heiko Schocher <hs@denx.de> > --- > > drivers/video/ipu_common.c | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/drivers/video/ipu_common.c b/drivers/video/ipu_common.c > index 5873531..1a209d4 100644 > --- a/drivers/video/ipu_common.c > +++ b/drivers/video/ipu_common.c > @@ -210,9 +210,13 @@ static struct clk ipu_clk = { > .usecount = 0, > }; > > +#if !defined CONFIG_SYS_LDB_CLOCK You might want to count the zeros here: > +#define CONFIG_SYS_LDB_CLOCK 650000000 That big a number will break existing code. > +#endif > + > static struct clk ldb_clk = { > .name = "ldb_clk", > - .rate = 65000000, > + .rate = CONFIG_SYS_LDB_CLOCK, > .usecount = 0, > }; > > Tested-by: Eric Nelson <eric.nelson@boundarydevices.com>
diff --git a/drivers/video/ipu_common.c b/drivers/video/ipu_common.c index 5873531..1a209d4 100644 --- a/drivers/video/ipu_common.c +++ b/drivers/video/ipu_common.c @@ -210,9 +210,13 @@ static struct clk ipu_clk = { .usecount = 0, }; +#if !defined CONFIG_SYS_LDB_CLOCK +#define CONFIG_SYS_LDB_CLOCK 650000000 +#endif + static struct clk ldb_clk = { .name = "ldb_clk", - .rate = 65000000, + .rate = CONFIG_SYS_LDB_CLOCK, .usecount = 0, };
make the ldb_clock configurable through the new define CONFIG_SYS_LDB_CLOCK. This is needed as the ldb clock is not always 650000000, for example on the aristainetos2 board, where the ldb clock derives from PLL5 clock. Signed-off-by: Heiko Schocher <hs@denx.de> --- drivers/video/ipu_common.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-)