27c425ac70aa8ac4f835d633b8c63aa7ab53640b
[safe/jmp/linux-2.6] / Documentation / i2c / writing-clients
1 This is a small guide for those who want to write kernel drivers for I2C
2 or SMBus devices.
3
4 To set up a driver, you need to do several things. Some are optional, and
5 some things can be done slightly or completely different. Use this as a
6 guide, not as a rule book!
7
8
9 General remarks
10 ===============
11
12 Try to keep the kernel namespace as clean as possible. The best way to
13 do this is to use a unique prefix for all global symbols. This is 
14 especially important for exported symbols, but it is a good idea to do
15 it for non-exported symbols too. We will use the prefix `foo_' in this
16 tutorial, and `FOO_' for preprocessor variables.
17
18
19 The driver structure
20 ====================
21
22 Usually, you will implement a single driver structure, and instantiate
23 all clients from it. Remember, a driver structure contains general access 
24 routines, a client structure specific information like the actual I2C
25 address.
26
27 static struct i2c_driver foo_driver = {
28         .driver = {
29                 .name   = "foo",
30         },
31         .attach_adapter = &foo_attach_adapter,
32         .detach_client  = &foo_detach_client,
33         .command        = &foo_command /* may be NULL */
34 }
35  
36 The name field must match the driver name, including the case. It must not
37 contain spaces, and may be up to 31 characters long.
38
39 All other fields are for call-back functions which will be explained 
40 below.
41
42
43 Extra client data
44 =================
45
46 The client structure has a special `data' field that can point to any
47 structure at all. You can use this to keep client-specific data. You
48 do not always need this, but especially for `sensors' drivers, it can
49 be very useful.
50
51 An example structure is below.
52
53   struct foo_data {
54     struct i2c_client client;
55     struct semaphore lock; /* For ISA access in `sensors' drivers. */
56     int sysctl_id;         /* To keep the /proc directory entry for 
57                               `sensors' drivers. */
58     enum chips type;       /* To keep the chips type for `sensors' drivers. */
59    
60     /* Because the i2c bus is slow, it is often useful to cache the read
61        information of a chip for some time (for example, 1 or 2 seconds).
62        It depends of course on the device whether this is really worthwhile
63        or even sensible. */
64     struct semaphore update_lock; /* When we are reading lots of information,
65                                      another process should not update the
66                                      below information */
67     char valid;                   /* != 0 if the following fields are valid. */
68     unsigned long last_updated;   /* In jiffies */
69     /* Add the read information here too */
70   };
71
72
73 Accessing the client
74 ====================
75
76 Let's say we have a valid client structure. At some time, we will need
77 to gather information from the client, or write new information to the
78 client. How we will export this information to user-space is less 
79 important at this moment (perhaps we do not need to do this at all for
80 some obscure clients). But we need generic reading and writing routines.
81
82 I have found it useful to define foo_read and foo_write function for this.
83 For some cases, it will be easier to call the i2c functions directly,
84 but many chips have some kind of register-value idea that can easily
85 be encapsulated. Also, some chips have both ISA and I2C interfaces, and
86 it useful to abstract from this (only for `sensors' drivers).
87
88 The below functions are simple examples, and should not be copied
89 literally.
90
91   int foo_read_value(struct i2c_client *client, u8 reg)
92   {
93     if (reg < 0x10) /* byte-sized register */
94       return i2c_smbus_read_byte_data(client,reg);
95     else /* word-sized register */
96       return i2c_smbus_read_word_data(client,reg);
97   }
98
99   int foo_write_value(struct i2c_client *client, u8 reg, u16 value)
100   {
101     if (reg == 0x10) /* Impossible to write - driver error! */ {
102       return -1;
103     else if (reg < 0x10) /* byte-sized register */
104       return i2c_smbus_write_byte_data(client,reg,value);
105     else /* word-sized register */
106       return i2c_smbus_write_word_data(client,reg,value);
107   }
108
109 For sensors code, you may have to cope with ISA registers too. Something
110 like the below often works. Note the locking! 
111
112   int foo_read_value(struct i2c_client *client, u8 reg)
113   {
114     int res;
115     if (i2c_is_isa_client(client)) {
116       down(&(((struct foo_data *) (client->data)) -> lock));
117       outb_p(reg,client->addr + FOO_ADDR_REG_OFFSET);
118       res = inb_p(client->addr + FOO_DATA_REG_OFFSET);
119       up(&(((struct foo_data *) (client->data)) -> lock));
120       return res;
121     } else
122       return i2c_smbus_read_byte_data(client,reg);
123   }
124
125 Writing is done the same way.
126
127
128 Probing and attaching
129 =====================
130
131 Most i2c devices can be present on several i2c addresses; for some this
132 is determined in hardware (by soldering some chip pins to Vcc or Ground),
133 for others this can be changed in software (by writing to specific client
134 registers). Some devices are usually on a specific address, but not always;
135 and some are even more tricky. So you will probably need to scan several
136 i2c addresses for your clients, and do some sort of detection to see
137 whether it is actually a device supported by your driver.
138
139 To give the user a maximum of possibilities, some default module parameters
140 are defined to help determine what addresses are scanned. Several macros
141 are defined in i2c.h to help you support them, as well as a generic
142 detection algorithm.
143
144 You do not have to use this parameter interface; but don't try to use
145 function i2c_probe() if you don't.
146
147 NOTE: If you want to write a `sensors' driver, the interface is slightly
148       different! See below.
149
150
151
152 Probing classes
153 ---------------
154
155 All parameters are given as lists of unsigned 16-bit integers. Lists are
156 terminated by I2C_CLIENT_END.
157 The following lists are used internally:
158
159   normal_i2c: filled in by the module writer. 
160      A list of I2C addresses which should normally be examined.
161    probe: insmod parameter. 
162      A list of pairs. The first value is a bus number (-1 for any I2C bus), 
163      the second is the address. These addresses are also probed, as if they 
164      were in the 'normal' list.
165    ignore: insmod parameter.
166      A list of pairs. The first value is a bus number (-1 for any I2C bus), 
167      the second is the I2C address. These addresses are never probed. 
168      This parameter overrules the 'normal_i2c' list only.
169    force: insmod parameter. 
170      A list of pairs. The first value is a bus number (-1 for any I2C bus),
171      the second is the I2C address. A device is blindly assumed to be on
172      the given address, no probing is done. 
173
174 Additionally, kind-specific force lists may optionally be defined if
175 the driver supports several chip kinds. They are grouped in a
176 NULL-terminated list of pointers named forces, those first element if the
177 generic force list mentioned above. Each additional list correspond to an
178 insmod parameter of the form force_<kind>.
179
180 Fortunately, as a module writer, you just have to define the `normal_i2c' 
181 parameter. The complete declaration could look like this:
182
183   /* Scan 0x37, and 0x48 to 0x4f */
184   static unsigned short normal_i2c[] = { 0x37, 0x48, 0x49, 0x4a, 0x4b, 0x4c,
185                                          0x4d, 0x4e, 0x4f, I2C_CLIENT_END };
186
187   /* Magic definition of all other variables and things */
188   I2C_CLIENT_INSMOD;
189   /* Or, if your driver supports, say, 2 kind of devices: */
190   I2C_CLIENT_INSMOD_2(foo, bar);
191
192 If you use the multi-kind form, an enum will be defined for you:
193   enum chips { any_chip, foo, bar, ... }
194 You can then (and certainly should) use it in the driver code.
195
196 Note that you *have* to call the defined variable `normal_i2c',
197 without any prefix!
198
199
200 Attaching to an adapter
201 -----------------------
202
203 Whenever a new adapter is inserted, or for all adapters if the driver is
204 being registered, the callback attach_adapter() is called. Now is the
205 time to determine what devices are present on the adapter, and to register
206 a client for each of them.
207
208 The attach_adapter callback is really easy: we just call the generic
209 detection function. This function will scan the bus for us, using the
210 information as defined in the lists explained above. If a device is
211 detected at a specific address, another callback is called.
212
213   int foo_attach_adapter(struct i2c_adapter *adapter)
214   {
215     return i2c_probe(adapter,&addr_data,&foo_detect_client);
216   }
217
218 Remember, structure `addr_data' is defined by the macros explained above,
219 so you do not have to define it yourself.
220
221 The i2c_probe function will call the foo_detect_client
222 function only for those i2c addresses that actually have a device on
223 them (unless a `force' parameter was used). In addition, addresses that
224 are already in use (by some other registered client) are skipped.
225
226
227 The detect client function
228 --------------------------
229
230 The detect client function is called by i2c_probe. The `kind' parameter
231 contains -1 for a probed detection, 0 for a forced detection, or a positive
232 number for a forced detection with a chip type forced.
233
234 Below, some things are only needed if this is a `sensors' driver. Those
235 parts are between /* SENSORS ONLY START */ and /* SENSORS ONLY END */
236 markers. 
237
238 Returning an error different from -ENODEV in a detect function will cause
239 the detection to stop: other addresses and adapters won't be scanned.
240 This should only be done on fatal or internal errors, such as a memory
241 shortage or i2c_attach_client failing.
242
243 For now, you can ignore the `flags' parameter. It is there for future use.
244
245   int foo_detect_client(struct i2c_adapter *adapter, int address, 
246                         unsigned short flags, int kind)
247   {
248     int err = 0;
249     int i;
250     struct i2c_client *new_client;
251     struct foo_data *data;
252     const char *client_name = ""; /* For non-`sensors' drivers, put the real
253                                      name here! */
254    
255     /* Let's see whether this adapter can support what we need.
256        Please substitute the things you need here! 
257        For `sensors' drivers, add `! is_isa &&' to the if statement */
258     if (!i2c_check_functionality(adapter,I2C_FUNC_SMBUS_WORD_DATA |
259                                         I2C_FUNC_SMBUS_WRITE_BYTE))
260        goto ERROR0;
261
262     /* SENSORS ONLY START */
263     const char *type_name = "";
264     int is_isa = i2c_is_isa_adapter(adapter);
265
266     /* Do this only if the chip can additionally be found on the ISA bus
267        (hybrid chip). */
268
269     if (is_isa) {
270
271       /* Discard immediately if this ISA range is already used */
272       /* FIXME: never use check_region(), only request_region() */
273       if (check_region(address,FOO_EXTENT))
274         goto ERROR0;
275
276       /* Probe whether there is anything on this address.
277          Some example code is below, but you will have to adapt this
278          for your own driver */
279
280       if (kind < 0) /* Only if no force parameter was used */ {
281         /* We may need long timeouts at least for some chips. */
282         #define REALLY_SLOW_IO
283         i = inb_p(address + 1);
284         if (inb_p(address + 2) != i)
285           goto ERROR0;
286         if (inb_p(address + 3) != i)
287           goto ERROR0;
288         if (inb_p(address + 7) != i)
289           goto ERROR0;
290         #undef REALLY_SLOW_IO
291
292         /* Let's just hope nothing breaks here */
293         i = inb_p(address + 5) & 0x7f;
294         outb_p(~i & 0x7f,address+5);
295         if ((inb_p(address + 5) & 0x7f) != (~i & 0x7f)) {
296           outb_p(i,address+5);
297           return 0;
298         }
299       }
300     }
301
302     /* SENSORS ONLY END */
303
304     /* OK. For now, we presume we have a valid client. We now create the
305        client structure, even though we cannot fill it completely yet.
306        But it allows us to access several i2c functions safely */
307     
308     if (!(data = kzalloc(sizeof(struct foo_data), GFP_KERNEL))) {
309       err = -ENOMEM;
310       goto ERROR0;
311     }
312
313     new_client = &data->client;
314     i2c_set_clientdata(new_client, data);
315
316     new_client->addr = address;
317     new_client->adapter = adapter;
318     new_client->driver = &foo_driver;
319     new_client->flags = 0;
320
321     /* Now, we do the remaining detection. If no `force' parameter is used. */
322
323     /* First, the generic detection (if any), that is skipped if any force
324        parameter was used. */
325     if (kind < 0) {
326       /* The below is of course bogus */
327       if (foo_read(new_client,FOO_REG_GENERIC) != FOO_GENERIC_VALUE)
328          goto ERROR1;
329     }
330
331     /* SENSORS ONLY START */
332
333     /* Next, specific detection. This is especially important for `sensors'
334        devices. */
335
336     /* Determine the chip type. Not needed if a `force_CHIPTYPE' parameter
337        was used. */
338     if (kind <= 0) {
339       i = foo_read(new_client,FOO_REG_CHIPTYPE);
340       if (i == FOO_TYPE_1) 
341         kind = chip1; /* As defined in the enum */
342       else if (i == FOO_TYPE_2)
343         kind = chip2;
344       else {
345         printk("foo: Ignoring 'force' parameter for unknown chip at "
346                "adapter %d, address 0x%02x\n",i2c_adapter_id(adapter),address);
347         goto ERROR1;
348       }
349     }
350
351     /* Now set the type and chip names */
352     if (kind == chip1) {
353       type_name = "chip1"; /* For /proc entry */
354       client_name = "CHIP 1";
355     } else if (kind == chip2) {
356       type_name = "chip2"; /* For /proc entry */
357       client_name = "CHIP 2";
358     }
359    
360     /* Reserve the ISA region */
361     if (is_isa)
362       request_region(address,FOO_EXTENT,type_name);
363
364     /* SENSORS ONLY END */
365
366     /* Fill in the remaining client fields. */
367     strcpy(new_client->name,client_name);
368
369     /* SENSORS ONLY BEGIN */
370     data->type = kind;
371     /* SENSORS ONLY END */
372
373     data->valid = 0; /* Only if you use this field */
374     init_MUTEX(&data->update_lock); /* Only if you use this field */
375
376     /* Any other initializations in data must be done here too. */
377
378     /* Tell the i2c layer a new client has arrived */
379     if ((err = i2c_attach_client(new_client)))
380       goto ERROR3;
381
382     /* SENSORS ONLY BEGIN */
383     /* Register a new directory entry with module sensors. See below for
384        the `template' structure. */
385     if ((i = i2c_register_entry(new_client, type_name,
386                                     foo_dir_table_template,THIS_MODULE)) < 0) {
387       err = i;
388       goto ERROR4;
389     }
390     data->sysctl_id = i;
391
392     /* SENSORS ONLY END */
393
394     /* This function can write default values to the client registers, if
395        needed. */
396     foo_init_client(new_client);
397     return 0;
398
399     /* OK, this is not exactly good programming practice, usually. But it is
400        very code-efficient in this case. */
401
402     ERROR4:
403       i2c_detach_client(new_client);
404     ERROR3:
405     ERROR2:
406     /* SENSORS ONLY START */
407       if (is_isa)
408         release_region(address,FOO_EXTENT);
409     /* SENSORS ONLY END */
410     ERROR1:
411       kfree(data);
412     ERROR0:
413       return err;
414   }
415
416
417 Removing the client
418 ===================
419
420 The detach_client call back function is called when a client should be
421 removed. It may actually fail, but only when panicking. This code is
422 much simpler than the attachment code, fortunately!
423
424   int foo_detach_client(struct i2c_client *client)
425   {
426     int err,i;
427
428     /* SENSORS ONLY START */
429     /* Deregister with the `i2c-proc' module. */
430     i2c_deregister_entry(((struct lm78_data *)(client->data))->sysctl_id);
431     /* SENSORS ONLY END */
432
433     /* Try to detach the client from i2c space */
434     if ((err = i2c_detach_client(client)))
435       return err;
436
437     /* HYBRID SENSORS CHIP ONLY START */
438     if i2c_is_isa_client(client)
439       release_region(client->addr,LM78_EXTENT);
440     /* HYBRID SENSORS CHIP ONLY END */
441
442     kfree(i2c_get_clientdata(client));
443     return 0;
444   }
445
446
447 Initializing the module or kernel
448 =================================
449
450 When the kernel is booted, or when your foo driver module is inserted, 
451 you have to do some initializing. Fortunately, just attaching (registering)
452 the driver module is usually enough.
453
454   /* Keep track of how far we got in the initialization process. If several
455      things have to initialized, and we fail halfway, only those things
456      have to be cleaned up! */
457   static int __initdata foo_initialized = 0;
458
459   static int __init foo_init(void)
460   {
461     int res;
462     printk("foo version %s (%s)\n",FOO_VERSION,FOO_DATE);
463     
464     if ((res = i2c_add_driver(&foo_driver))) {
465       printk("foo: Driver registration failed, module not inserted.\n");
466       foo_cleanup();
467       return res;
468     }
469     foo_initialized ++;
470     return 0;
471   }
472
473   void foo_cleanup(void)
474   {
475     if (foo_initialized == 1) {
476       if ((res = i2c_del_driver(&foo_driver))) {
477         printk("foo: Driver registration failed, module not removed.\n");
478         return;
479       }
480       foo_initialized --;
481     }
482   }
483
484   /* Substitute your own name and email address */
485   MODULE_AUTHOR("Frodo Looijaard <frodol@dds.nl>"
486   MODULE_DESCRIPTION("Driver for Barf Inc. Foo I2C devices");
487
488   module_init(foo_init);
489   module_exit(foo_cleanup);
490
491 Note that some functions are marked by `__init', and some data structures
492 by `__init_data'.  Hose functions and structures can be removed after
493 kernel booting (or module loading) is completed.
494
495 Command function
496 ================
497
498 A generic ioctl-like function call back is supported. You will seldom
499 need this. You may even set it to NULL.
500
501   /* No commands defined */
502   int foo_command(struct i2c_client *client, unsigned int cmd, void *arg)
503   {
504     return 0;
505   }
506
507
508 Sending and receiving
509 =====================
510
511 If you want to communicate with your device, there are several functions
512 to do this. You can find all of them in i2c.h.
513
514 If you can choose between plain i2c communication and SMBus level
515 communication, please use the last. All adapters understand SMBus level
516 commands, but only some of them understand plain i2c!
517
518
519 Plain i2c communication
520 -----------------------
521
522   extern int i2c_master_send(struct i2c_client *,const char* ,int);
523   extern int i2c_master_recv(struct i2c_client *,char* ,int);
524
525 These routines read and write some bytes from/to a client. The client
526 contains the i2c address, so you do not have to include it. The second
527 parameter contains the bytes the read/write, the third the length of the
528 buffer. Returned is the actual number of bytes read/written.
529   
530   extern int i2c_transfer(struct i2c_adapter *adap, struct i2c_msg *msg,
531                           int num);
532
533 This sends a series of messages. Each message can be a read or write,
534 and they can be mixed in any way. The transactions are combined: no
535 stop bit is sent between transaction. The i2c_msg structure contains
536 for each message the client address, the number of bytes of the message
537 and the message data itself.
538
539 You can read the file `i2c-protocol' for more information about the
540 actual i2c protocol.
541
542
543 SMBus communication
544 -------------------
545
546   extern s32 i2c_smbus_xfer (struct i2c_adapter * adapter, u16 addr, 
547                              unsigned short flags,
548                              char read_write, u8 command, int size,
549                              union i2c_smbus_data * data);
550
551   This is the generic SMBus function. All functions below are implemented
552   in terms of it. Never use this function directly!
553
554
555   extern s32 i2c_smbus_write_quick(struct i2c_client * client, u8 value);
556   extern s32 i2c_smbus_read_byte(struct i2c_client * client);
557   extern s32 i2c_smbus_write_byte(struct i2c_client * client, u8 value);
558   extern s32 i2c_smbus_read_byte_data(struct i2c_client * client, u8 command);
559   extern s32 i2c_smbus_write_byte_data(struct i2c_client * client,
560                                        u8 command, u8 value);
561   extern s32 i2c_smbus_read_word_data(struct i2c_client * client, u8 command);
562   extern s32 i2c_smbus_write_word_data(struct i2c_client * client,
563                                        u8 command, u16 value);
564   extern s32 i2c_smbus_write_block_data(struct i2c_client * client,
565                                         u8 command, u8 length,
566                                         u8 *values);
567   extern s32 i2c_smbus_read_i2c_block_data(struct i2c_client * client,
568                                            u8 command, u8 *values);
569
570 These ones were removed in Linux 2.6.10 because they had no users, but could
571 be added back later if needed:
572
573   extern s32 i2c_smbus_read_block_data(struct i2c_client * client,
574                                        u8 command, u8 *values);
575   extern s32 i2c_smbus_write_i2c_block_data(struct i2c_client * client,
576                                             u8 command, u8 length,
577                                             u8 *values);
578   extern s32 i2c_smbus_process_call(struct i2c_client * client,
579                                     u8 command, u16 value);
580   extern s32 i2c_smbus_block_process_call(struct i2c_client *client,
581                                           u8 command, u8 length,
582                                           u8 *values)
583
584 All these transactions return -1 on failure. The 'write' transactions 
585 return 0 on success; the 'read' transactions return the read value, except 
586 for read_block, which returns the number of values read. The block buffers 
587 need not be longer than 32 bytes.
588
589 You can read the file `smbus-protocol' for more information about the
590 actual SMBus protocol.
591
592
593 General purpose routines
594 ========================
595
596 Below all general purpose routines are listed, that were not mentioned
597 before.
598
599   /* This call returns a unique low identifier for each registered adapter,
600    * or -1 if the adapter was not registered.
601    */
602   extern int i2c_adapter_id(struct i2c_adapter *adap);
603
604
605 The sensors sysctl/proc interface
606 =================================
607
608 This section only applies if you write `sensors' drivers.
609
610 Each sensors driver creates a directory in /proc/sys/dev/sensors for each
611 registered client. The directory is called something like foo-i2c-4-65.
612 The sensors module helps you to do this as easily as possible.
613
614 The template
615 ------------
616
617 You will need to define a ctl_table template. This template will automatically
618 be copied to a newly allocated structure and filled in where necessary when
619 you call sensors_register_entry.
620
621 First, I will give an example definition.
622   static ctl_table foo_dir_table_template[] = {
623     { FOO_SYSCTL_FUNC1, "func1", NULL, 0, 0644, NULL, &i2c_proc_real,
624       &i2c_sysctl_real,NULL,&foo_func },
625     { FOO_SYSCTL_FUNC2, "func2", NULL, 0, 0644, NULL, &i2c_proc_real,
626       &i2c_sysctl_real,NULL,&foo_func },
627     { FOO_SYSCTL_DATA, "data", NULL, 0, 0644, NULL, &i2c_proc_real,
628       &i2c_sysctl_real,NULL,&foo_data },
629     { 0 }
630   };
631
632 In the above example, three entries are defined. They can either be
633 accessed through the /proc interface, in the /proc/sys/dev/sensors/*
634 directories, as files named func1, func2 and data, or alternatively 
635 through the sysctl interface, in the appropriate table, with identifiers
636 FOO_SYSCTL_FUNC1, FOO_SYSCTL_FUNC2 and FOO_SYSCTL_DATA.
637
638 The third, sixth and ninth parameters should always be NULL, and the
639 fourth should always be 0. The fifth is the mode of the /proc file;
640 0644 is safe, as the file will be owned by root:root. 
641
642 The seventh and eighth parameters should be &i2c_proc_real and
643 &i2c_sysctl_real if you want to export lists of reals (scaled
644 integers). You can also use your own function for them, as usual.
645 Finally, the last parameter is the call-back to gather the data
646 (see below) if you use the *_proc_real functions. 
647
648
649 Gathering the data
650 ------------------
651
652 The call back functions (foo_func and foo_data in the above example)
653 can be called in several ways; the operation parameter determines
654 what should be done:
655
656   * If operation == SENSORS_PROC_REAL_INFO, you must return the
657     magnitude (scaling) in nrels_mag;
658   * If operation == SENSORS_PROC_REAL_READ, you must read information
659     from the chip and return it in results. The number of integers
660     to display should be put in nrels_mag;
661   * If operation == SENSORS_PROC_REAL_WRITE, you must write the
662     supplied information to the chip. nrels_mag will contain the number
663     of integers, results the integers themselves.
664
665 The *_proc_real functions will display the elements as reals for the
666 /proc interface. If you set the magnitude to 2, and supply 345 for
667 SENSORS_PROC_REAL_READ, it would display 3.45; and if the user would
668 write 45.6 to the /proc file, it would be returned as 4560 for
669 SENSORS_PROC_REAL_WRITE. A magnitude may even be negative!
670
671 An example function:
672
673   /* FOO_FROM_REG and FOO_TO_REG translate between scaled values and
674      register values. Note the use of the read cache. */
675   void foo_in(struct i2c_client *client, int operation, int ctl_name, 
676               int *nrels_mag, long *results)
677   {
678     struct foo_data *data = client->data;
679     int nr = ctl_name - FOO_SYSCTL_FUNC1; /* reduce to 0 upwards */
680     
681     if (operation == SENSORS_PROC_REAL_INFO)
682       *nrels_mag = 2;
683     else if (operation == SENSORS_PROC_REAL_READ) {
684       /* Update the readings cache (if necessary) */
685       foo_update_client(client);
686       /* Get the readings from the cache */
687       results[0] = FOO_FROM_REG(data->foo_func_base[nr]);
688       results[1] = FOO_FROM_REG(data->foo_func_more[nr]);
689       results[2] = FOO_FROM_REG(data->foo_func_readonly[nr]);
690       *nrels_mag = 2;
691     } else if (operation == SENSORS_PROC_REAL_WRITE) {
692       if (*nrels_mag >= 1) {
693         /* Update the cache */
694         data->foo_base[nr] = FOO_TO_REG(results[0]);
695         /* Update the chip */
696         foo_write_value(client,FOO_REG_FUNC_BASE(nr),data->foo_base[nr]);
697       }
698       if (*nrels_mag >= 2) {
699         /* Update the cache */
700         data->foo_more[nr] = FOO_TO_REG(results[1]);
701         /* Update the chip */
702         foo_write_value(client,FOO_REG_FUNC_MORE(nr),data->foo_more[nr]);
703       }
704     }
705   }