On Sun, 18 Mar 2012, Phil Dibowitz wrote:
> + ri.flash_mfg = 0x01;
> + ri.flash_id = 0x49;
>
> Do these need to be 0 for 1000 usbnets?
I don't know for sure, but I kind of doubt it. Perhaps we can find
someone who has a 1000 to capture some data for us.
> +const int connect_timeout = 2; // try to connect for 2 seconds
>
> Does 1 work well for the usbnets? Its not a big deal, but I think it'd be a
> better user experience for everyone else.
A value of 1 probably would work fine for the usbnets. I'll try that
tonight and see how well it works.
> + /* If this is a usbnet remote, abort before trying to detach drivers */
> + if (h_dev && (h_dev->descriptor.idProduct == 0xC11F)) {
> + return LC_ERROR_CONNECT;
> + }
> +
>
> Are you really trying to just prevent the attempt at detaching the drivers, or
> are you trying to short circuit FindRemote for usbnet devices. Presumably if
> we know we're a usbnet remote we should bail out regardless of OS, but your
> code is guarded by '#ifdef linux'
I am really just trying to prevent the detaching of the drivers, since it
actually unloads the zaurus driver, which we don't want it to do, if the
remote is a usbnet.
Scott
------------------------------------------------------------------------------
This SF email is sponsosred by:
Try Windows Azure free for 90 days Click Here
http://p.sf.net/sfu/sfd2d-msazure
_______________________________________________
concordance-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/concordance-devel
