From: Michael Trimarchi <michael@amarulasolutions.com> To: "Raphaël Poggi" <poggi.raph@gmail.com> Cc: perex@perex.cz, tiwai@suse.com, "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>, "alsa-devel@alsa-project.org" <alsa-devel@alsa-project.org>, LKML <linux-kernel@vger.kernel.org> Subject: Re: [alsa-devel] [PATCH 1/4] sound: codecs: pcm1792a: clean function exit Date: Sat, 21 Nov 2015 17:33:46 +0100 [thread overview] Message-ID: <20151121163336.GA27666@panicking> (raw) In-Reply-To: <CACqcpZACQiBkyj=a765rKjGi7Ft_MS4+t_0F90dQXGxecRm92Q@mail.gmail.com> Hi On Sat, Nov 21, 2015 at 05:16:53PM +0100, Raphaël Poggi wrote: > Hi, > > I know that almost all the codec do as before, but what is the goal of > using an extra condition when we can avoid it ? > If you use top posting I can not follow. I said that I'm agree on the first one chage and I suggest to do the same for the next one. Michael > Raphael > > 2015-11-21 10:31 GMT+01:00 Michael Trimarchi <michael@amarulasolutions.com>: > > Hi > > > > On Fri, Nov 20, 2015 at 11:09 AM, Raphael Poggi <poggi.raph@gmail.com> wrote: > >> From: Raphael Poggi <poggi.raph@gmail.com> > >> > >> Signed-off-by: Raphael Poggi <poggi.raph@gmail.com> > >> --- > >> sound/soc/codecs/pcm1792a.c | 14 ++++---------- > >> 1 file changed, 4 insertions(+), 10 deletions(-) > >> > >> diff --git a/sound/soc/codecs/pcm1792a.c b/sound/soc/codecs/pcm1792a.c > >> index 08bb486..cf42b24 100644 > >> --- a/sound/soc/codecs/pcm1792a.c > >> +++ b/sound/soc/codecs/pcm1792a.c > >> @@ -91,14 +91,9 @@ static int pcm1792a_digital_mute(struct snd_soc_dai *dai, int mute) > >> { > >> struct snd_soc_codec *codec = dai->codec; > >> struct pcm1792a_private *priv = snd_soc_codec_get_drvdata(codec); > >> - int ret; > >> > >> - ret = regmap_update_bits(priv->regmap, PCM1792A_SOFT_MUTE, > >> + return regmap_update_bits(priv->regmap, PCM1792A_SOFT_MUTE, > >> PCM1792A_MUTE_MASK, !!mute); > >> - if (ret < 0) > >> - return ret; > >> - > > > > Ok, I have seen that almost all the codecs use return regmap_upda... > > > >> - return 0; > >> } > >> > >> static int pcm1792a_hw_params(struct snd_pcm_substream *substream, > >> @@ -107,7 +102,8 @@ static int pcm1792a_hw_params(struct snd_pcm_substream *substream, > >> { > >> struct snd_soc_codec *codec = dai->codec; > >> struct pcm1792a_private *priv = snd_soc_codec_get_drvdata(codec); > >> - int val = 0, ret; > >> + int val = 0; > >> + int ret = 0; > >> > >> priv->rate = params_rate(params); > >> > >> @@ -147,10 +143,8 @@ static int pcm1792a_hw_params(struct snd_pcm_substream *substream, > >> > >> ret = regmap_update_bits(priv->regmap, PCM1792A_FMT_CONTROL, > >> PCM1792A_FMT_MASK | PCM1792A_ATLD_ENABLE, val); > >> - if (ret < 0) > >> - return ret; > >> > > > > Please do the same as before > > > > Michael > > > > > >> - return 0; > >> + return ret; > >> } > >> > >> static const struct snd_soc_dai_ops pcm1792a_dai_ops = { > >> -- > >> 2.1.0 > >> > >> _______________________________________________ > >> Alsa-devel mailing list > >> Alsa-devel@alsa-project.org > >> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel > > > > > > > > -- > > | Michael Nazzareno Trimarchi Amarula Solutions BV | > > | COO - Founder Cruquiuskade 47 | > > | +31(0)851119172 Amsterdam 1018 AM NL | > > | [`as] http://www.amarulasolutions.com | > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > > the body of a message to majordomo@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html > > Please read the FAQ at http://www.tux.org/lkml/ -- | Michael Nazzareno Trimarchi Amarula Solutions BV | | COO - Founder Cruquiuskade 47 | | +31(0)851119172 Amsterdam 1018 AM NL | | [`as] http://www.amarulasolutions.com |
WARNING: multiple messages have this Message-ID (diff)
From: Michael Trimarchi <michael@amarulasolutions.com> To: "Raphaël Poggi" <poggi.raph@gmail.com> Cc: "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>, "alsa-devel@alsa-project.org" <alsa-devel@alsa-project.org>, tiwai@suse.com, LKML <linux-kernel@vger.kernel.org> Subject: Re: [PATCH 1/4] sound: codecs: pcm1792a: clean function exit Date: Sat, 21 Nov 2015 17:33:46 +0100 [thread overview] Message-ID: <20151121163336.GA27666@panicking> (raw) In-Reply-To: <CACqcpZACQiBkyj=a765rKjGi7Ft_MS4+t_0F90dQXGxecRm92Q@mail.gmail.com> Hi On Sat, Nov 21, 2015 at 05:16:53PM +0100, Raphaël Poggi wrote: > Hi, > > I know that almost all the codec do as before, but what is the goal of > using an extra condition when we can avoid it ? > If you use top posting I can not follow. I said that I'm agree on the first one chage and I suggest to do the same for the next one. Michael > Raphael > > 2015-11-21 10:31 GMT+01:00 Michael Trimarchi <michael@amarulasolutions.com>: > > Hi > > > > On Fri, Nov 20, 2015 at 11:09 AM, Raphael Poggi <poggi.raph@gmail.com> wrote: > >> From: Raphael Poggi <poggi.raph@gmail.com> > >> > >> Signed-off-by: Raphael Poggi <poggi.raph@gmail.com> > >> --- > >> sound/soc/codecs/pcm1792a.c | 14 ++++---------- > >> 1 file changed, 4 insertions(+), 10 deletions(-) > >> > >> diff --git a/sound/soc/codecs/pcm1792a.c b/sound/soc/codecs/pcm1792a.c > >> index 08bb486..cf42b24 100644 > >> --- a/sound/soc/codecs/pcm1792a.c > >> +++ b/sound/soc/codecs/pcm1792a.c > >> @@ -91,14 +91,9 @@ static int pcm1792a_digital_mute(struct snd_soc_dai *dai, int mute) > >> { > >> struct snd_soc_codec *codec = dai->codec; > >> struct pcm1792a_private *priv = snd_soc_codec_get_drvdata(codec); > >> - int ret; > >> > >> - ret = regmap_update_bits(priv->regmap, PCM1792A_SOFT_MUTE, > >> + return regmap_update_bits(priv->regmap, PCM1792A_SOFT_MUTE, > >> PCM1792A_MUTE_MASK, !!mute); > >> - if (ret < 0) > >> - return ret; > >> - > > > > Ok, I have seen that almost all the codecs use return regmap_upda... > > > >> - return 0; > >> } > >> > >> static int pcm1792a_hw_params(struct snd_pcm_substream *substream, > >> @@ -107,7 +102,8 @@ static int pcm1792a_hw_params(struct snd_pcm_substream *substream, > >> { > >> struct snd_soc_codec *codec = dai->codec; > >> struct pcm1792a_private *priv = snd_soc_codec_get_drvdata(codec); > >> - int val = 0, ret; > >> + int val = 0; > >> + int ret = 0; > >> > >> priv->rate = params_rate(params); > >> > >> @@ -147,10 +143,8 @@ static int pcm1792a_hw_params(struct snd_pcm_substream *substream, > >> > >> ret = regmap_update_bits(priv->regmap, PCM1792A_FMT_CONTROL, > >> PCM1792A_FMT_MASK | PCM1792A_ATLD_ENABLE, val); > >> - if (ret < 0) > >> - return ret; > >> > > > > Please do the same as before > > > > Michael > > > > > >> - return 0; > >> + return ret; > >> } > >> > >> static const struct snd_soc_dai_ops pcm1792a_dai_ops = { > >> -- > >> 2.1.0 > >> > >> _______________________________________________ > >> Alsa-devel mailing list > >> Alsa-devel@alsa-project.org > >> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel > > > > > > > > -- > > | Michael Nazzareno Trimarchi Amarula Solutions BV | > > | COO - Founder Cruquiuskade 47 | > > | +31(0)851119172 Amsterdam 1018 AM NL | > > | [`as] http://www.amarulasolutions.com | > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > > the body of a message to majordomo@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html > > Please read the FAQ at http://www.tux.org/lkml/ -- | Michael Nazzareno Trimarchi Amarula Solutions BV | | COO - Founder Cruquiuskade 47 | | +31(0)851119172 Amsterdam 1018 AM NL | | [`as] http://www.amarulasolutions.com |
next prev parent reply other threads:[~2015-11-21 16:34 UTC|newest] Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-11-20 10:09 [PATCH 0/4] New feature and clean up for pcm1792a driver Raphael Poggi 2015-11-20 10:09 ` [PATCH 1/4] sound: codecs: pcm1792a: clean function exit Raphael Poggi 2015-11-20 10:09 ` Raphael Poggi 2015-11-21 9:31 ` [alsa-devel] " Michael Trimarchi 2015-11-21 9:31 ` Michael Trimarchi 2015-11-21 16:16 ` Raphaël Poggi 2015-11-21 16:33 ` Michael Trimarchi [this message] 2015-11-21 16:33 ` Michael Trimarchi 2015-11-20 10:09 ` [PATCH 2/4] sound: codecs: pcm1792a: add define for register Raphael Poggi 2015-11-21 9:36 ` [alsa-devel] " Michael Trimarchi 2015-11-20 10:09 ` [PATCH 3/4] dt-bindings: sound: add binding for pcm1792a Raphael Poggi 2015-11-20 10:09 ` Raphael Poggi 2015-11-20 14:42 ` Rob Herring 2015-11-20 15:04 ` [alsa-devel] " Michael Trimarchi 2015-11-20 15:04 ` Michael Trimarchi 2015-11-20 10:09 ` [PATCH 4/4] sound: codecs: pcm1792a: introduce channel side Raphael Poggi 2015-11-21 9:37 ` [alsa-devel] " Michael Trimarchi 2015-11-21 9:37 ` Michael Trimarchi 2015-11-24 8:21 ` Raphaël Poggi 2015-11-24 8:27 ` Michael Trimarchi 2015-11-24 16:04 ` Raphaël Poggi 2015-11-24 16:04 ` Raphaël Poggi 2015-11-24 20:10 ` Michael Trimarchi
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20151121163336.GA27666@panicking \ --to=michael@amarulasolutions.com \ --cc=alsa-devel@alsa-project.org \ --cc=devicetree@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=perex@perex.cz \ --cc=poggi.raph@gmail.com \ --cc=tiwai@suse.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.