Re: adm1026hm

[Julian Coleman <jdc%coris.org.uk@localhost>]

Hi Martin, Frank,

>                fan 0:      5578                                       RPM
>                fan 1:      5818                                       RPM
>                fan 2:      5818                                       RPM
>                fan 3:         0                                       RPM
>                fan 4:     32926                                       RPM
>                fan 5:     34615                                       RPM
>                fan 6:     32926                                       RPM
>                fan 7:     34615                                       RPM
> 
>                 V-12:    -3.375                                         V
> 
> ... the values for fan 3 and V-12 seem odd.
> Should I be concerned?

For me, the -12V rail showed -12.098.  Complete output is:

  https://ftp.netbsd.org/pub/NetBSD/misc/jdc/sparc64/v240.envstat

but I don't know if you should be concerned by this or not.  Neither of the
+12v nor the -12v inputs are monitored by the ALOM, so that wouldn't alert
on this.  For my output, I first ran `envstat -c <file>` with this config:

  https://ftp.netbsd.org/pub/NetBSD/misc/jdc/sparc64/v240-envsys.conf

However, that expects only 3 chassis fans ...

> > It think the V-12 thing is expected, but the fan is strange.
> 
> Oh wait, that is not one of the CPU fans but a chasis one, and I think the
> V240 only has 3 fans there, not 4. Julian will know for sure.

... Yes - the V240 has 3 fans, compared to 4 in the V210, so the output for
fan 3 here is (sort of) expected.  However, I'm confused as to why it appears
at all and also that the CPU fan speeds are doubled.  Looking at the code
in -current:

  https://nxr.netbsd.org/xref/src/sys/dev/i2c/adm1026.c#270
  https://nxr.netbsd.org/xref/src/sys/dev/i2c/adm1026.c#482

we can override the chip configuration to set which fans are present, and
we do this for the V240:

  https://nxr.netbsd.org/xref/src/sys/arch/sparc64/sparc64/ofw_patch.c#639

and also set the correct fan speed divisor there.  The patch routine is
called for each i2c device from:

  https://nxr.netbsd.org/xref/src/sys/arch/sparc64/sparc64/autoconf.c#1202

There's not much difference between the code in current and 11, because
changes to the pcfiic and adm1026 drivers were all pulled up, along with
OFW patch changes.

I know that set_i2c_bus_props() is called and the machine model is
SUNW,Sun-Fire-V240, because without those we won't have the adm1026hm nor
the lmtemp devices.  However, I'm puzzled why either we don't call
set_i2c_dev_props() or we don't match on pcfiic or on SUNW,Sun-Fire-V240.

I'm away from my machines for a couple of weeks, so I can look when I get
back.  If you're inclined, a debug printf in set_i2c_dev_props() to check
that it's actually called might show the problem.

Regards,

Julian

--