Well, too late...
I sent this night a patch that enabled PCI in the same way than you,
viraptor.
The problem is that this mail is waiting for moderator approval, because it
is too big (a screenshot make it more than 50KB).
I send here a modified one because I think it can be merged easily into your
code.

The difference is that I've created a PCIController class (that is the
equivalent of your PCI class)  that maintain an array of PCIDevice
instances. This controller is a stub class that is initialized at boot time.

PCIDevice is implementation independant and make call to PCIController
methods.
I've added a shell function 'pci' that list the detected devices.

Now, we have to implement a VendorID/DeviceID lookup table (such a list can
be found in the net), and to create specialized implementations of PCIDevice
using the PCIDevice.Class property.
I've already created an enumeration for class types and the 'pci' command
could be more descriptive if Enum.ToString() member exist.

Cedric

On Jan 9, 2008 3:03 AM, Bruce Markham <[EMAIL PROTECTED]> wrote:

> viraptor:
> Excellent!
>
> The CPUID patch is effective as of revision 708.
>
> The AOT patch is effective as of revision 709. (Honestly, I don't
> understand why that xor'ing needs to even take place, but since DarxKies is
> unavailable for comment, and your patch doesn't break any tests, I let it on
> through.)
>
> The PCI patch is effective as of revision 710. Your submitted patch
> omitted your changes to IO.cs, but I worked around that - as I've actually
> been doing some PCI work offline.
>
> I'm glad you (as in, at least someone), has put forth some PCI work, as my
> own personal pursuit in the manner involved me getting way in over my head.
> But hopefully now that object support is getting in, we can all sit down and
> figure out how we want to abstract hardware and start doing fun stuff with
> it.
>
> Regardless, the patches are most appreciatted, and we look forward to
> working with you some more!
> -Bruce
>  aka illuminus
>
> -------------------------------------------------------------------------
> Check out the new SourceForge.net Marketplace.
> It's the best place to buy or sell services for
> just about anything Open Source.
>
> http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
> _______________________________________________
> SharpOS-Developers mailing list
> SharpOS-Developers@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/sharpos-developers
>
>

Attachment: 20080109_PCI_2.zipped_patch
Description: Binary data

-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace
_______________________________________________
SharpOS-Developers mailing list
SharpOS-Developers@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sharpos-developers

Reply via email to