Date: Wed, 08 Oct 2008 11:15:52 +0200 From: Alexander Leidinger <Alexander@Leidinger.net> To: Poul-Henning Kamp <phk@phk.freebsd.dk> Cc: current@freebsd.org Subject: Re: umass quirk review ? Message-ID: <20081008111552.69843ndq7vkp4hgk@webmail.leidinger.net> In-Reply-To: <56363.1223454787@critter.freebsd.dk> References: <56363.1223454787@critter.freebsd.dk>
next in thread | previous in thread | raw e-mail | index | archive | help
Quoting Poul-Henning Kamp <phk@phk.freebsd.dk> (from Wed, 08 Oct 2008 =20
08:33:07 +0000):
>
> I'm vaguely aware that we have some admin process in place for reviewing
> quirkentries in umass, but I seem unable to find out who to send my
> patch to ?
AFAIR the process is to register a PR so that we have a trace. The PR =20
should give infos about the device and the problems without the =20
quirks, and ideally why each quirk is needed for this device ("quirk A =20
solves error X and quirk B solves error Y" or something like this).
After that feel free to commit and close the PR.
Bye,
Alexander.
> Unless anybody objects, this goes in soonish.
>
> And yes, an infrared camera is a neat toy:
>
> =09http://phk.freebsd.dk/misc/misling.png
>
> Poul-Henning
>
>
> Index: usbdevs
> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
> --- usbdevs=09(revision 183674)
> +++ usbdevs=09(working copy)
> @@ -1855,6 +1855,7 @@
> product OMNIVISION OV511PLUS=090xa511=09OV511+ Camera
>
> /* OnSpec Electronic, Inc. */
> +product ONSPEC SDS_HOTFIND_D=090x0400=09SDS-infrared.com Hotfind-D =20
> Infrared Camera
> product ONSPEC MDCFE_B_CF_READER=090xa000=09MDCFE-B USB CF Reader
> product ONSPEC CFMS_RW=09=090xa001=09SIIG/Datafab Memory Stick+CF Reader/=
Writer
> product ONSPEC READER=09=090xa003=09Datafab-based Reader
> Index: umass.c
> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
> --- umass.c=09(revision 183674)
> +++ umass.c=09(working copy)
> @@ -610,6 +610,10 @@
> =09 UMASS_PROTO_SCSI,
> =09 NO_QUIRKS
> =09},
> +=09{ USB_VENDOR_ONSPEC, USB_PRODUCT_ONSPEC_SDS_HOTFIND_D, RID_WILDCARD,
> +=09 UMASS_PROTO_SCSI | UMASS_PROTO_BBB,
> +=09 NO_GETMAXLUN | NO_SYNCHRONIZE_CACHE
> +=09},
> =09{ USB_VENDOR_ONSPEC, USB_PRODUCT_ONSPEC_MDCFE_B_CF_READER, RID_WILDCAR=
D,
> =09 UMASS_PROTO_SCSI,
> =09 NO_QUIRKS
> --
> Poul-Henning Kamp | UNIX since Zilog Zeus 3.20
> phk@FreeBSD.ORG | TCP/IP since RFC 956
> FreeBSD committer | BSD since 4.3-tahoe
> Never attribute to malice what can adequately be explained by incompetence=
.
> _______________________________________________
> freebsd-current@freebsd.org mailing list
> http://lists.freebsd.org/mailman/listinfo/freebsd-current
> To unsubscribe, send any mail to "freebsd-current-unsubscribe@freebsd.org"
>
>
--=20
I'm definitely not in Omaha!
http://www.Leidinger.net Alexander @ Leidinger.net: PGP ID =3D B0063FE7
http://www.FreeBSD.org netchild @ FreeBSD.org : PGP ID =3D 72077137
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20081008111552.69843ndq7vkp4hgk>