linux-kernel - Re: [PATCH v6] ASoC: tas2552: Support TI TAS2552 Amplifier

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Mon, 14 Jul 2014 20:17:11 +0100
From:	Mark Brown <broonie@...nel.org>
To:	"Murphy, Dan" <dmurphy@...com>
Cc:	"linux-sound@...r.kernel.org" <linux-sound@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"alsa-devel@...a-project.org" <alsa-devel@...a-project.org>,
	"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>
Subject: Re: [PATCH v6] ASoC: tas2552: Support TI TAS2552 Amplifier

On Mon, Jul 14, 2014 at 07:07:06PM +0000, Murphy, Dan wrote:
> On 07/14/2014 01:43 PM, Mark Brown wrote:

> >> +config SND_SOC_TAS2552
> >> +	tristate "Texas Instruments TAS2552 Mono Audio amplifier"
> >> +	depends on I2C
> >> +
> >>  endmenu

> > Keep this and Makefile sorted - since this is a proper CODEC driver it
> > should be in with the CODECs and also sorted in alphabetical order.

> OK.  I was trying to stay away from a conflict by adding to the
> bottom of the file.

The whole point is to avoid conflicts - if everyone adds at the bottom
of the file any two adds are pretty much guaranteed to conflict either
on application or on merge.  Sorting avoids most of these and makes any
that do occur much clearer.

Download attachment "signature.asc" of type "application/pgp-signature" (820 bytes)