Messages in this thread | | | Date | Wed, 13 Dec 2017 09:38:06 +0000 | From | Mark Brown <> | Subject | Re: [PATCH] spi: s3c64xx: add SPDX identifier |
| |
On Wed, Dec 13, 2017 at 10:48:54AM +0900, Andi Shyti wrote:
> > > +// > > > +// Copyright (c) 2009 Samsung Electronics Co., Ltd. > > > +// Jaswinder Singh <jassi.brar@samsung.com>
> it just looks ugly to me to have something like
> // SPDX...
> /* > * Copyright (c)... > * Jasw... > */
> I would make everything of the '//' style (I've also seen that > this is the trend also in other subsystems), but honestly I don't > really care. If you're strong on not moving entirely in '//' style
Yes, my preference is to avoid a mixture like you've got there. [unhandled content-type:application/pgp-signature] | |