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 |