From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eric Nelson Date: Wed, 15 Apr 2015 17:16:31 -0700 Subject: [U-Boot] [PATCH] video, ipu: make ldb_clock configurable In-Reply-To: <1428826758-484-1-git-send-email-hs@denx.de> References: <1428826758-484-1-git-send-email-hs@denx.de> Message-ID: <552EFF5F.9040100@boundarydevices.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 > --- > > 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