On Mon, Oct 7, 2024 at 12:26 PM Damien Le Moal <dlemoal@xxxxxxxxxx> wrote: > > On 10/7/24 19:12, Niklas Cassel wrote: > > On Mon, Oct 07, 2024 at 01:43:50PM +0900, Damien Le Moal wrote: > >> From: Damien Le Moal <damien.lemoal@xxxxxxxxxxxxxxxxxx> > > > > (snip) > > > >> Early versions of this driver code were based on an RFC submission by > >> Alan Mikhak <alan.mikhak@xxxxxxxxxx> (https://lwn.net/Articles/804369/). > >> The code however has since been completely rewritten. > > > > Here you state that the code has been completely rewritten... > > > > > >> > >> Co-developed-by: Rick Wertenbroek <rick.wertenbroek@xxxxxxxxx> > >> Signed-off-by: Rick Wertenbroek <rick.wertenbroek@xxxxxxxxx> > >> Signed-off-by: Damien Le Moal <damien.lemoal@xxxxxxxxxxxxxxxxxx> > >> --- > >> MAINTAINERS | 7 + > >> drivers/pci/endpoint/functions/Kconfig | 9 + > >> drivers/pci/endpoint/functions/Makefile | 1 + > >> drivers/pci/endpoint/functions/pci-epf-nvme.c | 2489 +++++++++++++++++ > >> 4 files changed, 2506 insertions(+) > >> create mode 100644 drivers/pci/endpoint/functions/pci-epf-nvme.c > > > > (snip) > > > >> --- /dev/null > >> +++ b/drivers/pci/endpoint/functions/pci-epf-nvme.c > >> @@ -0,0 +1,2489 @@ > >> +// SPDX-License-Identifier: GPL-2.0 > >> +/* > >> + * NVMe function driver for PCI Endpoint Framework > >> + * > >> + * Copyright (C) 2019 SiFive > > > > ...yet here you claim Copyright (C) SiFive. > > > > *If* the code has been completely rewritten, then you probably should > > put yourself and/or your current employeer as the copyright holder. > > Oops. One thing I forgot to rewrite :) > Will change the copyright in v2. Hello Damien. Thank you for acknowledging my contributions in the commit message. I would appreciate it if you could add both our names in the copyright for the v2, I don't mind not being in the MODULE_AUTHOR() but I would appreciate a mention in the header comment of the file, thank you. Best regards, Rick > > > > > > > Kind regards, > > Niklas > > > -- > Damien Le Moal > Western Digital Research