* [U-Boot] [PATCH] video, ipu: make ldb_clock configurable
@ 2015-04-12 8:19 Heiko Schocher
2015-04-16 0:16 ` Eric Nelson
0 siblings, 1 reply; 2+ messages in thread
From: Heiko Schocher @ 2015-04-12 8:19 UTC (permalink / raw)
To: u-boot
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
+#define CONFIG_SYS_LDB_CLOCK 650000000
+#endif
+
static struct clk ldb_clk = {
.name = "ldb_clk",
- .rate = 65000000,
+ .rate = CONFIG_SYS_LDB_CLOCK,
.usecount = 0,
};
--
2.1.0
^ permalink raw reply related [flat|nested] 2+ messages in thread* [U-Boot] [PATCH] video, ipu: make ldb_clock configurable
2015-04-12 8:19 [U-Boot] [PATCH] video, ipu: make ldb_clock configurable Heiko Schocher
@ 2015-04-16 0:16 ` Eric Nelson
0 siblings, 0 replies; 2+ messages in thread
From: Eric Nelson @ 2015-04-16 0:16 UTC (permalink / raw)
To: u-boot
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>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2015-04-16 0:16 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-04-12 8:19 [U-Boot] [PATCH] video, ipu: make ldb_clock configurable Heiko Schocher
2015-04-16 0:16 ` Eric Nelson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox