From mboxrd@z Thu Jan 1 00:00:00 1970 From: Philippe Ombredanne Subject: Re: [PATCH v4 09/12] clk: qcom: Add Krait clock controller driver Date: Fri, 8 Dec 2017 11:23:02 +0100 Message-ID: References: <1512725401-4944-1-git-send-email-sricharan@codeaurora.org> <1512725401-4944-10-git-send-email-sricharan@codeaurora.org> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Return-path: In-Reply-To: <1512725401-4944-10-git-send-email-sricharan@codeaurora.org> Sender: linux-pm-owner@vger.kernel.org To: Sricharan R , Stephen Boyd Cc: mturquette@linaro.org, "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , linux-pm@vger.kernel.org, linux-arm-msm@vger.kernel.org, LKML , viresh.kumar@linaro.org, "moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE" List-Id: devicetree@vger.kernel.org Sricharan, Stephen, On Fri, Dec 8, 2017 at 10:29 AM, Sricharan R wrote: > From: Stephen Boyd > > The Krait CPU clocks are made up of a primary mux and secondary > mux for each CPU and the L2, controlled via cp15 accessors. For > Kraits within KPSSv1 each secondary mux accepts a different aux > source, but on KPSSv2 each secondary mux accepts the same aux > source. > > Cc: > Signed-off-by: Stephen Boyd [] > --- /dev/null > +++ b/drivers/clk/qcom/krait-cc.c > @@ -0,0 +1,350 @@ > +/* Copyright (c) 2013-2015, The Linux Foundation. All rights reserved. > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License version 2 and > + * only version 2 as published by the Free Software Foundation. > + * > + * This program is distributed in the hope that it will be useful, > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + */ Have you considered using the new SPDX ids? Something like this: // SPDX-License-Identifier: GPL-2.0 // Copyright (c) 2013-2015, The Linux Foundation. All rights reserved. Beside being simpler and increasing the code to comment ratio by deleting 9 lines of boilerplate, this will also save a few trees over the long term each time that I print the source code of the kernel ;) (do not worry, I am NOT as insane as to really print the kernel sources, but someone more insane than me may well do it) You may wonder about the C++ // comment style I used here... Please see Linus posts on the topic as well as Thomas doc patches overall for instructions on using the SPDX ids. And if you were to do this for your past, present and future contributions (eventually these of your group), I would be quite grateful. Thank you for your kind consideration -- Cordially Philippe Ombredanne