Opened 7 years ago

Closed 7 years ago

#8615 closed enhancement (fixed)

Intel 810 Video Driver Addition

Reported by: gerald.zajac Owned by: nobody
Priority: normal Milestone: R1/alpha4
Component: Drivers/Graphics/intel_810 Version: R1/Development
Keywords: Cc: umccullough
Blocked By: Blocking:
Has a Patch: no Platform: x86

Description (last modified by diver)

This is a new video driver which supports the Intel 810 & 815 video chips which have the following ID's and names:

0x7121 810,
0x7123 810-dc100,
0x7125 810e,
0x1132 815,

The driver has been tested under Haiku alpha3 with all of the above chips except the 810-dc100 chip.

The attached zip file contains all the source, header, and jam files for this driver.

Attachments (1)

intel_810.zip (34.5 KB) - added by gerald.zajac 7 years ago.
Source, header, and jam files for Intel 810 video driver.

Download all attachments as: .zip

Change History (12)

Changed 7 years ago by gerald.zajac

Attachment: intel_810.zip added

Source, header, and jam files for Intel 810 video driver.

comment:1 Changed 7 years ago by gerald.zajac

Description: modified (diff)

comment:2 Changed 7 years ago by umccullough

Cc: umccullough added

Spectacular!

I'll try to locate my i8xx-based machines tonight to test this with :)

comment:3 Changed 7 years ago by kallisti5

Milestone: R1R1/alpha4
Version: R1/alpha3R1/Development

I gave it a quick run over... it all looks really good. There were a few quirks with 3 vs two 2 returns before and after functions, however otherwise it passes the style smell test.

I do worry a little about some of the licensing in the i810_* files, changing as-is copyrights to mit can be tricky. (although you did document the original license, so those concerns may be moot)

Great work!!! umccullough, interested in your test results :)

Setting to development with a milestone of alpha 4.

Last edited 7 years ago by kallisti5 (previous) (diff)

comment:4 Changed 7 years ago by umccullough

It seems I'm out of luck this evening, my i810-based machine is not here. I suspect it's in storage at my mother's house.

comment:5 Changed 7 years ago by kallisti5

commited in hrev44212 (and cleaned up)

I'm leaving this ticket open for testing. I don't have any i810 machines laying around so someone else needs to verify.

comment:6 in reply to:  5 Changed 7 years ago by gerald.zajac

Replying to kallisti5:

commited in hrev44212 (and cleaned up)

I'm leaving this ticket open for testing. I don't have any i810 machines laying around so someone else needs to verify.

kallisti5; I downloaded nightly image hrev44215, and it does not contain the intel_810 driver or accelerant. If the nightly images don't contain the intel_810 driver or accelerant, it will be difficult for anyone the test it unless they build and install the driver and accelerant themselves.

comment:7 Changed 7 years ago by kallisti5

Yup, as noted it hasn't been put into the images yet.

I was waiting for umccullough to do a quick test run before putting it into the image.. give it a few days, if he cannot I'll put it into the nightlies.

comment:8 Changed 7 years ago by umccullough

It will be more than a few days at this point - I probably won't get a chance to locate my i810-based machine for a couple weeks at least.

I suggest throwing it in the nightlies and requesting on haiku-dev list that people test it once it gets there.

comment:9 Changed 7 years ago by anevilyak

Added in hrev44219.

comment:10 in reply to:  7 Changed 7 years ago by axeld

Replying to kallisti5:

Yup, as noted it hasn't been put into the images yet.

I was waiting for umccullough to do a quick test run before putting it into the image.. give it a few days, if he cannot I'll put it into the nightlies.

Unless you know that the driver is broken, and will cause problems, it's a good idea to add it to the image to prove you wrong :-)

That's pretty much what the nightlies are for, anyway.

comment:11 Changed 7 years ago by diver

Component: Drivers/GraphicsDrivers/Graphics/intel_810
Description: modified (diff)
Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.