Re: [PATCH 3/3] USB: gadget: refactor the MIDI device code

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Sep 20, 2011 at 06:59:27PM +0200, Daniel Mack wrote:
> Make use of the newly added MIDI function in f_midi.c and strip down
> the MIDI gadget code radically. Also use the generic framework function
> to avoid code duplication and rename some symbols to bring them in sync
> with other code in the gadget framework.
> 
> Signed-off-by: Daniel Mack <zonque@xxxxxxxxx>
> Cc: Felipe Balbi <balbi@xxxxxx>
> Cc: Greg Kroah-Hartman <gregkh@xxxxxxx>
> Cc: Ben Williamson <ben.williamson@xxxxxxxxxxxxxxxxxx>
> ---
>  drivers/usb/gadget/midi.c | 1286 +++------------------------------------------
>  1 files changed, 80 insertions(+), 1206 deletions(-)
> 
> diff --git a/drivers/usb/gadget/midi.c b/drivers/usb/gadget/midi.c
> index 8b9220e..b3d93ac 100644
> --- a/drivers/usb/gadget/midi.c
> +++ b/drivers/usb/gadget/midi.c
> @@ -36,134 +36,35 @@
>  
>  #include "gadget_chips.h"
>  
> -
> -/*
> - * Kbuild is not very cooperative with respect to linking separately
> - * compiled library objects into one module.  So for now we won't use
> - * separate compilation ... ensuring init/exit sections work to shrink
> - * the runtime footprint, and giving us at least some parts of what
> - * a "gcc --combine ... part1.c part2.c part3.c ... " build would.
> - */
> +#include "composite.c"
>  #include "usbstring.c"
>  #include "config.c"
>  #include "epautoconf.c"
> +#include "f_midi.c"
>  
>  /*-------------------------------------------------------------------------*/
>  
> -
> -MODULE_AUTHOR("Ben Williamson");
> +MODULE_AUTHOR("Ben Williamson, Daniel Mack");

two entries. BTW, if your just moving code around, how can you be the
author ?? Similarly to previous patch.

-- 
balbi

Attachment: signature.asc
Description: Digital signature


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux