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.

Sign up for GitHub

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

Jump to bottom

RTL8168 Implementation #1652

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

RTL8168 Implementation #1652

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
65c39b8
Add driver + card initialization
valentinbreiz Jan 22, 2021
312db3b
Merge branch 'master' into rtl8168
valentinbreiz Apr 9, 2021
0e26427
Merge branch 'master' into rtl8168
valentinbreiz Apr 15, 2021
8d5651f
Merge branch 'master' into rtl8168
MishaTy Nov 14, 2021
4679538
Merge branch 'master' into rtl8168
MishaTy Feb 5, 2022
f01229f
Merge branch 'master' into rtl8168
MishaTy Feb 8, 2022
59028ec
Merge branch 'master' into rtl8168
MishaTy Feb 9, 2022
7ae17cf
Merge branch 'master' into rtl8168
MishaTy Mar 3, 2022
9908de8
Merge branch 'master' into rtl8168
MishaTy Feb 3, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
86 changes: 86 additions & 0 deletions source/Cosmos.Core/Ports.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,86 @@
using System;
using System.Collections.Generic;
using System.Text;

namespace Cosmos.Core
{
public static class Ports
{
/// <summary>
/// Reads a byte
/// </summary>
/// <param name="port"></param>
/// <returns></returns>
public static byte InB(ushort port)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I prefer we go with the same approach we have for other drivers as well, which is that we define all the the IOPorts we need and then access the one we actually need. For example see

Cosmos/source/Cosmos.Core/IOGroup/ATA.cs

Line 13 in a3d0a07

public class ATA : IOGroup

That approach should be quicker since we arnt constantly creating new objects and less error prone than calculating the port address every time.

{
var io = new IOPort(port);
return io.Byte;
}

/// <summary>
/// Reads a 32 bit word
/// </summary>
/// <param name="port"></param>
/// <returns></returns>
public static uint InD(ushort port)
{
var io = new IOPort(port);
return io.DWord;
}

/// <summary>
/// Reads a 16 bit word
/// </summary>
/// <param name="port"></param>
/// <returns></returns>
public static ushort InW(ushort port)
{
var io = new IOPort(port);
return io.Word;
}

/// <summary>
/// Writes a byte
/// </summary>
/// <param name="port"></param>
/// <param name="data"></param>
public static void OutB(ushort port, byte data)
{
var io = new IOPort(port);
io.Byte = data;
}

/// <summary>
/// Writes a 32 bit word
/// </summary>
/// <param name="port"></param>
/// <param name="data"></param>
public static void OutD(ushort port, byte data)
{
var io = new IOPort(port);
io.DWord = data;
}

/// <summary>
/// Writes a 32 bit word
/// </summary>
/// <param name="port"></param>
/// <param name="data"></param>
public static void OutD(ushort port, uint data)
{
var io = new IOPort(port);
io.DWord = data;
}

/// <summary>
/// Writes a 16 bit word
/// </summary>
/// <param name="port"></param>
/// <param name="data"></param>
public static void OutW(ushort port, ushort data)
{
var io = new IOPort(port);
io.Word = data;
}
}
}
Loading