Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754042AbbHXH0t (ORCPT ); Mon, 24 Aug 2015 03:26:49 -0400 Received: from mail-la0-f43.google.com ([209.85.215.43]:36075 "EHLO mail-la0-f43.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753746AbbHXH0s (ORCPT ); Mon, 24 Aug 2015 03:26:48 -0400 Date: Mon, 24 Aug 2015 09:26:36 +0200 From: Johan Hovold To: "Liu.Zhao" Cc: johan@kernel.org, linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, gregkh@linuxfoundation.org, konstantin@linuxfoundation.org, 398817832@qq.com, 278883616@qq.com, yang.haojun3@zte.com.cn Subject: Re: [PATCH v3 1/1] USB:option:add ZTE PIDs Message-ID: <20150824072636.GD14209@localhost> References: <1439999477-3447-1-git-send-email-lzsos369@163.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1439999477-3447-1-git-send-email-lzsos369@163.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4837 Lines: 103 On Wed, Aug 19, 2015 at 08:51:17AM -0700, Liu.Zhao wrote: > > This is intended to add ZTE device PIDs on kernel. > > > > Signed-off-by: Liu.Zhao > --- > drivers/usb/serial/option.c | 36 ++++++++++++++++++++++++++++-------- > 1 file changed, 28 insertions(+), 8 deletions(-) > > diff --git a/drivers/usb/serial/option.c b/drivers/usb/serial/option.c > index 876423b..e26db28 100644 > --- a/drivers/usb/serial/option.c > +++ b/drivers/usb/serial/option.c > @@ -278,13 +278,17 @@ static void option_instat_callback(struct urb *urb); > #define ZTE_PRODUCT_MF622 0x0001 > #define ZTE_PRODUCT_MF628 0x0015 > #define ZTE_PRODUCT_MF626 0x0031 > -#define ZTE_PRODUCT_AC2726 0xfff1 > -#define ZTE_PRODUCT_MG880 0xfffd > -#define ZTE_PRODUCT_CDMA_TECH 0xfffe > -#define ZTE_PRODUCT_AC8710T 0xffff > +#define ZTE_PRODUCT_ZM8620_X 0x0396 > +#define ZTE_PRODUCT_ME3620_MBIM 0x0426 > +#define ZTE_PRODUCT_ME3620_X 0x1432 > +#define ZTE_PRODUCT_ME3620_L 0x1433 > #define ZTE_PRODUCT_MC2718 0xffe8 > #define ZTE_PRODUCT_AD3812 0xffeb > #define ZTE_PRODUCT_MC2716 0xffed > +#define ZTE_PRODUCT_AC2726 0xfff1 > +#define ZTE_PRODUCT_MG880 0xfffd > +#define ZTE_PRODUCT_CDMA_TECH 0xfffe > +#define ZTE_PRODUCT_AC8710T 0xffff Reordering the current entries is not needed, but if you do it then you must mention it in the commit message above. Also make sure to not change any formatting; checkpatch is now complaining about whitespace issues (always run checkpatch before submitting). > #define BENQ_VENDOR_ID 0x04a5 > #define BENQ_PRODUCT_H10 0x4068 > @@ -544,6 +548,14 @@ static const struct option_blacklist_info zte_mc2716_z_blacklist = { > .sendsetup = BIT(1) | BIT(2) | BIT(3), > }; > > +static const struct option_blacklist_info zte_me3620andzm8620_xl_blacklist = { > + .reserved = BIT(3) | BIT(4) | BIT(5), > +}; Use two structs for this: zte_me3620_blacklist and zm8620_xl_blacklist even if they reserve the same ports. > + > +static const struct option_blacklist_info zte_me3620_mbim_blacklist = { > + .reserved = BIT(2) | BIT(3) | BIT(4), > +}; > + > static const struct option_blacklist_info huawei_cdc12_blacklist = { > .reserved = BIT(1) | BIT(2), > }; > @@ -1581,16 +1593,24 @@ static const struct usb_device_id option_ids[] = { > { USB_DEVICE_AND_INTERFACE_INFO(ZTE_VENDOR_ID, 0xfff9, 0xff, 0xff, 0xff) }, > { USB_DEVICE_AND_INTERFACE_INFO(ZTE_VENDOR_ID, 0xfffb, 0xff, 0xff, 0xff) }, > { USB_DEVICE_AND_INTERFACE_INFO(ZTE_VENDOR_ID, 0xfffc, 0xff, 0xff, 0xff) }, > - { USB_DEVICE_AND_INTERFACE_INFO(ZTE_VENDOR_ID, ZTE_PRODUCT_MG880, 0xff, 0xff, 0xff) }, > - { USB_DEVICE_AND_INTERFACE_INFO(ZTE_VENDOR_ID, ZTE_PRODUCT_CDMA_TECH, 0xff, 0xff, 0xff) }, > { USB_DEVICE_AND_INTERFACE_INFO(ZTE_VENDOR_ID, ZTE_PRODUCT_AC2726, 0xff, 0xff, 0xff) }, > { USB_DEVICE_AND_INTERFACE_INFO(ZTE_VENDOR_ID, ZTE_PRODUCT_AC8710T, 0xff, 0xff, 0xff) }, > - { USB_DEVICE_AND_INTERFACE_INFO(ZTE_VENDOR_ID, ZTE_PRODUCT_MC2718, 0xff, 0xff, 0xff), > - .driver_info = (kernel_ulong_t)&zte_mc2718_z_blacklist }, > { USB_DEVICE_AND_INTERFACE_INFO(ZTE_VENDOR_ID, ZTE_PRODUCT_AD3812, 0xff, 0xff, 0xff), > .driver_info = (kernel_ulong_t)&zte_ad3812_z_blacklist }, > + { USB_DEVICE_AND_INTERFACE_INFO(ZTE_VENDOR_ID, ZTE_PRODUCT_CDMA_TECH, 0xff, 0xff, 0xff) }, > { USB_DEVICE_AND_INTERFACE_INFO(ZTE_VENDOR_ID, ZTE_PRODUCT_MC2716, 0xff, 0xff, 0xff), > .driver_info = (kernel_ulong_t)&zte_mc2716_z_blacklist }, > + { USB_DEVICE_AND_INTERFACE_INFO(ZTE_VENDOR_ID, ZTE_PRODUCT_MC2718, 0xff, 0xff, 0xff), > + .driver_info = (kernel_ulong_t)&zte_mc2718_z_blacklist }, > + { USB_DEVICE_AND_INTERFACE_INFO(ZTE_VENDOR_ID, ZTE_PRODUCT_MG880, 0xff, 0xff, 0xff) }, > + { USB_DEVICE(ZTE_VENDOR_ID, ZTE_PRODUCT_ME3620_L), > + .driver_info = (kernel_ulong_t)&zte_me3620andzm8620_xl_blacklist }, > + { USB_DEVICE(ZTE_VENDOR_ID, ZTE_PRODUCT_ME3620_X), > + .driver_info = (kernel_ulong_t)&zte_me3620andzm8620_xl_blacklist }, > + { USB_DEVICE(ZTE_VENDOR_ID, ZTE_PRODUCT_ZM8620_X), > + .driver_info = (kernel_ulong_t)&zte_me3620andzm8620_xl_blacklist }, > + { USB_DEVICE(ZTE_VENDOR_ID, ZTE_PRODUCT_ME3620_MBIM), > + .driver_info = (kernel_ulong_t)&zte_me3620_mbim_blacklist }, If really want to reorder the current entries, then at least make sure the end result is indeed sorted. > { USB_VENDOR_AND_INTERFACE_INFO(ZTE_VENDOR_ID, 0xff, 0x02, 0x01) }, > { USB_VENDOR_AND_INTERFACE_INFO(ZTE_VENDOR_ID, 0xff, 0x02, 0x05) }, > { USB_VENDOR_AND_INTERFACE_INFO(ZTE_VENDOR_ID, 0xff, 0x86, 0x10) }, Johan -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/