[sane-devel] Canon PIXMA MG6350

[Rolf Bensch]

Hi Theo,

Many thanks for reporting new Canon Pixma MG6350.

I added 2 patch files:

(1) pixma_mp150.c.diff0: This is your patch. Please note that the MG6350
scans up to 2400 x 4800 dpi. SANE supports only symmetric scan
resolutions. So the MG6350 should support up to 2400 x 2400 dpi.

(2) pixma_mp150.c.diff1: This should fix scan problems for 1200 dpi and
2400 dpi. I assume that the MG6350 has the same behaviour as the MG5350.

Please report if patch (2) fixes your scan problems.

Cheers,
Rolf


Am 15.03.2013 00:13, schrieb Theo:
> I have a Canon PIXMA MG6350 connected via USB and Ethernet. This model is not yet listed in the supported devices and I gladly help to promote it to status complete.
>
> This is the info I have so far:
>
> $ scanimage -L
> device `pixma:MG6300_192.168.1.4' is a CANON Canon PIXMA MG6300 multi-function peripheral
> device `pixma:04A91765_20A814' is a CANON Canon PIXMA MG6300 multi-function peripheral
>
> $ lsusb
> Bus 001 Device 005: ID 04a9:1765 Canon, Inc. 
>
> I did a beginners attempt to patch the pixma backend by copying these lines of the MG4200:
>
> diff --git a/backend/pixma_mp150.c b/backend/pixma_mp150.c
> index 376623c..14e9433 100644
> --- a/backend/pixma_mp150.c
> +++ b/backend/pixma_mp150.c
> @@ -185,6 +185,7 @@
>  #define MX890_PID 0x175E
>  #define E600_PID 0x175A
>  #define MG4200_PID 0x1763
> +#define MG6300_PID 0x1765
>  
>  
>  /* Generation 4 XML messages that encapsulates the Pixma protocol messages */
> @@ -1707,6 +1708,7 @@ const pixma_config_t pixma_mp150_devices[] = {
>    DEVICE ("Canon PIXMA MX890 Series", "MX890", MX890_PID, 2400, 0, 0, 638, 877, PIXMA_CAP_CIS | PIXMA_CAP_ADFDUP),
>    DEVICE ("Canon PIXMA E600 Series",  "E600",  E600_PID,  1200, 0, 0, 638, 877, PIXMA_CAP_CIS | PIXMA_CAP_ADF),
>    DEVICE ("Canon PIXMA MG4200", "MG4200", MG4200_PID, 1200, 0, 0, 638, 877, PIXMA_CAP_CIS),
> +  DEVICE ("Canon PIXMA MG6300", "MG6300", MG6300_PID, 1200, 0, 0, 638, 877, PIXMA_CAP_CIS),
>  
>    END_OF_DEVICE_LIST
>  };
>
> With this update, I can scan images up to 600 dpi OK, at 1200 pdi, the resulting image is distorted.
>
> I hope this is helpful. 
> -- 
> Kind regards, /Theo van Rijn/

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.alioth.debian.org/pipermail/sane-devel/attachments/20130315/64cd0acd/attachment.html>
-------------- next part --------------
--- ./pixma_mp150.c	2013-03-15 16:39:01.000000000 +0100
+++ ../sane-backends/backend/pixma_mp150.c	2013-03-15 16:51:43.000000000 +0100
@@ -186,6 +186,9 @@
 #define E600_PID 0x175A
 #define MG4200_PID 0x1763
 
+/* 2013 new devices (untested) */
+#define MG6300_PID 0x1765
+
 
 /* Generation 4 XML messages that encapsulates the Pixma protocol messages */
 #define XML_START_1   \
@@ -1708,5 +1711,8 @@
   DEVICE ("Canon PIXMA E600 Series",  "E600",  E600_PID,  1200, 0, 0, 638, 877, PIXMA_CAP_CIS | PIXMA_CAP_ADF),
   DEVICE ("Canon PIXMA MG4200", "MG4200", MG4200_PID, 1200, 0, 0, 638, 877, PIXMA_CAP_CIS),
 
+  /* Latest devices (2013) Generation 4 CIS */
+  DEVICE ("Canon PIXMA MG6300", "MG6300", MG6300_PID, 2400, 0, 0, 638, 877, PIXMA_CAP_CIS),
+
   END_OF_DEVICE_LIST
 };
-------------- next part --------------
--- ./pixma_mp150.c	2013-03-15 16:55:21.000000000 +0100
+++ ../sane-backends/backend/pixma_mp150.c	2013-03-15 16:55:34.000000000 +0100
@@ -1102,7 +1102,7 @@
           /* Color plane and stripes shift needed by e.g. CCD */
           /*PDBG (pixma_dbg (4, "*post_process_image_data***** Processing with c=%u, n=%u, m=%u, w=%i, line_size=%u ***** \n",
 	        c, n, m, s->param->wx, line_size));*/
-          if (s->cfg->pid != MG5300_PID && c >= 3)
+          if (s->cfg->pid != MG5300_PID && s->cfg->pid != MG6300_PID && c >= 3)
             dptr = shift_colors (dptr, sptr, 
                                  s->param->wx, s->param->xdpi, s->cfg->pid, c,
                                  mp->shift, mp->stripe_shift);
@@ -1116,7 +1116,8 @@
               && s->cfg->pid != MX890_PID
               && s->cfg->pid != MG3100_PID
               && s->cfg->pid != MG2100_PID
-              && s->cfg->pid != MG5300_PID)
+              && s->cfg->pid != MG5300_PID
+              && s->cfg->pid != MG6300_PID)
               reorder_pixels (mp->linebuf, sptr, c, n, m, s->param->wx, line_size);
           
           /* Crop line to selected borders */