[sane-devel] Canon LiDE 210

Olaf Meeuwissen paddy-hack at member.fsf.org
Thu Aug 3 09:45:54 UTC 2017


Hi Jörg,

Jörg Frings-Fürst writes:

> Hello,
>
> I forward this bug from Debian[1]. Please can someone check the
> included patch?

> diff --git a/backend/genesys.c b/backend/genesys.c
> index 984cead..eb8695a 100644
> --- a/backend/genesys.c
> +++ b/backend/genesys.c
> @@ -210,6 +210,8 @@ sanei_genesys_init_structs (Genesys_Device * dev)
> dev->model->motor_type);
> }
>
> +dev->usb_mode = 0;
> +
> /* set up initial line distance shift */
> dev->ld_shift_r = dev->model->ld_shift_r;
> dev->ld_shift_g = dev->model->ld_shift_g;

Patch looks simple enough, but I think a better place to set the initial
value of usb_mode is where dev is created, in attach() itself.

I'll attach my suggested patch to the Debian bug report[1] and follow up
there.

 [1]: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=869673

Hope this helps,
--
Olaf Meeuwissen, LPIC-2            FSF Associate Member since 2004-01-27
 GnuPG key: F84A2DD9/B3C0 2F47 EA19 64F4 9F13  F43E B8A4 A88A F84A 2DD9
 Support Free Software                        https://my.fsf.org/donate
 Join the Free Software Foundation              https://my.fsf.org/join



More information about the sane-devel mailing list