Add more Samsung MFP scanners support (Xerox WorkCentre 3119, Samsung SCX-4220)

[Roman Yepishev]

Xerox WorkCentre 3119 scanner can be made fully supported by Ubuntu with minor additions to the configs:

/lib/udev/rules.d/40-libsane.rules:
# Xerox WorkCentre 3119
ATTRS{idVendor}=="0924", ATTRS{idProduct}=="4265", ENV{libsane_matched}="yes"

/etc/sane.d/xerox_mfp.conf:
#Xerox WorkCentre 3119.
usb 0x0924 0x4265

Since Samsung SCX-4220 is fully supported by sane, I don't see the reason why Xerox WorkCentre 4220 should not be supported since the latter is a clone of SCX-4220.

ProblemType: Bug
Architecture: i386
CheckboxSubmission: b16b943d4712f4613c50f12b0ffe0cc5
CheckboxSystem: 1fd1d69a420d7665c5bbb30cf0881c53
Date: Sun Nov 1 23:33:53 2009
DistroRelease: Ubuntu 9.10
NonfreeKernelModules: nvidia
Package: libsane 1.0.20-4ubuntu3 [modified: lib/udev/rules.d/40-libsane.rules]
ProcEnviron:
 SHELL=/bin/bash
 PATH=(custom, user)
 LANG=en_US.UTF-8
ProcVersionSignature: Ubuntu 2.6.31-14.48-generic
SourcePackage: sane-backends
Uname: Linux 2.6.31-14-generic i686

[Roman Yepishev (rye)]

Patch for sane-backends-1.0.20

[Roman Yepishev (rye)]

Oops, I just found out that Samsung SCX-4220 is not supported out-of-the-box as well.
Will add the patch for sane-backends later today.

[Roman Yepishev (rye)]

SANE backends already contains more supported MFPs in their HEAD branch, so it may be worth looking there
http://git.debian.org/?p=sane/sane-backends.git;a=blob_plain;f=doc/descriptions/xerox_mfp.desc;hb=HEAD

So far I am attaching the patch that will enable Xerox Workcentre 3119 and Samsung SCX-4220 (distributed primarily in East Europe/Asia) - these are the only Samsung MFPs I have access to.

[dave b. (d+b)]

Can you please add the scx 4200(to /etc/sane.d/xerox_mfp.conf), as i have access to it and it does work.

"
#scx4200
usb 0x04e8 0x341b
"

By the way, if a device is marked "good" upstream , why not enable it ? ....

[Vadym Abramchuck (abramzzz)]

Hi all.
I've added 4220's lines to xerox_mfp.conf and /lib/udev/*libsane.* . Worked like a charm!

[Roman Yepishev (rye)]

Attached dpatch for xerox_mfp git20100208.
This is not only config change now but the code has some fixes as well:
 * Additional error check to (stop scanadf) + fix of it[1]
 * Standardize including local sane/ include files[2]

The best possible resolution of the problem is to have sane release new version of sane-backends, otherwise lucid lynx users will need to recompile the package for their scanners to be supported.

The configuration changes are not enough. Each time libsane is updated the udev rules are rewritten w/o any confirmation and confirmation about xerox_mfp.conf replacement is presented to the end user.

Even if the latter is preserved the permissions are not set properly for usb devices for non-root accounts leading to:

rtg@buzz:~$ sane-find-scanner
found USB scanner (vendor=0x0924, product=0x4265) at libusb:001:002
rtg@buzz:~$ scanimage -L

No scanners were identified. If you were expecting something different,
check that the scanner is plugged in, turned on and detected by the
sane-find-scanner tool (if appropriate). Please read the documentation
which came with this software (README, FAQ, manpages).

[1] http://git.debian.org/?p=sane/sane-backends.git;a=commit;h=dda029430eed0226d0d91188dfd5ffa4cc8b3041
[2] http://git.debian.org/?p=sane/sane-backends.git;a=commit;h=4d9e8138af0f399e72599631854fa266cb18a4da

[Launchpad Janitor (janitor)]

This bug was fixed in the package sane-backends - 1.0.20-13ubuntu1

---------------
sane-backends (1.0.20-13ubuntu1) lucid; urgency=low

  * Merge from Debian testing, remaining changes:
    - Add 42_add_epson_cx_6000.dpatch: Add Epson CX6000. (LP: #314485)
    - Add 43_scanadf_error_check.dpatch: Backport from Debian's
      sane-backends git repo adding additional error checking.
      (LP: #469540)
    - Add 44_add_epson_mfp_devs.dpatch: Add additional device id.
      (LP: #469540)
    - Don't apply 11_udev_147, because ubuntu_udev_noperm.dpatch is
      more appropriate for Ubuntu:
      + Do not change device permissions (already covered by udev-extra's
        70-acl.rules)
      + Do not cover SCSI scanners (already covered by udev-extra's
        70-acl.rules)
      + Simplify description and structure.
    - debian/control:
      + Demote libsane-extras from Depends: to Suggests:, as long as
        this package is in universe.
      + Demote sane-utils and avahi-daemon from Recommends to Suggests.
        We do not want to have an init script for saned laying around,
        which is disabled by default anyway.
    - debian/libsane.dirs.linux: remove
    - debian/{libsane.install.in,libsane-dev.install,rules}: Install
      documentation into libsane-dev instead of libsane to save CD space.
    - debian/rules:
      + Link using -Bsymbolic-functions (startup time optimization).
      + /etc/modprobe.d/libsane blacklists two modules that don't exist
        anymore, dropped creation of this file
    - debian{rules,sane-utils.saned.init}: saned init script does not need
      to be started before gdm, and does not need to be called on shutdown.
    - debian/libsane.postinst: Do not create the scanner group.
    - debian/{sane-utils.config,sane-utils.postinst,sane-utils.templates}:
      Drop the support for saned in the scanner group, since we don't have
      a scanner group anymore!
    - debian/libsane.preinst:
      + Remove the old blacklist-scanner modprobe file. LP: #340873.
      + Remove the old file if unmodified
    - debian/rules: Install udev rules again, since we now don't rely on
      hal any more and thus we need the ENV{libsane_matched}=="yes" flag.

sane-backends (1.0.20-13) unstable; urgency=low

  * debian/patches/34_genesys_gl841_cal_fix.dpatch:
    + Added; fix calibration on gl841-based scanners (closes: #563163).

  * debconf translations:
    + ja.po: courtesy of Hideki Yamane (closes: #564294).

sane-backends (1.0.20-12) unstable; urgency=low

  * debian/patches/33_epson2_update.dpatch:
    + Update epson2 from git HEAD (closes: #534746).

  * debconf translations:
    + de.po: courtesy of Helge Kreutzmann (closes: #562709).
    + sv.po: courtesy of Martin Bagge (closes: #562931).
    + ru.po: courtesy of Yuri Kozlov (closes: #563076).
    + pt.po: courtesy of Américo Monteiro (closes: #563174).

sane-backends (1.0.20-11) unstable; urgency=medium

  * Urgency set to medium to unbreak saned in testing.

  * debian/patches/12_saned_polling_fix.dpatch:
    + Added; unbreak saned's polling loop for more than 1 fds (closes: #562248).
  * debian/patches/13_saned_aliasing_fix.dpatch:
    + Added; fix strict alias...