Messages in this thread | | | Date | Fri, 19 Oct 2018 12:26:22 +0100 | From | Mark Brown <> | Subject | Re: [PATCH v3 4/5] regulator: lochnagar: Add support for the Cirrus Logic Lochnagar |
| |
On Fri, Oct 19, 2018 at 10:50:02AM +0100, Charles Keepax wrote:
Please do not submit new versions of already applied patches, please submit incremental updates to the existing code. Modifying existing commits creates problems for other users building on top of those commits so it's best practice to only change pubished git commits if absolutely essential.
> +++ b/drivers/regulator/lochnagar-regulator.c > @@ -0,0 +1,255 @@ > +// SPDX-License-Identifier: GPL-2.0 > +/* > + * Lochnagar regulator driver
Please don't mix C and C++ comments like this in the same block, just have it be a C++ block so it looks consistent. [unhandled content-type:application/pgp-signature] | |