From: Florian Fainelli <f.fainelli@gmail.com>
To: "Scott Branden" <scott.branden@broadcom.com>,
"Clément Péron" <peron.clem@gmail.com>,
devicetree@vger.kernel.org
Cc: "Rob Herring" <robh+dt@kernel.org>,
"Mark Rutland" <mark.rutland@arm.com>,
"Russell King" <linux@armlinux.org.uk>,
"Ray Jui" <rjui@broadcom.com>,
"Scott Branden" <sbranden@broadcom.com>,
"Jon Mason" <jonmason@broadcom.com>,
bcm-kernel-feedback-list@broadcom.com,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org,
"Clément Peron" <clement.peron@devialet.com>
Subject: Re: [PATCH] ARM: dts: cygnus: Add HWRNG node
Date: Wed, 6 Jun 2018 09:47:12 -0700 [thread overview]
Message-ID: <c7990001-00e6-d828-bafd-3d472fcfee20@gmail.com> (raw)
In-Reply-To: <c8836130-1d96-b813-4e87-5011db679ba8@broadcom.com>
On 06/06/2018 09:03 AM, Scott Branden wrote:
> Hi Clement,
>
>
> On 18-06-06 02:34 AM, Clément Péron wrote:
>> From: Clément Peron <clement.peron@devialet.com>
>>
>> There is a HWRNG in Broadcom Cygnus SoC, so enable it.
>>
>> Signed-off-by: Clément Peron <clement.peron@devialet.com>
> Thanks for upstreaming some missing Cygnus components.
>
> But, the problem is the tarball release from Broadcom you are extracting
> these changes from does not contain git history; so, you are missing the
> original authors and signed-off's.
> I checked our internal git repository and for this commit the author is:
> Mohamed Ismail Abdul Packir Mohamed <mohamed-ismail.abdul@broadcom.com>
>
> Please adjust author and signed-off appropriately. If there are other
> changes you are extracting from the source tarballs you have please
> contact me so we can construct patch appropriately.
If you want the original author's Signed-off-by to be preserved, why
don't you extract it from your internal git tree and submit the patch on
Mohamed's behalf?
AFAICT what Clement is doing here is permissible given the Linux
developer certificate of origin though I am not a lawyer of course.
--
Florian
next prev parent reply other threads:[~2018-06-06 16:47 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-06-06 9:34 [PATCH] ARM: dts: cygnus: Add HWRNG node Clément Péron
2018-06-06 16:03 ` Scott Branden
2018-06-06 16:47 ` Florian Fainelli [this message]
2018-06-06 17:06 ` Clément Péron
2018-06-06 17:31 ` Scott Branden
2018-06-06 17:14 ` Scott Branden
2018-06-06 17:32 ` Florian Fainelli
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=c7990001-00e6-d828-bafd-3d472fcfee20@gmail.com \
--to=f.fainelli@gmail.com \
--cc=bcm-kernel-feedback-list@broadcom.com \
--cc=clement.peron@devialet.com \
--cc=devicetree@vger.kernel.org \
--cc=jonmason@broadcom.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@armlinux.org.uk \
--cc=mark.rutland@arm.com \
--cc=peron.clem@gmail.com \
--cc=rjui@broadcom.com \
--cc=robh+dt@kernel.org \
--cc=sbranden@broadcom.com \
--cc=scott.branden@broadcom.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).