Re: [tegrarcm PATCH v1 1/4] Add option "--pkc"

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 03/08/2016 05:50 PM, Jimmy Zhang wrote:
Stephen Warren wrote at Monday, March 07, 2016 11:56 AM:
On 03/04/2016 04:44 PM, Jimmy Zhang wrote:
Add the support code needed to sign the RCM messages with RSA-PSS as
needed to communicate with secured production devices. This mode is
enabled by passing a key via the --pkc command line argument. If such
a key is set the RCM messages will be signed with it as well as the
bootloader.

Signed-off-by: Alban Bedel <alban.bedel@...>

Part of that s-o-b line has been corrupted.

If Alban wrote this, there should be a "From:" line for Alban at the top of the
email. Check that "git log" locally shows Alban as the git author of the patch,
and "git format-patch" will do the right thing automatically.


I tried not making any changes on Alban's patch. Seems you are suggesting me to make minor changes.

You have changed Alban's patch, and I'm asking you to undo that. If you run "git log", you will see that the commit has your name as "Author:" whereas it should have Alban's name. Make sure you applied Alban's patch using "git am" not "patch", and everything will be correct automatically. You can fix this by doing a "git rebase -i" and re-applying Alban's patch.

Your s-o-b line is missing. It needs to be present even for patches you didn't
author, but are simply passing on.

Sure.

There's no need to reply to points you agree with; just make the change an include it in V2 along with a changelog entry for it.
--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux