Thread (12 messages) flat view 12 messages, 3 authors, 2016-08-16

RE: [RFC PATCH v5 3/3] net: phy: Add gmiitorgmii converter support

From: Appana Durga Kedareswara Rao <hidden>
Date: 2016-08-16 06:30:13
Also in: linux-arm-kernel, linux-devicetree, lkml

Hi Andrew,
quoted
Signed-off-by: Kedareswara rao Appana <redacted>
---
Thanks a lot Andrew for your inputs.
Changes for v5:
--> Fixed return values in the probe as suggested by punnaiah.
--> Added a mask for the converter speed as suggested by punnaiah.
+/* Xilinx GMII2RGMII Converter driver
+ *
+ * Copyright (C) 2016 Xilinx, Inc.
+ *
+ * Author: Kedareswara rao Appana [off-list ref]
Not cool

https://github.com/lunn/linux/commit/03d375489ceb56e171056f44d0fe9c34ca
5a098e

Notice the

Not-Signed-off-by: Andrew Lunn [off-list ref]

and the Copyright i added?

In various emails i gave you the basic idea how this should be done, a framework
of code, and then took your code which did not even compile and made the core
of it work. This is as much my code as your code, so i expect my Copyright to be
kept on this code. And since this is my code, i need to give a Signed-off-by,
which i was not yet ready to give since the code was not complete or working.

Please put both my Copyright back and Not-Signed-Off-by back and post the
next version as RFC. Once i'm happy this code is O.K, i will give you a Signed-off-
by.
Sorry I should have been included your copyright.

I missed it...
I didn't included your SOF because.
AFAIK without explicit permission of the other person we shouldn't include SOF.
That's why I didn't included your SOF and thanked for your efforts after SOF.
Sorry if my understanding is wrong.
 
There are few comments for this patch series and I fixed it and sent the v6 version.
 V6 version of patch series already got merged on the net-next.

Sent the patch by updating the driver with your copy right 
"net: phy: Update copyright info".

Sorry for the noise will fix these type of issues next time onwards...

Regards,
Kedar.

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help