[AMD Official Use Only - General] > -----Original Message----- > From: Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx> > Sent: Friday, July 29, 2022 7:17 PM > To: Shubhrajyoti Datta <shubhrajyoti.datta@xxxxxxxxxx> > Cc: linux-serial@xxxxxxxxxxxxxxx; michal.simek@xxxxxxxxxx; > shubhrajyoti.datta@xxxxxxxxx; git (AMD-Xilinx) <git@xxxxxxx> > 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@xxxxxxxxxx> > > > > 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@xxxxxxxxxx> > > Signed-off-by: Shubhrajyoti Datta <shubhrajyoti.datta@xxxxxxxxxx> > > --- > > 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
