> From: sven falempin <sven.falem...@gmail.com>
> Date: Mon, 5 Jan 2015 13:29:41 -0500
> 
> On Thu, Dec 11, 2014 at 7:47 AM, Stuart Henderson <st...@openbsd.org> wrote:
> >
> > On 2014/12/11 07:41, sven falempin wrote:
> > > Thank you all,
> > >
> > > I grep(ed) -r NCT6 in sys and didn't find wbsio, I guess those
> > > christmass holydays will be much welcome !
> > >
> > > Does the wbsio detect the watchdog in the apu card ?
> >
> > No, as the manual says, "Only the hardware monitoring function is
> > currently supported", and at present, and this driver doesn't attach
> > on the APU.
> 
> 
> 
> First of all , Happy new gregorian Year 2015 !
> I hope you all survived the longest night of the year :-)
> 
> Jan  5 12:36:56 unicornD /bsd: HOIHOIHOI wbsio_probe: id 0xc4
> Jan  5 12:36:56 unicornD /bsd: >>> probe for wbsio* clone into wbsio0
> Jan  5 12:36:56 unicornD /bsd: wbsio0 at isa0 port 0x2e/2 rev 0x52
> Jan  5 12:36:56 unicornD /bsd: >>> probing for lm*
> Jan  5 12:36:56 unicornD /bsd: >>> lm probe returned 0
> 
> Nothing really interesting so far , but I can read the revision number .
> 
> What's next ?

Hmm, chris@ just added the NCT5104D chip, which happens to have the
same ID :(.  That will conflict with this diff.

Seems the NCT5104D is basically a stripped down NCT6106D that has just
the watchdog timer, 4 UARTs and GPIO.  Perhaps some sort of probe is
needed to see if a logical device is actually there.  I guess we
already have one in place for the hardware monitoring function since
we check the I/O address before attaching lm(4).

Perhaps a next step is to add support for this chip to lm(4).

> Index: isa/wbsio.c
> ===================================================================
> RCS file: /cvs/src/sys/dev/isa/wbsio.c,v
> retrieving revision 1.8
> diff -u -p -r1.8 wbsio.c
> --- isa/wbsio.c 1 Jul 2012 02:15:09 -0000       1.8
> +++ isa/wbsio.c 5 Jan 2015 17:53:00 -0000
> @@ -112,6 +112,7 @@ wbsio_probe(struct device *parent, void
>         case WBSIO_ID_W83637HF:
>         case WBSIO_ID_W83697HF:
>         case WBSIO_ID_NCT6776F:
> +       case WBSIO_ID_NCT6106D:
>                 ia->ipa_nio = 1;
>                 ia->ipa_io[0].length = WBSIO_IOSIZE;
>                 ia->ipa_nmem = 0;
> @@ -172,6 +173,9 @@ wbsio_attach(struct device *parent, stru
>                 break;
>         case WBSIO_ID_NCT6776F:
>                 printf(": NCT6776F");
> +               break;
> +       case WBSIO_ID_NCT6106D:
> +               printf(": NCT6106D");
>                 break;
>         }
> Index: isa/wbsioreg.h
> ===================================================================
> RCS file: /cvs/src/sys/dev/isa/wbsioreg.h,v
> retrieving revision 1.3
> diff -u -p -r1.3 wbsioreg.h
> --- isa/wbsioreg.h      1 Jul 2012 02:15:09 -0000       1.3
> +++ isa/wbsioreg.h      5 Jan 2015 17:53:00 -0000
> @@ -43,6 +43,7 @@
>  #define WBSIO_ID_W83637HF      0x70
>  #define WBSIO_ID_W83697HF      0x60
>  #define WBSIO_ID_NCT6776F      0xc3
> +#define WBSIO_ID_NCT6106D      0xc4
>  /* Logical Device Number (LDN) Assignments */
>  #define WBSIO_LDN_HM           0x0b
> -- 
> ---------------------------------------------------------------------------------------------------------------------
> () ascii ribbon campaign - against html e-mail
> /\
> 
> 

Reply via email to