[alsa-devel] [PATCH] ASoC: max98373: Added Amplifier Driver

[Ryan Lee]

Hi Philppe Ombredanne,

>-----Original Message-----
>From: Philippe Ombredanne [mailto:pombredanne at nexb.com]
>Sent: Friday, December 22, 2017 12:39 AM
>To: Ryan Lee <RyanS.Lee at maximintegrated.com>
>Cc: Liam Girdwood <lgirdwood at gmail.com>; Mark <broonie at kernel.org>;
>Rob Herring <robh+dt at kernel.org>; Mark Rutland <mark.rutland at arm.com>;
>Jaroslav Kysela <perex at perex.cz>; Takashi Iwai <tiwai at suse.com>; Arnd
>Bergmann <arnd at arndb.de>; afd at ti.com; robert.jarzmik at free.fr;
>supercraig0719 at gmail.com; jbrunet at baylibre.com; dannenberg at ti.com;
>Romain Perier <romain.perier at collabora.com>;
>bryce.ferguson at rockwellcollins.com; kuninori.morimoto.gx at renesas.com; m-
>stecklein at ti.com; ALSA <alsa-devel at alsa-project.org>; open list:OPEN
>FIRMWARE AND FLATTENED DEVICE TREE BINDINGS
><devicetree at vger.kernel.org>; LKML <linux-kernel at vger.kernel.org>;
>ryan.lee.maxim at gmail.com
>Subject: Re: [PATCH] ASoC: max98373: Added Amplifier Driver
>
>EXTERNAL EMAIL
>
>
>
>Ryan,
>
>On Fri, Dec 22, 2017 at 2:43 AM, Ryan Lee <ryans.lee at maximintegrated.com>
>wrote:
>> Signed-off-by: Ryan Lee <ryans.lee at maximintegrated.com>
>> ---
>>
>> Created max98373 amplifier driver.
>>
>
><snip>
>
>> --- /dev/null
>> +++ b/sound/soc/codecs/max98373.c
>> @@ -0,0 +1,996 @@
>> +/*
>> + * max98373.c  --  MAX98373 ALSA Soc Audio driver
>> + *
>> + * Copyright (C) 2017 Maxim Integrated Products
>> + * Author: Ryan Lee <ryans.lee at maximintegrated.com>
>> + *
>> + *  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 tags instead of this fine but long
>legalese?
>
>And if other contributors in your team could follow suit and you could spread
>the word that would be even better!
>
>See Thomas doc patches [1] for details.
>Thanks!
>
>[1]
>https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Flkml.org%
>2Flkml%2F2017%2F12%2F4%2F934&data=01%7C01%7Cryans.lee%40maximint
>egrated.com%7Cd26ec4859b2240fb0a6008d549178c4f%7Cfbd909dfea69478
>8a554f24b7854ad03%7C0&sdata=dTVH044XPox7EPR0WU1IGMlLxaWBDl80%
>2B%2FVYhwu51y0%3D&reserved=0
>
Thanks for your feedback. I will follow this.
>--
>Cordially
>Philippe Ombredanne