devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: James Calligeros <jcalligeros99@gmail.com>
To: "Martin Povišer" <povik+lin@cutebit.org>,
	"James Calligeros" <jcalligeros99@gmail.com>,
	"David Rhodes" <david.rhodes@cirrus.com>,
	"Richard Fitzgerald" <rf@opensource.cirrus.com>,
	"Liam Girdwood" <lgirdwood@gmail.com>,
	"Mark Brown" <broonie@kernel.org>,
	"Rob Herring" <robh@kernel.org>,
	"Krzysztof Kozlowski" <krzk+dt@kernel.org>,
	"Conor Dooley" <conor+dt@kernel.org>,
	"Jaroslav Kysela" <perex@perex.cz>,
	"Takashi Iwai" <tiwai@suse.com>
Cc: asahi@lists.linux.dev, linux-sound@vger.kernel.org,
	 patches@opensource.cirrus.com, devicetree@vger.kernel.org,
	 linux-kernel@vger.kernel.org, Hector Martin <marcan@marcan.st>,
	 Neal Gompa <neal@gompa.dev>
Subject: [PATCH v2 0/3] ASoC: add CS42L84 codec driver
Date: Sun, 20 Oct 2024 00:47:30 +1000	[thread overview]
Message-ID: <20241020-cs42l84-v2-0-37ba2b6721d9@gmail.com> (raw)

Hi all,

This is version 2 of the Cirrus Logic CS42L84 ASoC codec driver.

This patch set adds a driver for the Cirrus Logic CS42L84 codec. This chip
is (so far) found only on Apple Silicon Macs. In keeping with proud Apple
tradition, the CS42L84 is essentially just a CS42L42 with a different
regmap and no publicly available datasheet. It may also be missing its
parent's S/PDIF capabilities as none of Apple's devices support S/PDIF out,
however this cannot be positively confirmed.

This driver has lived in the downstream Asahi tree for quite a while now,
and gained some refinements along the way. I have squashed most of these
into the initial driver commit as they were small changes like tweaking
msleep()s or filling out TLVs, but left seperate a larger change to
tip/ring sense IRQ handling as it differs significantly from what is found
in the CS42L42 driver.

---
Changes in v2:
- Fixed failing DT binding checks. Sorry! (Patch 1)

- Added Martin's name to the DT binding maintainer list (Patch 1)

- Documented #sound-dai-cells property in DT binding (Patch 1)

- Dropped superfluous DT binding descriptions (Patch 1)

- Fixed DT binding description whitespace  (Patch 1)

- Fixed licence comment format in cs42l42.h (Patch 2)

- Fixed erroneous whitespace in cs42l84.h (Patch 2)

- Switched driver to use maple tree for regcache (Patch 2)

- Lowered max_register property to address of highest observed register (Patch 2)

- Fixed switch/case statement style (Patches 2, 3)

- Picked up Neal's Reviewed-by

- Link to v1: https://lore.kernel.org/r/20241016-cs42l84-v1-0-8d7e9d437d2d@gmail.com

---
James Calligeros (1):
      ASoC: cs42l84: leverage ring sense IRQs to correctly detect headsets

Martin Povišer (2):
      dt-bindings: sound: Add CS42L84 codec
      ASoC: cs42l84: Add new codec driver

 .../bindings/sound/cirrus,cs42l84.yaml   |   56 ++
 MAINTAINERS                              |    2 +
 sound/soc/codecs/Kconfig                 |    7 +
 sound/soc/codecs/Makefile                |    2 +
 sound/soc/codecs/cs42l84.c               | 1112 +++++++++++++++++++++++++
 sound/soc/codecs/cs42l84.h               |  210 +++++
 6 files changed, 1389 insertions(+)
---
base-commit: 469819cc17368702a6f68cec2148f518d3f3679b
change-id: 20241016-cs42l84-f38348d21c88

Best regards,
-- 
James Calligeros <jcalligeros99@gmail.com>


             reply	other threads:[~2024-10-19 14:49 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-10-19 14:47 James Calligeros [this message]
2024-10-19 14:47 ` [PATCH v2 1/3] dt-bindings: sound: Add CS42L84 codec James Calligeros
2024-10-21 19:26   ` Rob Herring
2024-10-21 22:15     ` Mark Brown
2024-10-22  7:14       ` James Calligeros
2024-10-24 14:59   ` Rob Herring (Arm)
2024-10-24 15:09   ` Mark Brown
2024-10-19 14:47 ` [PATCH v2 2/3] ASoC: cs42l84: Add new codec driver James Calligeros
2024-10-19 14:47 ` [PATCH v2 3/3] ASoC: cs42l84: leverage ring sense IRQs to correctly detect headsets James Calligeros
2024-10-24 20:16 ` [PATCH v2 0/3] ASoC: add CS42L84 codec driver 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=20241020-cs42l84-v2-0-37ba2b6721d9@gmail.com \
    --to=jcalligeros99@gmail.com \
    --cc=asahi@lists.linux.dev \
    --cc=broonie@kernel.org \
    --cc=conor+dt@kernel.org \
    --cc=david.rhodes@cirrus.com \
    --cc=devicetree@vger.kernel.org \
    --cc=krzk+dt@kernel.org \
    --cc=lgirdwood@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-sound@vger.kernel.org \
    --cc=marcan@marcan.st \
    --cc=neal@gompa.dev \
    --cc=patches@opensource.cirrus.com \
    --cc=perex@perex.cz \
    --cc=povik+lin@cutebit.org \
    --cc=rf@opensource.cirrus.com \
    --cc=robh@kernel.org \
    --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;
as well as URLs for NNTP newsgroup(s).