mbox series

[0/3] Add support for AW8898 Amplifier

Message ID 20250406-aw8898-v1-0-58a2d554693f@lucaweiss.eu
Headers show
Series Add support for AW8898 Amplifier | expand

Message

Luca Weiss April 6, 2025, 1:03 p.m. UTC
Add devicetree bindings and a driver for the AW8898 Amplifier, and add
it to the devicetree for the Fairphone 3 smartphone.

Note, that this is the first audio driver I've worked on, so please
complain if something doesn't quite look right, it probably wasn't done
on purpose.

Unfortunately we can't wire up the full audio stack yet since the SoC
side is not upstream yet, the last revision from July 2024 can be found
here:

https://lore.kernel.org/linux-arm-msm/20240731-msm8953-msm8976-asoc-v3-0-163f23c3a28d@gmail.com/

Signed-off-by: Luca Weiss <luca@lucaweiss.eu>
---
Luca Weiss (3):
      ASoC: dt-bindings: Document Awinic AW8898 amplifier
      ASoC: codecs: Add aw8898 amplifier driver
      arm64: dts: qcom: sdm632-fairphone-fp3: Add AW8898 amplifier

 .../devicetree/bindings/sound/awinic,aw8898.yaml   |  75 +++
 arch/arm64/boot/dts/qcom/sdm632-fairphone-fp3.dts  |  20 +
 sound/soc/codecs/Kconfig                           |  10 +
 sound/soc/codecs/Makefile                          |   1 +
 sound/soc/codecs/snd-soc-aw8898.c                  | 583 +++++++++++++++++++++
 5 files changed, 689 insertions(+)
---
base-commit: 287e43781488fd6ca5f4306dbb4f2a88c5564d79
change-id: 20250406-aw8898-06988582d2e0

Best regards,

Comments

Krzysztof Kozlowski April 6, 2025, 1:53 p.m. UTC | #1
On 06/04/2025 15:03, Luca Weiss wrote:
> +static const struct i2c_device_id aw8898_id[] = {
> +	{ "aw8898" },
> +	{ /* sentinel */ }
> +};
> +MODULE_DEVICE_TABLE(i2c, aw8898_id);
> +
> +static const struct of_device_id aw8898_of_match[] = {
> +	{ .compatible = "awinic,aw8898" },
> +	{ /* sentinel */ }
> +};
> +MODULE_DEVICE_TABLE(of, aw8898_of_match);
> +
> +static struct i2c_driver aw8898_driver = {
> +	.driver = {
> +		.name = "aw8898",
> +		.of_match_table = of_match_ptr(aw8898_of_match),


Drop of_match_ptr.

Best regards,
Krzysztof