From mboxrd@z Thu Jan 1 00:00:00 1970 From: Philippe Ombredanne Subject: Re: [PATCH v2 2/2] drm/tinydrm: add driver for ST7735R panels Date: Mon, 11 Dec 2017 10:06:50 +0100 Message-ID: References: <1512943833-31352-1-git-send-email-david@lechnology.com> <1512943833-31352-3-git-send-email-david@lechnology.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Return-path: In-Reply-To: <1512943833-31352-3-git-send-email-david@lechnology.com> Sender: linux-kernel-owner@vger.kernel.org To: David Lechner Cc: dri-devel , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , =?UTF-8?Q?Noralf_Tr=C3=B8nnes?= , limor@ladyada.net, Linus Walleij , Rob Herring , Mark Rutland , LKML List-Id: devicetree@vger.kernel.org David, On Sun, Dec 10, 2017 at 11:10 PM, David Lechner wrote: > This adds a new driver for Sitronix ST7735R display panels. > > This has been tested using an Adafruit 1.8" TFT. > > Signed-off-by: David Lechner > --- /dev/null > +++ b/drivers/gpu/drm/tinydrm/st7735r.c > @@ -0,0 +1,219 @@ > +/* > + * DRM driver for Sitronix ST7735R panels > + * > + * Copyright 2017 David Lechner > + * > + * 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. > + */ Have you considered using the new SPDX ids? Check the doc patches from Thomas for details. This could come out this way if you are using the C++ comment style all the way, a style that you should at least use for the license id as requested and commented by Linus: > +// SPDX-License-Identifier: GPL-2.0+ > +// Copyright 2017 David Lechner > +// DRM driver for Sitronix ST7735R panels -- Cordially Philippe Ombredanne