On 04/19/2012 04:55 PM, Julia Lawall wrote: > From: Julia Lawall <Julia.Lawall@xxxxxxx> > > Release bridge, as done on all other paths out of the function. > > Signed-off-by: Julia Lawall <Julia.Lawall@xxxxxxx> Looks good to me, applied. Thanks, Florian Tobias Schandinat > > --- > drivers/video/intelfb/intelfbdrv.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/video/intelfb/intelfbdrv.c b/drivers/video/intelfb/intelfbdrv.c > index 02fd226..bdcbfba 100644 > --- a/drivers/video/intelfb/intelfbdrv.c > +++ b/drivers/video/intelfb/intelfbdrv.c > @@ -680,6 +680,7 @@ static int __devinit intelfb_pci_register(struct pci_dev *pdev, > + dinfo->fb.size); > if (!dinfo->aperture.virtual) { > ERR_MSG("Cannot remap FB region.\n"); > + agp_backend_release(bridge); > cleanup(dinfo); > return -ENODEV; > } > @@ -689,6 +690,7 @@ static int __devinit intelfb_pci_register(struct pci_dev *pdev, > INTEL_REG_SIZE); > if (!dinfo->mmio_base) { > ERR_MSG("Cannot remap MMIO region.\n"); > + agp_backend_release(bridge); > cleanup(dinfo); > return -ENODEV; > } > > -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html