[Letux-kernel] [Internal RFC 00/16] Modernize sound support for OpenPandora

H. Nikolaus Schaller hns at goldelico.com
Sat Sep 17 19:16:42 CEST 2022


Hi all,
any comments before I try to submit upstream?

One major piece is missing: updating bindings doc for the pandora sound driver [14/16].
Currently it is just a copy of a simple soundcard which is not what it is meant
to be...

Anyone volunteering to work on this?

BR and thanks,
Nikolaus


> Am 07.09.2022 um 20:05 schrieb H. Nikolaus Schaller <hns at goldelico.com>:
> 
> Hi all,
> I found some hours to work on this and here is the result.
> Mainly I have fixed most of the comments from checkpatch
> which meant splitting some patches into two and do some reformatting.
> And adding bindings documentation.
> 
> Otherwise I have changed the opnd, prefix to openpandora,
> and renamed the amp-gpio.
> 
> Please comment the individual patches (or provide an update)
> so that I can collect for a RFC publication on the kernel mailing list.
> 
> BR and thanks,
> Nikolaus
> 
> 
> 
>> Am 07.09.2022 um 20:01 schrieb H. Nikolaus Schaller <hns at goldelico.com>:
>> 
>> Internal RFC V1 2022-09-07 20:01:55:
>> Tony Lindgren wrote (https://marc.info/?l=alsa-devel&m=165632351319716&w=2):
>> 
>> Subject: Pandora audio using machine_is_omap3_pandora() check
>> 
>> Arnd noticed that pandora audio is using machine_is_omap3_pandora() check
>> that never succeeds for devicetree booting machines. Looks like this has
>> been broken at some point many years ago with the devicetree conversion.
>> 
>> Does anybody have an interest in fixing this driver?
>> 
>> Regards,
>> 
>> Tony
>> 
>> ----
>> 
>> So far, the kernel driver (sound/soc/ti/omp3pandora.c) is an
>> old style sound board file. So we rewrite it to be a platform
>> driver, add a pcm1773 codec driver, add some special handling
>> for the twl4030-regen output and adapt the pandora device tree
>> files.
>> 
>> Notes:
>> - tested on OpenPandora
>> - we know that some .yaml files are missing to describe the ti,pcm1773 codec
>> and the opnd,omap3pandora-sound
>> and perhaps twl-regulator.yaml should be there instead of fixing twl-regulator.txt
>> 
>> This work was done by multiple authors:
>> Grond <grond66 at riseup.net>
>> Stefan Leichter <sle85276 at gmx.de>
>> H. Nikolaus Schaller <hns at goldelico.com>
>> 
>> 
>> Grond (10):
>> arm: dts: omap3pandora: Don't use DMA channels for unused SPI masters
>> ASoC: pcm1773-codec: write a driver for the PCM1773 chip from TI
>> arm: dts: omap3pandora: Add device node for PCM1773 codec
>> ASoC: twl4030-codec: Allow setting APLL rate through the .set_sysclk()
>>   interface
>> dt-bindings: twl-regulator: Expose the TWL4030s REGEN signal as a
>>   regulator
>> regulator: twl: Expose the TWL4030's REGEN signal as a regulator
>> arm: dts: twl4030: add twl4030-regen child
>> arm: dts: omap3pandora: Populate DT data for the TWL4030's REGEN
>>   regulator
>> ASoC: omap3pandora: Rewrite sound card driver as a platform driver
>> arm: dts: omap3pandora: Create new DT node for the sound card
>> 
>> H. Nikolaus Schaller (3):
>> dt-bindings: vendor-prefixes: add OpenPandora GmbH
>> ASoC: dt-bindings: add TI PCM1773
>> ASoC: dt-bindings: add OpenPandora Sound Card [NEEDS TO BE WORKED OUT]
>> 
>> Stefan Leichter (3):
>> ASoC: omap: omap3pandora: Fix module load for multi SBC kernel
>> ASoC: omap: omap3pandora: Correct the cpu_dai and platform for DT boot
>> ARM: dts: omap3-pandora-common: Enable audio in/out (mcbsp4/2)
>> 
>> .../bindings/regulator/twl-regulator.txt      |  12 +
>> .../sound/openpandora,omap3pandora-sound.yaml | 499 ++++++++++++++++++
>> .../devicetree/bindings/sound/pcm1773.yaml    |  26 +
>> .../devicetree/bindings/vendor-prefixes.yaml  |   2 +
>> arch/arm/boot/dts/omap3-pandora-common.dtsi   |  41 ++
>> arch/arm/boot/dts/twl4030.dtsi                |  14 +
>> drivers/regulator/twl-regulator.c             |  52 ++
>> include/linux/mfd/twl.h                       |   3 +
>> include/linux/mfd/twl4030-audio.h             |   3 +
>> sound/soc/codecs/Kconfig                      |   5 +
>> sound/soc/codecs/Makefile                     |   2 +
>> sound/soc/codecs/pcm1773.c                    | 177 +++++++
>> sound/soc/codecs/twl4030.c                    | 140 +++--
>> sound/soc/ti/omap3pandora.c                   | 438 +++++++++++----
>> 14 files changed, 1271 insertions(+), 143 deletions(-)
>> create mode 100644 Documentation/devicetree/bindings/sound/openpandora,omap3pandora-sound.yaml
>> create mode 100644 Documentation/devicetree/bindings/sound/pcm1773.yaml
>> create mode 100644 sound/soc/codecs/pcm1773.c
>> 
>> -- 
>> 2.33.0
>> 
> 



More information about the Letux-kernel mailing list