(subset) [PATCH v12 0/4] ASoC: codecs: add rk817 support
Heiko Stuebner
heiko at sntech.de
Thu Jun 3 02:56:17 PDT 2021
On Wed, 19 May 2021 15:37:50 -0500, Chris Morgan wrote:
> This series is to add support for the Rockchip rk817 audio codec which
> is present in all Rockchip rk817 MFD chips.
>
> Changes in v12:
> - Updated codec cell in MFD driver to a single line.
> Changes in v11:
> - Stopped checking for presence of parent node in codec driver, as
> driver should never be called if parent device doesn't exist to call
> it.
> - Made codec subnode optional and stopped returning errors if it is
> missing.
> - Downgraded missing codec clock message from dev_err to dev_dbg.
> Users who choose to use the rk817 PMIC but do not choose to use the
> codec will not get an error message unless they have debugging
> enabled, whereas users who wish to use the codec but don't have the
> clock defined will not get an error message unless they have
> debug messages enabled. This seems like a fair compromise between
> mututally exclusive users. Note that errors with the clock or codec
> will still result in a dev_err.
> - Updated DT documentation to note that codec subnode is optional if
> no properties are to exist beneath it.
> Changes in v10:
> - Correct order of test/ack/signed-off to chronological order.
> - Removed ifdef from mfd driver keep cell from being added when the
> codec was not to be built.
> - Changed codec DT parsing messages from error to debug to prevent
> errors in dmesg log in the rare but valid case a user has the rk817
> PMIC but does not use the codec.
> Changes in v9:
> - Add cover letter.
> - Remove documentation for interrupt parent per Rob Herring's request.
> - Remove unused MODULE_DEVICE_TABLE to fix a bug identified by kernel test
> robot.
> Changes in v8:
> - Added additional documentation for missing properties of
> #sound-dai-cells, interrupt-parent, and wakeup-source for mfd
> documentation.
> - Corrected order of elements descriptions in device tree documentation.
> - Changed name of "mic-in-differential" to
> "rockchip,mic-in-differential".
> - Changed name of sound card from "rockchip,rk817-codec" to "Analog".
> - Removed unused resets and reset-names from the i2s1_2ch node.
> Changes in v7:
> - Removed ifdef around register definitions for MFD.
> - Replaced codec documentation with updates to MFD documentation.
> - Reordered elements in example to comply with upstream rules.
> - Added binding update back for Odroid Go Advance as requested.
> - Submitting patches from gmail now.
> Changes in v6:
> - Included additional project maintainers for correct subsystems.
> - Removed unneeded compatible from DT documentation.
> - Removed binding update for Odroid Go Advance (will do in separate
> series).
> Changes in v5:
> - Move register definitions from rk817_codec.h to main rk808.h register
> definitions.
> - Add volatile register for codec bits.
> - Add default values for codec bits.
> - Removed of_compatible from mtd driver (not necessary).
> - Switched to using parent regmap instead of private regmap for codec.
> Changes in v4:
> - Created set_pll() call.
> - Created user visible gain control in mic.
> - Check for return value of clk_prepare_enable().
> - Removed duplicate clk_prepare_enable().
> - Split DT documentation to separate commit.
> Changes in v3:
> - Use DAPM macros to set audio path.
> - Updated devicetree binding (as every rk817 has this codec chip).
> - Changed documentation to yaml format.
> - Split MFD changes to separate commit.
> Changes in v2:
> - Fixed audio path registers to solve some bugs.
>
> [...]
Applied, thanks!
[4/4] arm64: dts: rockchip: add rk817 codec to Odroid Go
commit: 18d5c7bf50c6d820c366c2a23d71d468b14c87d6
Best regards,
--
Heiko Stuebner <heiko at sntech.de>
More information about the Linux-rockchip
mailing list