[RFC PATCH v5 3/3] net: phy: Add gmiitorgmii converter support
Appana Durga Kedareswara Rao
appana.durga.rao at xilinx.com
Mon Aug 15 23:30:00 PDT 2016
Hi Andrew,
>
> > Signed-off-by: Kedareswara rao Appana <appanad at xilinx.com>
> > ---
> > 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 <appanad at xilinx.com>
>
> Not cool
>
> https://github.com/lunn/linux/commit/03d375489ceb56e171056f44d0fe9c34ca
> 5a098e
>
> Notice the
>
> Not-Signed-off-by: Andrew Lunn <andrew at lunn.ch>
>
> 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.
More information about the linux-arm-kernel
mailing list