[sane-devel] [patch] mf4570dn

[Rolf Bensch]

Hi Chrysn,

We should avoid multiple declarations in the code.

I'll simply replace "Canon i-SENSYS MF4550d" with "Canon i-SENSYS MF4500
Series" in the code and commit the changes, soon.

It's correct, there are MF4500 Series scanners with and without
wireless. But we cannot differ wireless from wired only devices because
we identify scanners by their USB PID and they have the same.

Many thanks for your help.

Cheers,
Rolf

Am 21.02.2014 03:18, schrieb chrysn:
> On Mon, Feb 17, 2014 at 11:45:50AM +0100, louis at fazant.net wrote:
>>> $ scanimage -d pixma:mf4570dn_clio
>> Unless you have more than one scanner, there is no need to select the
>> device on the command line. The name you are using here is almost for sure
>> incorrect.
> `scanimage -d pixma` didn't improve the results either, but adding the
> device to the device list did.
>
> the attached patch adds an "i-SENSYS MF4500 Series" line to the list of
> known devices. ("Canon MF4500 Series" is what my MF4570dn reported). the
> usb pid is the same as for the already existing mf4550d (MF4550_PID), so
> i reused that. as a result of the sequence, all 4500 devices should show
> up as "Canon MF4500 Series" when connected by usb, but the 4550 stays
> discernable when connected via network (provided i understand all that
> discovery right).
>
> capability-wise, the new device / device group is no different than all
> the others already present. i've adapted the documentation to match that
> we now probably support all MF4500 series devices. (at least on usb;
> some might still use individual model numbers on network).
>
> side note: the i-SENSYS MF4570dn seems to be identical with the
> imageCLASS MF4570dw except it's not *w*ireless (only regular *n*etwork).
> apart from generalization that could be done from that: it looks like it
> has PIXMA_CAP_ADF.
>
>
> please review and accept the patch.
>
> best regards
> chrysn
>