Re: [RFC PATCH v12 01/17] dlb: add skeleton for DLB driver

On Tue, Dec 21, 2021 at 02:42:10PM +0000, Chen, Mike Ximing wrote:

quoted

-----Original Message-----
From: Chen, Mike Ximing <redacted>
Sent: Tuesday, December 21, 2021 9:26 AM
To: Greg KH <gregkh@linuxfoundation.org>
Cc: linux-kernel@vger.kernel.org; arnd@arndb.de; Williams, Dan J <redacted>; pierre-
louis.bossart@linux.intel.com; netdev@vger.kernel.org; davem@davemloft.net; kuba@kernel.org
Subject: RE: [RFC PATCH v12 01/17] dlb: add skeleton for DLB driver

quoted

-----Original Message-----
From: Greg KH <gregkh@linuxfoundation.org>
Sent: Tuesday, December 21, 2021 3:57 AM
To: Chen, Mike Ximing <redacted>
Cc: linux-kernel@vger.kernel.org; arnd@arndb.de; Williams, Dan J
[off-list ref]; pierre- louis.bossart@linux.intel.com;
netdev@vger.kernel.org; davem@davemloft.net; kuba@kernel.org
Subject: Re: [RFC PATCH v12 01/17] dlb: add skeleton for DLB driver

On Tue, Dec 21, 2021 at 08:12:00AM +0100, Greg KH wrote:

quoted

On Tue, Dec 21, 2021 at 12:50:31AM -0600, Mike Ximing Chen wrote:

quoted

+/* Copyright(C) 2016-2020 Intel Corporation. All rights reserved.
+*/

So you did not touch this at all in 2021?  And it had a
copyrightable changed added to it for every year, inclusive, from 2016-2020?

Please run this past your lawyers on how to do this properly.

Ah, this was a "throw it over the fence at the community to handle for
me before I go on vacation" type of posting, based on your autoresponse email that happened when I

sent this.

quoted

That too isn't the most kind thing, would you want to be the reviewer
of this if it were sent to you?  Please take some time and start doing
patch reviews for the char/misc drivers on the mailing list before submitting any more new code.

Also, this patch series goes agains the internal rules that I know
your company has, why is that?  Those rules are there for a good
reason, and by ignoring them, it's going to make it much harder to get patches to be reviewed.

I assume that you referred to the "Reviewed-by" rule from Intel. Since this is a RFC and we are seeking for
comments and guidance on our code structure, we thought it was appropriate to send out patch set out
with a full endorsement from our internal reviewers. The questions I posted in the cover letter (patch
00/17) are from the discussions with our internal reviewers.

.
"we thought it was appropriate to send out the patch set out without* a full endorsement from our
Internal reviewers"  --- sorry for misspelling.

I think you mean something like "we had an internal deadline to meet so
we are willing to ignore our rules and throw it over the wall and let
the community worry about it."

Congratulations, I now feel like the hours I spent last week talking to
hundreds of Intel developers about this very topic were totally wasted.

{sigh}

Intel now owes me another bottle of liquor...

greg k-h