SCM

[#300643] sm3600: endian bug when sending calibration data to the scanner

View Trackers | Bugs | Export CSV

Date:
2004-04-13 06:59
Priority:
3
State:
Closed
Submitted by:
Mattias Ellert (ellert-guest)
Assigned to:
Marian Eichholz (eichholz-guest)
Category:
backends (drivers)
Group:
Resolution:
Fixed
Summary:
sm3600: endian bug when sending calibration data to the scanner

Detailed description
The backend sends the calibration data to the scanner with the wrong endian order if the computer and the scanner don't use the same endianness.

Interrestingly, it does correct for this problem when sending the gamma table data.

The attached patch corrects this problem for the calibration data in the same way as is already done for the gamma table data.

Followups: Sort comments antichronologically

Message
Date: 2004-04-14 08:58
Sender: Marian Eichholz

Logged In: YES user_id=3129yes, since the whole usblib was not meant to be platform independent, there is no dedicated provision for correct byte sex handling in the backend. Are there any on-intel-sm3600 users????

I'll fix it when I get some spare time. Thank You, Matthias!

- Marian
Date: 2004-04-14 10:28
Sender: Mattias Ellert

Inloggad: JA user_id=2944If you intended to write 'non-intel-sm3600 users', then the answer is yes. This problem was spotted on MacOS X.
Date: 2004-04-16 08:42
Sender: Marian Eichholz

Logged In: YES user_id=3129Fixed in CVS.

A new gain upload function in sm3600-scanmtek.c serves both color and gray scan modes. It is not beautiful but seems to do the job. Thanks Mattias for the inspiration.
Date: 2004-06-20 01:06
Sender: Mattias Ellert

Inloggad: JA user_id=2944fixed in 1.0.14 - closing bug report

Attached Files:

Changes:

Field Old Value Date By
status_idOpen2004-06-20 01:06ellert-guest
close_date2004-06-20 01:062004-06-20 01:06ellert-guest
ResolutionNone2004-04-16 09:00eichholz-guest
File Deleted75: 2004-04-16 08:42eichholz-guest
assigned_tonone2004-04-13 16:35hmg-guest
File Added75: sane-sm3600-endian.patch2004-04-13 06:59ellert-guest
Powered By FusionForge