From: Herve Codina <herve.codina@bootlin.com>
To: 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>,
Alexander Sverdlin <alexander.sverdlin@gmail.com>,
Nikita Shubin <nikita.shubin@maquefel.me>,
Axel Lin <axel.lin@ingics.com>,
Brian Austin <brian.austin@cirrus.com>
Cc: linux-sound@vger.kernel.org, patches@opensource.cirrus.com,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
Herve Codina <herve.codina@bootlin.com>
Subject: [PATCH v2 0/4] Add support for an external Master Clock in the Cirrus CS4271 codec
Date: Wed, 29 Oct 2025 10:39:16 +0100 [thread overview]
Message-ID: <20251029093921.624088-1-herve.codina@bootlin.com> (raw)
The Cirrus CS4271 codec can have its Master Clock provided by an
external clock when no crystal is used.
This series adds support for this external Master clock.
The first patch in the series is not related to the clock but fixes an
issue related to module loading and MODULE_DEVICE_TABLE() due to a
driver split between i2c part and spi part.
The next patch fixes regulator handling in component_probe() error path.
The last two patches document the Master clock in the binding and
implement this clock handling in the existing driver.
Best regards,
Hervé
Changes v1 -> v2
v1: https://lore.kernel.org/lkml/20251016130340.1442090-1-herve.codina@bootlin.com/
- Patch 1:
Add missing MODULE_DEVICE_TABLE(spi, ...) in spi part.
- Patch 2 (new in v2)
Disable regulators in component_probe() error path
- Patch 3 (2 in v1)
Add 'Reviewed-by: Krzysztof Kozlowski'
- Patch 4 (3 in v1):
Remove fsleep() call.
Disable/enable the clock at suspend/resume.
Remove the reset line assertion on errors (not needed).
Herve Codina (4):
ASoC: cs4271: Fix cs4271 I2C and SPI drivers automatic module loading
ASoC: cs4271: Disable regulators in component_probe() error path
ASoC: dt-bindings: cirrus,cs4271: Document mclk clock
ASoC: cs4271: Add support for the external mclk
.../bindings/sound/cirrus,cs4271.yaml | 10 +++++
sound/soc/codecs/cs4271-i2c.c | 6 +++
sound/soc/codecs/cs4271-spi.c | 13 ++++++
sound/soc/codecs/cs4271.c | 42 +++++++++++++------
sound/soc/codecs/cs4271.h | 1 -
5 files changed, 59 insertions(+), 13 deletions(-)
--
2.51.0
next reply other threads:[~2025-10-29 9:39 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-10-29 9:39 Herve Codina [this message]
2025-10-29 9:39 ` [PATCH v2 1/4] ASoC: cs4271: Fix cs4271 I2C and SPI drivers automatic module loading Herve Codina
2025-10-29 11:20 ` Alexander Sverdlin
2025-10-30 13:43 ` Herve Codina
2025-10-30 13:52 ` Mark Brown
2025-10-30 13:54 ` Alexander Sverdlin
2025-10-30 14:01 ` Alexander Sverdlin
2025-10-29 9:39 ` [PATCH v2 2/4] ASoC: cs4271: Disable regulators in component_probe() error path Herve Codina
2025-10-29 11:14 ` Alexander Sverdlin
2025-10-29 9:39 ` [PATCH v2 3/4] ASoC: dt-bindings: cirrus,cs4271: Document mclk clock Herve Codina
2025-10-29 9:39 ` [PATCH v2 4/4] ASoC: cs4271: Add support for the external mclk Herve Codina
2025-10-29 11:24 ` Alexander Sverdlin
2025-10-30 16:50 ` (subset) [PATCH v2 0/4] Add support for an external Master Clock in the Cirrus CS4271 codec 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=20251029093921.624088-1-herve.codina@bootlin.com \
--to=herve.codina@bootlin.com \
--cc=alexander.sverdlin@gmail.com \
--cc=axel.lin@ingics.com \
--cc=brian.austin@cirrus.com \
--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=nikita.shubin@maquefel.me \
--cc=patches@opensource.cirrus.com \
--cc=perex@perex.cz \
--cc=rf@opensource.cirrus.com \
--cc=robh@kernel.org \
--cc=thomas.petazzoni@bootlin.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;
as well as URLs for NNTP newsgroup(s).