* [PATCH 2/6] pxa: frame buffer support both pxa168 and pxa910
@ 2009-11-03 6:45 ` Jun Nie
0 siblings, 0 replies; 6+ messages in thread
From: Jun Nie @ 2009-11-03 6:45 UTC (permalink / raw)
To: linux-arm-kernel, linux-fbdev-devel
pxa: frame buffer support both pxa168 and pxa910
Signed-off-by: Jun Nie <njun@marvell.com>
---
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
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH 2/6] pxa: frame buffer support both pxa168 and pxa910
@ 2009-11-03 6:45 ` Jun Nie
0 siblings, 0 replies; 6+ messages in thread
From: Jun Nie @ 2009-11-03 6:45 UTC (permalink / raw)
To: linux-arm-kernel
pxa: frame buffer support both pxa168 and pxa910
Signed-off-by: Jun Nie <njun@marvell.com>
---
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
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH 2/6] pxa: frame buffer support both pxa168 and pxa910
2009-11-03 6:45 ` Jun Nie
@ 2009-11-03 7:29 ` Jun Nie
-1 siblings, 0 replies; 6+ messages in thread
From: Jun Nie @ 2009-11-03 7:29 UTC (permalink / raw)
To: linux-arm-kernel, linux-fbdev-devel, linux-fbdev
2009/11/3 Jun Nie <niej0001@gmail.com>:
> pxa: frame buffer support both pxa168 and pxa910
>
> Signed-off-by: Jun Nie <njun@marvell.com>
> ---
> 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 linux-fbdev@vger.kernel.org
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 2/6] pxa: frame buffer support both pxa168 and pxa910
@ 2009-11-03 7:29 ` Jun Nie
0 siblings, 0 replies; 6+ messages in thread
From: Jun Nie @ 2009-11-03 7:29 UTC (permalink / raw)
To: linux-arm-kernel
2009/11/3 Jun Nie <niej0001@gmail.com>:
> pxa: frame buffer support both pxa168 and pxa910
>
> Signed-off-by: Jun Nie <njun@marvell.com>
> ---
> ?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 linux-fbdev at vger.kernel.org
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 2/6] pxa: frame buffer support both pxa168 and pxa910
2009-11-03 6:45 ` Jun Nie
@ 2009-11-03 9:32 ` Eric Miao
-1 siblings, 0 replies; 6+ messages in thread
From: Eric Miao @ 2009-11-03 9:32 UTC (permalink / raw)
To: Jun Nie; +Cc: linux-fbdev-devel, linux-arm-kernel
On Tue, Nov 3, 2009 at 2:45 PM, Jun Nie <niej0001@gmail.com> wrote:
> pxa: frame buffer support both pxa168 and pxa910
>
> Signed-off-by: Jun Nie <njun@marvell.com>
> ---
> 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
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>
------------------------------------------------------------------------------
Come build with us! The BlackBerry(R) Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay
ahead of the curve. Join us from November 9 - 12, 2009. Register now!
http://p.sf.net/sfu/devconference
_______________________________________________
Linux-fbdev-devel mailing list
Linux-fbdev-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-fbdev-devel
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 2/6] pxa: frame buffer support both pxa168 and pxa910
@ 2009-11-03 9:32 ` Eric Miao
0 siblings, 0 replies; 6+ messages in thread
From: Eric Miao @ 2009-11-03 9:32 UTC (permalink / raw)
To: linux-arm-kernel
On Tue, Nov 3, 2009 at 2:45 PM, Jun Nie <niej0001@gmail.com> wrote:
> pxa: frame buffer support both pxa168 and pxa910
>
> Signed-off-by: Jun Nie <njun@marvell.com>
> ---
> ?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
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2009-11-03 9:32 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-11-03 6:45 [PATCH 2/6] pxa: frame buffer support both pxa168 and pxa910 Jun Nie
2009-11-03 6:45 ` Jun Nie
2009-11-03 7:29 ` Jun Nie
2009-11-03 7:29 ` Jun Nie
2009-11-03 9:32 ` Eric Miao
2009-11-03 9:32 ` Eric Miao
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.