Page MenuHomeFreeBSD

Cadence GEM driver style clean-up.
ClosedPublic

Authored by skibo on Mar 30 2020, 6:30 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Oct 7, 1:51 AM
Unknown Object (File)
Sep 13 2024, 12:24 AM
Unknown Object (File)
Sep 12 2024, 6:01 PM
Unknown Object (File)
Sep 7 2024, 2:17 PM
Unknown Object (File)
Aug 25 2024, 11:19 AM
Unknown Object (File)
Aug 16 2024, 9:14 PM
Unknown Object (File)
May 30 2024, 8:22 AM
Unknown Object (File)
May 20 2024, 7:47 AM

Details

Summary

I have significant changes to the GEM driver coming but the changes include
lots of style and white-space clean-up so I am submitting these separately
to perhaps make it easier to review the functional changes.

Test Plan

I have diffed the disassembly of if_cgem.o before and after these changes
and the only difference is a small change in string literal addresses
because I have tweaked a couple printf's.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 30190
Build 27982: arc lint + arc unit

Event Timeline

Ok, roast me!

sys/dev/cadence/if_cgem_hw.h
302

Another issue I noticed is that I am not consistent using upper or lower case letters in hex literals. Is there an accepted standard?

This revision was not accepted when it landed; it landed in state Needs Review.Apr 2 2020, 5:57 PM
This revision was automatically updated to reflect the committed changes.