Re: [for-next V2 06/10] net/mlx5: Add interface to get reference to a UAR

On Mon, Jan 9, 2017 at 5:47 PM, David Miller [off-list ref] wrote:

From: Saeed Mahameed <redacted>
Date: Mon, 9 Jan 2017 10:31:36 +0200

quoted

We will submit an incremental patch for this, as checkpatch doesn't
complain about such minor things.

Please fix this and resubmit the series.

Sure, will do this.

Checkpatch not complaining is not an argument for fixing up coding
style issues reported to you in feedback.

I just thought that this pull request is sitting in the mailing list
for too long and we need to move on.
Such minor issues can be fixed in incremental patches that doesn't
block net-next and rdma submissions.

Anyway will fix and submit v3.

thanks,
Saeed.
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

From: Saeed Mahameed <redacted>
Date: Mon, 9 Jan 2017 18:05:16 +0200

I just thought that this pull request is sitting in the mailing list
for too long and we need to move on.

Nobody reviewed it for any element of it's substance, and we had an
intervening weekend.

It did not sit for too long but any measure whatsoever.