linux-kernel - Re: [PATCH v3] Fix missing PCI ID; Necessary for Lenovo Yoga 720-15 Touchpad.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Wed, 16 Aug 2017 15:58:38 +1000
From:   Andrew Donnellan <andrew.donnellan@....ibm.com>
To:     Hoeze <git.ich@...oelzlwimmer.de>, lee.jones@...aro.org,
        linux-kernel@...r.kernel.org
Cc:     coolstarorganization@...il.com, mykesx@...il.com, bjodah@...il.com,
        dja@...ens.net, Hoeze <Hoeze@...rs.noreply.github.com>
Subject: Re: [PATCH v3] Fix missing PCI ID; Necessary for Lenovo Yoga 720-15
 Touchpad.

Hi Florian,

As Daniel said, good work on figuring out how to get this far!

There's a couple of minor issues with the formatting of this patch.

Firstly, as Lee said in response to the last version, you should change 
the summary line of the commit to indicate the contents of the patch 
more accurately.

Based on what I see in the git log, I'd suggest:

	"mfd: intel-lpss: Add missing PCI ID"

The previous commits to this file also mention the type of chipset - I 
*think* this is Sunrise Point, but I don't work for Intel. :)

Don't mention the Lenovo touchpad in the summary - in the main body of 
the commit message is better.

On 13/08/17 06:16, Hoeze wrote:
> From: Hoeze <Hoeze@...rs.noreply.github.com>

The name and email here is what's going to appear in the git commit log 
as the author of the commit. Generally, it should be the same as both 
the details in your Signed-off-by line, and the email should be the same 
as the email address you're sending from. If the name/email address in 
the commit is the same as the name/email address you're sending from, 
git send-email will actually skip this line completely.

You should probably try:

	$ git config --global user.name "Florian R. Hölzlwimmer"
	$ git config --global user.email "git.ich@...oelzlwimmer.de"
	$ git commit --amend --reset-author

This will set your details in your git configuration, and then re-create 
the last commit using the correct authorship details.

If you then re-send it, it should skip this From: line and everything 
should be fine.

Andrew

-- 
Andrew Donnellan              OzLabs, ADL Canberra
andrew.donnellan@....ibm.com  IBM Australia Limited