Commit 99e11ab2 authored by Finn Thain's avatar Finn Thain Committed by Geert Uytterhoeven

macfb: fix 24-bit visual and stuff

Lots of changes. Remove volatile keywords. Take the advice in the comments
and change video_slot to slot_addr. Factor out fifteen or so "channel >>
8" shifts into three shifts higher up the call graph. Make
csc_setpalette() atomic, pass the correct color values. Check for
ioremap() failure. Add missing break statement, thus fixing 24 bit console
visual (blue background bug). Remove some uninformative printk() noise.
Add a description for powerbook 140/170 graphics.
Signed-off-by: default avatarFinn Thain <fthain@telegraphics.com.au>
Signed-off-by: default avatarGeert Uytterhoeven <geert@linux-m68k.org>
parent 34c41d0a
...@@ -120,8 +120,6 @@ struct jet_cmap_regs { ...@@ -120,8 +120,6 @@ struct jet_cmap_regs {
#define PIXEL_TO_MM(a) (((a)*10)/28) /* width in mm at 72 dpi */ #define PIXEL_TO_MM(a) (((a)*10)/28) /* width in mm at 72 dpi */
static int video_slot = 0;
static struct fb_var_screeninfo macfb_defined = { static struct fb_var_screeninfo macfb_defined = {
.bits_per_pixel = 8, .bits_per_pixel = 8,
.activate = FB_ACTIVATE_NOW, .activate = FB_ACTIVATE_NOW,
...@@ -139,6 +137,7 @@ static struct fb_fix_screeninfo macfb_fix = { ...@@ -139,6 +137,7 @@ static struct fb_fix_screeninfo macfb_fix = {
.accel = FB_ACCEL_NONE, .accel = FB_ACCEL_NONE,
}; };
static void *slot_addr;
static struct fb_info fb_info; static struct fb_info fb_info;
static u32 pseudo_palette[16]; static u32 pseudo_palette[16];
static int inverse; static int inverse;
...@@ -157,10 +156,6 @@ static int dafb_setpalette(unsigned int regno, unsigned int red, ...@@ -157,10 +156,6 @@ static int dafb_setpalette(unsigned int regno, unsigned int red,
static int lastreg = -1; static int lastreg = -1;
unsigned long flags; unsigned long flags;
red >>= 8;
green >>= 8;
blue >>= 8;
local_irq_save(flags); local_irq_save(flags);
/* /*
...@@ -205,10 +200,6 @@ static int v8_brazil_setpalette(unsigned int regno, unsigned int red, ...@@ -205,10 +200,6 @@ static int v8_brazil_setpalette(unsigned int regno, unsigned int red,
struct fb_info *info) struct fb_info *info)
{ {
unsigned int bpp = info->var.bits_per_pixel; unsigned int bpp = info->var.bits_per_pixel;
unsigned char _red =red>>8;
unsigned char _green=green>>8;
unsigned char _blue =blue>>8;
unsigned char _regno;
unsigned long flags; unsigned long flags;
if (bpp > 8) if (bpp > 8)
...@@ -222,16 +213,16 @@ static int v8_brazil_setpalette(unsigned int regno, unsigned int red, ...@@ -222,16 +213,16 @@ static int v8_brazil_setpalette(unsigned int regno, unsigned int red,
* In 4bpp, the regnos are 0x0f, 0x1f, 0x2f, etc, etc * In 4bpp, the regnos are 0x0f, 0x1f, 0x2f, etc, etc
* In 2bpp, the regnos are 0x3f, 0x7f, 0xbf, 0xff * In 2bpp, the regnos are 0x3f, 0x7f, 0xbf, 0xff
*/ */
_regno = (regno << (8 - bpp)) | (0xFF >> bpp); regno = (regno << (8 - bpp)) | (0xFF >> bpp);
nubus_writeb(_regno, &v8_brazil_cmap_regs->addr); nubus_writeb(regno, &v8_brazil_cmap_regs->addr);
nop(); nop();
/* send one color channel at a time */ /* send one color channel at a time */
nubus_writeb(_red, &v8_brazil_cmap_regs->lut); nubus_writeb(red, &v8_brazil_cmap_regs->lut);
nop(); nop();
nubus_writeb(_green, &v8_brazil_cmap_regs->lut); nubus_writeb(green, &v8_brazil_cmap_regs->lut);
nop(); nop();
nubus_writeb(_blue, &v8_brazil_cmap_regs->lut); nubus_writeb(blue, &v8_brazil_cmap_regs->lut);
local_irq_restore(flags); local_irq_restore(flags);
return 0; return 0;
...@@ -242,11 +233,6 @@ static int rbv_setpalette(unsigned int regno, unsigned int red, ...@@ -242,11 +233,6 @@ static int rbv_setpalette(unsigned int regno, unsigned int red,
unsigned int green, unsigned int blue, unsigned int green, unsigned int blue,
struct fb_info *info) struct fb_info *info)
{ {
/* use MSBs */
unsigned char _red =red>>8;
unsigned char _green=green>>8;
unsigned char _blue =blue>>8;
unsigned char _regno;
unsigned long flags; unsigned long flags;
if (info->var.bits_per_pixel > 8) if (info->var.bits_per_pixel > 8)
...@@ -258,22 +244,22 @@ static int rbv_setpalette(unsigned int regno, unsigned int red, ...@@ -258,22 +244,22 @@ static int rbv_setpalette(unsigned int regno, unsigned int red,
* regno #254 and #255 are the important ones for 1-bit color, * regno #254 and #255 are the important ones for 1-bit color,
* regno #252-255 are the important ones for 2-bit color, etc. * regno #252-255 are the important ones for 2-bit color, etc.
*/ */
_regno = regno + (256-(1 << info->var.bits_per_pixel)); regno += 256 - (1 << info->var.bits_per_pixel);
/* reset clut? (VideoToolbox sez "not necessary") */ /* reset clut? (VideoToolbox sez "not necessary") */
nubus_writeb(0xFF, &rbv_cmap_regs->cntl); nubus_writeb(0xFF, &rbv_cmap_regs->cntl);
nop(); nop();
/* tell clut which address to use. */ /* tell clut which address to use. */
nubus_writeb(_regno, &rbv_cmap_regs->addr); nubus_writeb(regno, &rbv_cmap_regs->addr);
nop(); nop();
/* send one color channel at a time. */ /* send one color channel at a time. */
nubus_writeb(_red, &rbv_cmap_regs->lut); nubus_writeb(red, &rbv_cmap_regs->lut);
nop(); nop();
nubus_writeb(_green, &rbv_cmap_regs->lut); nubus_writeb(green, &rbv_cmap_regs->lut);
nop(); nop();
nubus_writeb(_blue, &rbv_cmap_regs->lut); nubus_writeb(blue, &rbv_cmap_regs->lut);
local_irq_restore(flags); local_irq_restore(flags);
return 0; return 0;
...@@ -284,25 +270,19 @@ static int mdc_setpalette(unsigned int regno, unsigned int red, ...@@ -284,25 +270,19 @@ static int mdc_setpalette(unsigned int regno, unsigned int red,
unsigned int green, unsigned int blue, unsigned int green, unsigned int blue,
struct fb_info *info) struct fb_info *info)
{ {
volatile struct mdc_cmap_regs *cmap_regs = struct mdc_cmap_regs *cmap_regs = slot_addr;
nubus_slot_addr(video_slot);
/* use MSBs */
unsigned char _red =red>>8;
unsigned char _green=green>>8;
unsigned char _blue =blue>>8;
unsigned char _regno=regno;
unsigned long flags; unsigned long flags;
local_irq_save(flags); local_irq_save(flags);
/* the nop's are there to order writes. */ /* the nop's are there to order writes. */
nubus_writeb(_regno, &cmap_regs->addr); nubus_writeb(regno, &cmap_regs->addr);
nop(); nop();
nubus_writeb(_red, &cmap_regs->lut); nubus_writeb(red, &cmap_regs->lut);
nop(); nop();
nubus_writeb(_green, &cmap_regs->lut); nubus_writeb(green, &cmap_regs->lut);
nop(); nop();
nubus_writeb(_blue, &cmap_regs->lut); nubus_writeb(blue, &cmap_regs->lut);
local_irq_restore(flags); local_irq_restore(flags);
return 0; return 0;
...@@ -313,25 +293,24 @@ static int toby_setpalette(unsigned int regno, unsigned int red, ...@@ -313,25 +293,24 @@ static int toby_setpalette(unsigned int regno, unsigned int red,
unsigned int green, unsigned int blue, unsigned int green, unsigned int blue,
struct fb_info *info) struct fb_info *info)
{ {
volatile struct toby_cmap_regs *cmap_regs = struct toby_cmap_regs *cmap_regs = slot_addr;
nubus_slot_addr(video_slot);
unsigned int bpp = info->var.bits_per_pixel; unsigned int bpp = info->var.bits_per_pixel;
/* use MSBs */
unsigned char _red =~(red>>8);
unsigned char _green=~(green>>8);
unsigned char _blue =~(blue>>8);
unsigned char _regno = (regno << (8 - bpp)) | (0xFF >> bpp);
unsigned long flags; unsigned long flags;
red = ~red;
green = ~green;
blue = ~blue;
regno = (regno << (8 - bpp)) | (0xFF >> bpp);
local_irq_save(flags); local_irq_save(flags);
nubus_writeb(_regno, &cmap_regs->addr); nubus_writeb(regno, &cmap_regs->addr);
nop(); nop();
nubus_writeb(_red, &cmap_regs->lut); nubus_writeb(red, &cmap_regs->lut);
nop(); nop();
nubus_writeb(_green, &cmap_regs->lut); nubus_writeb(green, &cmap_regs->lut);
nop(); nop();
nubus_writeb(_blue, &cmap_regs->lut); nubus_writeb(blue, &cmap_regs->lut);
local_irq_restore(flags); local_irq_restore(flags);
return 0; return 0;
...@@ -342,23 +321,18 @@ static int jet_setpalette(unsigned int regno, unsigned int red, ...@@ -342,23 +321,18 @@ static int jet_setpalette(unsigned int regno, unsigned int red,
unsigned int green, unsigned int blue, unsigned int green, unsigned int blue,
struct fb_info *info) struct fb_info *info)
{ {
volatile struct jet_cmap_regs *cmap_regs = struct jet_cmap_regs *cmap_regs = slot_addr;
nubus_slot_addr(video_slot);
/* use MSBs */
unsigned char _red = (red>>8);
unsigned char _green = (green>>8);
unsigned char _blue = (blue>>8);
unsigned long flags; unsigned long flags;
local_irq_save(flags); local_irq_save(flags);
nubus_writeb(regno, &cmap_regs->addr); nubus_writeb(regno, &cmap_regs->addr);
nop(); nop();
nubus_writeb(_red, &cmap_regs->lut); nubus_writeb(red, &cmap_regs->lut);
nop(); nop();
nubus_writeb(_green, &cmap_regs->lut); nubus_writeb(green, &cmap_regs->lut);
nop(); nop();
nubus_writeb(_blue, &cmap_regs->lut); nubus_writeb(blue, &cmap_regs->lut);
local_irq_restore(flags); local_irq_restore(flags);
return 0; return 0;
...@@ -383,10 +357,6 @@ static int civic_setpalette(unsigned int regno, unsigned int red, ...@@ -383,10 +357,6 @@ static int civic_setpalette(unsigned int regno, unsigned int red,
if (info->var.bits_per_pixel > 8) if (info->var.bits_per_pixel > 8)
return 1; /* failsafe */ return 1; /* failsafe */
red >>= 8;
green >>= 8;
blue >>= 8;
local_irq_save(flags); local_irq_save(flags);
/* Set the register address */ /* Set the register address */
...@@ -462,11 +432,17 @@ static int csc_setpalette(unsigned int regno, unsigned int red, ...@@ -462,11 +432,17 @@ static int csc_setpalette(unsigned int regno, unsigned int red,
unsigned int green, unsigned int blue, unsigned int green, unsigned int blue,
struct fb_info *info) struct fb_info *info)
{ {
mdelay(1); unsigned long flags;
local_irq_save(flags);
udelay(1); /* mklinux on PB 5300 waits for 260 ns */
nubus_writeb(regno, &csc_cmap_regs->clut_waddr); nubus_writeb(regno, &csc_cmap_regs->clut_waddr);
nubus_writeb(red, &csc_cmap_regs->clut_data); nubus_writeb(red, &csc_cmap_regs->clut_data);
nubus_writeb(green, &csc_cmap_regs->clut_data); nubus_writeb(green, &csc_cmap_regs->clut_data);
nubus_writeb(blue, &csc_cmap_regs->clut_data); nubus_writeb(blue, &csc_cmap_regs->clut_data);
local_irq_restore(flags);
return 0; return 0;
} }
...@@ -493,8 +469,8 @@ static int macfb_setcolreg(unsigned regno, unsigned red, unsigned green, ...@@ -493,8 +469,8 @@ static int macfb_setcolreg(unsigned regno, unsigned red, unsigned green,
case 4: case 4:
case 8: case 8:
if (macfb_setpalette) if (macfb_setpalette)
macfb_setpalette(regno, red, green, blue, macfb_setpalette(regno, red >> 8, green >> 8,
fb_info); blue >> 8, fb_info);
else else
return 1; return 1;
break; break;
...@@ -611,7 +587,10 @@ static int __init macfb_init(void) ...@@ -611,7 +587,10 @@ static int __init macfb_init(void)
* those mappings are set up, so this is in fact the safest * those mappings are set up, so this is in fact the safest
* way to ensure that this driver will work on every possible Mac * way to ensure that this driver will work on every possible Mac
*/ */
fb_info.screen_base = ioremap(mac_bi_data.videoaddr, macfb_fix.smem_len); fb_info.screen_base = ioremap(mac_bi_data.videoaddr,
macfb_fix.smem_len);
if (!fb_info.screen_base)
return -ENODEV;
printk("macfb: framebuffer at 0x%08lx, mapped to 0x%p, size %dk\n", printk("macfb: framebuffer at 0x%08lx, mapped to 0x%p, size %dk\n",
macfb_fix.smem_start, fb_info.screen_base, macfb_fix.smem_start, fb_info.screen_base,
...@@ -620,17 +599,13 @@ static int __init macfb_init(void) ...@@ -620,17 +599,13 @@ static int __init macfb_init(void)
macfb_defined.xres, macfb_defined.yres, macfb_defined.xres, macfb_defined.yres,
macfb_defined.bits_per_pixel, macfb_fix.line_length); macfb_defined.bits_per_pixel, macfb_fix.line_length);
/* /* Fill in the available video resolution */
* Fill in the available video resolution
*/
macfb_defined.xres_virtual = macfb_defined.xres; macfb_defined.xres_virtual = macfb_defined.xres;
macfb_defined.yres_virtual = macfb_defined.yres; macfb_defined.yres_virtual = macfb_defined.yres;
macfb_defined.height = PIXEL_TO_MM(macfb_defined.yres); macfb_defined.height = PIXEL_TO_MM(macfb_defined.yres);
macfb_defined.width = PIXEL_TO_MM(macfb_defined.xres); macfb_defined.width = PIXEL_TO_MM(macfb_defined.xres);
printk("macfb: scrolling: redraw\n"); /* Some dummy values for timing to make fbset happy */
/* some dummy values for timing to make fbset happy */
macfb_defined.pixclock = 10000000 / macfb_defined.xres * macfb_defined.pixclock = 10000000 / macfb_defined.xres *
1000 / macfb_defined.yres; 1000 / macfb_defined.yres;
macfb_defined.left_margin = (macfb_defined.xres / 8) & 0xf8; macfb_defined.left_margin = (macfb_defined.xres / 8) & 0xf8;
...@@ -666,8 +641,6 @@ static int __init macfb_init(void) ...@@ -666,8 +641,6 @@ static int __init macfb_init(void)
macfb_defined.green.length = 5; macfb_defined.green.length = 5;
macfb_defined.blue.offset = 0; macfb_defined.blue.offset = 0;
macfb_defined.blue.length = 5; macfb_defined.blue.length = 5;
printk("macfb: directcolor: "
"size=1:5:5:5, shift=15:10:5:0\n");
video_cmap_len = 16; video_cmap_len = 16;
/* /*
* Should actually be FB_VISUAL_DIRECTCOLOR, but this * Should actually be FB_VISUAL_DIRECTCOLOR, but this
...@@ -683,10 +656,9 @@ static int __init macfb_init(void) ...@@ -683,10 +656,9 @@ static int __init macfb_init(void)
macfb_defined.green.length = 8; macfb_defined.green.length = 8;
macfb_defined.blue.offset = 0; macfb_defined.blue.offset = 0;
macfb_defined.blue.length = 8; macfb_defined.blue.length = 8;
printk("macfb: truecolor: "
"size=0:8:8:8, shift=0:16:8:0\n");
video_cmap_len = 16; video_cmap_len = 16;
macfb_fix.visual = FB_VISUAL_TRUECOLOR; macfb_fix.visual = FB_VISUAL_TRUECOLOR;
break;
default: default:
video_cmap_len = 0; video_cmap_len = 0;
macfb_fix.visual = FB_VISUAL_MONO01; macfb_fix.visual = FB_VISUAL_MONO01;
...@@ -704,16 +676,17 @@ static int __init macfb_init(void) ...@@ -704,16 +676,17 @@ static int __init macfb_init(void)
* code is really broken :-) * code is really broken :-)
*/ */
while ((ndev = nubus_find_type(NUBUS_CAT_DISPLAY, NUBUS_TYPE_VIDEO, ndev)) while ((ndev = nubus_find_type(NUBUS_CAT_DISPLAY,
!= NULL) NUBUS_TYPE_VIDEO, ndev)))
{ {
if (!(mac_bi_data.videoaddr >= ndev->board->slot_addr unsigned long base = ndev->board->slot_addr;
&& (mac_bi_data.videoaddr <
(unsigned long)nubus_slot_addr(ndev->board->slot+1)))) if (mac_bi_data.videoaddr < base ||
mac_bi_data.videoaddr - base > 0xFFFFFF)
continue; continue;
video_is_nubus = 1; video_is_nubus = 1;
/* We should probably just use the slot address... */ slot_addr = (unsigned char *)base;
video_slot = ndev->board->slot;
switch(ndev->dr_hw) { switch(ndev->dr_hw) {
case NUBUS_DRHW_APPLE_MDC: case NUBUS_DRHW_APPLE_MDC:
...@@ -882,6 +855,9 @@ static int __init macfb_init(void) ...@@ -882,6 +855,9 @@ static int __init macfb_init(void)
* really support that. * really support that.
*/ */
/*
* Slot 0 ROM says TIM. No external video. B&W.
*/
case MAC_MODEL_PB140: case MAC_MODEL_PB140:
case MAC_MODEL_PB145: case MAC_MODEL_PB145:
case MAC_MODEL_PB170: case MAC_MODEL_PB170:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment