Antonino A. Daplas wrote:
> Knut Petersen wrote:
>
> Wrong boolean check? Should be if (vesafb & 4). Or might as
> well get rid of this check, it's redundant.
>
>> + release_mem_region(info->fix.smem_start,
>> + info->fix.smem_len);
>> iounmap(io_virt);
>> errout_mmio_remap:
>> release_mem_region(info->fix.mmio_start,
>> info->fix.mmio_len);
>> errout_mmio_reqmem:
>> -// release_region(0x3c0,32);
>> + if (!(vesafb & 1))
>> + release_region(0x3c0,32);
>> errout_enable:
>> + kfree(info->pixmap.addr);
>> + errout_alloc_pixmap:
>> framebuffer_release(info);
>> - errout_alloc:
>> + errout_alloc_info:
>> output("CyblaFB version %s aborting init.\n",VERSION);
>> return -ENODEV;
>> }
>> @@ -1359,9 +1468,15 @@ static void __devexit cybla_pci_remove(s
>> unregister_framebuffer(info);
>> iounmap(io_virt);
>> iounmap(info->screen_base);
>> - release_mem_region(info->fix.smem_start,info->fix.smem_len);
>> + if (!(vesafb & 4))
>
> Shouldn't this be if (vesafb & 4)?
>
>> + release_mem_region(info->fix.smem_start,info->fix.smem_len);
>> release_mem_region(info->fix.mmio_start,info->fix.mmio_len);
>> fb_dealloc_cmap(&info->cmap);
>> + if (!(vesafb & 2))
>
> and this...
>
>> + release_region(GEBase,0x100);
>> + if (!(vesafb & 1))
>
> and this...?
>
Disregard the last comments, you're right.
Tony
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]