diff mbox

usb: phy: fix phy-fsl-usb.c's copyright issue

Message ID 1531808756-989-1-git-send-email-pengbo.mu@nxp.com (mailing list archive)
State New, archived
Headers show

Commit Message

Pengbo Mu July 17, 2018, 6:25 a.m. UTC
From: "yinbo.zhu" <yinbo.zhu@nxp.com>

Signed-off-by: yinbo.zhu <yinbo.zhu@nxp.com>
---
 drivers/usb/phy/phy-fsl-usb.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Greg Kroah-Hartman July 17, 2018, 6:40 a.m. UTC | #1
On Tue, Jul 17, 2018 at 02:25:56PM +0800, Pengbo Mu wrote:
> From: "yinbo.zhu" <yinbo.zhu@nxp.com>
> 
> Signed-off-by: yinbo.zhu <yinbo.zhu@nxp.com>
> ---
>  drivers/usb/phy/phy-fsl-usb.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/usb/phy/phy-fsl-usb.c b/drivers/usb/phy/phy-fsl-usb.c
> index 900875f..dcb531c 100644
> --- a/drivers/usb/phy/phy-fsl-usb.c
> +++ b/drivers/usb/phy/phy-fsl-usb.c
> @@ -1,6 +1,6 @@
>  // SPDX-License-Identifier: GPL-2.0+
>  /*
> - * Copyright (C) 2007,2008 Freescale semiconductor, Inc.
> + * Copyright 2007,2008 Freescale Semiconductor, Inc.

Huh?  WHat exact "issue" are you fixing here?  Be specific please.

And I can't take patches without any changelog text, sorry.

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Greg Kroah-Hartman July 17, 2018, 6:40 a.m. UTC | #2
On Tue, Jul 17, 2018 at 02:25:56PM +0800, Pengbo Mu wrote:
> From: "yinbo.zhu" <yinbo.zhu@nxp.com>
> 
> Signed-off-by: yinbo.zhu <yinbo.zhu@nxp.com>

Also, you need to sign off on it, _and_ we need a valid name here for
the author and sign-off.

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Pengbo Mu July 17, 2018, 7:16 a.m. UTC | #3
Hi Greg,

Sorry for missing the changelog text & sign .
BTW , it's my first send email to upstream. 
Thanks for your advices.

Regards,
Pengbo Mu

-----Original Message-----
From: Greg Kroah-Hartman [mailto:gregkh@linuxfoundation.org] 

Sent: 2018年7月17日 14:41
To: Pengbo Mu <pengbo.mu@nxp.com>
Cc: Felipe Balbi <balbi@kernel.org>; linux-kernel@vger.kernel.org; linux-usb@vger.kernel.org; Yinbo Zhu <yinbo.zhu@nxp.com>
Subject: Re: [PATCH] usb: phy: fix phy-fsl-usb.c's copyright issue

On Tue, Jul 17, 2018 at 02:25:56PM +0800, Pengbo Mu wrote:
> From: "yinbo.zhu" <yinbo.zhu@nxp.com>

> 

> Signed-off-by: yinbo.zhu <yinbo.zhu@nxp.com>


Also, you need to sign off on it, _and_ we need a valid name here for the author and sign-off.

thanks,

greg k-h
Greg Kroah-Hartman July 17, 2018, 7:41 a.m. UTC | #4
On Tue, Jul 17, 2018 at 07:16:40AM +0000, Pengbo Mu wrote:
> Hi Greg,
> 
> Sorry for missing the changelog text & sign .
> BTW , it's my first send email to upstream. 

Do not make your first patch to a driver be a legal one, unless you have
a very strong legal background and can properly answer the questions
asked of such a change.  Try doing real bug fixes instead.

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Pengbo Mu July 17, 2018, 8:04 a.m. UTC | #5
Hi Felipe , Greg,

After our discussion , this patch would not been sent temporarily .
I'll pay more attention to learn the upsteam.
Thanks Felipe , Greg .

Regards,
Pengbo Mu

-----Original Message-----
From: Greg Kroah-Hartman [mailto:gregkh@linuxfoundation.org] 

Sent: 2018年7月17日 15:42
To: Pengbo Mu <pengbo.mu@nxp.com>
Cc: Felipe Balbi <balbi@kernel.org>; linux-kernel@vger.kernel.org; linux-usb@vger.kernel.org
Subject: Re: [PATCH] usb: phy: fix phy-fsl-usb.c's copyright issue

On Tue, Jul 17, 2018 at 07:16:40AM +0000, Pengbo Mu wrote:
> Hi Greg,

> 

> Sorry for missing the changelog text & sign .

> BTW , it's my first send email to upstream. 


Do not make your first patch to a driver be a legal one, unless you have a very strong legal background and can properly answer the questions asked of such a change.  Try doing real bug fixes instead.

thanks,

greg k-h
diff mbox

Patch

diff --git a/drivers/usb/phy/phy-fsl-usb.c b/drivers/usb/phy/phy-fsl-usb.c
index 900875f..dcb531c 100644
--- a/drivers/usb/phy/phy-fsl-usb.c
+++ b/drivers/usb/phy/phy-fsl-usb.c
@@ -1,6 +1,6 @@ 
 // SPDX-License-Identifier: GPL-2.0+
 /*
- * Copyright (C) 2007,2008 Freescale semiconductor, Inc.
+ * Copyright 2007,2008 Freescale Semiconductor, Inc.
  *
  * Author: Li Yang <LeoLi@freescale.com>
  *         Jerry Huang <Chang-Ming.Huang@freescale.com>