From: Sascha Hauer <s.hauer@pengutronix.de>
To: barebox@lists.infradead.org
Subject: [PATCH] video imx: specify num_modes in platform_data
Date: Thu, 5 Jan 2012 10:23:57 +0100 [thread overview]
Message-ID: <1325755439-19534-4-git-send-email-s.hauer@pengutronix.de> (raw)
In-Reply-To: <1325755439-19534-1-git-send-email-s.hauer@pengutronix.de>
Without num_modes the imx fb driver won't work. Specify this
in the boards and also bail out in the driver when num_modes
is unspecified.
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
arch/arm/boards/eukrea_cpuimx25/eukrea_cpuimx25.c | 1 +
arch/arm/boards/eukrea_cpuimx27/eukrea_cpuimx27.c | 1 +
arch/arm/boards/guf-neso/board.c | 1 +
arch/arm/boards/imx21ads/imx21ads.c | 1 +
arch/arm/boards/karo-tx25/board.c | 1 +
arch/arm/boards/pcm038/pcm038.c | 1 +
drivers/video/imx.c | 5 +++++
7 files changed, 11 insertions(+), 0 deletions(-)
diff --git a/arch/arm/boards/eukrea_cpuimx25/eukrea_cpuimx25.c b/arch/arm/boards/eukrea_cpuimx25/eukrea_cpuimx25.c
index 162c117..73e54f2 100644
--- a/arch/arm/boards/eukrea_cpuimx25/eukrea_cpuimx25.c
+++ b/arch/arm/boards/eukrea_cpuimx25/eukrea_cpuimx25.c
@@ -116,6 +116,7 @@ static struct imx_fb_videomode imxfb_mode = {
static struct imx_fb_platform_data eukrea_cpuimx25_fb_data = {
.mode = &imxfb_mode,
+ .num_modes = 1,
.pwmr = 0x00A903FF,
.lscr1 = 0x00120300,
.dmacr = 0x80040060,
diff --git a/arch/arm/boards/eukrea_cpuimx27/eukrea_cpuimx27.c b/arch/arm/boards/eukrea_cpuimx27/eukrea_cpuimx27.c
index c419c73..e2766fb 100644
--- a/arch/arm/boards/eukrea_cpuimx27/eukrea_cpuimx27.c
+++ b/arch/arm/boards/eukrea_cpuimx27/eukrea_cpuimx27.c
@@ -116,6 +116,7 @@ static struct imx_fb_videomode imxfb_mode = {
static struct imx_fb_platform_data eukrea_cpuimx27_fb_data = {
.mode = &imxfb_mode,
+ .num_modes = 1,
.pwmr = 0x00A903FF,
.lscr1 = 0x00120300,
.dmacr = 0x00020010,
diff --git a/arch/arm/boards/guf-neso/board.c b/arch/arm/boards/guf-neso/board.c
index e524b70..62d82f2 100644
--- a/arch/arm/boards/guf-neso/board.c
+++ b/arch/arm/boards/guf-neso/board.c
@@ -105,6 +105,7 @@ static void neso_fb_enable(int enable)
static struct imx_fb_platform_data neso_fb_data = {
.mode = &imxfb_mode,
+ .num_modes = 1,
.pwmr = 0x00000000, /* doesn't matter */
.lscr1 = 0x00120300, /* doesn't matter */
/* dynamic mode -> using the reset values (as recommended in the datasheet) */
diff --git a/arch/arm/boards/imx21ads/imx21ads.c b/arch/arm/boards/imx21ads/imx21ads.c
index fc34709..fde6774 100644
--- a/arch/arm/boards/imx21ads/imx21ads.c
+++ b/arch/arm/boards/imx21ads/imx21ads.c
@@ -70,6 +70,7 @@ static struct imx_fb_videomode imx_fb_modedata = {
static struct imx_fb_platform_data imx_fb_data = {
.mode = &imx_fb_modedata,
+ .num_modes = 1,
.cmap_greyscale = 0,
.cmap_inverse = 0,
.cmap_static = 0,
diff --git a/arch/arm/boards/karo-tx25/board.c b/arch/arm/boards/karo-tx25/board.c
index 1eece1d..bfccd0c 100644
--- a/arch/arm/boards/karo-tx25/board.c
+++ b/arch/arm/boards/karo-tx25/board.c
@@ -239,6 +239,7 @@ static void tx25_fb_enable(int enable)
static struct imx_fb_platform_data tx25_fb_data = {
.mode = &stk5_fb_mode,
+ .num_modes = 1,
.dmacr = 0x80040060,
.enable = tx25_fb_enable,
};
diff --git a/arch/arm/boards/pcm038/pcm038.c b/arch/arm/boards/pcm038/pcm038.c
index 6b8897e..8dd6521 100644
--- a/arch/arm/boards/pcm038/pcm038.c
+++ b/arch/arm/boards/pcm038/pcm038.c
@@ -104,6 +104,7 @@ static struct imx_fb_videomode imxfb_mode = {
static struct imx_fb_platform_data pcm038_fb_data = {
.mode = &imxfb_mode,
+ .num_modes = 1,
.pwmr = 0x00A903FF,
.lscr1 = 0x00120300,
.dmacr = 0x00020010,
diff --git a/drivers/video/imx.c b/drivers/video/imx.c
index 78179af..452e558 100644
--- a/drivers/video/imx.c
+++ b/drivers/video/imx.c
@@ -552,6 +552,11 @@ static int imxfb_probe(struct device_d *dev)
writel(readl(IMX_CCM_BASE + CCM_CGCR1) & ~(1 << 29),
IMX_CCM_BASE + CCM_CGCR1);
#endif
+ if (!pdata->num_modes) {
+ dev_err(dev, "no modes. bailing out\n");
+ return -EINVAL;
+ }
+
mode_list = xzalloc(sizeof(*mode_list) * pdata->num_modes);
for (i = 0; i < pdata->num_modes; i++)
mode_list[i] = pdata->mode[i].mode;
--
1.7.7.3
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
next prev parent reply other threads:[~2012-01-05 9:24 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-01-05 9:23 compile patches Sascha Hauer
2012-01-05 9:23 ` [PATCH 1/3] ARM AT91: Add missing on_flash_bbt in atmel_nand_data Sascha Hauer
2012-01-05 13:14 ` Jean-Christophe PLAGNIOL-VILLARD
2012-01-06 8:32 ` Sascha Hauer
2012-01-06 12:09 ` Jean-Christophe PLAGNIOL-VILLARD
2012-01-06 12:44 ` Sascha Hauer
2012-01-05 9:23 ` [PATCH] ARM: move exception vectors away from start of binary Sascha Hauer
2012-01-05 13:17 ` Jean-Christophe PLAGNIOL-VILLARD
2012-01-06 8:34 ` Sascha Hauer
2012-01-05 9:23 ` Sascha Hauer [this message]
2012-01-05 9:23 ` [PATCH 2/3] ARM omap beagle xload: update defconfig Sascha Hauer
2012-01-05 9:23 ` [PATCH 3/3] fat fs: Fix compile warning Sascha Hauer
-- strict thread matches above, loose matches on Subject: below --
2012-01-04 14:21 [PATCH] video imx: specify num_modes in platform_data Sascha Hauer
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1325755439-19534-4-git-send-email-s.hauer@pengutronix.de \
--to=s.hauer@pengutronix.de \
--cc=barebox@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox