Thanks Philippe for review, > -----Original Message----- > From: Philippe Ombredanne [mailto:pombreda...@nexb.com] > Sent: Tuesday, January 09, 2018 4:48 AM > To: Jolly Shah <jol...@xilinx.com> > Cc: Matthias Brugger <matthias....@gmail.com>; Andy Gross > <andy.gr...@linaro.org>; Shawn Guo <shawn...@kernel.org>; > geert+rene...@glider.be; Bjorn Andersson <bjorn.anders...@linaro.org>; Sean > Wang <sean.w...@mediatek.com>; Marek Szyprowski > <m.szyprow...@samsung.com>; Michal Simek <michal.si...@xilinx.com>; > moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE <linux-arm- > ker...@lists.infradead.org>; LKML <linux-kernel@vger.kernel.org>; Jolly Shah > <jol...@xilinx.com>; Rajan Vaja <raj...@xilinx.com> > Subject: Re: [RFC PATCH] drivers: soc: xilinx: Add ZynqMP power domain driver > > Jolly, > > On Mon, Jan 8, 2018 at 11:12 PM, Jolly Shah <jolly.s...@xilinx.com> wrote: > > The zynqmp-genpd driver communicates the usage requirements for > > logical power domains / devices to the platform FW. > > FW is responsible for choosing appropriate power states, taking Linux' > > usage information into account. > > > > Signed-off-by: Jolly Shah <jol...@xilinx.com> > > Signed-off-by: Rajan Vaja <raj...@xilinx.com> > > --- > > <snip> > > > --- /dev/null > > +++ b/drivers/soc/xilinx/zynqmp/pm_domains.c > > @@ -0,0 +1,343 @@ > > +/* > > + * ZynqMP Generic PM domain support > > + * > > + * Copyright (C) 2014-2017 Xilinx, Inc. > > + * > > + * Davorin Mista <davorin.mi...@aggios.com> > > + * Jolly Shah <jol...@xilinx.com> > > + * Rajan Vaja <raj...@xilinx.com> > > + * > > + * SPDX-License-Identifier: GPL-2.0+ > > + */ > > This tag should be on the fist line as this: > > // SPDX-License-Identifier: GPL-2.0+ > > -- > Cordially > Philippe Ombredanne
Will fix it in next version
