MA-USB Host driver provides USB connectivity over an available
network, allowing host device to access remote USB devices attached
to one or more MA USB devices (accessible via network).
This driver has been developed to enable the host to communicate
with DisplayLink products supporting MA USB protocol (MA USB device,
in terms of MA USB Specification).
MA-USB protocol used by MA-USB Host driver has been implemented in
accordance with MA USB Specification Release 1.0b.
This driver depends on the functions provided by DisplayLink's
user-space driver.
Signed-off-by: Vladimir Stankovic <[email protected]>
---
drivers/usb/Kconfig | 2 +
drivers/usb/Makefile | 2 +
drivers/usb/mausb_host/Kconfig | 15 +
drivers/usb/mausb_host/Makefile | 30 +
.../usb/mausb_host/include/common/ma_usb.h | 873 +++++++++
.../mausb_host/include/common/mausb_address.h | 38 +
.../include/common/mausb_driver_status.h | 21 +
.../mausb_host/include/common/mausb_event.h | 228 +++
drivers/usb/mausb_host/include/hcd/hub.h | 115 ++
drivers/usb/mausb_host/include/hcd/vhcd.h | 79 +
.../usb/mausb_host/include/hpal/data_common.h | 66 +
drivers/usb/mausb_host/include/hpal/data_in.h | 18 +
.../usb/mausb_host/include/hpal/data_out.h | 22 +
drivers/usb/mausb_host/include/hpal/hpal.h | 184 ++
.../usb/mausb_host/include/hpal/isoch_in.h | 22 +
.../usb/mausb_host/include/hpal/isoch_out.h | 20 +
.../mausb_host/include/hpal/mausb_events.h | 99 +
.../include/hpal/network_callbacks.h | 20 +
.../mausb_host/include/link/mausb_ip_link.h | 87 +
.../include/utils/mausb_data_iterator.h | 52 +
.../usb/mausb_host/include/utils/mausb_logs.h | 35 +
.../usb/mausb_host/include/utils/mausb_mmap.h | 21 +
.../include/utils/mausb_ring_buffer.h | 67 +
.../mausb_host/include/utils/mausb_version.h | 17 +
drivers/usb/mausb_host/src/hcd/hub.c | 1684 +++++++++++++++++
drivers/usb/mausb_host/src/hcd/module_init.c | 219 +++
drivers/usb/mausb_host/src/hcd/vhcd.c | 291 +++
drivers/usb/mausb_host/src/hpal/data_common.c | 141 ++
drivers/usb/mausb_host/src/hpal/data_in.c | 73 +
drivers/usb/mausb_host/src/hpal/data_out.c | 202 ++
drivers/usb/mausb_host/src/hpal/hpal.c | 1306 +++++++++++++
drivers/usb/mausb_host/src/hpal/isoch_in.c | 241 +++
drivers/usb/mausb_host/src/hpal/isoch_out.c | 361 ++++
.../usb/mausb_host/src/hpal/mausb_events.c | 659 +++++++
.../mausb_host/src/hpal/network_callbacks.c | 160 ++
.../usb/mausb_host/src/link/mausb_ip_link.c | 354 ++++
.../src/utils/mausb_data_iterator.c | 295 +++
drivers/usb/mausb_host/src/utils/mausb_mmap.c | 381 ++++
.../mausb_host/src/utils/mausb_ring_buffer.c | 168 ++
39 files changed, 8668 insertions(+)
create mode 100644 drivers/usb/mausb_host/Kconfig
create mode 100644 drivers/usb/mausb_host/Makefile
create mode 100644 drivers/usb/mausb_host/include/common/ma_usb.h
create mode 100644 drivers/usb/mausb_host/include/common/mausb_address.h
create mode 100644
drivers/usb/mausb_host/include/common/mausb_driver_status.h
create mode 100644 drivers/usb/mausb_host/include/common/mausb_event.h
create mode 100644 drivers/usb/mausb_host/include/hcd/hub.h
create mode 100644 drivers/usb/mausb_host/include/hcd/vhcd.h
create mode 100644 drivers/usb/mausb_host/include/hpal/data_common.h
create mode 100644 drivers/usb/mausb_host/include/hpal/data_in.h
create mode 100644 drivers/usb/mausb_host/include/hpal/data_out.h
create mode 100644 drivers/usb/mausb_host/include/hpal/hpal.h
create mode 100644 drivers/usb/mausb_host/include/hpal/isoch_in.h
create mode 100644 drivers/usb/mausb_host/include/hpal/isoch_out.h
create mode 100644 drivers/usb/mausb_host/include/hpal/mausb_events.h
create mode 100644 drivers/usb/mausb_host/include/hpal/network_callbacks.h
create mode 100644 drivers/usb/mausb_host/include/link/mausb_ip_link.h
create mode 100644
drivers/usb/mausb_host/include/utils/mausb_data_iterator.h
create mode 100644 drivers/usb/mausb_host/include/utils/mausb_logs.h
create mode 100644 drivers/usb/mausb_host/include/utils/mausb_mmap.h
create mode 100644
drivers/usb/mausb_host/include/utils/mausb_ring_buffer.h
create mode 100644 drivers/usb/mausb_host/include/utils/mausb_version.h
create mode 100644 drivers/usb/mausb_host/src/hcd/hub.c
create mode 100644 drivers/usb/mausb_host/src/hcd/module_init.c
create mode 100644 drivers/usb/mausb_host/src/hcd/vhcd.c
create mode 100644 drivers/usb/mausb_host/src/hpal/data_common.c
create mode 100644 drivers/usb/mausb_host/src/hpal/data_in.c
create mode 100644 drivers/usb/mausb_host/src/hpal/data_out.c
create mode 100644 drivers/usb/mausb_host/src/hpal/hpal.c
create mode 100644 drivers/usb/mausb_host/src/hpal/isoch_in.c
create mode 100644 drivers/usb/mausb_host/src/hpal/isoch_out.c
create mode 100644 drivers/usb/mausb_host/src/hpal/mausb_events.c
create mode 100644 drivers/usb/mausb_host/src/hpal/network_callbacks.c
create mode 100644 drivers/usb/mausb_host/src/link/mausb_ip_link.c
create mode 100644 drivers/usb/mausb_host/src/utils/mausb_data_iterator.c
create mode 100644 drivers/usb/mausb_host/src/utils/mausb_mmap.c
create mode 100644 drivers/usb/mausb_host/src/utils/mausb_ring_buffer.c
On Tue, Feb 11, 2020 at 04:21:24PM +0000, Vladimir Stankovic wrote:
> 39 files changed, 8668 insertions(+)
This is a bit hard, if not impossible, to review all in one huge patch.
Can you resend this as a patch series, breaking it down into logical
chunks, like all other kernel patches have?
Also, why so many individual files? For only 8k lines, 39 files seems
like a huge number.
thanks,
greg k-h
On Tue, Feb 11, 2020 at 04:21:24PM +0000, Vladimir Stankovic wrote:
> MA-USB Host driver provides USB connectivity over an available
> network, allowing host device to access remote USB devices attached
> to one or more MA USB devices (accessible via network).
>
> This driver has been developed to enable the host to communicate
> with DisplayLink products supporting MA USB protocol (MA USB device,
> in terms of MA USB Specification).
>
> MA-USB protocol used by MA-USB Host driver has been implemented in
> accordance with MA USB Specification Release 1.0b.
>
> This driver depends on the functions provided by DisplayLink's
> user-space driver.
>
> Signed-off-by: Vladimir Stankovic <[email protected]>
> ---
> drivers/usb/Kconfig | 2 +
> drivers/usb/Makefile | 2 +
> drivers/usb/mausb_host/Kconfig | 15 +
> drivers/usb/mausb_host/Makefile | 30 +
> .../usb/mausb_host/include/common/ma_usb.h | 873 +++++++++
> .../mausb_host/include/common/mausb_address.h | 38 +
> .../include/common/mausb_driver_status.h | 21 +
> .../mausb_host/include/common/mausb_event.h | 228 +++
> drivers/usb/mausb_host/include/hcd/hub.h | 115 ++
> drivers/usb/mausb_host/include/hcd/vhcd.h | 79 +
> .../usb/mausb_host/include/hpal/data_common.h | 66 +
> drivers/usb/mausb_host/include/hpal/data_in.h | 18 +
> .../usb/mausb_host/include/hpal/data_out.h | 22 +
> drivers/usb/mausb_host/include/hpal/hpal.h | 184 ++
> .../usb/mausb_host/include/hpal/isoch_in.h | 22 +
> .../usb/mausb_host/include/hpal/isoch_out.h | 20 +
> .../mausb_host/include/hpal/mausb_events.h | 99 +
> .../include/hpal/network_callbacks.h | 20 +
> .../mausb_host/include/link/mausb_ip_link.h | 87 +
> .../include/utils/mausb_data_iterator.h | 52 +
> .../usb/mausb_host/include/utils/mausb_logs.h | 35 +
> .../usb/mausb_host/include/utils/mausb_mmap.h | 21 +
> .../include/utils/mausb_ring_buffer.h | 67 +
> .../mausb_host/include/utils/mausb_version.h | 17 +
> drivers/usb/mausb_host/src/hcd/hub.c | 1684 +++++++++++++++++
> drivers/usb/mausb_host/src/hcd/module_init.c | 219 +++
> drivers/usb/mausb_host/src/hcd/vhcd.c | 291 +++
> drivers/usb/mausb_host/src/hpal/data_common.c | 141 ++
> drivers/usb/mausb_host/src/hpal/data_in.c | 73 +
> drivers/usb/mausb_host/src/hpal/data_out.c | 202 ++
> drivers/usb/mausb_host/src/hpal/hpal.c | 1306 +++++++++++++
> drivers/usb/mausb_host/src/hpal/isoch_in.c | 241 +++
> drivers/usb/mausb_host/src/hpal/isoch_out.c | 361 ++++
> .../usb/mausb_host/src/hpal/mausb_events.c | 659 +++++++
> .../mausb_host/src/hpal/network_callbacks.c | 160 ++
> .../usb/mausb_host/src/link/mausb_ip_link.c | 354 ++++
> .../src/utils/mausb_data_iterator.c | 295 +++
> drivers/usb/mausb_host/src/utils/mausb_mmap.c | 381 ++++
> .../mausb_host/src/utils/mausb_ring_buffer.c | 168 ++
> 39 files changed, 8668 insertions(+)
> create mode 100644 drivers/usb/mausb_host/Kconfig
> create mode 100644 drivers/usb/mausb_host/Makefile
> create mode 100644 drivers/usb/mausb_host/include/common/ma_usb.h
> create mode 100644 drivers/usb/mausb_host/include/common/mausb_address.h
> create mode 100644
> drivers/usb/mausb_host/include/common/mausb_driver_status.h
> create mode 100644 drivers/usb/mausb_host/include/common/mausb_event.h
> create mode 100644 drivers/usb/mausb_host/include/hcd/hub.h
> create mode 100644 drivers/usb/mausb_host/include/hcd/vhcd.h
> create mode 100644 drivers/usb/mausb_host/include/hpal/data_common.h
> create mode 100644 drivers/usb/mausb_host/include/hpal/data_in.h
> create mode 100644 drivers/usb/mausb_host/include/hpal/data_out.h
> create mode 100644 drivers/usb/mausb_host/include/hpal/hpal.h
> create mode 100644 drivers/usb/mausb_host/include/hpal/isoch_in.h
> create mode 100644 drivers/usb/mausb_host/include/hpal/isoch_out.h
> create mode 100644 drivers/usb/mausb_host/include/hpal/mausb_events.h
> create mode 100644 drivers/usb/mausb_host/include/hpal/network_callbacks.h
> create mode 100644 drivers/usb/mausb_host/include/link/mausb_ip_link.h
> create mode 100644
> drivers/usb/mausb_host/include/utils/mausb_data_iterator.h
> create mode 100644 drivers/usb/mausb_host/include/utils/mausb_logs.h
> create mode 100644 drivers/usb/mausb_host/include/utils/mausb_mmap.h
> create mode 100644
> drivers/usb/mausb_host/include/utils/mausb_ring_buffer.h
> create mode 100644 drivers/usb/mausb_host/include/utils/mausb_version.h
> create mode 100644 drivers/usb/mausb_host/src/hcd/hub.c
> create mode 100644 drivers/usb/mausb_host/src/hcd/module_init.c
> create mode 100644 drivers/usb/mausb_host/src/hcd/vhcd.c
> create mode 100644 drivers/usb/mausb_host/src/hpal/data_common.c
> create mode 100644 drivers/usb/mausb_host/src/hpal/data_in.c
> create mode 100644 drivers/usb/mausb_host/src/hpal/data_out.c
> create mode 100644 drivers/usb/mausb_host/src/hpal/hpal.c
> create mode 100644 drivers/usb/mausb_host/src/hpal/isoch_in.c
> create mode 100644 drivers/usb/mausb_host/src/hpal/isoch_out.c
> create mode 100644 drivers/usb/mausb_host/src/hpal/mausb_events.c
> create mode 100644 drivers/usb/mausb_host/src/hpal/network_callbacks.c
> create mode 100644 drivers/usb/mausb_host/src/link/mausb_ip_link.c
> create mode 100644 drivers/usb/mausb_host/src/utils/mausb_data_iterator.c
> create mode 100644 drivers/usb/mausb_host/src/utils/mausb_mmap.c
> create mode 100644 drivers/usb/mausb_host/src/utils/mausb_ring_buffer.c
>
>
> From 0324b3a49d0ef41480a0a42ab27f70edeb007980 Mon Sep 17 00:00:00 2001
> From: Vladimir Stankovic <[email protected]>
> Date: Tue, 11 Feb 2020 16:03:43 +0100
> Subject: [PATCH] usb: Add MA USB Host driver
>
> MA-USB Host driver provides USB connectivity over an available
> network, allowing host device to access remote USB devices attached
> to one or more MA USB devices (accessible via network).
<snip>
You attached the patch, as well as put the description above in the
email as well, so I couldn't apply this even if I wanted to :(
Please just submit the patch all inline, like all other kernel patches
are.
thanks,
greg k-h
On Tue, Feb 11, 2020 at 04:21:24PM +0000, Vladimir Stankovic wrote:
> +++ b/drivers/usb/mausb_host/include/common/ma_usb.h
> @@ -0,0 +1,873 @@
> +/* SPDX-License-Identifier: GPL-2.0 */
Nice.
> +/*
> + * Copyright (c) 2019 - 2020 DisplayLink (UK) Ltd.
> + *
> + * This file is subject to the terms and conditions of the GNU General Public
> + * License v2. See the file COPYING in the main directory of this archive for
> + * more details.
Note, COPYING does not say what you think it says here :)
Please remove all "license boilerplate" from all of these files, with
the use of SPDX you do not need it, and, as is the case here, it can
often be wrong.
thanks,
greg k-h
On 11.2.20. 17:39, [email protected] wrote:
> On Tue, Feb 11, 2020 at 04:21:24PM +0000, Vladimir Stankovic wrote:
>> 39 files changed, 8668 insertions(+)
>
> This is a bit hard, if not impossible, to review all in one huge patch.
>
> Can you resend this as a patch series, breaking it down into logical
> chunks, like all other kernel patches have?
>
> Also, why so many individual files? For only 8k lines, 39 files seems
> like a huge number.
>
> thanks,
>
> greg k-h
>
Will break it down into patch series and resend.
In regards of the file count, our intention was to ease the
troubleshooting efforts during development and have a clear separation
between logical parts of MA-USB implementation (i.e data in/out, isoch
in/out, etc.; each source file representing logical chunk).
Regards,
Vladimir.
On 11.2.20. 17:41, [email protected] wrote:
> On Tue, Feb 11, 2020 at 04:21:24PM +0000, Vladimir Stankovic wrote:
>> +++ b/drivers/usb/mausb_host/include/common/ma_usb.h
>> @@ -0,0 +1,873 @@
>> +/* SPDX-License-Identifier: GPL-2.0 */
>
> Nice.
>
>> +/*
>> + * Copyright (c) 2019 - 2020 DisplayLink (UK) Ltd.
>> + *
>> + * This file is subject to the terms and conditions of the GNU General Public
>> + * License v2. See the file COPYING in the main directory of this archive for
>> + * more details.
>
> Note, COPYING does not say what you think it says here :)
>
> Please remove all "license boilerplate" from all of these files, with
> the use of SPDX you do not need it, and, as is the case here, it can
> often be wrong.
>
> thanks,
>
> greg k-h
>
Agree. Will address.
Regards,
Vladimir.
On Wed, Feb 12, 2020 at 09:41:12AM +0000, Vladimir Stankovic wrote:
> On 11.2.20. 17:39, [email protected] wrote:
> > On Tue, Feb 11, 2020 at 04:21:24PM +0000, Vladimir Stankovic wrote:
> >> 39 files changed, 8668 insertions(+)
> >
> > This is a bit hard, if not impossible, to review all in one huge patch.
> >
> > Can you resend this as a patch series, breaking it down into logical
> > chunks, like all other kernel patches have?
> >
> > Also, why so many individual files? For only 8k lines, 39 files seems
> > like a huge number.
> >
> > thanks,
> >
> > greg k-h
> >
> Will break it down into patch series and resend.
>
> In regards of the file count, our intention was to ease the
> troubleshooting efforts during development and have a clear separation
> between logical parts of MA-USB implementation (i.e data in/out, isoch
> in/out, etc.; each source file representing logical chunk).
Yeah, but at the very least, only have a single .h "internal" file, you
have whole files for just 1 or 2 function prototypes, that's totally
overkill.
Simple is good, you don't want to have to bounce around through multiple
files for a simple 8k line driver, that's not a good idea.
thanks,
greg k-h