Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove dependency on Elbox's pci.library #1

Closed
rkujawa opened this issue Mar 3, 2015 · 8 comments
Closed

Remove dependency on Elbox's pci.library #1

rkujawa opened this issue Mar 3, 2015 · 8 comments
Assignees

Comments

@rkujawa
Copy link
Member

rkujawa commented Mar 3, 2015

Problems

  • Currently sonnet.library is relying on a modified version of Elbox's pci.library.
  • Currently sonnet.library requires an access to graphics card memory, which in some cases require prior card-specific setup (for example initialisation of memory controller on Voodoo 3).
  • Elbox does provide development kit only under restrictive NDA.
    • We don't have an access to that SDK.
    • Such restrictive NDA is incompatible with open source nature of this project.

Proposed solution

  • Write a new, open source implementation of PCI library - codename KlozetPCI.
  • Use NetBSD Mediator drivers as a basis.
  • Provide bus_space(4)-like API.
  • Optionally add a compatibility layer that would allow to use existing drivers (OpenPCI or Elbox pci.library API reimplementation).
@rkujawa rkujawa self-assigned this Mar 3, 2015
@rkujawa
Copy link
Member Author

rkujawa commented Mar 11, 2015

This is being worked on in KlozetPCI project:
https://github.com/Sakura-IT/KlozetPCI

@DvdBoon
Copy link
Contributor

DvdBoon commented Mar 15, 2015

Dependency could already be made to a minimum for at least the sonnet card if voodoo3 gfx memory initialization code could be added to the sonnet.library (and other gfx card memory initialization code). However, all other mediator functionality (graphics, network, usb etc.) will still be dependent on the pci.library in this case.

Until now the way to initialize the voodoo3 gfx mem eludes me.

@rkujawa
Copy link
Member Author

rkujawa commented Jul 20, 2016

Did you perhaps investigate the supposed new functionality of pci.library 11.0? I.e. "New pci.library gives the possibility to reallocate (free and allocate) PCI spaces from the driver for the specific PCI card".

Does the 11.0 library still lock up the system when Sonnet is installed?

@DvdBoon
Copy link
Contributor

DvdBoon commented Jul 20, 2016

Not yet. But I can confirm that the Amiga starts up with pci.library 11.0
and a sonnet card installed. No infinite loop during PCI memory allocation
anymore. Sonnet card is correctly identified by pciinfo and the A1200 PPC
setup code starts the PPC processor correctly.

2016-07-20 9:25 GMT+02:00 Radosław Kujawa notifications@github.com:

Did you perhaps investigate the supposed new functionality of pci.library
11.0? I.e. "New pci.library gives the possibility to reallocate (free and
allocate) PCI spaces from the driver for the specific PCI card".

Does the 11.0 library still lock up the system when Sonnet is installed?


You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#1 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKdWEHxIXj23x6vdkl1TxojYVG5MvwMmks5qXc1jgaJpZM4DpED9
.

@rkujawa
Copy link
Member Author

rkujawa commented Jul 20, 2016

Since they have eliminated the problems with startup, maybe it would be possible use stock non-patched pci.library?

@DvdBoon
Copy link
Contributor

DvdBoon commented Jul 20, 2016

I think so, yes. I've installed it on my A1200 and also made a quick
posting on EAB about it. Have not tested it running real software yet, but
on the A1200 the amiga starts, pciinfo sees the card correctly and the PPC
boots correctly.

Cannot really run anything on it as I cannot load any software yet (that is
where the mmu library comes in), but the PPC kernel is running.

Op 20 jul. 2016 11:16 a.m. schreef "Radosław Kujawa" <
notifications@github.com>:

Since they have eliminated the problems with startup, maybe it would be
possible use stock non-patched pci.library?


You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#1 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKdWEG_v2iFxRFG60INqWH9bsGvRWpJ2ks5qXedygaJpZM4DpED9
.

@DvdBoon
Copy link
Contributor

DvdBoon commented Oct 25, 2016

11.0 seems to work. Also I think I have enough info to replace the pci library on the big box Amigas, but I'm lacking time and it's prioritized rather low on my list.

@rkujawa
Copy link
Member Author

rkujawa commented Feb 3, 2018

In an unexpected turn of events, Elbox started cooperating with Dennis on updating pci.library to fully support Sonnet (and other future accelerators). Therefore this task seem unnecessary and would be a pure waste of resources.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants