[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20181019120638.GG5895@sirena.org.uk>
Date: Fri, 19 Oct 2018 13:06:38 +0100
From: Mark Brown <broonie@...nel.org>
To: Richard Fitzgerald <rf@...nsource.cirrus.com>
Cc: Charles Keepax <ckeepax@...nsource.cirrus.com>,
lee.jones@...aro.org, mturquette@...libre.com, sboyd@...nel.org,
linus.walleij@...aro.org, robh+dt@...nel.org, mark.rutland@....com,
lgirdwood@...il.com, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org, patches@...nsource.cirrus.com,
linux-clk@...r.kernel.org, linux-gpio@...r.kernel.org
Subject: Re: [PATCH v3 4/5] regulator: lochnagar: Add support for the Cirrus
Logic Lochnagar
On Fri, Oct 19, 2018 at 01:02:45PM +0100, Richard Fitzgerald wrote:
> Most SPDX headers on C files that I've looked at have it this way with a
> C++ style comment above a C-style comment, though some don't. license-rules.rst
> doesn't define how or if a SPDX comment line should be merged with the following
> file header comment. I've had a bunch of patches in different subsystems all
> accepted with this mixed format (copied from existing files). Doing the same as
Right, a lot of the conversions seem to have been done very
mechanically.
> Maybe someone should update license-rules.rst to make a definite statement of the
> style instead of leaving it to become another style that varies across the kernel
> and between files.
Well volunteered!
Download attachment "signature.asc" of type "application/pgp-signature" (489 bytes)
Powered by blists - more mailing lists