# Re: \[RFC PATCH v2\] Resurrect Intel740 driver: i740fb

- From: Dave Airlie \<airlied@gmail.com\>
- Date: 2011-08-23 09:10:04 
- List: linux-fbdev
- Page: https://ratatoskr.run/linux-fbdev/2011/08/3755792
- Thread: https://ratatoskr.run/linux-fbdev/2011/08/3755165/t.md

```text
On Sun, Aug 21, 2011 at 9:42 PM, Ondrej Zary [ref] wrote:
> Hello,
> this is an v2 attempt to resurrect an old (like 2.4.19) out-of-tree driver for
> Intel740 graphics cards and modify it for recent kernels. The old driver is
> located at: http://sourceforge.net/projects/i740fbdev/files/
>
> It was easier to create a new driver based on skeletonfb, using most of the
> low level HW code from the old driver. The DDC code is completely new.
>
> The driver was tested on two 8MB cards: ManLi and Diamond Stealth II G460
>
> The i740_calc_fifo() function formatting is nice but does not pass
> checkpatch. Making it checkpatch-compliant makes the code look ugly. How to
> make it both nice and compatible with Linux coding style?
> (the remaining // comments mark things that need attention - missing xpan
> and suspend/resume)
>
> Changes in v2:
> - fixed 15bpp (16bpp 555) mode
> - not working 1280x1024-32 is not a bug but card's feature, it's now handled
>  correctly by the driver
> - 24bpp modes still don't work in fbtest but it looks like fbtest bug!
>
> Is there any better way to test fb drivers than fbtest?
>
> no signed-off-by yet
>
> --- linux-2.6.39-rc2-orig/drivers/video/Kconfig 2011-04-06 03:30:43.000000000 +0200
> +++ linux-2.6.39-rc2/drivers/video/Kconfig      2011-07-30 11:34:10.000000000 +0200
> @@ -1117,6 +1117,17 @@ config FB_RIVA_BACKLIGHT
>        help
>          Say Y here if you want to control the backlight of your display.
>
> +config FB_I740
> +       tristate "Intel740 support (EXPERIMENTAL)"
> +       depends on EXPERIMENTAL && FB && PCI && X86_32 && AGP_INTEL
> +       select FB_MODE_HELPERS
> +       select FB_CFB_FILLRECT
> +       select FB_CFB_COPYAREA
> +       select FB_CFB_IMAGEBLIT
> +       select VGASTATE
> +       help
> +         This driver supports graphics cards based on Intel740 chip.
> +

It shouldn't need AGP_INTEL, as its an AGP card not an integrated
chipset, it might need just AGP.

Dave.
```