Hi, On Wed, 29 Jun 2022 at 10:58, <teik.heng.ch...@intel.com> wrote: > > From: Teik Heng Chong <teik.heng.ch...@intel.com> > > All the source code of clk-mem-n5x.c and clk-n5x.c are from Intel, > update the license to use both GPL2.0 and BSD-3 Clause because this > copy of code may used for open source and internal project. > > Signed-off-by: Teik Heng Chong <teik.heng.ch...@intel.com> > --- > drivers/clk/altera/clk-mem-n5x.c | 4 ++-- > drivers/clk/altera/clk-mem-n5x.h | 4 ++-- > drivers/clk/altera/clk-n5x.c | 4 ++-- > drivers/clk/altera/clk-n5x.h | 4 ++-- > 4 files changed, 8 insertions(+), 8 deletions(-) > > diff --git a/drivers/clk/altera/clk-mem-n5x.c > b/drivers/clk/altera/clk-mem-n5x.c > index ca44998641..9bbe2cd0ca 100644 > --- a/drivers/clk/altera/clk-mem-n5x.c > +++ b/drivers/clk/altera/clk-mem-n5x.c > @@ -1,6 +1,6 @@ > -// SPDX-License-Identifier: GPL-2.0 > +// SPDX-License-Identifier: GPL-2.0 OR BSD-3-Clause
+Tom Rini OR is not a license so I think you should drop that word. Also U-Boot is GPL so seems strange to use a different license for these files. If it was wrong initially, perhaps add a 'Fixes' tag? Regards, Simon