2007-12-14 12:16:40

by Nicolas Mailhot

[permalink] [raw]
Subject: Re: [argyllcms] Re: [PATCH] usb/hid: Blacklist the Gretag-Macbeth Huey display colorimeter


Le vendredi 14 décembre 2007 à 22:46 +1100, Graeme Gill a écrit :
> Nicolas Mailhot wrote:
>
> >>Care to make up a patch as per the Documentation/SubmittingPatches file
> >>and send it to this list and the HID driver maintainer? That way you
> >>get the proper credit for this change.
> >
> > The things I do as a public service, I don't even have the damn Huey
> > myself:(
>
> Actually (to put it frankly), I'm amazed that this exceptions list is
> compiled into the driver. I would have thought that such a list should
> at least be in a configuration file that an installed application can
> add or delete, if not something more sophisticated.

Something more sophisticated will happen someday, this is what we have
now.

> What will happen if the
> HID driver is fixed to allow arbitrary messages, and I want to switch
> back to using it rather than libusb ?

Then the person doing the HID code fix should remove the blacklist entry
at the same time.

> And the question still remains as to why usb_detach_kernel_driver_np()
> works fine on some systems (ie. the one I'm testing on), and not (apparently)
> others.

I'm not saying it does not work, I'm saying I don't want my users to
fool with kernel modules when we know the default should not be HID for
huey right now. If you prefer enhancing the kernel HID driver, and
switching argyll to it, just do it. I don't care as long as I don't have
to maintain workarounds distro-side.

--
Nicolas Mailhot


Attachments:
signature.asc (197.00 B)
Ceci est une partie de message num?riquement sign

2007-12-16 16:59:41

by Jiri Kosina

[permalink] [raw]
Subject: Re: [argyllcms] Re: [PATCH] usb/hid: Blacklist the Gretag-Macbeth Huey display colorimeter

On Fri, 14 Dec 2007, Nicolas Mailhot wrote:

> > Actually (to put it frankly), I'm amazed that this exceptions list is
> > compiled into the driver. I would have thought that such a list should
> > at least be in a configuration file that an installed application can
> > add or delete, if not something more sophisticated.
> Something more sophisticated will happen someday, this is what we have
> now.

There is also possibility (for quite a few releases already) to change the
quirk list in runtime. See the 'quirk' parameter to the usbhid module.

> > What will happen if the HID driver is fixed to allow arbitrary
> > messages, and I want to switch back to using it rather than libusb ?

What exactly is the problem here? I didn't seem to catch the beginning of
the thread (or it happened off the list I am subscribed to).

I will apply the patch to my tree, thanks.

--
Jiri Kosina
SUSE Labs

2007-12-17 11:07:28

by Nicolas Mailhot

[permalink] [raw]
Subject: Re: [argyllcms] Re: [PATCH] usb/hid: Blacklist the Gretag-Macbeth Huey display colorimeter

[Replying twice because I replied to the wrong message at first]

Le dimanche 16 décembre 2007 à 17:59 +0100, Jiri Kosina a écrit :

> What exactly is the problem here? I didn't seem to catch the beginning of
> the thread (or it happened off the list I am subscribed to).

For more info on usb colorimeter problems you can check
http://www.freelists.org/archives/argyllcms/12-2007/

It seems Graeme Gill, the Argyll CMS author has hit several linux usb
problems over the years but has not coordinated well with the Linux usb
maintainers. We're finding all the ugly bits now that we're trying to
meger Argyll CMS in various distributions (Fedora, Mandriva, Ubuntu,
etc)

It's a bit difficult to follow this thread because the argyllcms list is
killing CCs and not everyone is putting them back in.

Regards,

--
Nicolas Mailhot


Attachments:
signature.asc (197.00 B)
Ceci est une partie de message num?riquement sign