RE: [PATCH 09/15] Introduce Cadence USBSSP DRD Driver - added debugfs files.

From: Pawel Laszczak
Date: Thu Jul 05 2018 - 05:44:37 EST


> > Signed-off-by: Laszczak Pawel <pawell@xxxxxxxxxxx>
> > ---
>
> Again for all of these, you need a changelog text.
>
> > drivers/usb/usbssp/gadget-debugfs.c | 514
> ++++++++++++++++++++++++++++
> > drivers/usb/usbssp/gadget-debugfs.h | 137 ++++++++
> > 2 files changed, 651 insertions(+)
> > create mode 100644 drivers/usb/usbssp/gadget-debugfs.c
> > create mode 100644 drivers/usb/usbssp/gadget-debugfs.h
> >
> > diff --git a/drivers/usb/usbssp/gadget-debugfs.c
> b/drivers/usb/usbssp/gadget-debugfs.c
> > new file mode 100644
> > index 000000000000..c0d1e96728e6
> > --- /dev/null
> > +++ b/drivers/usb/usbssp/gadget-debugfs.c
> > @@ -0,0 +1,514 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > +/*
> > + * USBSSP device controller driver
> > + *
> > + * Copyright (C) 2018 Cadence.
> > + *
> > + * Author: Pawel Laszczak
> > + * Some code borrowed from the Linux XHCI driver.
>
> What code was borrowed? Please acknowledge that copyright.

USBSSP controller is very similar to XHCI controller and
is mostly compliance with XHCI specification.
In result, driver for device mode based on
drivers/usb/host/xhci* driver.

Especially xhci.c, xhci-ring.c, xhci-mem.c , xhci-debugfs.c
and xhci-trace.h was re-used. By borrowed I means that these
files have been taken from xhci driver and have been adopted
in such way that it can be use for the device side.
Probably this sentence should looks like:
" A lot of code based on Linux XHCI driver" ?

You know, the most of algorithm for handling transfer and
memory management are the same as for XHCI driver.

Because the USB gadget API in kernel completely different
from Host USB API, so files had to been changed very much.

What do you think by "Please acknowledge that copyright" ?

"
The standard header for xhci driver looks like:
// SPDX-License-Identifier: GPL-2.0
/*
* xHCI host controller driver PCI Bus Glue.
*
* Copyright (C) 2008 Intel Corp.
*
* Author: Sarah Sharp
* Some code borrowed from the Linux EHCI driver.
*/
"
Should I add In my header:
Copyright (C) 2018 Cadence.
Copyright (C) 2008 Intel Corp.

or something like this:
....
A lot of code based on Linux XHCI driver
Origin: Copyright (C) 2008 Intel Corp

Thanks very much for all advice and comments.
Pawel Laszczak