Re: [EXT] Re: [PATCH] include: mtd: spi-nor: change HWCPAS to HWCAPS

From: Tudor.Ambarus
Date: Fri Feb 08 2019 - 07:27:56 EST

On 02/08/2019 01:39 PM, Bean Huo (beanhuo) wrote:
> Hi, Tudor
>> please update your remote and send a new patch.
> Which branch do you mean? I checked linux-mtd, not renamed yet.


>> The patch prefix is wrong, it should be prefixed with "mtd: spi-nor:". When in
>> doubt, you can check how were prefixed previous patches on the same file by
>> running something like:
>> git log --oneline include/linux/mtd/spi-nor.h
>> On 02/07/2019 03:46 PM, Bean Huo (beanhuo) wrote:
>> This looks like a blank line, it's not needed, remove it please.
>>> Maybe this is wrong abbreviation, change from HWCPAS to HWCAPS.
>> Indeed, it is, use imperative, something like: "Fix wrong abbreviation,
>>> Signed-off-by: Bean Huo <beanhuo@xxxxxxxxxx>
>> Your S-o-b tag doesn't match the author name, please fix this as well.
> It is automatically added by git, what is correct way?

Your author name "Bean Huo (beanhuo)" is different than your S-o-b name, "Bean
Huo". Did you add your S-o-b tag by hand? The author name is automatically added
by git, it looks into your ~/.gitconfig file, at [user] name. S-o-b tag can also
be added by git, just use the "-s" option when committing the change: "git
commit -s".