Page MenuHomeFreeBSD
Differential D8495

add infineon adm6996fc support on etherswitch
ClosedPublic
Actions

Authored by yamori813_yahoo.co.jp on Nov 11 2016, 12:59 AM.
Tags
None
Referenced Files
F149398198: D8495.id22137.diff
Tue, Mar 24, 4:47 AM
Unknown Object (File)
Fri, Mar 20, 3:19 PM
Unknown Object (File)
Thu, Mar 19, 6:39 PM
Unknown Object (File)
Tue, Mar 17, 5:32 PM
Unknown Object (File)
Tue, Mar 17, 12:03 PM
Unknown Object (File)
Sat, Mar 7, 8:32 PM
Unknown Object (File)
Mon, Mar 2, 3:04 PM
Unknown Object (File)
Mon, Mar 2, 2:30 AM
View All Files
Subscribers
mizhka

Details

Reviewers
mizhka
adrian
Commits
rS308744: [etherswitch] add infineon adm6996fc support on etherswitch
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 updated this revision to Diff 22137.Nov 11 2016, 12:59 AM
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)
mizhka added a subscriber: mizhka.Nov 11 2016, 5:20 AM
  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 ;)

yamori813_yahoo.co.jp updated this revision to Diff 22191.Nov 14 2016, 4:07 AM
  • review refict
yamori813_yahoo.co.jp updated this revision to Diff 22192.Nov 14 2016, 5:47 AM
  • add empty line
yamori813_yahoo.co.jp added a comment.Nov 14 2016, 5:59 AM
  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 accepted this revision.Nov 16 2016, 11:43 PM
adrian added a reviewer: adrian.
This revision is now accepted and ready to land.Nov 16 2016, 11:43 PM
mizhka accepted this revision.Nov 17 2016, 7:31 AM
mizhka added a reviewer: mizhka.
Closed by commit rS308744: [etherswitch] add infineon adm6996fc support on etherswitch (authored by mizhka). · Explain WhyNov 17 2016, 7:34 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
dev/
etherswitch/
infineon/
858 lines

Diff 22274

View Options

head/sys/dev/etherswitch/infineon/adm6996fc.c

Loading...