On Fri, 2014-03-28 at 11:37 -0500, Dan Williams wrote: > On Fri, 2014-03-28 at 15:25 +0800, AceLan Kao wrote: > > Fixed the product id typo and replace whitespace with tab. > > > > Signed-off-by: AceLan Kao <acelan.kao@xxxxxxxxxxxxx> > > --- > > drivers/usb/serial/sierra.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/usb/serial/sierra.c b/drivers/usb/serial/sierra.c > > index a9eb622..b78fd4e 100644 > > --- a/drivers/usb/serial/sierra.c > > +++ b/drivers/usb/serial/sierra.c > > @@ -291,7 +291,7 @@ static const struct usb_device_id id_table[] = { > > { USB_DEVICE(0x0f3d, 0x68A3), /* Airprime/Sierra Wireless Direct IP modems */ > > .driver_info = (kernel_ulong_t)&direct_ip_interface_blacklist > > }, > > - { USB_DEVICE(0x413C, 0x08133) }, /* Dell Computer Corp. Wireless 5720 VZW Mobile Broadband (EVDO Rev-A) Minicard GPS Port */ > > + { USB_DEVICE(0x413C, 0x8133) }, /* Dell Computer Corp. Wireless 5720 VZW Mobile Broadband (EVDO Rev-A) Minicard GPS Port */ > > The Dell 5720 (Novatel E725) is not a Sierra device, so it should > certainly not remain in the 'sierra' driver and the original commit that > did this was in error. It's device ID should be moved to the 'option' > driver instead. So could you resend this patch, but move the device to > option.c and fix the typo there? Nevermind, Johan nailed it; we should do the revert instead. Dan -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html