On 2025-09-02 11:57, Paolo Abeni wrote:

On 9/2/25 11:33 AM, Brett Sheffield wrote:

quoted

On 2025-09-02 10:49, Paolo Abeni wrote:

quoted

On 8/28/25 1:42 PM, Oscar Maes wrote:

quoted

Add test to check the broadcast ethernet destination field is set
correctly.

This test sends a broadcast ping, captures it using tcpdump and
ensures that all bits of the 6 octet ethernet destination address
are correctly set by examining the output capture file.

Signed-off-by: Oscar Maes <redacted>
Co-authored-by: Brett A C Sheffield [off-list ref]

I'm sorry for nit-picking, but the sob/tag-chain is wrong, please have a
look at:

https://elixir.bootlin.com/linux/v6.16.4/source/Documentation/process/submitting-patches.rst#L516

Thanks Paolo. So, something like:

Co-developed-by: Brett A C Sheffield <redacted>
Signed-off-by: Brett A C Sheffield <redacted>
Co-developed-by: Oscar Maes <redacted>
Signed-off-by: Oscar Maes <redacted>

with the last sign-off by Oscar because he is submitting?

Actually my understanding is:

Co-developed-by: Brett A C Sheffield <redacted>
Signed-off-by: Brett A C Sheffield <redacted>
Signed-off-by: Oscar Maes <redacted>

(if the patch is submitted by Oscar.) Basically the first examples in
the doc, with the only differences that such examples lists 3 co-developers.

Ah yes, you are correct. I missed the "in addition to the author attributed by
the From: tag" bit.

Thanks again.


Brett