Dear Greg:
> -----Original Message-----
> From: Greg KH [mailto:g...@kroah.com]
> Sent: Saturday, January 19, 2013 7:44 AM
> To: Fangxiaozhi (Franko)
> Cc: linux-usb@vger.kernel.org; linux-ker...@vger.kernel.org; Xueguiying
> (Zihan);
> Linlei (Lei Lin); Yili (Neil); Wangyuhua (Roger, Credit); Huqiao (C);
> ba...@ti.com;
> mdharm-...@one-eyed-alien.net; sebast...@breakpoint.cc
> Subject: Re: USB: storage: optimize the matching rules and support new switch
> command for Huawei USB storage devices
>
> On Mon, Jan 14, 2013 at 10:55:48AM +0800, fangxiaozhi 00110321 wrote:
> >
> > From: fangxiaozhi <huana...@huawei.com>
> >
> > 1. Optimize the matching rules with new macro for Huawei USB storage
> > devices, to avoid to load USB storage driver for the modem interface
> > with Huawei devices.
> > 2. Add to support new switch command for new Huawei USB dongles.
> >
> > Signed-off-by: fangxiaozhi <huana...@huawei.com>
>
> This patch breaks the build, did you test it out?
>
> I get the following errors:
>
> drivers/usb/storage/unusual_devs.h:1530:1: error: implicit declaration of
> function ‘UNUSUAL_VENDOR_INTF’ [-Werror=implicit-function-declaration]
> drivers/usb/storage/unusual_devs.h:1534:3: warning: missing braces around
> initializer [-Wmissing-braces]
> drivers/usb/storage/unusual_devs.h:1534:3: warning: (near initialization for
> ‘us_unusual_dev_list[186]’) [-Wmissing-braces]
> drivers/usb/storage/unusual_devs.h:1534:3: error: initializer element is not
> constant
> drivers/usb/storage/unusual_devs.h:1534:3: error: (near initialization for
> ‘us_unusual_dev_list[186].vendorName’)
> drivers/usb/storage/unusual_devs.h:1537:1: warning: braces around scalar
> initializer [enabled by default]
>
> And it goes on and on...
----------The macro define, please see another patch:
[PATCH 1/1]linux-usb:Define a new macro for USB storage match rules
http://www.spinics.net/lists/linux-usb/msg76629.html
> Care to fix this up and resend it?
>
> thanks,
>
> greg k-h
Best Regards,
Franko Fang
