On Sat, Aug 22, 2020 at 02:36:37AM +0200, Thomas Gleixner wrote:

CAUTION: This email originated from outside of the organization. Do not click links or open attachments unless you can confirm the sender and know the content is safe.



On Fri, Aug 21 2020 at 22:27, Thomas Gleixner wrote:

quoted

Add a new quirk flag IRQCHIP_SHUTDOWN_ON_SUSPEND and add support for
it the core interrupt suspend/resume paths.

Changelog:
v1->v2: Corrected the author's name to tglx@

Can you please move that Changelog part below the --- seperator next
time because that's really not part of the final commit messaage and the
maintainer has then to strip it off manually

Ack.

quoted

Signed-off-by: Anchal Agarwal <redacted>
Signed-off-by: Thomas Gleixner <redacted>

These SOB lines are just wrongly ordered as they suggest:

     Anchal has authored the patch and Thomas transported it

which is clearly not the case. So the right order is:

I must admit I wasn't aware of that. Will fix.

Signed-off-by: Thomas Gleixner <redacted>
Signed-off-by: Anchal Agarwal <redacted>

And that needs another tweak at the top of the change log. The first
line in the mail body wants to be:

From: Thomas Gleixner <redacted>

Yes I accidentally missed that in this patch.
Others have that line on all the patches and even v2 for this patch
has. Will fix.

followed by an empty new line before the actual changelog text
starts. That way the attribution of the patch when applying it will be
correct.

Documentation/process/ is there for a reason and following the few
simple rules to get that straight is not rocket science.

Thanks,

        tglx

Anchal