Skip to content

Commit

Permalink
[VIDEO]: Do not prom_halt() in cg3 and bw2 device probe.
Browse files Browse the repository at this point in the history
Just give a normal kernel log message of the problem and
return failure.

Based upon a patch from Mark Fortescue.

Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
David S. Miller committed Aug 27, 2007
1 parent 1bd4b28 commit 6c8f5b9
Show file tree
Hide file tree
Showing 2 changed files with 23 additions and 16 deletions.
22 changes: 13 additions & 9 deletions drivers/video/bw2.c
Expand Up @@ -233,9 +233,9 @@ static u8 bw2regs_66hz[] __devinitdata = {
0x10, 0x20, 0
};

static void __devinit bw2_do_default_mode(struct bw2_par *par,
struct fb_info *info,
int *linebytes)
static int __devinit bw2_do_default_mode(struct bw2_par *par,
struct fb_info *info,
int *linebytes)
{
u8 status, mon;
u8 *p;
Expand Down Expand Up @@ -266,17 +266,18 @@ static void __devinit bw2_do_default_mode(struct bw2_par *par,
break;

case BWTWO_SR_ID_NOCONN:
return;
return 0;

default:
prom_printf("bw2: can't handle SR %02x\n",
status);
prom_halt();
printk(KERN_ERR "bw2: can't handle SR %02x\n",
status);
return -EINVAL;
}
for ( ; *p; p += 2) {
u8 __iomem *regp = &((u8 __iomem *)par->regs)[p[0]];
sbus_writeb(p[1], regp);
}
return 0;
}

static int __devinit bw2_probe(struct of_device *op, const struct of_device_id *match)
Expand Down Expand Up @@ -312,8 +313,11 @@ static int __devinit bw2_probe(struct of_device *op, const struct of_device_id *
if (!par->regs)
goto out_release_fb;

if (!of_find_property(dp, "width", NULL))
bw2_do_default_mode(par, info, &linebytes);
if (!of_find_property(dp, "width", NULL)) {
err = bw2_do_default_mode(par, info, &linebytes);
if (err)
goto out_unmap_regs;
}

par->fbsize = PAGE_ALIGN(linebytes * info->var.yres);

Expand Down
17 changes: 10 additions & 7 deletions drivers/video/cg3.c
Expand Up @@ -315,7 +315,7 @@ static u_char cg3_dacvals[] __devinitdata = {
4, 0xff, 5, 0x00, 6, 0x70, 7, 0x00, 0
};

static void __devinit cg3_do_default_mode(struct cg3_par *par)
static int __devinit cg3_do_default_mode(struct cg3_par *par)
{
enum cg3_type type;
u8 *p;
Expand All @@ -332,10 +332,9 @@ static void __devinit cg3_do_default_mode(struct cg3_par *par)
else
type = CG3_AT_66HZ;
} else {
prom_printf("cgthree: can't handle SR %02x\n",
status);
prom_halt();
return;
printk(KERN_ERR "cgthree: can't handle SR %02x\n",
status);
return -EINVAL;
}
}

Expand All @@ -351,6 +350,7 @@ static void __devinit cg3_do_default_mode(struct cg3_par *par)
regp = (u8 __iomem *)&par->regs->cmap.control;
sbus_writeb(p[1], regp);
}
return 0;
}

static int __devinit cg3_probe(struct of_device *op,
Expand Down Expand Up @@ -400,8 +400,11 @@ static int __devinit cg3_probe(struct of_device *op,

cg3_blank(0, info);

if (!of_find_property(dp, "width", NULL))
cg3_do_default_mode(par);
if (!of_find_property(dp, "width", NULL)) {
err = cg3_do_default_mode(par);
if (err)
goto out_unmap_screen;
}

if (fb_alloc_cmap(&info->cmap, 256, 0))
goto out_unmap_screen;
Expand Down

0 comments on commit 6c8f5b9

Please sign in to comment.