Re: [PATCH] Staging: android: ashmem: changed struct file_operations to const file_operations

From: Greg KH
Date: Fri Oct 16 2020 - 02:19:23 EST


A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?
A: Top-posting.
Q: What is the most annoying thing in e-mail?

A: No.
Q: Should I include quotations after my reply?

http://daringfireball.net/2007/07/on_top

On Fri, Oct 16, 2020 at 01:38:47AM -0400, Kiran Suren wrote:
> Hi,
>
> My apologies about the last couple emails, I test built completely the
> wrong file. This is my first attempt at doing patch so I’m still trying to
> get a hang of how all this works. Also, I’m not sure what you mean by the
> signed off line? I believe I did put my actual full name? Again, apologies
> for any inconveniences just trying to get a hang of how the entire process
> works.

For the signed-off-by line, look at your email below:

>
> On Fri, Oct 16, 2020 at 1:12 AM kiransuren <kirank.suren@xxxxxxxxx> wrote:
>
> > From: kiransuren <kirank.suren@xxxxxxxxx>

That has one name and then:

> >
> > Fixed a coding style issue.
> >
> > Signed-off-by: Kiran Surendran <kirank.suren@xxxxxxxxx>

This has another.

They have to match, otherwise it doesn't make any sense, right?

thanks,

greg k-h