Page MenuHomeFreeBSD

Cleanup of the ENA driver header file
ClosedPublic

Authored by mk_semihalf.com on Oct 31 2017, 1:44 PM.
Tags
None
Referenced Files
F82810581: D12864.id34988.diff
Thu, May 2, 7:42 PM
F82809730: D12864.diff
Thu, May 2, 7:33 PM
F82809546: D12864.diff
Thu, May 2, 7:30 PM
F82809149: D12864.id34988.diff
Thu, May 2, 7:24 PM
F82808725: D12864.diff
Thu, May 2, 7:20 PM
Unknown Object (File)
Mon, Apr 15, 4:55 AM
Unknown Object (File)
Mar 31 2024, 2:12 PM
Unknown Object (File)
Mar 10 2024, 11:33 PM

Details

Summary

Remove unused macros and fields - some of them were only initialized,
but had no further uses.

Minor style fixes and added required comments.

Missing TX completions counter was existing, but was not used in driver
code - the required line had been added.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

Always nice to have minus diffs, assuming it still compiles :)

sys/dev/ena/ena.c
3407

Stray patch?

sys/dev/ena/ena.h
73

Can delete this too?

240

The leading whitespace looks funny in phabricator, can you double check it here?

sys/dev/ena/ena.c
3407

I wrote about this in the commit log. Should i split it into 2 patches?

mk_semihalf.com marked 2 inline comments as done.

Removed unused macro and additional whitespace

This revision is now accepted and ready to land.Nov 8 2017, 11:27 PM
This revision was automatically updated to reflect the committed changes.