From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paul Bolle Subject: Re: [PATCH v4 14/15] ASoC: rockchip/rockchip-hdmi-audio: add sound driver for hdmi audio Date: Mon, 02 Mar 2015 10:07:27 +0100 Message-ID: <1425287247.24292.82.camel@x220> References: <1425175834-24661-1-git-send-email-ykk@rock-chips.com> <1425179070-2736-1-git-send-email-ykk@rock-chips.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1425179070-2736-1-git-send-email-ykk@rock-chips.com> Sender: linux-kernel-owner@vger.kernel.org To: Yakir Yang Cc: Liam Girdwood , djkurtz@chromium.org, dianders@chromium.org, linux-rockchip@lists.infradead.org, David Airlie , Philipp Zabel , Russell King , Andy Yan , Greg Kroah-Hartman , Fabio Estevam , dri-devel@lists.freedesktop.org, Mark Brown , Jaroslav Kysela , Takashi Iwai , Lars-Peter Clausen , Brian Austin , Bard Liao , Oder Chiou , Max Filippov , Axel Lin , Arnd Bergmann , Jyri Sarha , Sean Cross , Ben Zhang , linux-kernel@vger.kernel.org, alsa-devel@alsa- List-Id: dri-devel@lists.freedesktop.org On Sat, 2015-02-28 at 22:04 -0500, Yakir Yang wrote: > --- /dev/null > +++ b/sound/soc/rockchip/rockchip_hdmi_audio.c > @@ -0,0 +1,169 @@ > +/* > + * rockchip-hdmi-card.c Doesn't match the filename. Is this line needed anyway? > + * > + * ROCKCHIP ALSA SoC DAI driver for HDMI audio on rockchip processors. > + * Copyright (c) 2014, ROCKCHIP CORPORATION. All rights reserved. > + * Authors: Yakir Yang > + * > + * This program is free software; you can redistribute it and/or modify it > + * under the terms and conditions of the GNU General Public License, > + * version 2, as published by the Free Software Foundation. > + * > + * This program is distributed in the hope 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. > + * > + * You should have received a copy of the GNU General Public License > + * along with this program. If not, see .* > + * > + */ This states the license is plain GPL v2. > +MODULE_LICENSE("GPL"); So you probably want MODULE_LICENSE("GPL v2"); here. Paul Bolle