Page MenuHomeFreeBSD

Mechanically convert neta(4) to DrvAPI
ClosedPublic

Authored by jhibbits on Dec 22 2022, 3:32 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, May 28, 4:23 AM
Unknown Object (File)
May 11 2024, 6:28 PM
Unknown Object (File)
May 11 2024, 6:23 PM
Unknown Object (File)
May 11 2024, 6:23 PM
Unknown Object (File)
May 11 2024, 6:23 PM
Unknown Object (File)
May 11 2024, 6:23 PM
Unknown Object (File)
May 8 2024, 6:15 PM
Unknown Object (File)
May 4 2024, 7:49 AM
Subscribers

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

mw requested changes to this revision.Jan 9 2023, 3:04 PM
mw added inline comments.
sys/dev/neta/if_mvneta.c
936

Erroneous line removal.

943

ditto

2095

Can these 2 lines be wrapped into 1:
if_setcapenablebit(ifp, IFCAP_HWCSUM & ifr->ifr_reqcap, IFCAP_HWCSUM);
?

2154

While at it, can you please fix line breaking?

3266

While at it, please add missing spaces around |

This revision now requires changes to proceed.Jan 9 2023, 3:04 PM
sys/dev/neta/if_mvneta.c
936

For other CTR* entries ifp->if_xname was updated to if_name(sc->ifp) - please do the same here for consistency.

943

For other CTR* entries ifp->if_xname was updated to if_name(sc->ifp) - please do the same here for consistency.

2095

?

sys/dev/neta/if_mvneta.c
2095

Not as it's currently implemented. It currently sets the "set" mask, then clears the "clear" mask, which is the opposite direction of this.

This revision is now accepted and ready to land.Jan 10 2023, 9:25 AM
This revision was automatically updated to reflect the committed changes.