On Mon, Nov 10, 2014 at 06:09:41PM -0800, Stephanie Wallick wrote:
> Signed-off-by: Sean O. Stalley <sean.stal...@intel.com>
> Signed-off-by: Stephanie Wallick <stephanie.s.wall...@intel.com>

No changelog entry?


> ---
>  MAINTAINERS                            |  7 +++++++
>  drivers/staging/Kconfig                |  2 ++
>  drivers/staging/Makefile               |  1 +
>  drivers/staging/mausb/Kconfig          | 16 ++++++++++++++++
>  drivers/staging/mausb/Makefile         |  2 ++
>  drivers/staging/mausb/TODO             |  5 +++++
>  drivers/staging/mausb/drivers/Kconfig  | 34 
> ++++++++++++++++++++++++++++++++++
>  drivers/staging/mausb/drivers/Makefile | 18 ++++++++++++++++++
>  8 files changed, 85 insertions(+)
>  create mode 100644 drivers/staging/mausb/Kconfig
>  create mode 100644 drivers/staging/mausb/Makefile
>  create mode 100644 drivers/staging/mausb/TODO
>  create mode 100644 drivers/staging/mausb/drivers/Kconfig
>  create mode 100644 drivers/staging/mausb/drivers/Makefile
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index c3cfa1b..bd52ec2 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -8721,6 +8721,13 @@ W:     http://www.lirc.org/
>  S:   Odd Fixes
>  F:   drivers/staging/media/lirc/
>  
> +STAGING - MEDIA AGNOSTIC USB DRIVERS
> +M:   Sean O. Stalley <sean.stal...@intel.com>
> +M:   Stephanie Wallick <stephanie.s.wall...@intel.com>
> +L:   linux-...@vger.kernel.org
> +S:   Maintained
> +F:   drivers/staging/mausb
> +
>  STAGING - NVIDIA COMPLIANT EMBEDDED CONTROLLER INTERFACE (nvec)
>  M:   Julian Andres Klode <j...@jak-linux.org>
>  M:   Marc Dietrich <marvi...@gmx.de>
> diff --git a/drivers/staging/Kconfig b/drivers/staging/Kconfig
> index 35b494f..f57621b 100644
> --- a/drivers/staging/Kconfig
> +++ b/drivers/staging/Kconfig
> @@ -24,6 +24,8 @@ menuconfig STAGING
>  
>  if STAGING
>  
> +source "drivers/staging/mausb/Kconfig"
> +
>  source "drivers/staging/et131x/Kconfig"
>  
>  source "drivers/staging/slicoss/Kconfig"

Please put yourself at the end, not the top of this file.
> --- /dev/null
> +++ b/drivers/staging/mausb/TODO
> @@ -0,0 +1,5 @@
> +TODO:
> +     - checkpatch.pl cleanups
> +     - address miscellaneous "TODO" statements in code
> +     - add support for multiple media agnostic (MA) devices
> +     - add/improve support for unimplemented packet types

What about the other comments you already received such as:
        - unify with usbip



> diff --git a/drivers/staging/mausb/drivers/Kconfig 
> b/drivers/staging/mausb/drivers/Kconfig
> new file mode 100644
> index 0000000..9e12e22
> --- /dev/null
> +++ b/drivers/staging/mausb/drivers/Kconfig
> @@ -0,0 +1,34 @@
> +config MA_CORE
> +     tristate "MA USB core"
> +     ---help---
> +       This builds ma_core module.
> +
> +config MAUSB_HOST
> +     tristate "MA USB host"
> +     depends on MA_CORE
> +     ---help---
> +       This builds MA USB host driver module.
> +
> +config MAUSB_DEVICE
> +     tristate "MA USB device"
> +     depends on MA_CORE && USB_GADGET
> +     ---help---
> +       This builds MA USB device driver module.
> +
> +config MATCP_CORE
> +     tristate "MA USB tcp core"
> +     ---help---
> +       This builds tcp_core module.
> +
> +config MATCP_HOST
> +     tristate "MA USB host tcp"
> +     depends on MATCP_CORE
> +     ---help---
> +       This builds tcp_host module.
> +
> +config MATCP_DEVICE
> +     tristate "MA USB device tcp"
> +     depends on MATCP_CORE
> +     ---help---
> +       This builds tcp_dev module.
> +
> diff --git a/drivers/staging/mausb/drivers/Makefile 
> b/drivers/staging/mausb/drivers/Makefile
> new file mode 100644
> index 0000000..47f3222
> --- /dev/null
> +++ b/drivers/staging/mausb/drivers/Makefile
> @@ -0,0 +1,18 @@
> +obj-$(CONFIG_MA_CORE) += ma_core.o
> +ma_core-y := mausb_pkt.o mausb_tx.o mausb_msapi.o mausb_mem.o mausb_mgmt.o
> +
> +obj-$(CONFIG_MAUSB_HOST) += mausb.o
> +mausb-y := mausb_hcd.o mausb_hub.o mausb_tx-host.o mausb_mem-host.o
> +
> +obj-$(CONFIG_MAUSB_DEVICE) += maudc.o
> +maudc-y := mausb_udc.o mausb_tx-device.o
> +
> +obj-$(CONFIG_MATCP_HOST) += matcp_host.o
> +matcp_host-y := mausb_tcp-host.o
> +
> +obj-$(CONFIG_MATCP_DEVICE) += matcp_dev.o
> +matcp_dev-y := mausb_tcp-device.o
> +
> +obj-$(CONFIG_MATCP_CORE) += matcp_core.o
> +matcp_core-y := mausb_tcp.o mausb_ioctl.o

Why so many different modules?  Can't you merge most of these together
as you can't do anything with just a few of them alone.

thanks,

greg k-h
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to