On Wed, Nov 11, 2020 at 09:17:37AM -0800, Dan Williams wrote:
> > > +config CXL_MEM
> > > + tristate "CXL.mem Device Support"
> > > + depends on PCI && CXL_BUS_PROVIDER != n
> >
> > depend on PCI && CXL_BUS_PROVIDER
> >
> > > + default m if CXL_BUS_PROVIDER
> >
> > Please don't set weird defaults for new code. Especially not default
> > to module crap like this.
>
> This goes back to what people like Dave C. asked for LIBNVDIMM / DAX,
> a way to blanket turn on a subsystem without needing to go hunt down
> individual configs.
Then at least do a
default CXL_BUS_PROVIDER
but we really don't do this elsewhere. E.g. we don't default the scsi
disk driver on if there is some host adapter selected.
> > > +MODULE_AUTHOR("Intel Corporation");
> >
> > A module author is not a company.
>
> At least I don't have a copyright assignment clause, I don't agree
> with the vanity of listing multiple people here especially when
> MAINTAINERS has the contact info, and I don't want to maintain a list
> as people do drive-by contributions and we need to figure out at what
> level of contribution mandates a new MODULE_AUTHOR line. Now, that
> said I would be ok to duplicate the MAINTAINERS as MODULE_AUTHOR
> lines, but I otherwise expect MAINTAINERS is the central source for
> module contact info.
IMHO MODULE_AUTHOR is completely pointless. I haven't used for ~15
years. Especially as the concept that a module has a single author
is a rather strange one.
