Re: [PATCH 30/32] staging: gasket: gasket_mmap use PAGE_MASK

From: Dan Carpenter
Date: Tue Jul 17 2018 - 06:45:09 EST


On Mon, Jul 16, 2018 at 07:09:24PM -0700, Todd Poynor wrote:
> From: Todd Poynor <toddpoynor@xxxxxxxxxx>
>
> Instead of math on PAGE_SIZE.
>

The commit message is sort of distinct from the first line. The first
line is like a title, not the start of a sentence. On like marc.info
when you're looking at patches, the Subject and the commit message are
sort of far apart so it looks weird. It looks weird on my email as
well.

Sorry, we're really hitting you with a bunch of nit-picky stuff. If you
were a one off contributer I'd just let a lot of this stuff slide but
since you're going to be a maintainer for a while then it's good to
learn all these micro rules at the begining.

regards,
dan carpenter