diff mbox

[U-Boot,2/3] mx5: Rename mx51_fb_init()

Message ID 1336698455-11921-2-git-send-email-festevam@gmail.com
State Accepted
Commit a1b0e190a3c0c7e1a40877218804a656e45194be
Delegated to: Anatolij Gustschin
Headers show

Commit Message

Fabio Estevam May 11, 2012, 1:07 a.m. UTC
From: Fabio Estevam <fabio.estevam@freescale.com>

The ipuv3 driver is currently only used on mx51, but it can be extended to work
on mx53 and mx6 as well.

Rename mx51_fb_init(), so that it can be used by other SoCs. 

Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
 board/freescale/mx51evk/mx51evk.c |    2 +-
 board/ttcontrol/vision2/vision2.c |    2 +-
 drivers/video/mxc_ipuv3_fb.c      |    2 +-
 include/ipu_pixfmt.h              |    2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

Comments

Liu Hui-R64343 May 11, 2012, 1:50 a.m. UTC | #1
>-----Original Message-----
>From: Fabio Estevam [mailto:festevam@gmail.com]
>Sent: Friday, May 11, 2012 9:08 AM
>To: u-boot@lists.denx.de
>Cc: sbabic@denx.de; Liu Hui-R64343; Estevam Fabio-R49496
>Subject: [PATCH 2/3] mx5: Rename mx51_fb_init()
>
>From: Fabio Estevam <fabio.estevam@freescale.com>
>
>The ipuv3 driver is currently only used on mx51, but it can be extended to
>work on mx53 and mx6 as well.
>
>Rename mx51_fb_init(), so that it can be used by other SoCs.
>
>Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>

Acked-by: Jason Liu <r64343@freescale.com>

