On Wed 2018-12-05 09:28:21, Stephen Boyd wrote: > Quoting Matti Vaittinen (2018-12-04 03:39:38) > > diff --git a/drivers/clk/clk-bd718x7.c b/drivers/clk/clk-bd718x7.c > > new file mode 100644 > > index 000000000000..d486859526ed > > --- /dev/null > > +++ b/drivers/clk/clk-bd718x7.c > > @@ -0,0 +1,131 @@ > > +// SPDX-License-Identifier: GPL-2.0 > > +// Copyright (C) 2018 ROHM Semiconductors > > +// bd71837.c -- ROHM BD71837MWV clock driver > > This isn't even the name of the file. Please remove. Also, only the SPDX > tag is supposed to have // on it and the other things should be normal > /* */ comment style. Actually whole block commented by // is now considered ok. Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html