public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot] [PATCH V2 1/2] EXYNOS5: Make all display related code dependent on CONFIG_LCD
@ 2013-01-11  7:06 Ajay Kumar
  2013-01-11  7:06 ` [U-Boot] [PATCH V2 2/2] EXYNOS5: Enable console multiplexing in u-boot Ajay Kumar
  2013-01-11  8:20 ` [U-Boot] [PATCH V2 1/2] EXYNOS5: Make all display related code dependent on CONFIG_LCD Minkyu Kang
  0 siblings, 2 replies; 4+ messages in thread
From: Ajay Kumar @ 2013-01-11  7:06 UTC (permalink / raw)
  To: u-boot

u-boot compilation fails for smdk5250 when we deselect CONFIG_LCD
from the main config file.

Following error was observed:
drivers/video/libvideo.o: In function `exynos_lcd_init':
/home/ajay/u-boot-samsung/drivers/video/exynos_fb.c:68: undefined reference to `lcd_set_flush_dcache'

This is because exynos video drivers have dependency on CONFIG_LCD.

Signed-off-by: Ajay Kumar <ajaykumar.rs@samsung.com>
Acked-by: Simon Glass <sjg@chromium.org>
---
Changes in V2:  Add compilation error message to commit log.

 board/samsung/smdk5250/smdk5250.c | 2 ++
 include/configs/exynos5250-dt.h   | 2 ++
 2 files changed, 4 insertions(+)

diff --git a/board/samsung/smdk5250/smdk5250.c b/board/samsung/smdk5250/smdk5250.c
index 9c4bf9b..7a5f132 100644
--- a/board/samsung/smdk5250/smdk5250.c
+++ b/board/samsung/smdk5250/smdk5250.c
@@ -281,6 +281,7 @@ int board_early_init_f(void)
 }
 #endif
 
+#ifdef CONFIG_LCD
 void cfg_lcd_gpio(void)
 {
 	struct exynos5_gpio_part1 *gpio1 =
@@ -374,3 +375,4 @@ void init_panel_info(vidinfo_t *vid)
 
 	exynos_set_dp_platform_data(&dp_platform_data);
 }
+#endif
diff --git a/include/configs/exynos5250-dt.h b/include/configs/exynos5250-dt.h
index 07bca1d..298bae9 100644
--- a/include/configs/exynos5250-dt.h
+++ b/include/configs/exynos5250-dt.h
@@ -299,10 +299,12 @@
 
 /* Display */
 #define CONFIG_LCD
+#ifdef CONFIG_LCD
 #define CONFIG_EXYNOS_FB
 #define CONFIG_EXYNOS_DP
 #define LCD_XRES			2560
 #define LCD_YRES			1600
 #define LCD_BPP			LCD_COLOR16
+#endif
 
 #endif	/* __CONFIG_H */
-- 
1.8.0

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [U-Boot] [PATCH V2 2/2] EXYNOS5: Enable console multiplexing in u-boot
  2013-01-11  7:06 [U-Boot] [PATCH V2 1/2] EXYNOS5: Make all display related code dependent on CONFIG_LCD Ajay Kumar
