diff mbox series

[v2,2/7] tty: xilinx_uartps: Update copyright text to correct format

Message ID 20220729114748.18332-3-shubhrajyoti.datta@xilinx.com
State New
Headers show
Series tty: xilinx_uartps: fixes and improvements | expand

Commit Message

Shubhrajyoti Datta July 29, 2022, 11:47 a.m. UTC
From: Michal Simek <michal.simek@xilinx.com>

Based on recommended guidance Copyright term should be also present in
front of (c). That's why aligned drivers to match this pattern.
It helps automated tools with source code scanning.

Signed-off-by: Michal Simek <michal.simek@xilinx.com>
Signed-off-by: Shubhrajyoti Datta <shubhrajyoti.datta@xilinx.com>
---
 drivers/tty/serial/xilinx_uartps.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Shubhrajyoti Datta Aug. 2, 2022, 7:29 a.m. UTC | #1
[AMD Official Use Only - General]



> -----Original Message-----
> From: Greg KH <gregkh@linuxfoundation.org>
> Sent: Friday, July 29, 2022 7:17 PM
> To: Shubhrajyoti Datta <shubhrajyoti.datta@xilinx.com>
> Cc: linux-serial@vger.kernel.org; michal.simek@xilinx.com;
> shubhrajyoti.datta@gmail.com; git (AMD-Xilinx) <git@amd.com>
> Subject: Re: [PATCH v2 2/7] tty: xilinx_uartps: Update copyright text to
> correct format
> 
> [CAUTION: External Email]
> 
> On Fri, Jul 29, 2022 at 05:17:43PM +0530, Shubhrajyoti Datta wrote:
> > From: Michal Simek <michal.simek@xilinx.com>
> >
> > Based on recommended guidance Copyright term should be also present in
> > front of (c). That's why aligned drivers to match this pattern.
> > It helps automated tools with source code scanning.
> >
> > Signed-off-by: Michal Simek <michal.simek@xilinx.com>
> > Signed-off-by: Shubhrajyoti Datta <shubhrajyoti.datta@xilinx.com>
> > ---
> >  drivers/tty/serial/xilinx_uartps.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/tty/serial/xilinx_uartps.c
> > b/drivers/tty/serial/xilinx_uartps.c
> > index 51fd09e14eda..8d2c21718869 100644
> > --- a/drivers/tty/serial/xilinx_uartps.c
> > +++ b/drivers/tty/serial/xilinx_uartps.c
> > @@ -2,7 +2,7 @@
> >  /*
> >   * Cadence UART driver (found in Xilinx Zynq)
> >   *
> > - * 2011 - 2014 (C) Xilinx Inc.
> > + * Copyright (c) 2011 - 2014 Xilinx, Inc.
> 
> This is great, but as you did just touch this thing in 2022, shouldn't you write
> this as:
>         Copyright (c) 2011 - 2014, 2022 Xilinx, Inc.
> 
> You can do an add-on patch for that if you want.

I will do a add on  patch for the same.

> 
> thanks,
> 
> greg k-h
diff mbox series

Patch

diff --git a/drivers/tty/serial/xilinx_uartps.c b/drivers/tty/serial/xilinx_uartps.c
index 51fd09e14eda..8d2c21718869 100644
--- a/drivers/tty/serial/xilinx_uartps.c
+++ b/drivers/tty/serial/xilinx_uartps.c
@@ -2,7 +2,7 @@ 
 /*
  * Cadence UART driver (found in Xilinx Zynq)
  *
- * 2011 - 2014 (C) Xilinx Inc.
+ * Copyright (c) 2011 - 2014 Xilinx, Inc.
  *
  * This driver has originally been pushed by Xilinx using a Zynq-branding. This
  * still shows in the naming of this file, the kconfig symbols and some symbols