All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ricard Wanderlof <ricardw@axis.com>
To: Mark Brown <broonie@kernel.org>
Cc: alsa-devel <alsa-devel@alsa-project.org>,
	Liam Girdwood <lgirdwood@gmail.com>
Subject: Re: [PATCH v6 0/2] Codec driver for TLV320ADC3001 and 3101
Date: Wed, 1 Dec 2021 14:58:12 +0100	[thread overview]
Message-ID: <alpine.DEB.2.21.2112011454470.30606@lnxricardw1.se.axis.com> (raw)
In-Reply-To: <Yad8CVmZKUDtpQaX@sirena.org.uk>


On Wed, 1 Dec 2021, Mark Brown wrote:

> On Wed, Dec 01, 2021 at 01:43:54PM +0100, Ricard Wanderlof wrote:
> 
> > Changes v5->v6:
> > 
> > - Added missing "depends on I2C" in the Kconfig
> >   (reported by lkp@intel.com).
> > - Fold include file into source as it is not needed elsewhere.
> > - Fix SPDX-License-Identifier which was using a deprecated license.
> > - Clean up authorship information in file header, removing history.
> > - Remove useless GPIOLIB ifdefs.
> > - Use devm_gpiod_get for reset gpio rather than devm_gpio_request etc.
> > - Handle EPROBE_DEFER properly for reset gpio request.
> > - Use dev_err_probe rather than explicit EPROBE_DEFER checking.
> > - Properly check value from DT parsing for mic bias and codec gpio pins.
> > 
> > No changes to the devicetree bindings.
> 
> I had review comments on the DT bindings?

Yes, they were addressed in v1. The list above only refers to v5->v6.  
Perhaps I should have expressed the changelog in some other way?

/Ricard
-- 
Ricard Wolf Wanderlof                           ricardw(at)axis.com
Axis Communications AB, Lund, Sweden            www.axis.com
Phone +46 46 272 2016                           Fax +46 46 13 61 30

      reply	other threads:[~2021-12-01 13:59 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-12-01 12:43 [PATCH v6 0/2] Codec driver for TLV320ADC3001 and 3101 Ricard Wanderlof
2021-12-01 13:43 ` Mark Brown
2021-12-01 13:58   ` Ricard Wanderlof [this message]

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=alpine.DEB.2.21.2112011454470.30606@lnxricardw1.se.axis.com \
    --to=ricardw@axis.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=broonie@kernel.org \
    --cc=lgirdwood@gmail.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: link
Be 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.