public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* Patch: Plustek scanner driver (pt_drv) port to 2.6, correction for 2.6.3
@ 2004-03-04 15:19 large
  2004-03-04 20:12 ` Gerhard Jaeger
  0 siblings, 1 reply; 2+ messages in thread
From: large @ 2004-03-04 15:19 UTC (permalink / raw)
  To: Linux Kernel Mailing List; +Cc: gerhard, Domen Puncer

[-- Attachment #1: Type: text/plain, Size: 1271 bytes --]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Hi,

I recently updated from 2.6.0-test4 to 2.6.3 and noticed that my pt_drv
won't work any more. It did not load, telling me something like
missing symbol: kdev_t_to_nr

I investigated and found the problem:
the inline function kdev_t_to_nr does not exist any longer, I think it's
depr. due to the kdev device changes in 2.6.3 (2.6.2?).

ptdrv.c:
255: int minor = MINOR(kdev_t_to_nr(ip->i_rdev));

Anyway, I could not figure out what function might be it's successor, so
I simply removed the function call, leading to:

ptdrv.c:
225: int minor = MINOR(ip->i_rdev);

All kdev_t.h functions existing in 2.6.3 did only bad stuff to the
numbers so think this might be correct.

I just removed it, as it works for me (and should logically, reviewing
how the MINOR inline works now). I could not test it with more than one
scanner device, so it might still be false.

Anyone out there who can proof whether this is OK like this?

Attached the changed patch...

cu,
~  Lars
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.3 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://enigmail.mozdev.org

iD8DBQFAR0kO7V+LJJ0V3jwRAmM6AJkBzA2VciANIfuSUsP+8TqZfpvJ0ACfdfGs
+AR+ncqsDvRuqZZXGk4L5a8=
=4DJ7
-----END PGP SIGNATURE-----

[-- Attachment #2: plustek-0.45-2.6.3.diff.gz --]
[-- Type: application/gzip, Size: 4293 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: Patch: Plustek scanner driver (pt_drv) port to 2.6, correction for 2.6.3
  2004-03-04 15:19 Patch: Plustek scanner driver (pt_drv) port to 2.6, correction for 2.6.3 large
@ 2004-03-04 20:12 ` Gerhard Jaeger
  0 siblings, 0 replies; 2+ messages in thread
From: Gerhard Jaeger @ 2004-03-04 20:12 UTC (permalink / raw)
  To: large, Linux Kernel Mailing List; +Cc: Domen Puncer

Hmm,

I don't think that this patch is necessary anymore. The latest driver
is contained in the sane-backends package. The backend is called
plustek_pp can can either be used completely from userspace or if
you want to as kernel-module (including all the patches for 2.6
kernel, except devfs support).

Ciao,
  Gerhard

On Thursday 04 March 2004 16:19, large wrote:
> Hi,
>
> I recently updated from 2.6.0-test4 to 2.6.3 and noticed that my pt_drv
> won't work any more. It did not load, telling me something like
> missing symbol: kdev_t_to_nr
>
> I investigated and found the problem:
> the inline function kdev_t_to_nr does not exist any longer, I think it's
> depr. due to the kdev device changes in 2.6.3 (2.6.2?).
>
> ptdrv.c:
> 255: int minor = MINOR(kdev_t_to_nr(ip->i_rdev));
>
> Anyway, I could not figure out what function might be it's successor, so
> I simply removed the function call, leading to:
>
> ptdrv.c:
> 225: int minor = MINOR(ip->i_rdev);
>
> All kdev_t.h functions existing in 2.6.3 did only bad stuff to the
> numbers so think this might be correct.
>
> I just removed it, as it works for me (and should logically, reviewing
> how the MINOR inline works now). I could not test it with more than one
> scanner device, so it might still be false.
>
> Anyone out there who can proof whether this is OK like this?
>
> Attached the changed patch...
>
> cu,
> ~  Lars


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2004-03-04 20:14 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-03-04 15:19 Patch: Plustek scanner driver (pt_drv) port to 2.6, correction for 2.6.3 large
2004-03-04 20:12 ` Gerhard Jaeger

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox