From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1033075AbbKEP1x (ORCPT ); Thu, 5 Nov 2015 10:27:53 -0500 Received: from mezzanine.sirena.org.uk ([106.187.55.193]:48568 "EHLO mezzanine.sirena.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1033007AbbKEP1v (ORCPT ); Thu, 5 Nov 2015 10:27:51 -0500 Date: Thu, 5 Nov 2015 15:27:36 +0000 From: Mark Brown To: Adam Thomson Cc: Liam Girdwood , Jaroslav Kysela , Takashi Iwai , Rob Herring , Pawel Moll , Mark Rutland , Ian Campbell , Kumar Gala , alsa-devel@alsa-project.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Support Opensource Message-ID: <20151105152736.GO18409@sirena.org.uk> References: <01a9b6e96b66327cf9a27781c13aa50545380acb.1446718490.git.Adam.Thomson.Opensource@diasemi.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="2/+Vq7w28QOSGzSM" Content-Disposition: inline In-Reply-To: <01a9b6e96b66327cf9a27781c13aa50545380acb.1446718490.git.Adam.Thomson.Opensource@diasemi.com> X-Cookie: Put your trust in those who are worthy. User-Agent: Mutt/1.5.24 (2015-08-30) X-SA-Exim-Connect-IP: 94.197.121.29 X-SA-Exim-Mail-From: broonie@sirena.org.uk Subject: Re: [PATCH 2/2] ASoC: codecs: Add da7218 codec driver X-SA-Exim-Version: 4.2.1 (built Mon, 26 Dec 2011 16:24:06 +0000) X-SA-Exim-Scanned: Yes (on mezzanine.sirena.org.uk) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --2/+Vq7w28QOSGzSM Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Nov 05, 2015 at 10:43:19AM +0000, Adam Thomson wrote: > +/* ALC */ > +static void da7218_alc_calib(struct snd_soc_codec *codec) > +{ > + struct da7218_priv *da7218 =3D snd_soc_codec_get_drvdata(codec); > + u8 calib_ctrl; > + int i =3D 0; > + bool calibrated =3D false; > + > + /* Bypass cache so it saves current settings */ > + regcache_cache_bypass(da7218->regmap, true); What ensures that nothing else is running at the same time this is? =20 > +static int da7218_mic_lvl_det_sw_put(struct snd_kcontrol *kcontrol, > + struct snd_ctl_elem_value *ucontrol) > +{ Why is this a user visible control? > + /* Default all mixers off */ > + snd_soc_write(codec, DA7218_DROUTING_OUTDAI_1L, 0); > + snd_soc_write(codec, DA7218_DROUTING_OUTDAI_1R, 0); > + snd_soc_write(codec, DA7218_DROUTING_OUTDAI_2L, 0); > + snd_soc_write(codec, DA7218_DROUTING_OUTDAI_2R, 0); > + snd_soc_write(codec, DA7218_DROUTING_OUTFILT_1L, 0); > + snd_soc_write(codec, DA7218_DROUTING_OUTFILT_1R, 0); > + snd_soc_write(codec, DA7218_DROUTING_ST_OUTFILT_1L, 0); > + snd_soc_write(codec, DA7218_DROUTING_ST_OUTFILT_1R, 0); We generally just use the device defaults, why change them? --2/+Vq7w28QOSGzSM Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQEcBAEBCAAGBQJWO3VnAAoJECTWi3JdVIfQOoIH/3WmoI/XdUvAlul5oKTGsLf1 kE2Ms8ZIDfm3SgVXqi35qdMWfn16wXNE1tBGpbwqBdI53LtKVMwaTUc+upuhXOm7 tuJCndsPIdBREvSMgpmplELfpB3ffUA3uIMHCJS+/u8Yzogy3wN0tGGF8eGajN/t 9TeVWVedW1i0x8w4kCt2UBjpNzcMP/87Gb20hjL8IdG9bJ/IDlNgE4vUhDU28utI XkdpZp++WaJAvsvx2HbGsXbWX+pwSvXI74srMxKFRfVBVDAnP5JKErtkWceZ/n0R FIsd2LJJ4chk6RXbfT5VTsPH5RmL9SBVWT9FdeXyvo2OZKTOYFzCV7LSmeW1aAw= =FMP8 -----END PGP SIGNATURE----- --2/+Vq7w28QOSGzSM--