[sane-devel] Lexmark backend / Dell A920

Fred Odendaal freshshelf at rogers.com
Fri Jan 27 02:01:35 UTC 2006


See below for my response.

Stéphane VOLTZ wrote:

>	Hello,
>	I have borrowed a Dell A920 which happens to be a relabelled Lexmark X1100 
>series scanner. As expected, it is slightly different from the X1100 model 
>for which the lexmark backend works. So I'm about to add support for the A920 
>in this backend. For that I think doing the following changes:
>	- move the shadow_registers table in device struct in case someone has
>	    several of theses scanners plugged int the same computer.
Yes. This makes sense as I've moved just about everything else under the 
device struct for the same reason.

> 	- detect scanner model by using USB ids
Sounds good. I assume the Dell A920 has a different ID than  0x043d 
0x007c and that there's an example of how to do this in other backends.

>	- add a struct holding flags and parameters on a model basis like the gt68xx 
>	  or genesy backend
I haven't looked at these backends. I suppose this would be okay. What 
flags and parameters would differ between models?

>	- move constant register values to sanei_lexmark_x1100_init(), and initialize
>	  differently according to scanner model
Hmmmm. This is something I should have probably done earlier as I think 
I still set  the constant register values per scan operation. Care will 
have to be taken that nothing breaks when moving this to 

>	- finally set up the about 20 registers (sensor and motor related) that 
>	 differs between the X110 and A920 in each scan opertion
I assume this will be handled conditionally (if x1100 do this..., else 
if A920 do this...) in the appropriate function for each scan operation.

>	Fred Odendaal, what do you think of it.? If it is the correct route to take, 
>how do you prefer these changes being done ? Through patches sent to the 
>list, or in a copy of the backend in the experimental CVS staging area ?
I'd rather do it through the experimental CVS staging area. Do we need 
to do a branch?

Fred O.

>	Stef

More information about the sane-devel mailing list