[lm-sensors] Address configuration for scx200_acb

[jim.cromie@gmail.com (Jim Cromie)]

Thomas Andrews wrote:
> On Thu, Apr 27, 2006 at 11:41:14PM +0200, Jean Delvare wrote:
>
>   
>> Can you please comment on this?
>> http://bugzilla.kernel.org/show_bug.cgi?idd45#c15
>>
>> This is a propsal to make the scx200_acb driver's default base
>> addresses configurable through Kconfig. What do you think?
>>
>> Another possibility would be to detect them at runtime depending on
>> some platform data. Sounds more complex, if possible at all.
>>
>> Or we can stick to the current state (default addresses, can be
>> overriden with module parameters).
>>
>> What makes the current default addresses (0x820 and 0x840) more
>> legitimate than what Alexander has (0x810 and 0x820)?
>>     
>
> I've been wondering about this myself. I also use the wrap boards, and
> I've found that the address can be automatically set using something
> like this:
>
> static void detect_acb_base_addresses(int *acb1, int* acb2)
> {
>     unsigned char h1 = 0;
>     unsigned char l1 = 0;
>     unsigned char h2 = 0;
>     unsigned char l2 = 0;
>
>     /* Select ACB1  (LDN 5) */
>     outb(0x07,CFG_INDEX);
>     outb(0x05,CFG_DATA);
>     outb(0x60,CFG_INDEX);
>     h1 = inb(CFG_DATA);
>     outb(0x61,CFG_INDEX);
>     l1 = inb(CFG_DATA);
>
>     /* ACB2 (LDN 6) */
>     outb(0x07,CFG_INDEX);
>     outb(0x06,CFG_DATA);
>     outb(0x60,CFG_INDEX);
>     h2 = inb(CFG_DATA);
>     outb(0x61,CFG_INDEX);
>     l2 = inb(CFG_DATA);
>     *acb1 = (h1 << 8) | l1;
>     *acb2 = (h2 << 8) | l2;    
>     DEBUG(1,printk("ACB1 at %x, ACB2 at %x",*acb1,*acb2));
> }
>
> Would this not work for the whole family ?
>
> Regards,
> Thomas
>   

I have a soekris net4801 which has the same sc-1100.
On my board, ISA addys 2e, 2f talk to a separate Super-IO chip, pc87360.
which IIRC has an acb unit also.
Back when I got the board, the presence of 2 superio devices was somewhat
confusing, until Id read both datasheets - 2x.

FWIW, the pc87360 has these (from a pre-proto-patch I still have)

+char* funcblocks[] = {
+       "Floppy Disk Controller (FDC)",
+       "Parallel Port (PP)",
+       "Serial Port 2 with IR (SP2)",
+       "Serial Port 1 (SP1)",
+       "System Wake-Up Control (SWC)",
+       "Keyboard and Mouse Controller (KBC) - Mouse interface",
+       "Keyboard and Mouse Controller (KBC) - Keyboard interface",
+       "General-Purpose I/O (GPIO) Ports",
+       "ACCESS.bus Interface (ACB)",
+       "Fan Speed Control and Monitor (FSCM)",
+       "WATCHDOG Timer (WDT)",
+       "Game Port (GMP)",
+       "Musical Instrument Digital Interface (MIDI) Port",
+       "Voltage Level Monitor (VLM)",
+       "Temperature Sensor (TMS)",
+       NULL
+};
+
+
+struct pc87366_unit pc87366_units[] = {
+       { 8,    "Floppy Disk Controller (FDC)" },
+       { 0x10, "Parallel Port (PP)" },
+       { 8, "Serial Port 2 with IR (SP2)" },
+       { 8, "Serial Port 1 (SP1)" },
+       { 0x10, "System Wake-Up Control (SWC)" },
+       { 0x10, "Keyboard and Mouse Controller (KBC) - Mouse interface" },
+       { 0x10, "Keyboard and Mouse Controller (KBC) - Keyboard 
interface" },
+       { 0x0c, "General-Purpose I/O (GPIO) Ports" },
+       { 6,    "ACCESS.bus Interface (ACB)" },
+       { 0,    "Fan Speed Control and Monitor (FSCM)" },
+       { 4,    "WATCHDOG Timer (WDT)" },
+       { 0x0d, "Game Port (GMP)" },
+       { 3,    "Musical Instrument Digital Interface (MIDI) Port" },
+       { 0x10, "Voltage Level Monitor (VLM)" },
+       { 0x0f, "Temperature Sensor (TMS)" },
+       { 0,    NULL }
+};


the sc-1100 has fewer functional units; I dimly recall poking at them 
this same way,
but I cant dig up any old bones.