@ 2013-01-11  7:06 ` Ajay Kumar
  2013-01-11  8:20   ` Minkyu Kang
  2013-01-11  8:20 ` [U-Boot] [PATCH V2 1/2] EXYNOS5: Make all display related code dependent on CONFIG_LCD Minkyu Kang
  1 sibling, 1 reply; 4+ messages in thread
From: Ajay Kumar @ 2013-01-11  7:06 UTC (permalink / raw)
  To: u-boot

We enable console multiplexing and use both serial and LCD for stdout/stderr.
Initially, u-boot output console is observed via serial port.
If you also have a DP panel connected onto your SMDK5250 board,
you can switch to LCD console by typing "setenv stdout lcd".
You can always switch back to serial using "setenv stdout serial".
You can switch error console(stderr) as well, using similar commands.

Signed-off-by: Ajay Kumar <ajaykumar.rs@samsung.com>
Acked-by: Simon Glass <sjg@chromium.org>
---
Changes in V2:  We use CONFIG_CONSOLE_MUX always. In V1, CONFIG_CONSOLE_MUX
		was enabled only when CONFIG_LCD is selected.

 include/configs/exynos5250-dt.h | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/include/configs/exynos5250-dt.h b/include/configs/exynos5250-dt.h
index 298bae9..6912936 100644
--- a/include/configs/exynos5250-dt.h
+++ b/include/configs/exynos5250-dt.h
@@ -77,6 +77,17 @@
 #define CONFIG_BAUDRATE			115200
 #define EXYNOS5_DEFAULT_UART_OFFSET	0x010000
 
+/* Console configuration */
+#define CONFIG_CONSOLE_MUX
+#define CONFIG_SYS_CONSOLE_IS_IN_ENV
+#define EXYNOS_DEVICE_SETTINGS \
+		"stdin=serial\0" \
+		"stdout=serial,lcd\0" \
+		"stderr=serial,lcd\0"
+
+#define CONFIG_EXTRA_ENV_SETTINGS \
+	EXYNOS_DEVICE_SETTINGS
+
 #define TZPC_BASE_OFFSET		0x10000
 
 /* SD/MMC configuration */
-- 
1.8.0

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [U-Boot] [PATCH V2 1/2] EXYNOS5: Make all display related code dependent on CONFIG_LCD
  2013-01-11  7:06 [U-Boot] [PATCH V2 1/2] EXYNOS5: Make all display related code dependent on CONFIG_LCD Ajay Kumar
  2013-01-11  7:06 ` [U-Boot] [PATCH V2 2/2] EXYNOS5: Enable console multiplexing in u-boot Ajay Kumar
@ 2013-01-11  8:20 ` Minkyu Kang
  1 sibling, 0 replies; 4+ messages in thread
From: Minkyu Kang @ 2013-01-11  8:20 UTC (permalink / raw)
  To: u-boot

On 11/01/13 16:06, Ajay Kumar wrote:
> u-boot compilation fails for smdk5250 when we deselect CONFIG_LCD
> from the main config file.
> 
> Following error was observed:
> drivers/video/libvideo.o: In function `exynos_lcd_init':
> /home/ajay/u-boot-samsung/drivers/video/exynos_fb.c:68: undefined reference to `lcd_set_flush_dcache'
> 
> This is because exynos video drivers have dependency on CONFIG_LCD.
> 
> Signed-off-by: Ajay Kumar <ajaykumar.rs@samsung.com>
> Acked-by: Simon Glass <sjg@chromium.org>

applied to u-boot-samsung.

Thanks,
Minkyu Kang.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [U-Boot] [PATCH V2 2/2] EXYNOS5: Enable console multiplexing in u-boot
  2013-01-11  7:06 ` [U-Boot] [PATCH V2 2/2] EXYNOS5: Enable console multiplexing in u-boot Ajay Kumar
@ 2013-01-11  8:20   ` Minkyu Kang
  0 siblings, 0 replies; 4+ messages in thread
From: Minkyu Kang @ 2013-01-11  8:20 UTC (permalink / raw)
  To: u-boot

On 11/01/13 16:06, Ajay Kumar wrote:
> We enable console multiplexing and use both serial and LCD for stdout/stderr.
> Initially, u-boot output console is observed via serial port.
> If you also have a DP panel connected onto your SMDK5250 board,
> you can switch to LCD console by typing "setenv stdout lcd".
> You can always switch back to serial using "setenv stdout serial".
> You can switch error console(stderr) as well, using similar commands.
> 
> Signed-off-by: Ajay Kumar <ajaykumar.rs@samsung.com>
> Acked-by: Simon Glass <sjg@chromium.org>
> ---
> Changes in V2:  We use CONFIG_CONSOLE_MUX always. In V1, CONFIG_CONSOLE_MUX
> 		was enabled only when CONFIG_LCD is selected.
> 
>  include/configs/exynos5250-dt.h | 11 +++++++++++
>  1 file changed, 11 insertions(+)
> 

applied to u-boot-samsung.

Thanks,
Minkyu Kang.

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2013-01-11  8:20 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-01-11  7:06 [U-Boot] [PATCH V2 1/2] EXYNOS5: Make all display related code dependent on CONFIG_LCD Ajay Kumar
2013-01-11  7:06 ` [U-Boot] [PATCH V2 2/2] EXYNOS5: Enable console multiplexing in u-boot Ajay Kumar
2013-01-11  8:20   ` Minkyu Kang
2013-01-11  8:20 ` [U-Boot] [PATCH V2 1/2] EXYNOS5: Make all display related code dependent on CONFIG_LCD Minkyu Kang

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox