[PATCH v2 0/4] Add support for an external Master Clock in the Cirrus CS4271 codec

Herve Codina posted 4 patches 1 month, 2 weeks ago
.../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(-)
[PATCH v2 0/4] Add support for an external Master Clock in the Cirrus CS4271 codec
Posted by Herve Codina 1 month, 2 weeks ago
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

Re: (subset) [PATCH v2 0/4] Add support for an external Master Clock in the Cirrus CS4271 codec
Posted by Mark Brown 1 month, 2 weeks ago
On Wed, 29 Oct 2025 10:39:16 +0100, Herve Codina wrote:
> 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.
> 
> [...]

Applied to

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next

Thanks!

[2/4] ASoC: cs4271: Disable regulators in component_probe() error path
      commit: 1e5351ba60f5355809f30c61bbd27e97611d2be9
[3/4] ASoC: dt-bindings: cirrus,cs4271: Document mclk clock
      commit: 3cd523ba270665861647304aa30500f238ebf26e
[4/4] ASoC: cs4271: Add support for the external mclk
      commit: cf6bf51b53252284bafc7377a4d8dbf10f048b4d

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark