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

Add link for bonw15 Ethernet #34

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add link for bonw15 Ethernet #34

wants to merge 1 commit into from

Conversation

ahoneybun
Copy link
Member

This is needed for working Ethernet as reported and confirmed by a customer.

@leviport leviport requested review from a team June 11, 2024 15:02
@leviport
Copy link
Member

Why link to Intel's site, rather than download to this repo and link it there? Also don't we usually get these elsewhere?

@ahoneybun
Copy link
Member Author

The org of this repo is not very clear and I guess I should just ping @crawfxrd on how he wants to do this addition.

@crawfxrd
Copy link
Member

I think we get all the Intel drivers from the private software kits, preferably the Best Known Configuration kits.

@crawfxrd
Copy link
Member

The Ethernet drivers in BKC are for I219 and I225-I226. They do not provider installers, just the driver files.

@ahoneybun
Copy link
Member Author

The Ethernet drivers in BKC are for I219 and I225-I226. They do not provider installers, just the driver files.

In that case is there a reason to not link to official Intel docs that provide the driver(s)?

@crawfxrd
Copy link
Member

Have we had any questions about Intel Ethernet drivers for other systems?

These use I219 and would need the driver link as well:

  • addw3
  • darp10
  • gaze16
  • gaze17

@ahoneybun
Copy link
Member Author

I don't believe so but I don't see a reason to not list the driver for those models in this PR or another one.

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

Successfully merging this pull request may close these issues.

3 participants