Skip to content

Add RTL8211F driver#58

Draft
ZigaMahne wants to merge 1 commit into
ARM-software:mainfrom
ZigaMahne:main
Draft

Add RTL8211F driver#58
ZigaMahne wants to merge 1 commit into
ARM-software:mainfrom
ZigaMahne:main

Conversation

@ZigaMahne

Copy link
Copy Markdown

No description provided.

@RobertRostohar
RobertRostohar marked this pull request as draft July 16, 2026 11:42
#include "Driver_ETH_PHY.h"

#ifndef ARM_ETH_INTERFACE_RGMII
#define ARM_ETH_INTERFACE_RGMII 3U /* CMSIS-Driver 2.x has no RGMII ID */

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This should be removed once it is part of the updated ETH driver API.


switch (interface) {
case ARM_ETH_INTERFACE_MII:
case ARM_ETH_INTERFACE_RGMII:

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This will need to be reworked with new defines for media interfaces once ETH driver API is updated.

Comment thread ARM.CMSIS-Driver.pdsc
</files>
</component>

<component Cvendor="Keil" Cclass="CMSIS Driver" Cgroup="Ethernet PHY" Csub="RTL8211F" Capiversion="2.2.0" Cversion="1.0.0" condition="Device CMSIS_Core">

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Capiversion will be higher (probably "2.3.0") once ETH Driver API is updated.

Comment thread ARM.CMSIS-Driver.pdsc

<requirements>
<packages>
<package vendor="ARM" name="CMSIS" version="5.9.0-0"/>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The latest CMSIS pack with updated ETH Driver API will be required.

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.

2 participants