Hi Philippe, On Tue, Feb 20, 2018 at 4:36 PM, Philippe Ombredanne <pombredanne@xxxxxxxx> wrote: > On Tue, Feb 20, 2018 at 5:44 AM, Alexandre Courbot > <acourbot@xxxxxxxxxxxx> wrote: >> Add a v4l2 request entity data structure that takes care of storing the >> request-related state of a V4L2 device ; in this case, its controls. >> >> Signed-off-by: Alexandre Courbot <acourbot@xxxxxxxxxxxx> > > <snip> > >> --- /dev/null >> +++ b/drivers/media/v4l2-core/v4l2-request.c >> @@ -0,0 +1,178 @@ >> +/* >> + * Media requests support for V4L2 >> + * >> + * Copyright (C) 2018, The Chromium OS Authors. 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 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. >> + */ > > Do you mind using SPDX tags per [1] rather that this fine but long > legalese. (Here and in the whole patch series) > > [1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/license-rules.rst I need to check what is the stance of Chromium OS regarding these, but will gladly comply if possible.