mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Richard Fitzgerald <rf@opensource.cirrus.com>
To: "Martin Povišer" <povik+lin@cutebit.org>,
	"James Schulman" <james.schulman@cirrus.com>,
	"David Rhodes" <david.rhodes@cirrus.com>,
	"Lucas Tanure" <tanureal@opensource.cirrus.com>,
	"Liam Girdwood" <lgirdwood@gmail.com>,
	"Mark Brown" <broonie@kernel.org>,
	"Rob Herring" <robh+dt@kernel.org>,
	"Krzysztof Kozlowski" <krzysztof.kozlowski+dt@linaro.org>,
	"Jaroslav Kysela" <perex@perex.cz>,
	"Takashi Iwai" <tiwai@suse.com>
Cc: Charles Keepax <ckeepax@opensource.cirrus.com>,
	ChiYuan Huang <cy_huang@richtek.com>,
	Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>,
	Lukas Bulwahn <lukas.bulwahn@gmail.com>,
	Matt Flax <flatmax@flatmax.com>,
	- <patches@opensource.cirrus.com>, <alsa-devel@alsa-project.org>,
	<devicetree@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
	<asahi@lists.linux.dev>
Subject: Re: [PATCH 00/10] Support for CS42L83 on Apple machines
Date: Fri, 9 Sep 2022 17:16:48 +0100	[thread overview]
Message-ID: <a7c2df88-766a-4657-8379-649a5ae93ac4@opensource.cirrus.com> (raw)
In-Reply-To: <20220909135334.98220-1-povik+lin@cutebit.org>

On 09/09/2022 14:53, Martin Povišer wrote:
> Hi all,
> 
> there's a CS42L83 headphone jack codec found in Apple computers (in the
> recent 'Apple Silicon' ones as well as in earlier models, one example
> [1]). The part isn't publicly documented, but it appears almost
> identical to CS42L42, for which we have a driver in kernel. This series
> adapts the CS42L42 driver to the new part, and makes one change in
> anticipation of a machine driver for the Apple computers.
> 
> Patch 1 adds new compatible to the cs42l42 schema.
> 
> Patches 2 to 7 are taken from Richard's recent series [2] adding
> soundwire support to cs42l42. They are useful refactorings to build on
> in later patches, and also this way our work doesn't diverge. I made
> one fix: I added a call of common_remove at the end of i2c_probe should
> the cs42l42_init call fail (both before and after the split to
> cs42l42-i2c.c). Also s/Soundwire/SoundWire/ in the changelogs.
> 

Mark: I've no objection to you taking my patches from this chain instead
of waiting for me to re-send them myself. I can rebase my remaining
patches onto this chain. But I do have comments on patches #4 and #7.

I've been very busy and don't have time right now to deal with
re-sending my original patch chain.

  parent reply	other threads:[~2022-09-09 16:18 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-09-09 13:53 Martin Povišer
2022-09-09 13:53 ` [PATCH 01/10] ASoC: dt-bindings: cs42l42: Add 'cs42l83' compatible Martin Povišer
2022-09-09 14:48   ` Krzysztof Kozlowski
2022-09-09 13:53 ` [PATCH 02/10] ASoC: cs42l42: Add bitclock frequency argument to cs42l42_pll_config() Martin Povišer
2022-09-09 13:53 ` [PATCH 03/10] ASoC: cs42l42: Use cs42l42->dev instead of &i2c_client->dev Martin Povišer
2022-09-09 13:53 ` [PATCH 04/10] ASoC: cs42l42: Split probe() and remove() into stages Martin Povišer
2022-09-09 15:55   ` Richard Fitzgerald
2022-09-15  9:08   ` Martin Povišer
2022-09-09 13:53 ` [PATCH 05/10] ASoC: cs42l42: Split cs42l42_resume into two functions Martin Povišer
2022-09-09 13:53 ` [PATCH 06/10] ASoC: cs42l42: Pass component and dai defs into common probe Martin Povišer
2022-09-09 13:53 ` [PATCH 07/10] ASoC: cs42l42: Split I2C identity into separate module Martin Povišer
2022-09-09 15:40   ` Richard Fitzgerald
2022-09-09 15:44     ` Martin Povišer
2022-09-09 16:00       ` Richard Fitzgerald
2022-09-09 13:53 ` [PATCH 08/10] ASoC: cs42l42: Export regmap elements to core namespace Martin Povišer
2022-09-09 13:53 ` [PATCH 09/10] ASoC: cs42l83: Extend CS42L42 support to new part Martin Povišer
2022-09-09 14:47   ` Krzysztof Kozlowski
2022-09-09 15:10     ` Martin Povišer
2022-09-09 15:33       ` Krzysztof Kozlowski
2022-09-09 13:53 ` [PATCH 10/10] ASoC: cs42l42: Implement 'set_bclk_ratio' Martin Povišer
2022-09-09 16:16 ` Richard Fitzgerald [this message]
2022-09-09 17:27   ` [PATCH 00/10] Support for CS42L83 on Apple machines Mark Brown

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=a7c2df88-766a-4657-8379-649a5ae93ac4@opensource.cirrus.com \
    --to=rf@opensource.cirrus.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=asahi@lists.linux.dev \
    --cc=broonie@kernel.org \
    --cc=ckeepax@opensource.cirrus.com \
    --cc=cy_huang@richtek.com \
    --cc=david.rhodes@cirrus.com \
    --cc=devicetree@vger.kernel.org \
    --cc=flatmax@flatmax.com \
    --cc=james.schulman@cirrus.com \
    --cc=krzysztof.kozlowski+dt@linaro.org \
    --cc=lgirdwood@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lukas.bulwahn@gmail.com \
    --cc=patches@opensource.cirrus.com \
    --cc=perex@perex.cz \
    --cc=pierre-louis.bossart@linux.intel.com \
    --cc=povik+lin@cutebit.org \
    --cc=robh+dt@kernel.org \
    --cc=tanureal@opensource.cirrus.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: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®