On 07/30/2015 08:45 AM, Todd Sperry wrote:
This patch contains extensions for the NCT6683 hwmon driver for Ethernet Bypass, sometimes referred
to as Fail To Wire (FTW). FTW is a new feature on SuperMicro motherboards supported by the Nuvoton
NCT6683D SuperIO chip. The NCT6683D supports controlling a set of motherboard relays that open or
close pairs of Ethernet ports. The extensions found here have been shipping in productions systems for
at least the last 18 months.
Signed-off-by: Todd Sperry <todd.sperry@xxxxxxxxxxx <mailto:todd.sperry@xxxxxxxxxxx>>
Todd,
I don't necessarily object to moving the driver into its own subdirectory. If you do so,
please to it in two steps, one to move the driver and one with your additions.
That is a minor problem, however. Checkpatch reports
total: 405 errors, 78 warnings, 3175 lines checked
NOTE: Whitespace errors detected.
You may wish to use scripts/cleanpatch or scripts/cleanfile
checkpatch --strict reports another 56 check messages.
Please fix this up before resubmitting. Documentation/hwmon/submitting-patches
might be quite useful to provide guidelines.
Note that it may make more sense to separate the guide-to-wire code into a completely
separate driver (possibly by converting the driver into an mfd driver, with hwmon
and guide-to-wire slave drivers). That is also secondary, though; first your code
will have to be in a shape that lets us review it without getting stuck with coding
stye problems.
Thanks,
Guenter
_______________________________________________
lm-sensors mailing list
lm-sensors@xxxxxxxxxxxxxx
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