>---
> board/freescale/mx51evk/mx51evk.c |    2 +-
> board/ttcontrol/vision2/vision2.c |    2 +-
> drivers/video/mxc_ipuv3_fb.c      |    2 +-
> include/ipu_pixfmt.h              |    2 +-
> 4 files changed, 4 insertions(+), 4 deletions(-)
>
>diff --git a/board/freescale/mx51evk/mx51evk.c
>b/board/freescale/mx51evk/mx51evk.c
>index bc7f057..514a7ac 100644
>--- a/board/freescale/mx51evk/mx51evk.c
>+++ b/board/freescale/mx51evk/mx51evk.c
>@@ -502,7 +502,7 @@ void lcd_iomux(void)
>
> void lcd_enable(void)
> {
>-	int ret = mx51_fb_init(&claa_wvga, 1, IPU_PIX_FMT_RGB565);
>+	int ret = ipuv3_fb_init(&claa_wvga, 1, IPU_PIX_FMT_RGB565);
> 	if (ret)
> 		printf("LCD cannot be configured: %d\n", ret);  } diff --git
>a/board/ttcontrol/vision2/vision2.c b/board/ttcontrol/vision2/vision2.c
>index 282de95..d68bef7 100644
>--- a/board/ttcontrol/vision2/vision2.c
>+++ b/board/ttcontrol/vision2/vision2.c
>@@ -604,7 +604,7 @@ void lcd_enable(void)
> 	gpio_set_value(2, 1);
> 	mxc_request_iomux(MX51_PIN_GPIO1_2, IOMUX_CONFIG_ALT0);
>
>-	ret = mx51_fb_init(&nec_nl6448bc26_09c, 0, IPU_PIX_FMT_RGB666);
>+	ret = ipuv3_fb_init(&nec_nl6448bc26_09c, 0, IPU_PIX_FMT_RGB666);
> 	if (ret)
> 		puts("LCD cannot be configured\n");
> }
>diff --git a/drivers/video/mxc_ipuv3_fb.c b/drivers/video/mxc_ipuv3_fb.c
>index 1bee54c..c38e22d 100644
>--- a/drivers/video/mxc_ipuv3_fb.c
>+++ b/drivers/video/mxc_ipuv3_fb.c
>@@ -599,7 +599,7 @@ void video_set_lut(unsigned int index, /* color
>number */
> 	return;
> }
>
>-int mx51_fb_init(struct fb_videomode *mode, uint8_t disp, uint32_t pixfmt)
>+int ipuv3_fb_init(struct fb_videomode *mode, uint8_t disp, uint32_t
>+pixfmt)
> {
> 	gmode = mode;
> 	gdisp = disp;
>diff --git a/include/ipu_pixfmt.h b/include/ipu_pixfmt.h index
>656f605..0019898 100644
>--- a/include/ipu_pixfmt.h
>+++ b/include/ipu_pixfmt.h
>@@ -76,6 +76,6 @@
> #define IPU_PIX_FMT_YVU422P fourcc('Y', 'V', '1', '6')	/*< 16 YVU 4:2:2 */
> #define IPU_PIX_FMT_YUV422P fourcc('4', '2', '2', 'P')	/*< 16 YUV 4:2:2 */
>
>-int mx51_fb_init(struct fb_videomode *mode, uint8_t disp, uint32_t pixfmt);
>+int ipuv3_fb_init(struct fb_videomode *mode, uint8_t disp, uint32_t
>+pixfmt);
>
> #endif
>--
>1.7.1
>
Anatolij Gustschin May 25, 2012, 8:42 a.m. UTC | #2
Hi,

On Thu, 10 May 2012 22:07:34 -0300
Fabio Estevam <festevam@gmail.com> wrote:

> From: Fabio Estevam <fabio.estevam@freescale.com>
> 
> The ipuv3 driver is currently only used on mx51, but it can be extended to work
> on mx53 and mx6 as well.
> 
> Rename mx51_fb_init(), so that it can be used by other SoCs. 
> 
> Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
> ---
>  board/freescale/mx51evk/mx51evk.c |    2 +-
>  board/ttcontrol/vision2/vision2.c |    2 +-
>  drivers/video/mxc_ipuv3_fb.c      |    2 +-
>  include/ipu_pixfmt.h              |    2 +-
>  4 files changed, 4 insertions(+), 4 deletions(-)

Applied to u-boot-video/master, thanks!

Anatolij
diff mbox

Patch

diff --git a/board/freescale/mx51evk/mx51evk.c b/board/freescale/mx51evk/mx51evk.c
index bc7f057..514a7ac 100644
--- a/board/freescale/mx51evk/mx51evk.c
+++ b/board/freescale/mx51evk/mx51evk.c
@@ -502,7 +502,7 @@  void lcd_iomux(void)
 
 void lcd_enable(void)
 {
-	int ret = mx51_fb_init(&claa_wvga, 1, IPU_PIX_FMT_RGB565);
+	int ret = ipuv3_fb_init(&claa_wvga, 1, IPU_PIX_FMT_RGB565);
 	if (ret)
 		printf("LCD cannot be configured: %d\n", ret);
 }
diff --git a/board/ttcontrol/vision2/vision2.c b/board/ttcontrol/vision2/vision2.c
index 282de95..d68bef7 100644
--- a/board/ttcontrol/vision2/vision2.c
+++ b/board/ttcontrol/vision2/vision2.c
@@ -604,7 +604,7 @@  void lcd_enable(void)
 	gpio_set_value(2, 1);
 	mxc_request_iomux(MX51_PIN_GPIO1_2, IOMUX_CONFIG_ALT0);
 
-	ret = mx51_fb_init(&nec_nl6448bc26_09c, 0, IPU_PIX_FMT_RGB666);
+	ret = ipuv3_fb_init(&nec_nl6448bc26_09c, 0, IPU_PIX_FMT_RGB666);
 	if (ret)
 		puts("LCD cannot be configured\n");
 }
diff --git a/drivers/video/mxc_ipuv3_fb.c b/drivers/video/mxc_ipuv3_fb.c
index 1bee54c..c38e22d 100644
--- a/drivers/video/mxc_ipuv3_fb.c
+++ b/drivers/video/mxc_ipuv3_fb.c
@@ -599,7 +599,7 @@  void video_set_lut(unsigned int index, /* color number */
 	return;
 }
 
-int mx51_fb_init(struct fb_videomode *mode, uint8_t disp, uint32_t pixfmt)
+int ipuv3_fb_init(struct fb_videomode *mode, uint8_t disp, uint32_t pixfmt)
 {
 	gmode = mode;
 	gdisp = disp;
diff --git a/include/ipu_pixfmt.h b/include/ipu_pixfmt.h
index 656f605..0019898 100644
--- a/include/ipu_pixfmt.h
+++ b/include/ipu_pixfmt.h
@@ -76,6 +76,6 @@ 
 #define IPU_PIX_FMT_YVU422P fourcc('Y', 'V', '1', '6')	/*< 16 YVU 4:2:2 */
 #define IPU_PIX_FMT_YUV422P fourcc('4', '2', '2', 'P')	/*< 16 YUV 4:2:2 */
 
-int mx51_fb_init(struct fb_videomode *mode, uint8_t disp, uint32_t pixfmt);
+int ipuv3_fb_init(struct fb_videomode *mode, uint8_t disp, uint32_t pixfmt);
 
 #endif