|
From: Eric M. <eri...@gm...> - 2009-11-03 09:32:57
|
On Tue, Nov 3, 2009 at 2:45 PM, Jun Nie <nie...@gm...> wrote:
> pxa: frame buffer support both pxa168 and pxa910
>
> Signed-off-by: Jun Nie <nj...@ma...>
> ---
> drivers/video/pxa168fb.c | 6 ++++++
> 1 files changed, 6 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/video/pxa168fb.c b/drivers/video/pxa168fb.c
> index 27bdf2b..2ba1444 100644
> --- a/drivers/video/pxa168fb.c
> +++ b/drivers/video/pxa168fb.c
> @@ -875,12 +875,18 @@ failed:
> return ret;
> }
>
> + static const struct platform_device_id mmpfb_id_table[] = {
> + {"pxa168-fb", 0},
> + {"pxa910-fb", 1},
> + };
> +
Not sure '0' and '1' means anything here? Otherwise can just be '0'.
Intended usage could be some flags difference between processors,
let's say pxa910-fb supports YUV, one can have something like:
{ "pxa168-fb", FB_SUPPORT_RGB },
{ "pxa910-fb", FB_SUPPORT_RGB | FB_SUPPORT_YUV },
Just examples.
> static struct platform_driver pxa168fb_driver = {
> .driver = {
> .name = "pxa168-fb",
> .owner = THIS_MODULE,
> },
> .probe = pxa168fb_probe,
> + .id_table = mmpfb_id_table,
> };
>
> static int __devinit pxa168fb_init(void)
> --
> 1.5.4.3
>
> _______________________________________________
> linux-arm-kernel mailing list
> lin...@li...
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>
|
|
From: Jun N. <nie...@gm...> - 2009-11-03 06:45:40
|
pxa: frame buffer support both pxa168 and pxa910
Signed-off-by: Jun Nie <nj...@ma...>
---
drivers/video/pxa168fb.c | 6 ++++++
1 files changed, 6 insertions(+), 0 deletions(-)
diff --git a/drivers/video/pxa168fb.c b/drivers/video/pxa168fb.c
index 27bdf2b..2ba1444 100644
--- a/drivers/video/pxa168fb.c
+++ b/drivers/video/pxa168fb.c
@@ -875,12 +875,18 @@ failed:
return ret;
}
+ static const struct platform_device_id mmpfb_id_table[] = {
+ {"pxa168-fb", 0},
+ {"pxa910-fb", 1},
+ };
+
static struct platform_driver pxa168fb_driver = {
.driver = {
.name = "pxa168-fb",
.owner = THIS_MODULE,
},
.probe = pxa168fb_probe,
+ .id_table = mmpfb_id_table,
};
static int __devinit pxa168fb_init(void)
--
1.5.4.3
|
|
From: Jun N. <nie...@gm...> - 2009-11-03 07:29:27
|
2009/11/3 Jun Nie <nie...@gm...>:
> pxa: frame buffer support both pxa168 and pxa910
>
> Signed-off-by: Jun Nie <nj...@ma...>
> ---
> drivers/video/pxa168fb.c | 6 ++++++
> 1 files changed, 6 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/video/pxa168fb.c b/drivers/video/pxa168fb.c
> index 27bdf2b..2ba1444 100644
> --- a/drivers/video/pxa168fb.c
> +++ b/drivers/video/pxa168fb.c
> @@ -875,12 +875,18 @@ failed:
> return ret;
> }
>
> + static const struct platform_device_id mmpfb_id_table[] = {
> + {"pxa168-fb", 0},
> + {"pxa910-fb", 1},
> + };
> +
> static struct platform_driver pxa168fb_driver = {
> .driver = {
> .name = "pxa168-fb",
> .owner = THIS_MODULE,
> },
> .probe = pxa168fb_probe,
> + .id_table = mmpfb_id_table,
> };
>
> static int __devinit pxa168fb_init(void)
> --
> 1.5.4.3
>
add lin...@vg...
|
|
From: Jun N. <nie...@gm...> - 2009-11-16 09:34:58
|
From: Jun Nie <nj...@ma...>
Date: Tue, 10 Nov 2009 09:43:57 +0800
Subject: [PATCH] pxa: frame buffer support both pxa168 and pxa910
Signed-off-by: Jun Nie <nj...@ma...>
---
drivers/video/pxa168fb.c | 6 ++++++
include/video/pxa168fb.h | 3 +++
2 files changed, 9 insertions(+), 0 deletions(-)
diff --git a/drivers/video/pxa168fb.c b/drivers/video/pxa168fb.c
index 6283c16..09ada72 100644
--- a/drivers/video/pxa168fb.c
+++ b/drivers/video/pxa168fb.c
@@ -882,12 +882,18 @@ failed:
return ret;
}
+ static const struct platform_device_id mmpfb_id_table[] = {
+ {"pxa168-fb", PXA168_FB},
+ {"pxa910-fb", PXA910_FB},
+ };
+
static struct platform_driver pxa168fb_driver = {
.driver = {
.name = "pxa168-fb",
.owner = THIS_MODULE,
},
.probe = pxa168fb_probe,
+ .id_table = mmpfb_id_table,
};
static int __devinit pxa168fb_init(void)
diff --git a/include/video/pxa168fb.h b/include/video/pxa168fb.h
index 53919fe..da83c94 100644
--- a/include/video/pxa168fb.h
+++ b/include/video/pxa168fb.h
@@ -87,6 +87,9 @@
#define PIX_FMT_PSEUDOCOLOR 20
#define PIX_FMT_UYVY422PACK (0x1000|PIX_FMT_YUV422PACK)
+#define PXA168_FB 0x0168
+#define PXA910_FB 0x0910
+
/*
* PXA LCD controller private state.
*/
--
1.5.4.3
|