Page MenuHomeFreeBSD

add infineon adm6996fc support on etherswitch
ClosedPublic

Authored by yamori813_yahoo.co.jp on Nov 11 2016, 12:59 AM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Dec 19, 8:31 PM
Unknown Object (File)
Mon, Dec 2, 12:27 PM
Unknown Object (File)
Thu, Nov 28, 7:21 PM
Unknown Object (File)
Mon, Nov 25, 4:31 PM
Unknown Object (File)
Oct 4 2024, 2:44 PM
Unknown Object (File)
Sep 22 2024, 6:36 AM
Unknown Object (File)
Sep 21 2024, 9:25 AM
Unknown Object (File)
Sep 17 2024, 7:00 AM
Subscribers

Details

Summary

add infineon adm6996fc support on etherswitch

Test Plan

check on Netgear WGR614Cv7

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

yamori813_yahoo.co.jp retitled this revision from to add infineon adm6996fc support on etherswitch.
yamori813_yahoo.co.jp updated this object.
yamori813_yahoo.co.jp edited the test plan for this revision. (Show Details)
  1. I see some "magic" constants. May be worth to add macros.
  2. DPRINTF usage: it prints function name on each call, is it useful/enough useful?
  3. "style(9)": ADM6996FC_READREG/WRITEREG - to be tab instead of space
  4. No checks on malloc results, may be NULL.
  5. Probe: can you add array of IDs to match. In future it will be easy to add IDs into array to bring support of another models.
sys/dev/etherswitch/infineon/adm6996fc.c
721โ€“722 โ†—(On Diff #22137)

one empty line is enough ;)

  1. add more macros.

2 This code is same as other ehterswitch code.

  1. add space.
  2. Other etherswitch code not check malloc also...
  3. Check only Product Code.
adrian added a reviewer: adrian.
This revision is now accepted and ready to land.Nov 16 2016, 11:43 PM
mizhka added a reviewer: mizhka.
This revision was automatically updated to reflect the committed changes.