Hi,
I would like to hear your opinions about the patchset below.
The Cell Broadband Engine contains a 64-bit PowerPC core with 2 hardware
threads (called PPEs) and 8 Synergistic Processing Engines (called SPEs).
When booting Linux, 2 penguins logos are shown on the graphical console.
To emphasize the existence of the SPEs (which can be used under Linux), we
added a second row of (smaller) helper Penguin logos, one for each SPE.
Summaries:
[PATCH 1/6] fbdev: Avoid vertical overflow when making space for the logo
[PATCH 2/6] fbdev: fb_do_show_logo() updates
[PATCH 3/6] fbdev: extract fb_show_logo_line()
[PATCH 4/6] fbdev: Add fb_append_extra_logo()
[PATCH 5/6] fbdev: SPE helper penguin logo
[PATCH 6/6] Cell: Draw SPE helper penguin logos
Thanks for your comments!
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- Sony Network and Software Technology Center Europe (NSCE)
Geert.Uytterhoeven@sonycom.com ------- The Corporate Village, Da Vincilaan 7-D1
Voice +32-2-7008453 Fax +32-2-7008622 ---------------- B-1935 Zaventem, Belgium
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
fbcon_prepare_logo(): Avoid vertical overflow when making space for the logo
Signed-off-by: Geert Uytterhoeven <redacted>
Signed-off-by: Geoff Levand <redacted>
---
drivers/video/console/fbcon.c | 9 +++++++--
1 files changed, 7 insertions(+), 2 deletions(-)
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- Sony Network and Software Technology Center Europe (NSCE)
Geert.Uytterhoeven@sonycom.com ------- The Corporate Village, Da Vincilaan 7-D1
Voice +32-2-7008453 Fax +32-2-7008622 ---------------- B-1935 Zaventem, Belgium
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
fb_do_show_logo() updates:
- Use width and height of the passed image instead of the global variable
fb_logo
- Explicitly pass the number of logos to draw
Signed-off-by: Geert Uytterhoeven <redacted>
Signed-off-by: Geoff Levand <redacted>
---
drivers/video/fbmem.c | 32 ++++++++++++++------------------
1 files changed, 14 insertions(+), 18 deletions(-)
@@ -533,7 +529,7 @@ int fb_show_logo(struct fb_info *info, ifb_rotate_logo(info,logo_rotate,&image,rotate);}-fb_do_show_logo(info,&image,rotate);+fb_do_show_logo(info,&image,rotate,num_online_cpus());kfree(palette);if(saved_pseudo_palette!=NULL)
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- Sony Network and Software Technology Center Europe (NSCE)
Geert.Uytterhoeven@sonycom.com ------- The Corporate Village, Da Vincilaan 7-D1
Voice +32-2-7008453 Fax +32-2-7008622 ---------------- B-1935 Zaventem, Belgium
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
@@ -473,21 +473,22 @@ int fb_prepare_logo(struct fb_info *inforeturnfb_logo.logo->height;}-intfb_show_logo(structfb_info*info,introtate)+staticintfb_show_logo_line(structfb_info*info,introtate,+conststructlinux_logo*logo,inty,intn){u32*palette=NULL,*saved_pseudo_palette=NULL;unsignedchar*logo_new=NULL,*logo_rotate=NULL;structfb_imageimage;/* Return if the frame buffer is not mapped or suspended */-if(fb_logo.logo==NULL||info->state!=FBINFO_STATE_RUNNING)+if(logo==NULL||info->state!=FBINFO_STATE_RUNNING)return0;image.depth=8;-image.data=fb_logo.logo->data;+image.data=logo->data;if(fb_logo.needs_cmapreset)-fb_set_logocmap(info,fb_logo.logo);+fb_set_logocmap(info,logo);if(fb_logo.needs_truepalette||fb_logo.needs_directpalette){
@@ -496,17 +497,16 @@ int fb_show_logo(struct fb_info *info, ireturn0;if(fb_logo.needs_truepalette)-fb_set_logo_truepalette(info,fb_logo.logo,palette);+fb_set_logo_truepalette(info,logo,palette);else-fb_set_logo_directpalette(info,fb_logo.logo,palette);+fb_set_logo_directpalette(info,logo,palette);saved_pseudo_palette=info->pseudo_palette;info->pseudo_palette=palette;}if(fb_logo.depth<=4){-logo_new=kmalloc(fb_logo.logo->width*fb_logo.logo->height,-GFP_KERNEL);+logo_new=kmalloc(logo->width*logo->height,GFP_KERNEL);if(logo_new==NULL){kfree(palette);if(saved_pseudo_palette)
@@ -514,29 +514,35 @@ int fb_show_logo(struct fb_info *info, ireturn0;}image.data=logo_new;-fb_set_logo(info,fb_logo.logo,logo_new,fb_logo.depth);+fb_set_logo(info,logo,logo_new,fb_logo.depth);}image.dx=0;-image.dy=0;-image.width=fb_logo.logo->width;-image.height=fb_logo.logo->height;+image.dy=y;+image.width=logo->width;+image.height=logo->height;if(rotate){-logo_rotate=kmalloc(fb_logo.logo->width*-fb_logo.logo->height,GFP_KERNEL);+logo_rotate=kmalloc(logo->width*+logo->height,GFP_KERNEL);if(logo_rotate)fb_rotate_logo(info,logo_rotate,&image,rotate);}-fb_do_show_logo(info,&image,rotate,num_online_cpus());+fb_do_show_logo(info,&image,rotate,n);kfree(palette);if(saved_pseudo_palette!=NULL)info->pseudo_palette=saved_pseudo_palette;kfree(logo_new);kfree(logo_rotate);-returnfb_logo.logo->height;+returnlogo->height;+}++intfb_show_logo(structfb_info*info,introtate)+{+returnfb_show_logo_line(info,rotate,fb_logo.logo,0,+num_online_cpus());}#elseintfb_prepare_logo(structfb_info*info,introtate){return0;}
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- Sony Network and Software Technology Center Europe (NSCE)
Geert.Uytterhoeven@sonycom.com ------- The Corporate Village, Da Vincilaan 7-D1
Voice +32-2-7008453 Fax +32-2-7008622 ---------------- B-1935 Zaventem, Belgium
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- Sony Network and Software Technology Center Europe (NSCE)
Geert.Uytterhoeven@sonycom.com ------- The Corporate Village, Da Vincilaan 7-D1
Voice +32-2-7008453 Fax +32-2-7008622 ---------------- B-1935 Zaventem, Belgium
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
@@ -14,6 +14,8 @@ obj-$(CONFIG_LOGO_SUPERH_VGA16) += logoobj-$(CONFIG_LOGO_SUPERH_CLUT224)+=logo_superh_clut224.oobj-$(CONFIG_LOGO_M32R_CLUT224)+=logo_m32r_clut224.o+obj-$(CONFIG_SPU_BASE)+=logo_spe_clut224.o+# How to generate logo's# Use logo-cfiles to retrieve list of .c files to be built---/dev/null+++geert-linux-src/drivers/video/logo/logo_spe_clut224.ppm
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- Sony Network and Software Technology Center Europe (NSCE)
Geert.Uytterhoeven@sonycom.com ------- The Corporate Village, Da Vincilaan 7-D1
Voice +32-2-7008453 Fax +32-2-7008622 ---------------- B-1935 Zaventem, Belgium
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
Let spu_management_ops.enumerate_spus() return the number of found SPEs
and use that information to draw some little helper penguin logos.
Signed-off-by: Geert Uytterhoeven <redacted>
Signed-off-by: Geoff Levand <redacted>
---
arch/powerpc/platforms/cell/spu_base.c | 11 +++++++++--
arch/powerpc/platforms/cell/spu_priv1_mmio.c | 4 +++-
arch/powerpc/platforms/ps3/spu.c | 6 ++++--
3 files changed, 16 insertions(+), 5 deletions(-)
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- Sony Network and Software Technology Center Europe (NSCE)
Geert.Uytterhoeven@sonycom.com ------- The Corporate Village, Da Vincilaan 7-D1
Voice +32-2-7008453 Fax +32-2-7008622 ---------------- B-1935 Zaventem, Belgium
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
You should not have the #ifdef nor the extern declaration in here.
A better way to do it would be to have in include/linux/linux_logo.h
(or similar):
extern const struct linux_logo logo_spe_clut224;
#ifdef CONFIG_LOGO
extern void fb_append_extra_logo(const struct linux_logo *logo, int n);
#else
static inline void fb_append_extra_logo(const struct linux_logo *logo, int n)
{
}
#endif
Arnd <><
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
On Tuesday 30 January 2007 19:00, Geert Uytterhoeven wrote:
The Cell Broadband Engine contains a 64-bit PowerPC core with 2 hardware
threads (called PPEs) and 8 Synergistic Processing Engines (called SPEs).
When booting Linux, 2 penguins logos are shown on the graphical console.
To emphasize the existence of the SPEs (which can be used under Linux), we
added a second row of (smaller) helper Penguin logos, one for each SPE.
I love the small penguins, they're cute!
You should probably run this patch set through linux-kernel@vger though,
so see if there are people around there objecting to the idea. The patches
all look good from a technical point, except for the fact that the existing
fb boot logo is a gross hack to start with, but I don't think anyone is
arguing for getting rid of that.
Arnd <><
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
From: Benjamin Herrenschmidt <benh@kernel.crashing.org> Date: 2007-01-31 01:44:41
On Wed, 2007-01-31 at 02:19 +0100, Arnd Bergmann wrote:
On Tuesday 30 January 2007 19:00, Geert Uytterhoeven wrote:
quoted
The Cell Broadband Engine contains a 64-bit PowerPC core with 2 hardware
threads (called PPEs) and 8 Synergistic Processing Engines (called SPEs).
When booting Linux, 2 penguins logos are shown on the graphical console.
To emphasize the existence of the SPEs (which can be used under Linux), we
added a second row of (smaller) helper Penguin logos, one for each SPE.
I love the small penguins, they're cute!
You should probably run this patch set through linux-kernel@vger though,
so see if there are people around there objecting to the idea. The patches
all look good from a technical point, except for the fact that the existing
fb boot logo is a gross hack to start with, but I don't think anyone is
arguing for getting rid of that.
Well, the fb boot logo might be a gross hack but didnt Geert write it in
the first place ? :-)
Ben.
-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV