Re: RFC: new USB quirk

2012-11-05 Thread Hans Petter Selasky
Hi,

Looks good. Go ahead and commit.

BTW: Is it possible to auto detect this quirk? Then you might consider adding 
some code to dev/usb/usb_msctest.c .

--HPS
___
freebsd-usb@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-usb
To unsubscribe, send any mail to freebsd-usb-unsubscr...@freebsd.org


Re: RFC: new USB quirk

2012-11-05 Thread Marcel Moolenaar

On Nov 5, 2012, at 9:58 AM, Hans Petter Selasky hsela...@c2i.net wrote:

 Hi,
 
 Looks good. Go ahead and commit.

Thanks.

 BTW: Is it possible to auto detect this quirk? Then you might consider adding 
 some code to dev/usb/usb_msctest.c .

I don't know enough to say anything with confidence. It looks to
me that you first want to know if the device has removable media
and whether there's a medium inserted before you reach conclusions
about whether a failed prevent/allow media removal command is
an indication that UQ_MSC_NO_PREVENT_ALLOW should be used for the
device.

-- 
Marcel Moolenaar
mar...@xcllnt.net


___
freebsd-usb@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-usb
To unsubscribe, send any mail to freebsd-usb-unsubscr...@freebsd.org


Re: RFC: new USB quirk

2012-11-04 Thread Hans Petter Selasky
On Saturday 03 November 2012 22:42:33 Marcel Moolenaar wrote:
 Hans,
 
 Juniper has a few products that use STmicro USB-NAND controllers
 (the ST72682 in particular) and they really don't like the SCSI
 command for prevent/allow medium removal. The attached patch
 adds a quirk for this to avoid spamming the console with errors
 or warnings.
 
 Does it look good to you?
 Ok to commit?
 
 Thanks,

Hi,

You also need to update this table in usb_quirk.c:

static const char *usb_quirk_str[USB_QUIRK_MAX] = {
[UQ_NONE]   = UQ_NONE,
 
Can you do that and send a new patch for review?

--HPS
___
freebsd-usb@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-usb
To unsubscribe, send any mail to freebsd-usb-unsubscr...@freebsd.org


Re: RFC: new USB quirk

2012-11-04 Thread Marcel Moolenaar

On Nov 4, 2012, at 2:47 AM, Hans Petter Selasky hsela...@c2i.net wrote:

 On Saturday 03 November 2012 22:42:33 Marcel Moolenaar wrote:
 Hans,
 
 Juniper has a few products that use STmicro USB-NAND controllers
 (the ST72682 in particular) and they really don't like the SCSI
 command for prevent/allow medium removal. The attached patch
 adds a quirk for this to avoid spamming the console with errors
 or warnings.
 
 Does it look good to you?
 Ok to commit?
 
 Thanks,
 
 Hi,
 
 You also need to update this table in usb_quirk.c:
 
 static const char *usb_quirk_str[USB_QUIRK_MAX] = {
[UQ_NONE]   = UQ_NONE,
 
 Can you do that and send a new patch for review?

See attached.
FYI,

-- 
Marcel Moolenaar
mar...@xcllnt.net



usb.diff
Description: Binary data
___
freebsd-usb@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-usb
To unsubscribe, send any mail to freebsd-usb-unsubscr...@freebsd.org