From: Philippe Ombredanne <pombredanne@nexb.com> To: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> Cc: "Michael Turquette" <mturquette@baylibre.com>, "Stephen Boyd" <sboyd@codeaurora.org>, "Andreas Färber" <afaerber@suse.de>, "Rob Herring" <robh+dt@kernel.org>, "Mark Rutland" <mark.rutland@arm.com>, liuwei@actions-semi.com, mp-cs@actions-semi.com, 96boards@ucrobotics.com, "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" <devicetree@vger.kernel.org>, "David S. Miller" <davem@davemloft.net>, "Mauro Carvalho Chehab" <mchehab@kernel.org>, daniel.thompson@linaro.org, amit.kucheria@linaro.org, LKML <linux-kernel@vger.kernel.org>, linux-clk@vger.kernel.org, "moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE" <linux-arm-kernel@lists.infradead.org>, "Viresh Kumar" <viresh.kumar@linaro.org> Subject: Re: [PATCH v3 01/11] dt-bindings: clock: Add Actions S900 clock bindings Date: Sat, 10 Feb 2018 08:36:21 +0100 [thread overview] Message-ID: <CAOFm3uFU9KPZqvRucm8U0ruy6ZZ6AoN55ELCdzp6+sqA6i_SaQ@mail.gmail.com> (raw) In-Reply-To: <20180210024120.27503-2-manivannan.sadhasivam@linaro.org> Dear Manivannan, On Sat, Feb 10, 2018 at 3:41 AM, Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> wrote: > Add Actions Semi S900 clock bindings. > > Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> > Acked-by: Rob Herring <robh@kernel.org> <snip> > diff --git a/include/dt-bindings/clock/actions,s900-cmu.h b/include/dt-bindings/clock/actions,s900-cmu.h > new file mode 100644 > index 000000000000..2fa94e19922b > --- /dev/null > +++ b/include/dt-bindings/clock/actions,s900-cmu.h > @@ -0,0 +1,139 @@ > +/* > + * Device Tree binding constants for Actions S900 Clock Management Unit > + * > + * Copyright (c) 2014 Actions Semi Inc. > + * Copyright (c) 2017 Linaro Ltd. > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License as published by > + * the Free Software Foundation; either version 2 of the License, or > + * (at your option) any later version. > + * > + * 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. > + */ Would you consider using the new SPDX license ids rather that this time-tested but rather boring legalese? The (still new and fresh) license documentation contributed by tglx --the only maintainer that I know that understands both the innards of Spectre and Meltdown and the beauty of reStructuredText -- is in: Documentation/process/license-rules.rst Practically this means replacing the above by a simple single line and getting rid of a whopping 8 comment lines! SPDX-License-Identifier: GPL-2.0+ You get to save a few tree as a bonus if you also do the same for all Linaro-copyrighted files. Yes this is saving trees because I will use less paper each time I print a listing of the kernel source code. Which is something that I rarely if ever do: but somebody must do it somewhere for sure. If I do the math: we have ~60K files in the kernel, and say we can remove roughly 5 lines of legalese per file on average. Each printed source code page is roughly 60 lines : this will mean a saving of about 6000 paper sheets saved on each printout! A letter-size paper ream is 500 pages, about 2.5 Kg and costs about ~$8. You can extract about 10K to 20k sheets of paper per tree [1]. Therefore my Fermi estimate is that using shorter legalese in the kernel will eventually save roughly ONE FULL smaller tree (6K pages) each time someone prints the kernel code: incredible, right? Thank you for helping make the kernel a mostly legalese-free codebase and saving trees at the same time! [1] https://www.sierraclub.org/sierra/2014-4-july-august/green-life/how-much-paper-does-one-tree-produce -- Cordially Philippe Ombredanne
WARNING: multiple messages have this Message-ID (diff)
From: pombredanne@nexb.com (Philippe Ombredanne) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH v3 01/11] dt-bindings: clock: Add Actions S900 clock bindings Date: Sat, 10 Feb 2018 08:36:21 +0100 [thread overview] Message-ID: <CAOFm3uFU9KPZqvRucm8U0ruy6ZZ6AoN55ELCdzp6+sqA6i_SaQ@mail.gmail.com> (raw) In-Reply-To: <20180210024120.27503-2-manivannan.sadhasivam@linaro.org> Dear Manivannan, On Sat, Feb 10, 2018 at 3:41 AM, Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> wrote: > Add Actions Semi S900 clock bindings. > > Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> > Acked-by: Rob Herring <robh@kernel.org> <snip> > diff --git a/include/dt-bindings/clock/actions,s900-cmu.h b/include/dt-bindings/clock/actions,s900-cmu.h > new file mode 100644 > index 000000000000..2fa94e19922b > --- /dev/null > +++ b/include/dt-bindings/clock/actions,s900-cmu.h > @@ -0,0 +1,139 @@ > +/* > + * Device Tree binding constants for Actions S900 Clock Management Unit > + * > + * Copyright (c) 2014 Actions Semi Inc. > + * Copyright (c) 2017 Linaro Ltd. > + * > + * This program is free software; you can redistribute it and/or modify > + * it under the terms of the GNU General Public License as published by > + * the Free Software Foundation; either version 2 of the License, or > + * (at your option) any later version. > + * > + * 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. > + */ Would you consider using the new SPDX license ids rather that this time-tested but rather boring legalese? The (still new and fresh) license documentation contributed by tglx --the only maintainer that I know that understands both the innards of Spectre and Meltdown and the beauty of reStructuredText -- is in: Documentation/process/license-rules.rst Practically this means replacing the above by a simple single line and getting rid of a whopping 8 comment lines! SPDX-License-Identifier: GPL-2.0+ You get to save a few tree as a bonus if you also do the same for all Linaro-copyrighted files. Yes this is saving trees because I will use less paper each time I print a listing of the kernel source code. Which is something that I rarely if ever do: but somebody must do it somewhere for sure. If I do the math: we have ~60K files in the kernel, and say we can remove roughly 5 lines of legalese per file on average. Each printed source code page is roughly 60 lines : this will mean a saving of about 6000 paper sheets saved on each printout! A letter-size paper ream is 500 pages, about 2.5 Kg and costs about ~$8. You can extract about 10K to 20k sheets of paper per tree [1]. Therefore my Fermi estimate is that using shorter legalese in the kernel will eventually save roughly ONE FULL smaller tree (6K pages) each time someone prints the kernel code: incredible, right? Thank you for helping make the kernel a mostly legalese-free codebase and saving trees at the same time! [1] https://www.sierraclub.org/sierra/2014-4-july-august/green-life/how-much-paper-does-one-tree-produce -- Cordially Philippe Ombredanne
next prev parent reply other threads:[~2018-02-10 7:37 UTC|newest] Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top 2018-02-10 2:41 [PATCH v3 00/11] Add clock driver for Actions S900 SoC Manivannan Sadhasivam 2018-02-10 2:41 ` Manivannan Sadhasivam 2018-02-10 2:41 ` Manivannan Sadhasivam 2018-02-10 2:41 ` [PATCH v3 01/11] dt-bindings: clock: Add Actions S900 clock bindings Manivannan Sadhasivam 2018-02-10 2:41 ` Manivannan Sadhasivam 2018-02-10 2:41 ` Manivannan Sadhasivam 2018-02-10 7:36 ` Philippe Ombredanne [this message] 2018-02-10 7:36 ` Philippe Ombredanne 2018-02-10 8:02 ` Manivannan Sadhasivam 2018-02-10 8:02 ` Manivannan Sadhasivam 2018-02-10 2:41 ` [PATCH v3 02/11] arm64: dts: actions: Add S900 clock management unit nodes Manivannan Sadhasivam 2018-02-10 2:41 ` Manivannan Sadhasivam 2018-02-10 2:41 ` [PATCH v3 03/11] arm64: dts: actions: Source CMU clock for UART5 Manivannan Sadhasivam 2018-02-10 2:41 ` Manivannan Sadhasivam 2018-02-10 2:41 ` [PATCH v3 04/11] clk: actions: Add common clock driver support Manivannan Sadhasivam 2018-02-10 2:41 ` Manivannan Sadhasivam 2018-02-10 2:41 ` [PATCH v3 05/11] clk: actions: Add gate clock support Manivannan Sadhasivam 2018-02-10 2:41 ` Manivannan Sadhasivam 2018-02-10 2:41 ` [PATCH v3 06/11] clk: actions: Add mux " Manivannan Sadhasivam 2018-02-10 2:41 ` Manivannan Sadhasivam 2018-02-10 2:41 ` [PATCH v3 07/11] clk: actions: Add divider " Manivannan Sadhasivam 2018-02-10 2:41 ` Manivannan Sadhasivam 2018-02-10 2:41 ` [PATCH v3 08/11] clk: actions: Add factor " Manivannan Sadhasivam 2018-02-10 2:41 ` Manivannan Sadhasivam 2018-02-10 2:41 ` [PATCH v3 09/11] clk: actions: Add composite " Manivannan Sadhasivam 2018-02-10 2:41 ` Manivannan Sadhasivam 2018-02-10 2:41 ` Manivannan Sadhasivam 2018-02-10 2:41 ` [PATCH v3 10/11] clk: actions: Add pll " Manivannan Sadhasivam 2018-02-10 2:41 ` Manivannan Sadhasivam 2018-02-10 2:41 ` [PATCH v3 11/11] clk: actions: Add S900 SoC " Manivannan Sadhasivam 2018-02-10 2:41 ` Manivannan Sadhasivam 2018-02-10 2:41 ` Manivannan Sadhasivam
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=CAOFm3uFU9KPZqvRucm8U0ruy6ZZ6AoN55ELCdzp6+sqA6i_SaQ@mail.gmail.com \ --to=pombredanne@nexb.com \ --cc=96boards@ucrobotics.com \ --cc=afaerber@suse.de \ --cc=amit.kucheria@linaro.org \ --cc=daniel.thompson@linaro.org \ --cc=davem@davemloft.net \ --cc=devicetree@vger.kernel.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-clk@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=liuwei@actions-semi.com \ --cc=manivannan.sadhasivam@linaro.org \ --cc=mark.rutland@arm.com \ --cc=mchehab@kernel.org \ --cc=mp-cs@actions-semi.com \ --cc=mturquette@baylibre.com \ --cc=robh+dt@kernel.org \ --cc=sboyd@codeaurora.org \ --cc=viresh.kumar@linaro.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.